diff mbox series

hwmon: stts751: Add "st" vendor prefix to "stts751" compatible string

Message ID 20240826191811.1416011-1-robh@kernel.org (mailing list archive)
State Accepted
Headers show
Series hwmon: stts751: Add "st" vendor prefix to "stts751" compatible string | expand

Commit Message

Rob Herring (Arm) Aug. 26, 2024, 7:18 p.m. UTC
The documented compatible string is "st,stts751", not "stts751". Even if
"stts751" was in use, there's no need to list "stts751" in the DT match
table. The I2C core will strip any vendor prefix and match against the
i2c_device_id table which has an "stts751" entry.

Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
---
 drivers/hwmon/stts751.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Guenter Roeck Aug. 27, 2024, 3:55 a.m. UTC | #1
On Mon, Aug 26, 2024 at 02:18:11PM -0500, Rob Herring (Arm) wrote:
> The documented compatible string is "st,stts751", not "stts751". Even if
> "stts751" was in use, there's no need to list "stts751" in the DT match
> table. The I2C core will strip any vendor prefix and match against the
> i2c_device_id table which has an "stts751" entry.
> 
> Signed-off-by: Rob Herring (Arm) <robh@kernel.org>

Applied.

Thanks,
Guenter
diff mbox series

Patch

diff --git a/drivers/hwmon/stts751.c b/drivers/hwmon/stts751.c
index e7632081a1d1..f9e8b2869164 100644
--- a/drivers/hwmon/stts751.c
+++ b/drivers/hwmon/stts751.c
@@ -77,7 +77,7 @@  static const struct i2c_device_id stts751_id[] = {
 };
 
 static const struct of_device_id __maybe_unused stts751_of_match[] = {
-	{ .compatible = "stts751" },
+	{ .compatible = "st,stts751" },
 	{ },
 };
 MODULE_DEVICE_TABLE(of, stts751_of_match);