Add bindings for SAM9X7's pmc.
Signed-off-by: Varshini Rajendran <[email protected]>
---
.../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
reg:
--
2.25.1
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 <[email protected]>
>> ---
>> .../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/[email protected]/
Best regards,
Krzysztof
On 28/07/2023 12:23, Varshini Rajendran wrote:
> Add bindings for SAM9X7's pmc.
>
> Signed-off-by: Varshini Rajendran <[email protected]>
> ---
> .../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/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 <[email protected]>
>> ---
>> .../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
>
--
Thanks and Regards,
Varshini Rajendran.
On 10/08/2023 07:31, [email protected] 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, [email protected] 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
>
--
Thanks and Regards,
Varshini Rajendran.