* [PATCH] hwmon: (emc2305) update of_emc2305_match_table
@ 2026-05-21 18:03 Alexander A. Klimov
2026-05-21 18:37 ` Guenter Roeck
0 siblings, 1 reply; 2+ messages in thread
From: Alexander A. Klimov @ 2026-05-21 18:03 UTC (permalink / raw)
To: Guenter Roeck, Frank Li, Florin Leotescu,
open list:HARDWARE MONITORING, open list
Cc: Alexander A. Klimov
Latter was added with only emc2305, in contrast to emc2305_ids
which also listed emc2301, emc2302 and emc2303.
So these three devices couldn't be probed based on device tree contents
until now. I fixed this.
Fixes: 882bd6de1a5b ("hwmon: emc2305: Add OF support")
Signed-off-by: Alexander A. Klimov <grandmaster@al2klimov.de>
---
drivers/hwmon/emc2305.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/hwmon/emc2305.c b/drivers/hwmon/emc2305.c
index 64b213e1451e..4c99018d0575 100644
--- a/drivers/hwmon/emc2305.c
+++ b/drivers/hwmon/emc2305.c
@@ -716,6 +716,9 @@ static int emc2305_probe(struct i2c_client *client)
static const struct of_device_id of_emc2305_match_table[] = {
{ .compatible = "microchip,emc2305", },
+ { .compatible = "microchip,emc2303", },
+ { .compatible = "microchip,emc2302", },
+ { .compatible = "microchip,emc2301", },
{},
};
MODULE_DEVICE_TABLE(of, of_emc2305_match_table);
--
2.54.0
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] hwmon: (emc2305) update of_emc2305_match_table
2026-05-21 18:03 [PATCH] hwmon: (emc2305) update of_emc2305_match_table Alexander A. Klimov
@ 2026-05-21 18:37 ` Guenter Roeck
0 siblings, 0 replies; 2+ messages in thread
From: Guenter Roeck @ 2026-05-21 18:37 UTC (permalink / raw)
To: Alexander A. Klimov, Frank Li, Florin Leotescu,
open list:HARDWARE MONITORING, open list
On 5/21/26 11:03, Alexander A. Klimov wrote:
> Latter was added with only emc2305, in contrast to emc2305_ids
> which also listed emc2301, emc2302 and emc2303.
> So these three devices couldn't be probed based on device tree contents
> until now. I fixed this.
>
> Fixes: 882bd6de1a5b ("hwmon: emc2305: Add OF support")
> Signed-off-by: Alexander A. Klimov <grandmaster@al2klimov.de>
NACK.
The devicetree files should specify microchip,emc2305 as compatible,
as documented in
Documentation/devicetree/bindings/hwmon/microchip,emc2305.yaml.
Upstream devicetree files do that. If downstream devicetree files don't,
they are at fault and will need to be fixed.
Guenter
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-05-21 18:37 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-05-21 18:03 [PATCH] hwmon: (emc2305) update of_emc2305_match_table Alexander A. Klimov
2026-05-21 18:37 ` Guenter Roeck
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®