Re: [PATCH] hwmon: (lm77) Provide of_match_table to allow module auto loading

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] hwmon: (lm77) Undocumented DT compatible string national,lm77
--

commit 9ee0f69ba3227a960f104981995c86d2458362e9
Author: Lukasz Majewski <lukma@nabladev.com>

hwmon: (lm77) Provide of_match_table to allow module auto loading

This commit adds an OF match table for "national,lm77" to the lm77 I2C
driver to enable automatic module loading on Devicetree systems.

> diff --git a/drivers/hwmon/lm77.c b/drivers/hwmon/lm77.c
> index 51bbc84d74844..e64dabfcda09e 100644
> --- a/drivers/hwmon/lm77.c
> +++ b/drivers/hwmon/lm77.c
> @@ -336,6 +336,12 @@ static int lm77_probe(struct i2c_client *client)
>  	return PTR_ERR_OR_ZERO(hwmon_dev);
>  }
>  
> +static const struct of_device_id lm77_of_match[] = {
> +	{ .compatible = "national,lm77" },
> +	{ }
> +};

[Severity: Low]
Is "national,lm77" documented in the Devicetree bindings?  It does not
appear in Documentation/devicetree/bindings/ (such as in
Documentation/devicetree/bindings/trivial-devices.yaml alongside other
National Semiconductor I2C sensors), which causes checkpatch.pl to warn
about an undocumented DT compatible string and dtbs_check to fail on
nodes using it.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261113130133.4094280-1-lukma@nabladev.com?part=1