| Message ID | 20230728102350.265520-1-varshini.rajendran@microchip.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v3,01/50] dt-bindings: microchip: atmel,at91rm9200-tcb: add sam9x60, sam9x7 compatible | expand |
On 28/07/2023 12:23, Varshini Rajendran wrote: > Add bindings for SAM9X7's pmc. > > Signed-off-by: Varshini Rajendran <varshini.rajendran@microchip.com> > --- > .../devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml > index c1bdcd9058ed..ce0d99503645 100644 > --- a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml > +++ b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml > @@ -43,6 +43,7 @@ properties: > - atmel,sama5d4-pmc > - microchip,sam9x60-pmc > - microchip,sama7g5-pmc > + - microchip,sam9x7-pmc > - const: syscon Where is the change in allOf:if:then:? No need for it? Why? Where is the driver change? Please do not send huge series to 50 different people and subsystems. With your lack of threading it is so difficult to review. Best regards, Krzysztof
On 28/07/2023 13:40, Krzysztof Kozlowski wrote: > On 28/07/2023 12:23, Varshini Rajendran wrote: >> Add bindings for SAM9X7's pmc. >> >> Signed-off-by: Varshini Rajendran <varshini.rajendran@microchip.com> >> --- >> .../devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >> index c1bdcd9058ed..ce0d99503645 100644 >> --- a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >> +++ b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >> @@ -43,6 +43,7 @@ properties: >> - atmel,sama5d4-pmc >> - microchip,sam9x60-pmc >> - microchip,sama7g5-pmc >> + - microchip,sam9x7-pmc >> - const: syscon > > Where is the change in allOf:if:then:? No need for it? Why? Where is the > driver change? > > Please do not send huge series to 50 different people and subsystems. > With your lack of threading it is so difficult to review. You already got this comment and not much improved: https://lore.kernel.org/all/f2f8cabf-ca4d-c6f3-5561-b24334be89d1@linaro.org/ Best regards, Krzysztof
On 28/07/23 5:10 pm, Krzysztof Kozlowski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On 28/07/2023 12:23, Varshini Rajendran wrote: >> Add bindings for SAM9X7's pmc. >> >> Signed-off-by: Varshini Rajendran <varshini.rajendran@microchip.com> >> --- >> .../devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >> index c1bdcd9058ed..ce0d99503645 100644 >> --- a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >> +++ b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >> @@ -43,6 +43,7 @@ properties: >> - atmel,sama5d4-pmc >> - microchip,sam9x60-pmc >> - microchip,sama7g5-pmc >> + - microchip,sam9x7-pmc >> - const: syscon > > Where is the change in allOf:if:then:? No need for it? Why? Where is the > driver change? Hi Krzysztof, This SoC has a new pmc driver and is not dependent on any other compatible as a fallback. So I think adding it in the enum makes sense. Correct me if I am wrong. The driver is in the series. Apologies for breaking the thread. I will send another version following your advice about splitting up the patch series. > > Please do not send huge series to 50 different people and subsystems. > With your lack of threading it is so difficult to review. > > Best regards, > Krzysztof >
On 10/08/2023 07:31, Varshini.Rajendran@microchip.com wrote: on/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>> index c1bdcd9058ed..ce0d99503645 100644 >>> --- a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>> +++ b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>> @@ -43,6 +43,7 @@ properties: >>> - atmel,sama5d4-pmc >>> - microchip,sam9x60-pmc >>> - microchip,sama7g5-pmc >>> + - microchip,sam9x7-pmc >>> - const: syscon >> >> Where is the change in allOf:if:then:? No need for it? Why? Where is the >> driver change? > > Hi Krzysztof, > > This SoC has a new pmc driver and is not dependent on any other > compatible as a fallback. So I think adding it in the enum makes sense. This I did not question. I was speaking about allOf:if:then: section. Best regards, Krzysztof
On 10/08/23 11:32 am, Krzysztof Kozlowski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > On 10/08/2023 07:31, Varshini.Rajendran@microchip.com wrote: > on/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml > b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>>> index c1bdcd9058ed..ce0d99503645 100644 >>>> --- a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>>> +++ b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>>> @@ -43,6 +43,7 @@ properties: >>>> - atmel,sama5d4-pmc >>>> - microchip,sam9x60-pmc >>>> - microchip,sama7g5-pmc >>>> + - microchip,sam9x7-pmc >>>> - const: syscon >>> >>> Where is the change in allOf:if:then:? No need for it? Why? Where is the >>> driver change? >> >> Hi Krzysztof, >> >> This SoC has a new pmc driver and is not dependent on any other >> compatible as a fallback. So I think adding it in the enum makes sense. > > This I did not question. > > I was speaking about allOf:if:then: section. Okay. Got your point. I will fix that in the next version. Thanks! > > > Best regards, > Krzysztof >
diff --git a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml index c1bdcd9058ed..ce0d99503645 100644 --- a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml +++ b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml @@ -43,6 +43,7 @@ properties: - atmel,sama5d4-pmc - microchip,sam9x60-pmc - microchip,sama7g5-pmc + - microchip,sam9x7-pmc - const: syscon reg:
Add bindings for SAM9X7's pmc. Signed-off-by: Varshini Rajendran <varshini.rajendran@microchip.com> --- .../devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml | 1 + 1 file changed, 1 insertion(+)