Messages in this thread | | | Date | Wed, 13 Apr 2022 14:42:56 +0200 | Subject | Re: [RESEND PATCH] arm: dts: at91: Fix boolean properties with values | From | Nicolas Ferre <> |
| |
On 06/04/2022 at 21:09, Rob Herring wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > Boolean properties in DT are present or not present and don't take a value. > A property such as 'foo = <0>;' evaluated to true. IOW, the value doesn't > matter. > > It may have been intended that 0 values are false, but there is no change > in behavior with this patch. > > Signed-off-by: Rob Herring <robh@kernel.org>
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
> --- > Can someone apply this for 5.18 please.
Yes, sure, it's now queued in the fixes branch for 5.18.
Thanks for your help Rob. Best regards, Nicolas
> > arch/arm/boot/dts/at91-kizbox3-hs.dts | 2 +- > arch/arm/boot/dts/at91-kizbox3_common.dtsi | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/arm/boot/dts/at91-kizbox3-hs.dts b/arch/arm/boot/dts/at91-kizbox3-hs.dts > index 2799b2a1f4d2..f7d90cf1bb77 100644 > --- a/arch/arm/boot/dts/at91-kizbox3-hs.dts > +++ b/arch/arm/boot/dts/at91-kizbox3-hs.dts > @@ -225,7 +225,7 @@ pinctrl_pio_zbe_rst: gpio_zbe_rst { > pinctrl_pio_io_reset: gpio_io_reset { > pinmux = <PIN_PB30__GPIO>; > bias-disable; > - drive-open-drain = <1>; > + drive-open-drain; > output-low; > }; > pinctrl_pio_input: gpio_input { > diff --git a/arch/arm/boot/dts/at91-kizbox3_common.dtsi b/arch/arm/boot/dts/at91-kizbox3_common.dtsi > index abe27adfa4d6..465664628419 100644 > --- a/arch/arm/boot/dts/at91-kizbox3_common.dtsi > +++ b/arch/arm/boot/dts/at91-kizbox3_common.dtsi > @@ -211,7 +211,7 @@ pinctrl_flx4_default: flx4_i2c6_default { > pinmux = <PIN_PD12__FLEXCOM4_IO0>, //DATA > <PIN_PD13__FLEXCOM4_IO1>; //CLK > bias-disable; > - drive-open-drain = <1>; > + drive-open-drain; > }; > > pinctrl_pwm0 { > -- > 2.32.0 >
-- Nicolas Ferre
| |