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

[PATCH] hwmon: stts751: Add "st" vendor prefix to "stts751" compatible string

Posted by Rob Herring (Arm) 1 year, 3 months ago

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(-)

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);
-- 
2.43.0

Re: [PATCH] hwmon: stts751: Add "st" vendor prefix to "stts751" compatible string

Posted by Guenter Roeck 1 year, 3 months ago

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