| Message ID | 20240822132708.51884-1-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | dt-bindings: i2c: aspeed: drop redundant multi-master | expand |
On Thu, Aug 22, 2024 at 03:27:08PM +0200, Krzysztof Kozlowski wrote: > 'multi-master' property is defined by core i2c-controller schema in > dtschema package, so binding which references it and has > unevaluatedProperties:false, does not need to mention it. It is > completely redundant here. > > Suggested-by: Andi Shyti <andi.shyti@kernel.org> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Acked-by: Conor Dooley <conor.dooley@microchip.com>
Hi Krzysztof, On Thu, Aug 22, 2024 at 03:27:08PM GMT, Krzysztof Kozlowski wrote: > 'multi-master' property is defined by core i2c-controller schema in > dtschema package, so binding which references it and has > unevaluatedProperties:false, does not need to mention it. It is > completely redundant here. > > Suggested-by: Andi Shyti <andi.shyti@kernel.org> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Thanks for taking care of this! Merged to i2c/i2c-host. Andi
On Thu, 2024-08-22 at 15:27 +0200, Krzysztof Kozlowski wrote: > 'multi-master' property is defined by core i2c-controller schema in > dtschema package, so binding which references it and has > unevaluatedProperties:false, does not need to mention it. It is > completely redundant here. > > Suggested-by: Andi Shyti <andi.shyti@kernel.org> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > Acked-by: Andrew Jeffery <andrew@codeconstruct.com.au>
diff --git a/Documentation/devicetree/bindings/i2c/aspeed,i2c.yaml b/Documentation/devicetree/bindings/i2c/aspeed,i2c.yaml index 6df27b47b922..5b9bd2feda3b 100644 --- a/Documentation/devicetree/bindings/i2c/aspeed,i2c.yaml +++ b/Documentation/devicetree/bindings/i2c/aspeed,i2c.yaml @@ -44,11 +44,6 @@ properties: description: frequency of the bus clock in Hz defaults to 100 kHz when not specified - multi-master: - type: boolean - description: - states that there is another master active on this bus - required: - reg - compatible
'multi-master' property is defined by core i2c-controller schema in dtschema package, so binding which references it and has unevaluatedProperties:false, does not need to mention it. It is completely redundant here. Suggested-by: Andi Shyti <andi.shyti@kernel.org> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Reference: https://github.com/devicetree-org/dt-schema/blob/main/dtschema/schemas/i2c/i2c-controller.yaml#L114 --- Documentation/devicetree/bindings/i2c/aspeed,i2c.yaml | 5 ----- 1 file changed, 5 deletions(-)