[PATCH v8 1/5] dt-bindings: i2c: qcom-cci: Document qcs8300 compatible

Nihal Kumar Gupta posted 5 patches 1 month, 2 weeks ago
There is a newer version of this series
[PATCH v8 1/5] dt-bindings: i2c: qcom-cci: Document qcs8300 compatible
Posted by Nihal Kumar Gupta 1 month, 2 weeks ago
The three instances of CCI found on the QCS8300 are functionally the same
as on a number of existing Qualcomm SoCs.

Introduce a new SoC-specific compatible string "qcom,qcs8300-cci" with a
common fallback.

Signed-off-by: Nihal Kumar Gupta <quic_nihalkum@quicinc.com>
Signed-off-by: Vikram Sharma <quic_vikramsa@quicinc.com>
Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
Reviewed-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>
---
 Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
index a3fe1eea6aec..399a09409e07 100644
--- a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
+++ b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
@@ -28,6 +28,7 @@ properties:
           - enum:
               - qcom,kaanapali-cci
               - qcom,qcm2290-cci
+              - qcom,qcs8300-cci
               - qcom,sa8775p-cci
               - qcom,sc7280-cci
               - qcom,sc8280xp-cci
@@ -133,6 +134,7 @@ allOf:
             enum:
               - qcom,kaanapali-cci
               - qcom,qcm2290-cci
+              - qcom,qcs8300-cci
               - qcom,sm8750-cci
     then:
       properties:
-- 
2.34.1
Re: [PATCH v8 1/5] dt-bindings: i2c: qcom-cci: Document qcs8300 compatible
Posted by Bjorn Andersson 2 weeks, 5 days ago
On Mon, Dec 22, 2025 at 03:29:10PM +0530, Nihal Kumar Gupta wrote:
> The three instances of CCI found on the QCS8300 are functionally the same
> as on a number of existing Qualcomm SoCs.
> 
> Introduce a new SoC-specific compatible string "qcom,qcs8300-cci" with a
> common fallback.
> 
> Signed-off-by: Nihal Kumar Gupta <quic_nihalkum@quicinc.com>
> Signed-off-by: Vikram Sharma <quic_vikramsa@quicinc.com>
> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
> Reviewed-by: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>

Maybe I'm not seeing it, but you didn't include either i2c maintainers
or the linux-i2c mailing list, so this is unlikely to move anywhere.

Please send this out again with the recipients that get_maintainers tell
you do include. (and skip the ones that aren't relevant).

And please adopt b4 (go/upstream tells you how) to avoid such (human)
mistakes going forward.

Thanks,
Bjorn

> ---
>  Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
> index a3fe1eea6aec..399a09409e07 100644
> --- a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
> +++ b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml
> @@ -28,6 +28,7 @@ properties:
>            - enum:
>                - qcom,kaanapali-cci
>                - qcom,qcm2290-cci
> +              - qcom,qcs8300-cci
>                - qcom,sa8775p-cci
>                - qcom,sc7280-cci
>                - qcom,sc8280xp-cci
> @@ -133,6 +134,7 @@ allOf:
>              enum:
>                - qcom,kaanapali-cci
>                - qcom,qcm2290-cci
> +              - qcom,qcs8300-cci
>                - qcom,sm8750-cci
>      then:
>        properties:
> -- 
> 2.34.1
>
Re: [PATCH v8 1/5] dt-bindings: i2c: qcom-cci: Document qcs8300 compatible
Posted by Nihal Kumar Gupta 2 weeks, 5 days ago

On 21-01-2026 19:30, Bjorn Andersson wrote:
> Maybe I'm not seeing it, but you didn't include either i2c maintainers
> or the linux-i2c mailing list, so this is unlikely to move anywhere.
> 
> Please send this out again with the recipients that get_maintainers tell
> you do include. (and skip the ones that aren't relevant).
Apologies, and thanks for pointing this out.
Posted v9, rebased on tip, with the I2C maintainers and the linux-i2c mailing list included.
-- 
Regards,
Nihal Kumar Gupta