diff mbox series

[1/2] HID: hid-goodix: drop unsupported and undocumented DT part

Message ID 20240925194921.18933-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State New
Delegated to: Jiri Kosina
Headers show
Series [1/2] HID: hid-goodix: drop unsupported and undocumented DT part | expand

Commit Message

Krzysztof Kozlowski Sept. 25, 2024, 7:49 p.m. UTC
Drop support for Devicetree from, because the binding is being reverted
(on basis of duplicating existing binding) and property was not added to
the original binding.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 drivers/hid/hid-goodix-spi.c | 9 ---------
 1 file changed, 9 deletions(-)

Comments

Krzysztof Kozlowski Sept. 25, 2024, 8:03 p.m. UTC | #1
On 25/09/2024 21:49, Krzysztof Kozlowski wrote:
> Drop support for Devicetree from, because the binding is being reverted
> (on basis of duplicating existing binding) and property was not added to

This probably is unclear. Should be:

...(on basis of duplicating existing binding) and the driver uses a
property was not added to the original binding.

> the original binding.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---

I really wished we did not have to drop parts of drivers, but after
three pings no one cared to fix this (author does not respond at all)
and it landed in Linus' affecting everyone.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/drivers/hid/hid-goodix-spi.c b/drivers/hid/hid-goodix-spi.c
index de655f745d3f..0e59663814dd 100644
--- a/drivers/hid/hid-goodix-spi.c
+++ b/drivers/hid/hid-goodix-spi.c
@@ -786,14 +786,6 @@  static const struct acpi_device_id goodix_spi_acpi_match[] = {
 MODULE_DEVICE_TABLE(acpi, goodix_spi_acpi_match);
 #endif
 
-#ifdef CONFIG_OF
-static const struct of_device_id goodix_spi_of_match[] = {
-	{ .compatible = "goodix,gt7986u", },
-	{ }
-};
-MODULE_DEVICE_TABLE(of, goodix_spi_of_match);
-#endif
-
 static const struct spi_device_id goodix_spi_ids[] = {
 	{ "gt7986u" },
 	{ },
@@ -804,7 +796,6 @@  static struct spi_driver goodix_spi_driver = {
 	.driver = {
 		.name = "goodix-spi-hid",
 		.acpi_match_table = ACPI_PTR(goodix_spi_acpi_match),
-		.of_match_table = of_match_ptr(goodix_spi_of_match),
 		.pm = pm_sleep_ptr(&goodix_spi_pm_ops),
 	},
 	.probe =	goodix_spi_probe,