On Friday, July 05, 2024 15:20 IST, Marek Vasut <marex@xxxxxxx> wrote: > The "ltr,ltrf216a" compatible string is not documented in DT binding > document, remove it. > > Signed-off-by: Marek Vasut <marex@xxxxxxx> > --- > Cc: Conor Dooley <conor+dt@xxxxxxxxxx> > Cc: Jonathan Cameron <jic23@xxxxxxxxxx> > Cc: Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx> > Cc: Lars-Peter Clausen <lars@xxxxxxxxxx> > Cc: Marek Vasut <marex@xxxxxxx> > Cc: Rob Herring <robh@xxxxxxxxxx> > Cc: Shreeya Patel <shreeya.patel@xxxxxxxxxxxxx> > Cc: devicetree@xxxxxxxxxxxxxxx > Cc: linux-iio@xxxxxxxxxxxxxxx > --- > drivers/iio/light/ltrf216a.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/iio/light/ltrf216a.c b/drivers/iio/light/ltrf216a.c > index 68dc48420a886..78fc910fcb18c 100644 > --- a/drivers/iio/light/ltrf216a.c > +++ b/drivers/iio/light/ltrf216a.c > @@ -528,7 +528,6 @@ MODULE_DEVICE_TABLE(i2c, ltrf216a_id); > > static const struct of_device_id ltrf216a_of_match[] = { > { .compatible = "liteon,ltrf216a" }, > - { .compatible = "ltr,ltrf216a" }, > {} This compatible string with a different vendor prefix was added for a specific reason. Please see the commit message of the following patch :- https://lore.kernel.org/all/20220511094024.175994-2-shreeya.patel@xxxxxxxxxxxxx/ We were very well aware that not documenting this was going to generate a warning so we tried to fix that with a deprecated tag but it was NAKd by Rob. What we understood from his last message was that it wasn't necessary to fix the DT warning. Thanks, Shreeya Patel > }; > MODULE_DEVICE_TABLE(of, ltrf216a_of_match); > -- > 2.43.0 >