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