Re: [PATCH] iio: light: ltrf216a: Drop undocumented ltr,ltrf216a compatible string

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 7/5/24 12:42 PM, Shreeya Patel wrote:
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.

From what I read in the aforementioned discussion thread, it seems Rob was very much opposed to this compatible string, so this shouldn't have gone in in the first place.

But it did ... so the question is, what now ?




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux