Re: [PATCH 32/43] dt-bindings: gpio: Add DT bindings ep93xx gpio

From: Linus Walleij
Date: Wed Apr 26 2023 - 16:48:40 EST


On Mon, Apr 24, 2023 at 6:32 PM Rob Herring <robh@xxxxxxxxxx> wrote:
> On Mon, Apr 24, 2023 at 03:34:48PM +0300, Nikita Shubin wrote:

> > Add YAML bindings for ep93xx SoC.
> >
> > Signed-off-by: Nikita Shubin <nikita.shubin@xxxxxxxxxxx>
(...)
> > + chip-label:
> > + maxItems: 1
> > + description: human readable name.
>
> Why do you need this? It's not standard and I don't see other GPIO
> controllers needing it.

Caught my eye too, Nikita can you live without this and just use dev_name()
or something to name the chip in Linux?

If it is to conform to EP93xx documentation naming I guess it should be
cirrus,ep93xx-gpio-chip-name = "..."; ?

Yours,
Linus Walleij