[PATCH v1 4/5] pwm: Unify coding style of of_device_id arrays

[Krzysztof Kozlowski]

On 06/07/2026 12:39, Uwe Kleine-König (The Capable Hub) wrote:
> Hello,
> 
> On Mon, Jul 06, 2026 at 10:28:16AM +0200, Philippe Mathieu-Daudé wrote:
>> Hi Uwe,
>>
>> On 6/7/26 09:19, Uwe Kleine-König (The Capable Hub) wrote:
>>> Hello Chen,
>>>
>>> On Mon, Jul 06, 2026 at 08:33:47AM +0800, Chen Wang wrote:
>>>> On 7/5/2026 11:14 PM, Uwe Kleine-König (The Capable Hub) wrote:
>>>>>    - Use a single space in a of_device_id array terminator; A single space
>>>>>      after the opening { and before the closing } in non-empty
>>>>>      initializers;
>>>>>    - No comma after an array terminator;
>>>>>    - Also no trailing comma after a named initializer iff the
>>>> iff -> if
>>>
>>> This is actually intended. iff = "if and only if", see e.g.
>>> https://en.wikipedia.org/wiki/If_and_only_if .
>>
>> I agree this scientific idiomatic expression is not well known
>> by non-native English speakers, and using the expanded form is
>> just simpler to review (thinking at non-native English speaker
>> reading your commit in the future).
> 
> fine for me, I'll fixup to 

Noooo, really, this is nitpicking on this commit. Not that I like that
commit - it's just churn - but 'iff' is widely known and widely used in
Linux kernel. It's completely correct, it is common and expected.

It is not scientific expression. It is common in the kernel docs.

Best regards,
Krzysztof