[RFC PATCH 1/6] dt-bindings: pinctrl: starfive: add JH8100 pinctrl bindings

Conor Dooley conor at kernel.org
Thu Jan 4 10:07:03 PST 2024


On Thu, Jan 04, 2024 at 03:12:31AM +0000, Yuklin Soo wrote:
> 
> 
> > -----Original Message-----
> > From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> > Sent: Thursday, December 21, 2023 11:45 PM
> > To: Linus Walleij <linus.walleij at linaro.org>; Yuklin Soo
> > <yuklin.soo at starfivetech.com>
> > Cc: Bartosz Golaszewski <bartosz.golaszewski at linaro.org>; Hal Feng
> > <hal.feng at starfivetech.com>; Leyfoon Tan <leyfoon.tan at starfivetech.com>;
> > Jianlong Huang <jianlong.huang at starfivetech.com>; Emil Renner Berthing
> > <kernel at esmil.dk>; Rob Herring <robh at kernel.org>; Krzysztof Kozlowski
> > <krzysztof.kozlowski+dt at linaro.org>; Conor Dooley <conor+dt at kernel.org>;
> > Drew Fustini <drew at beagleboard.org>; linux-gpio at vger.kernel.org; linux-
> > kernel at vger.kernel.org; devicetree at vger.kernel.org; linux-
> > riscv at lists.infradead.org; Paul Walmsley <paul.walmsley at sifive.com>; Palmer
> > Dabbelt <palmer at dabbelt.com>; Albert Ou <aou at eecs.berkeley.edu>
> > Subject: Re: [RFC PATCH 1/6] dt-bindings: pinctrl: starfive: add JH8100 pinctrl
> > bindings
> > 
> > On 21/12/2023 14:57, Linus Walleij wrote:
> > >> +          drive-strength:
> > >> +            enum: [ 2, 4, 8, 12 ]
> > >
> > > Milliamperes? Then spell that out in a description:
> > 
> > Or just use drive-strength-microamp
> 
> Suggest changing “drive-strength:” to “drive-strength: Drive strength in mA” since the unit is in mA.

Just call the property "drive-strength-microamp". We have existing users
of that property.

Cheers,
Conor.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20240104/8798d26e/attachment.sig>


More information about the linux-riscv mailing list