Re: [PATCH 3/7] pwm: dwc: add of/platform support

From: Krzysztof Kozlowski
Date: Wed Jul 13 2022 - 05:26:34 EST


On 13/07/2022 11:21, Ben Dooks wrote:
> On 12/07/2022 11:08, Krzysztof Kozlowski wrote:
>> On 12/07/2022 12:01, Ben Dooks wrote:
>>> The dwc pwm controller can be used in non-PCI systems, so allow
>>> either platform or OF based probing.
>>>
>>> Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxx>
>>> ---
>>> .../devicetree/bindings/pwm/pwm-synposys.yaml | 40 ++++++++++++++
>>
>> Bindings must be a separate patch. Preferably first in the series. Use
>> proper subject prefix matching the subsystem.
>>
>> Best regards,
>> Krzysztof
>
> Thanks, seems counter-intuitive that one change is split up like that.

Why? Bindings are not really related to driver implementation, they are
shared with other projects. Binding reviewers usually care less about
implementation, so combining it with drivers forces me to read carefully
each one of 300 emails I receive...

Anyway, that's the policy since years, so nothing new here.

Best regards,
Krzysztof