On 13/12/2022 12:56, Konrad Dybcio wrote: > Almost every compatible string in the CCI driver is a duplicate. Adjust > the bindings to include a common (first-soc-implementing-vX) compatible > to remove the need to keep adding superfluous compatible strings. > > Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> > --- > .../devicetree/bindings/i2c/qcom,i2c-cci.yaml | 25 +++++++++++++------ > 1 file changed, 17 insertions(+), 8 deletions(-) > > diff --git a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml > index cf9f8fda595f..65cf7860a9a0 100644 > --- a/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml > +++ b/Documentation/devicetree/bindings/i2c/qcom,i2c-cci.yaml > @@ -12,14 +12,23 @@ maintainers: > > properties: > compatible: > - enum: > - - qcom,msm8226-cci > - - qcom,msm8916-cci > - - qcom,msm8974-cci > - - qcom,msm8996-cci > - - qcom,sdm845-cci > - - qcom,sm8250-cci > - - qcom,sm8450-cci > + oneOf: Except missing test, how about also adding a comment to the driver, that list of compatibles should not grow when not needed? Best regards, Krzysztof
