[PATCH 0/2] of: add support for value "false" to of_property_read_bool
Rob Herring
robh at kernel.org
Tue Nov 12 05:46:09 PST 2024
On Tue, Nov 12, 2024 at 12:41 AM Josua Mayer <josua at solid-run.com> wrote:
>
> Boolean type properties are usually considered true if present and false
> when they do not exist. This works well for many in-tree board dts and
> existing drivers.
>
> When users need to overrride boolean values from included dts,
> /delete-property/ is recommend. This however does not work in overlays
> (addons).
As soon as someone needs to delete a non-boolean property, we're back
to the same problem. If you want to fix it, you need to fix it for any
property.
> Geert pointed out [1] that there are several invitations for using
> strings "true" and "false" on boolean properties: [1], [2], [3].
There's always bad examples...
> Add support for a string value "false" to be considered false on boolean
> properties by changing of_property_read_bool implementation.
Any existing s/w will treat 'foo = "false"' as true. It's an ABI.
Rob
More information about the linux-arm-kernel
mailing list