On 06/04/2023 21:19, Arınç ÜNAL wrote: > On 6.04.2023 22:08, Krzysztof Kozlowski wrote: >> On 06/04/2023 10:01, arinc9.unal@xxxxxxxxx wrote: >>> From: Arınç ÜNAL <arinc.unal@xxxxxxxxxx> >>> >>> The resets and reset-names properties are used only if mediatek,mcm is >>> used. Set them to false if mediatek,mcm is not used. >>> >>> Remove now unnecessary 'reset-names: false' from MT7988. >>> >>> Signed-off-by: Arınç ÜNAL <arinc.unal@xxxxxxxxxx> >>> --- >>> .../devicetree/bindings/net/dsa/mediatek,mt7530.yaml | 5 ++++- >>> 1 file changed, 4 insertions(+), 1 deletion(-) >>> >>> diff --git a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml >>> index 9d99f7303453..3fd953b1453e 100644 >>> --- a/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml >>> +++ b/Documentation/devicetree/bindings/net/dsa/mediatek,mt7530.yaml >>> @@ -282,6 +282,10 @@ allOf: >>> required: >>> - resets >>> - reset-names >>> + else: >>> + properties: >>> + resets: false >>> + reset-names: false >>> >>> - dependencies: >>> interrupt-controller: [ interrupts ] >>> @@ -324,7 +328,6 @@ allOf: >>> properties: >>> gpio-controller: false >>> mediatek,mcm: false >>> - reset-names: false >> >> I don't see such hunk in linux-next. > > This was added very recently so it's only on net-next at the moment. OK, I was checking one few days old. Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> Best regards, Krzysztof
