This series makes using (of|fwnode|device)_property_read_bool() on non-boolean properties give a warning when used with DT. This has been deprecated behavior for a while now. Soon we will add yet another variant of this function with a Rust binding[1]. Why not put the warning in the fwnode code? The fwnode code doesn't know the property's type as that depends on the firmware. For DT, a boolean is a property with no value. Perhaps the ACPI backend should have a warning too? I looked briefly at it, but I don't have a clue how or if that can be detected. Doesn't look like there's a specific type for booleans. In any case, that would be an additional change on top of this series. [1] https://lore.kernel.org/all/20241025-rust-platform-dev-v1-2-0df8dcf7c20b@xxxxxxxxxx/ Signed-off-by: Rob Herring (Arm) <robh@xxxxxxxxxx> --- Rob Herring (Arm) (2): device property: Split property reading bool and presence test ops of: Warn when of_property_read_bool() is used on non-boolean properties drivers/acpi/property.c | 1 + drivers/base/property.c | 38 ++++++++++++++++++++++++++++++++++++++ drivers/base/swnode.c | 1 + drivers/of/property.c | 33 +++++++++++++++++++++++++++++++++ include/linux/fwnode.h | 3 +++ include/linux/of.h | 29 ++++++++++------------------- include/linux/property.h | 15 +++------------ 7 files changed, 89 insertions(+), 31 deletions(-) --- base-commit: 40384c840ea1944d7c5a392e8975ed088ecf0b37 change-id: 20250109-dt-type-warnings-efe52d977bb2 Best regards, -- Rob Herring (Arm) <robh@xxxxxxxxxx>
