[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CACRpkdbJFNhE1acJ5eNncvfK0VZgW=pCB3716Rv3KahdK1dokQ@mail.gmail.com>
Date: Tue, 8 Nov 2022 13:11:42 +0100
From: Linus Walleij <linus.walleij@...aro.org>
To: Yinbo Zhu <zhuyinbo@...ngson.cn>
Cc: Bartosz Golaszewski <brgl@...ev.pl>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
WANG Xuerui <kernel@...0n.name>,
Jiaxun Yang <jiaxun.yang@...goat.com>,
Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
Juxin Gao <gaojuxin@...ngson.cn>,
Bibo Mao <maobibo@...ngson.cn>,
Yanteng Si <siyanteng@...ngson.cn>, linux-gpio@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
loongarch@...ts.linux.dev, linux-mips@...r.kernel.org,
richard.liu@...com, Arnaud Patard <apatard@...driva.com>,
Hongbing Hu <huhb@...ote.com>,
Huacai Chen <chenhuacai@...nel.org>
Subject: Re: [PATCH v1 2/2] dt-bindings: gpio: add loongson series gpio
Hi Yinbo,
thanks for your patch!
On Tue, Nov 8, 2022 at 10:21 AM Yinbo Zhu <zhuyinbo@...ngson.cn> wrote:
> Add the Loongson series gpio binding with DT schema format using
> json-schema.
>
> Signed-off-by: Yinbo Zhu <zhuyinbo@...ngson.cn>
> + gpio-ranges: true
So you are using GPIO ranges... and...
> + loongson,gpio_base:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description:
> + This option indicate the first GPIO number in this node.
Then you have added this to reimplement gpio ranges.
It also shows in the driver.
Drop gpio_base altogether (we do not encode linux-specific properties
into the device trees) and use gpio-ranges as they are intended to
map between the GPIO numberspace and the pin control pin number
space.
See
Documentation/devicetree/bindings/gpio/gpio.txt
for documentation on gpio-ranges, also look how other drivers are
using them.
Yours,
Linus Walleij
Powered by blists - more mailing lists