Re: [PATCH v7 1/6] dt-bindings: i2c: Describe SA8255p

From: Shawn Guo

Date: Fri Jul 03 2026 - 23:27:03 EST


On Wed, Jun 17, 2026 at 10:20:15AM +0530, Praveen Talari wrote:
[...]
> diff --git a/Documentation/devicetree/bindings/i2c/qcom,sa8255p-geni-i2c.yaml b/Documentation/devicetree/bindings/i2c/qcom,sa8255p-geni-i2c.yaml
> new file mode 100644
> index 000000000000..a61e40b5cbc1
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/i2c/qcom,sa8255p-geni-i2c.yaml
> @@ -0,0 +1,64 @@
> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/i2c/qcom,sa8255p-geni-i2c.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Qualcomm SA8255p QUP GENI I2C Controller
> +
> +maintainers:
> + - Praveen Talari <praveen.talari@xxxxxxxxxxxxxxxx>
> +
> +properties:
> + compatible:
> + const: qcom,sa8255p-geni-i2c

The compatible had already been defined by soc/qcom/qcom,sa8255p-geni-se-qup.yaml.
Now we get the same compatible defined by two binding files.

I spot this because I'm running check_dtbs for Nord SA8797P and getting
complaints on "qcom,sa8797p-geni-i2c" compatible which is already defined
by geni-se-qup yaml. But check_dtbs seems to be running the check against
sa8255p-geni-i2c yaml added by this patch.

Shawn