Re: [PATCH v5 5/6] dt-bindings: display: Add Synaptics R63455 panel support

[Krzysztof Kozlowski <krzk@kernel.org>]

On 29/09/2026 14:54, Linus Walleij wrote:
> On Mon, Jul 27, 2026 at 10:29 PM Krzysztof Kozlowski <krzk@kernel.org> wrote:
>> On Mon, Jul 27, 2026 at 04:08:44PM +0800, Jun Nie wrote:
> 
>>> +  reset-gpios:
>>> +    maxItems: 2
>>> +    description: 2 reset pins for 2 physical panels
>>
>> Won't work. gpio-consumer-common expects only one reset line. You need
>> two separate properties.
> 
> Hm that is a big confusion for the head.
> 
> reset-gpios is by it's very nature plural, because we structured

So GPIOs are plural, but not reset-gpios. Just like shutdown-gpios. If
you have two pins to turn off one device, how does it work? Maybe you
have simple two devices?

> all GPIOs like that, exactly in order to indicate that they can be
> arrays. :/

Yes, in many cases.

> 
> I think it's appropriate to patch gpio-consumer-common to accept
>> 1 gpios, in a separate patch, can even be sent outside
> of this series.
> 
> Unless the DT maintainers have some good reason to slap my
> hands for that suggestion...

Make a case, can be an exception in gpio-consumer-common. But if you
have two panels, why can't it go to the graph for each of them?

This entire binding seems like stitching two devices together, which
might be fine (I don't even remember this stuff... two months old) or
might be artificial grouping of separate devices.

Best regards,
Krzysztof