* [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match
@ 2024-03-29 7:56 Tzung-Bi Shih
2024-03-29 7:56 ` [PATCH 01/18] media: platform: cros-ec: " Tzung-Bi Shih
` (21 more replies)
0 siblings, 22 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Inspired by [1]. Turn all MODULE_ALIAS() in ChromeOS EC platform drivers into
proper platform_device_id table and MODULE_DEVICE_TABLE().
The series is basically looking for drivers from:
- `struct mfd_cell` in drivers/mfd/cros_ec_dev.c.
- grep -R MODULE_ALIAS drivers/platform/chrome/.
[1]: https://lore.kernel.org/chrome-platform/20240325-public-ucsi-h-v2-0-a6d716968bb1@chromium.org/T/#m2cc7d6f770cf0a48fb537bbaed655169c974f067
Tzung-Bi Shih (18):
media: platform: cros-ec: provide ID table for avoiding fallback match
gpio: cros-ec: provide ID table for avoiding fallback match
rtc: cros-ec: provide ID table for avoiding fallback match
platform/chrome: cros_ec_sensorhub: provide ID table for avoiding
fallback match
power: supply: cros_usbpd: provide ID table for avoiding fallback
match
platform/chrome: cros_usbpd_logger: provide ID table for avoiding
fallback match
platform/chrome: cros_usbpd_notify: provide ID table for avoiding
fallback match
platform/chrome: cros_ec_chardev: provide ID table for avoiding
fallback match
platform/chrome: cros_ec_debugfs: provide ID table for avoiding
fallback match
platform/chrome: cros_ec_sysfs: provide ID table for avoiding fallback
match
power: supply: cros_pchg: provide ID table for avoiding fallback match
platform/chrome: cros_ec_lightbar: provide ID table for avoiding
fallback match
platform/chrome: cros_ec_vbc: provide ID table for avoiding fallback
match
platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding
fallback match
platform/chrome: wilco_ec: telemetry: provide ID table for avoiding
fallback match
platform/chrome: wilco_ec: debugfs: provide ID table for avoiding
fallback match
platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS
platform/chrome/wilco_ec: core: provide ID table for avoiding fallback
match
drivers/gpio/gpio-cros-ec.c | 8 ++++++++
drivers/media/cec/platform/cros-ec/cros-ec-cec.c | 9 ++++++++-
drivers/platform/chrome/cros_ec_chardev.c | 9 ++++++++-
drivers/platform/chrome/cros_ec_debugfs.c | 9 ++++++++-
drivers/platform/chrome/cros_ec_lightbar.c | 9 ++++++++-
drivers/platform/chrome/cros_ec_sensorhub.c | 9 ++++++++-
drivers/platform/chrome/cros_ec_sysfs.c | 9 ++++++++-
drivers/platform/chrome/cros_ec_vbc.c | 9 ++++++++-
drivers/platform/chrome/cros_kbd_led_backlight.c | 11 +++++++++--
drivers/platform/chrome/cros_usbpd_logger.c | 9 ++++++++-
drivers/platform/chrome/cros_usbpd_notify.c | 9 ++++++++-
drivers/platform/chrome/wilco_ec/core.c | 9 ++++++++-
drivers/platform/chrome/wilco_ec/debugfs.c | 9 ++++++++-
drivers/platform/chrome/wilco_ec/event.c | 1 -
drivers/platform/chrome/wilco_ec/telemetry.c | 9 ++++++++-
drivers/power/supply/cros_peripheral_charger.c | 11 +++++++++--
drivers/power/supply/cros_usbpd-charger.c | 11 +++++++++--
drivers/rtc/rtc-cros-ec.c | 9 ++++++++-
18 files changed, 139 insertions(+), 20 deletions(-)
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply [flat|nested] 64+ messages in thread
* [PATCH 01/18] media: platform: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 20:05 ` Benson Leung
2024-03-29 7:56 ` [PATCH 02/18] gpio: " Tzung-Bi Shih
` (20 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/media/cec/platform/cros-ec/cros-ec-cec.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
index 48ed2993d2f0..8fbbb4091455 100644
--- a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
+++ b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
@@ -8,6 +8,7 @@
#include <linux/kernel.h>
#include <linux/module.h>
+#include <linux/mod_devicetable.h>
#include <linux/platform_device.h>
#include <linux/dmi.h>
#include <linux/pci.h>
@@ -573,6 +574,12 @@ static void cros_ec_cec_remove(struct platform_device *pdev)
}
}
+static const struct platform_device_id cros_ec_cec_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_cec_id);
+
static struct platform_driver cros_ec_cec_driver = {
.probe = cros_ec_cec_probe,
.remove_new = cros_ec_cec_remove,
@@ -580,6 +587,7 @@ static struct platform_driver cros_ec_cec_driver = {
.name = DRV_NAME,
.pm = &cros_ec_cec_pm_ops,
},
+ .id_table = cros_ec_cec_id,
};
module_platform_driver(cros_ec_cec_driver);
@@ -587,4 +595,3 @@ module_platform_driver(cros_ec_cec_driver);
MODULE_DESCRIPTION("CEC driver for ChromeOS ECs");
MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>");
MODULE_LICENSE("GPL");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 02/18] gpio: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
2024-03-29 7:56 ` [PATCH 01/18] media: platform: cros-ec: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
` (2 more replies)
2024-03-29 7:56 ` [PATCH 03/18] rtc: " Tzung-Bi Shih
` (19 subsequent siblings)
21 siblings, 3 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match. Also allow automatic module loading by adding
MODULE_DEVICE_TABLE().
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/gpio/gpio-cros-ec.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/gpio/gpio-cros-ec.c b/drivers/gpio/gpio-cros-ec.c
index 842e1c060414..0c09bb54dc0c 100644
--- a/drivers/gpio/gpio-cros-ec.c
+++ b/drivers/gpio/gpio-cros-ec.c
@@ -12,6 +12,7 @@
#include <linux/errno.h>
#include <linux/gpio/driver.h>
#include <linux/kernel.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_commands.h>
#include <linux/platform_data/cros_ec_proto.h>
@@ -197,11 +198,18 @@ static int cros_ec_gpio_probe(struct platform_device *pdev)
return devm_gpiochip_add_data(dev, gc, cros_ec);
}
+static const struct platform_device_id cros_ec_gpio_id[] = {
+ { "cros-ec-gpio", 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_gpio_id);
+
static struct platform_driver cros_ec_gpio_driver = {
.probe = cros_ec_gpio_probe,
.driver = {
.name = "cros-ec-gpio",
},
+ .id_table = cros_ec_gpio_id,
};
module_platform_driver(cros_ec_gpio_driver);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 03/18] rtc: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
2024-03-29 7:56 ` [PATCH 01/18] media: platform: cros-ec: " Tzung-Bi Shih
2024-03-29 7:56 ` [PATCH 02/18] gpio: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
` (2 more replies)
2024-03-29 7:56 ` [PATCH 04/18] platform/chrome: cros_ec_sensorhub: " Tzung-Bi Shih
` (18 subsequent siblings)
21 siblings, 3 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/rtc/rtc-cros-ec.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/rtc/rtc-cros-ec.c b/drivers/rtc/rtc-cros-ec.c
index 0cd397c04ff0..f57462c7b2c6 100644
--- a/drivers/rtc/rtc-cros-ec.c
+++ b/drivers/rtc/rtc-cros-ec.c
@@ -5,6 +5,7 @@
// Author: Stephen Barber <smbarber@chromium.org>
#include <linux/kernel.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_commands.h>
#include <linux/platform_data/cros_ec_proto.h>
@@ -392,6 +393,12 @@ static void cros_ec_rtc_remove(struct platform_device *pdev)
dev_err(dev, "failed to unregister notifier\n");
}
+static const struct platform_device_id cros_ec_rtc_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_rtc_id);
+
static struct platform_driver cros_ec_rtc_driver = {
.probe = cros_ec_rtc_probe,
.remove_new = cros_ec_rtc_remove,
@@ -399,6 +406,7 @@ static struct platform_driver cros_ec_rtc_driver = {
.name = DRV_NAME,
.pm = &cros_ec_rtc_pm_ops,
},
+ .id_table = cros_ec_rtc_id,
};
module_platform_driver(cros_ec_rtc_driver);
@@ -406,4 +414,3 @@ module_platform_driver(cros_ec_rtc_driver);
MODULE_DESCRIPTION("RTC driver for Chrome OS ECs");
MODULE_AUTHOR("Stephen Barber <smbarber@chromium.org>");
MODULE_LICENSE("GPL v2");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 04/18] platform/chrome: cros_ec_sensorhub: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (2 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 03/18] rtc: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 12:16 ` Krzysztof Kozlowski
2024-03-29 20:08 ` Benson Leung
2024-03-29 7:56 ` [PATCH 05/18] power: supply: cros_usbpd: " Tzung-Bi Shih
` (17 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_ec_sensorhub.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/cros_ec_sensorhub.c b/drivers/platform/chrome/cros_ec_sensorhub.c
index 31fb8bdaad5a..50cdae67fa32 100644
--- a/drivers/platform/chrome/cros_ec_sensorhub.c
+++ b/drivers/platform/chrome/cros_ec_sensorhub.c
@@ -8,6 +8,7 @@
#include <linux/init.h>
#include <linux/device.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_commands.h>
#include <linux/platform_data/cros_ec_proto.h>
@@ -247,17 +248,23 @@ static SIMPLE_DEV_PM_OPS(cros_ec_sensorhub_pm_ops,
cros_ec_sensorhub_suspend,
cros_ec_sensorhub_resume);
+static const struct platform_device_id cros_ec_sensorhub_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_sensorhub_id);
+
static struct platform_driver cros_ec_sensorhub_driver = {
.driver = {
.name = DRV_NAME,
.pm = &cros_ec_sensorhub_pm_ops,
},
.probe = cros_ec_sensorhub_probe,
+ .id_table = cros_ec_sensorhub_id,
};
module_platform_driver(cros_ec_sensorhub_driver);
-MODULE_ALIAS("platform:" DRV_NAME);
MODULE_AUTHOR("Gwendal Grignou <gwendal@chromium.org>");
MODULE_DESCRIPTION("ChromeOS EC MEMS Sensor Hub Driver");
MODULE_LICENSE("GPL");
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 05/18] power: supply: cros_usbpd: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (3 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 04/18] platform/chrome: cros_ec_sensorhub: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 12:18 ` Krzysztof Kozlowski
` (2 more replies)
2024-03-29 7:56 ` [PATCH 06/18] platform/chrome: cros_usbpd_logger: " Tzung-Bi Shih
` (16 subsequent siblings)
21 siblings, 3 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/power/supply/cros_usbpd-charger.c | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/drivers/power/supply/cros_usbpd-charger.c b/drivers/power/supply/cros_usbpd-charger.c
index b6c96376776a..8008e31c0c09 100644
--- a/drivers/power/supply/cros_usbpd-charger.c
+++ b/drivers/power/supply/cros_usbpd-charger.c
@@ -5,6 +5,7 @@
* Copyright (c) 2014 - 2018 Google, Inc
*/
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_commands.h>
#include <linux/platform_data/cros_ec_proto.h>
@@ -711,16 +712,22 @@ static int cros_usbpd_charger_resume(struct device *dev)
static SIMPLE_DEV_PM_OPS(cros_usbpd_charger_pm_ops, NULL,
cros_usbpd_charger_resume);
+static const struct platform_device_id cros_usbpd_charger_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_usbpd_charger_id);
+
static struct platform_driver cros_usbpd_charger_driver = {
.driver = {
.name = DRV_NAME,
.pm = &cros_usbpd_charger_pm_ops,
},
- .probe = cros_usbpd_charger_probe
+ .probe = cros_usbpd_charger_probe,
+ .id_table = cros_usbpd_charger_id,
};
module_platform_driver(cros_usbpd_charger_driver);
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("ChromeOS EC USBPD charger");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 06/18] platform/chrome: cros_usbpd_logger: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (4 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 05/18] power: supply: cros_usbpd: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:11 ` Benson Leung
2024-03-29 7:56 ` [PATCH 07/18] platform/chrome: cros_usbpd_notify: " Tzung-Bi Shih
` (15 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_usbpd_logger.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/cros_usbpd_logger.c b/drivers/platform/chrome/cros_usbpd_logger.c
index f618757f8b32..930c2f47269f 100644
--- a/drivers/platform/chrome/cros_usbpd_logger.c
+++ b/drivers/platform/chrome/cros_usbpd_logger.c
@@ -7,6 +7,7 @@
#include <linux/ktime.h>
#include <linux/math64.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_commands.h>
#include <linux/platform_data/cros_ec_proto.h>
@@ -249,6 +250,12 @@ static int __maybe_unused cros_usbpd_logger_suspend(struct device *dev)
static SIMPLE_DEV_PM_OPS(cros_usbpd_logger_pm_ops, cros_usbpd_logger_suspend,
cros_usbpd_logger_resume);
+static const struct platform_device_id cros_usbpd_logger_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_usbpd_logger_id);
+
static struct platform_driver cros_usbpd_logger_driver = {
.driver = {
.name = DRV_NAME,
@@ -256,10 +263,10 @@ static struct platform_driver cros_usbpd_logger_driver = {
},
.probe = cros_usbpd_logger_probe,
.remove_new = cros_usbpd_logger_remove,
+ .id_table = cros_usbpd_logger_id,
};
module_platform_driver(cros_usbpd_logger_driver);
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("Logging driver for ChromeOS EC USBPD Charger.");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 07/18] platform/chrome: cros_usbpd_notify: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (5 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 06/18] platform/chrome: cros_usbpd_logger: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
` (2 more replies)
2024-03-29 7:56 ` [PATCH 08/18] platform/chrome: cros_ec_chardev: " Tzung-Bi Shih
` (14 subsequent siblings)
21 siblings, 3 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_usbpd_notify.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/cros_usbpd_notify.c b/drivers/platform/chrome/cros_usbpd_notify.c
index aacad022f21d..c83f81d86483 100644
--- a/drivers/platform/chrome/cros_usbpd_notify.c
+++ b/drivers/platform/chrome/cros_usbpd_notify.c
@@ -6,6 +6,7 @@
*/
#include <linux/acpi.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_proto.h>
#include <linux/platform_data/cros_usbpd_notify.h>
@@ -218,12 +219,19 @@ static void cros_usbpd_notify_remove_plat(struct platform_device *pdev)
&pdnotify->nb);
}
+static const struct platform_device_id cros_usbpd_notify_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_usbpd_notify_id);
+
static struct platform_driver cros_usbpd_notify_plat_driver = {
.driver = {
.name = DRV_NAME,
},
.probe = cros_usbpd_notify_probe_plat,
.remove_new = cros_usbpd_notify_remove_plat,
+ .id_table = cros_usbpd_notify_id,
};
static int __init cros_usbpd_notify_init(void)
@@ -258,4 +266,3 @@ module_exit(cros_usbpd_notify_exit);
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("ChromeOS power delivery notifier device");
MODULE_AUTHOR("Jon Flatley <jflat@chromium.org>");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 08/18] platform/chrome: cros_ec_chardev: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (6 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 07/18] platform/chrome: cros_usbpd_notify: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:13 ` Benson Leung
2024-03-29 7:56 ` [PATCH 09/18] platform/chrome: cros_ec_debugfs: " Tzung-Bi Shih
` (13 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_ec_chardev.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/cros_ec_chardev.c b/drivers/platform/chrome/cros_ec_chardev.c
index 81950bb2c6da..7f034ead7ae4 100644
--- a/drivers/platform/chrome/cros_ec_chardev.c
+++ b/drivers/platform/chrome/cros_ec_chardev.c
@@ -14,6 +14,7 @@
#include <linux/device.h>
#include <linux/fs.h>
#include <linux/miscdevice.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/notifier.h>
#include <linux/platform_data/cros_ec_chardev.h>
@@ -403,17 +404,23 @@ static void cros_ec_chardev_remove(struct platform_device *pdev)
misc_deregister(&data->misc);
}
+static const struct platform_device_id cros_ec_chardev_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_chardev_id);
+
static struct platform_driver cros_ec_chardev_driver = {
.driver = {
.name = DRV_NAME,
},
.probe = cros_ec_chardev_probe,
.remove_new = cros_ec_chardev_remove,
+ .id_table = cros_ec_chardev_id,
};
module_platform_driver(cros_ec_chardev_driver);
-MODULE_ALIAS("platform:" DRV_NAME);
MODULE_AUTHOR("Enric Balletbo i Serra <enric.balletbo@collabora.com>");
MODULE_DESCRIPTION("ChromeOS EC Miscellaneous Character Driver");
MODULE_LICENSE("GPL");
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 09/18] platform/chrome: cros_ec_debugfs: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (7 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 08/18] platform/chrome: cros_ec_chardev: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:14 ` Benson Leung
2024-03-29 7:56 ` [PATCH 10/18] platform/chrome: cros_ec_sysfs: " Tzung-Bi Shih
` (12 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_ec_debugfs.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/cros_ec_debugfs.c b/drivers/platform/chrome/cros_ec_debugfs.c
index 6bf6f0e7b597..e1d313246beb 100644
--- a/drivers/platform/chrome/cros_ec_debugfs.c
+++ b/drivers/platform/chrome/cros_ec_debugfs.c
@@ -7,6 +7,7 @@
#include <linux/debugfs.h>
#include <linux/delay.h>
#include <linux/fs.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/mutex.h>
#include <linux/platform_data/cros_ec_commands.h>
@@ -564,6 +565,12 @@ static int __maybe_unused cros_ec_debugfs_resume(struct device *dev)
static SIMPLE_DEV_PM_OPS(cros_ec_debugfs_pm_ops,
cros_ec_debugfs_suspend, cros_ec_debugfs_resume);
+static const struct platform_device_id cros_ec_debugfs_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_debugfs_id);
+
static struct platform_driver cros_ec_debugfs_driver = {
.driver = {
.name = DRV_NAME,
@@ -572,10 +579,10 @@ static struct platform_driver cros_ec_debugfs_driver = {
},
.probe = cros_ec_debugfs_probe,
.remove_new = cros_ec_debugfs_remove,
+ .id_table = cros_ec_debugfs_id,
};
module_platform_driver(cros_ec_debugfs_driver);
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("Debug logs for ChromeOS EC");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 10/18] platform/chrome: cros_ec_sysfs: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (8 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 09/18] platform/chrome: cros_ec_debugfs: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 12:37 ` Krzysztof Kozlowski
2024-03-29 20:15 ` Benson Leung
2024-03-29 7:56 ` [PATCH 11/18] power: supply: cros_pchg: " Tzung-Bi Shih
` (11 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_ec_sysfs.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/cros_ec_sysfs.c b/drivers/platform/chrome/cros_ec_sysfs.c
index 93e67ab4af06..9c944146ee50 100644
--- a/drivers/platform/chrome/cros_ec_sysfs.c
+++ b/drivers/platform/chrome/cros_ec_sysfs.c
@@ -8,6 +8,7 @@
#include <linux/device.h>
#include <linux/fs.h>
#include <linux/kobject.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_commands.h>
#include <linux/platform_data/cros_ec_proto.h>
@@ -347,16 +348,22 @@ static void cros_ec_sysfs_remove(struct platform_device *pd)
sysfs_remove_group(&ec_dev->class_dev.kobj, &cros_ec_attr_group);
}
+static const struct platform_device_id cros_ec_sysfs_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_sysfs_id);
+
static struct platform_driver cros_ec_sysfs_driver = {
.driver = {
.name = DRV_NAME,
},
.probe = cros_ec_sysfs_probe,
.remove_new = cros_ec_sysfs_remove,
+ .id_table = cros_ec_sysfs_id,
};
module_platform_driver(cros_ec_sysfs_driver);
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("Expose the ChromeOS EC through sysfs");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 11/18] power: supply: cros_pchg: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (9 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 10/18] platform/chrome: cros_ec_sysfs: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:16 ` Benson Leung
2024-03-29 7:56 ` [PATCH 12/18] platform/chrome: cros_ec_lightbar: " Tzung-Bi Shih
` (10 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/power/supply/cros_peripheral_charger.c | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/drivers/power/supply/cros_peripheral_charger.c b/drivers/power/supply/cros_peripheral_charger.c
index a204f2355be4..d406f2a78449 100644
--- a/drivers/power/supply/cros_peripheral_charger.c
+++ b/drivers/power/supply/cros_peripheral_charger.c
@@ -5,6 +5,7 @@
* Copyright 2020 Google LLC.
*/
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/notifier.h>
#include <linux/platform_data/cros_ec_commands.h>
@@ -367,16 +368,22 @@ static int __maybe_unused cros_pchg_resume(struct device *dev)
static SIMPLE_DEV_PM_OPS(cros_pchg_pm_ops, NULL, cros_pchg_resume);
+static const struct platform_device_id cros_pchg_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_pchg_id);
+
static struct platform_driver cros_pchg_driver = {
.driver = {
.name = DRV_NAME,
.pm = &cros_pchg_pm_ops,
},
- .probe = cros_pchg_probe
+ .probe = cros_pchg_probe,
+ .id_table = cros_pchg_id,
};
module_platform_driver(cros_pchg_driver);
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("ChromeOS EC peripheral device charger");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 12/18] platform/chrome: cros_ec_lightbar: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (10 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 11/18] power: supply: cros_pchg: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:18 ` Benson Leung
2024-03-29 7:56 ` [PATCH 13/18] platform/chrome: cros_ec_vbc: " Tzung-Bi Shih
` (9 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_ec_lightbar.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/cros_ec_lightbar.c b/drivers/platform/chrome/cros_ec_lightbar.c
index 6677cc6c4984..1e69f61115a4 100644
--- a/drivers/platform/chrome/cros_ec_lightbar.c
+++ b/drivers/platform/chrome/cros_ec_lightbar.c
@@ -9,6 +9,7 @@
#include <linux/fs.h>
#include <linux/kobject.h>
#include <linux/kstrtox.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_commands.h>
#include <linux/platform_data/cros_ec_proto.h>
@@ -594,6 +595,12 @@ static int __maybe_unused cros_ec_lightbar_suspend(struct device *dev)
static SIMPLE_DEV_PM_OPS(cros_ec_lightbar_pm_ops,
cros_ec_lightbar_suspend, cros_ec_lightbar_resume);
+static const struct platform_device_id cros_ec_lightbar_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_lightbar_id);
+
static struct platform_driver cros_ec_lightbar_driver = {
.driver = {
.name = DRV_NAME,
@@ -602,10 +609,10 @@ static struct platform_driver cros_ec_lightbar_driver = {
},
.probe = cros_ec_lightbar_probe,
.remove_new = cros_ec_lightbar_remove,
+ .id_table = cros_ec_lightbar_id,
};
module_platform_driver(cros_ec_lightbar_driver);
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("Expose the Chromebook Pixel's lightbar to userspace");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 13/18] platform/chrome: cros_ec_vbc: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (11 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 12/18] platform/chrome: cros_ec_lightbar: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:55 ` Krzysztof Kozlowski
2024-03-29 20:19 ` Benson Leung
2024-03-29 7:56 ` [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: " Tzung-Bi Shih
` (8 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_ec_vbc.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/cros_ec_vbc.c b/drivers/platform/chrome/cros_ec_vbc.c
index 274ea0c64b33..787a19db4911 100644
--- a/drivers/platform/chrome/cros_ec_vbc.c
+++ b/drivers/platform/chrome/cros_ec_vbc.c
@@ -6,6 +6,7 @@
#include <linux/of.h>
#include <linux/platform_device.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/cros_ec_commands.h>
#include <linux/platform_data/cros_ec_proto.h>
@@ -133,16 +134,22 @@ static void cros_ec_vbc_remove(struct platform_device *pd)
&cros_ec_vbc_attr_group);
}
+static const struct platform_device_id cros_ec_vbc_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, cros_ec_vbc_id);
+
static struct platform_driver cros_ec_vbc_driver = {
.driver = {
.name = DRV_NAME,
},
.probe = cros_ec_vbc_probe,
.remove_new = cros_ec_vbc_remove,
+ .id_table = cros_ec_vbc_id,
};
module_platform_driver(cros_ec_vbc_driver);
MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("Expose the vboot context nvram to userspace");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (12 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 13/18] platform/chrome: cros_ec_vbc: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:53 ` Krzysztof Kozlowski
2024-03-29 20:20 ` Benson Leung
2024-03-29 7:56 ` [PATCH 15/18] platform/chrome: wilco_ec: telemetry: " Tzung-Bi Shih
` (7 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
Also shrink the name for fitting to [2].
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
[2]: https://elixir.bootlin.com/linux/v6.8/source/include/linux/mod_devicetable.h#L608
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/cros_kbd_led_backlight.c | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/drivers/platform/chrome/cros_kbd_led_backlight.c b/drivers/platform/chrome/cros_kbd_led_backlight.c
index 793fd3f1015d..b83e4f328620 100644
--- a/drivers/platform/chrome/cros_kbd_led_backlight.c
+++ b/drivers/platform/chrome/cros_kbd_led_backlight.c
@@ -9,6 +9,7 @@
#include <linux/init.h>
#include <linux/kernel.h>
#include <linux/leds.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/of.h>
#include <linux/platform_data/cros_ec_commands.h>
@@ -247,17 +248,23 @@ static const struct of_device_id keyboard_led_of_match[] = {
MODULE_DEVICE_TABLE(of, keyboard_led_of_match);
#endif
+static const struct platform_device_id keyboard_led_id[] = {
+ { "cros-keyboard-leds", 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, keyboard_led_id);
+
static struct platform_driver keyboard_led_driver = {
.driver = {
- .name = "chromeos-keyboard-leds",
+ .name = "cros-keyboard-leds",
.acpi_match_table = ACPI_PTR(keyboard_led_acpi_match),
.of_match_table = of_match_ptr(keyboard_led_of_match),
},
.probe = keyboard_led_probe,
+ .id_table = keyboard_led_id,
};
module_platform_driver(keyboard_led_driver);
MODULE_AUTHOR("Simon Que <sque@chromium.org>");
MODULE_DESCRIPTION("ChromeOS Keyboard backlight LED Driver");
MODULE_LICENSE("GPL");
-MODULE_ALIAS("platform:chromeos-keyboard-leds");
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 15/18] platform/chrome: wilco_ec: telemetry: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (13 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:52 ` Krzysztof Kozlowski
2024-03-29 20:21 ` Benson Leung
2024-03-29 7:56 ` [PATCH 16/18] platform/chrome: wilco_ec: debugfs: " Tzung-Bi Shih
` (6 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/wilco_ec/telemetry.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/wilco_ec/telemetry.c b/drivers/platform/chrome/wilco_ec/telemetry.c
index b7c616f3d179..21d4cbbb009a 100644
--- a/drivers/platform/chrome/wilco_ec/telemetry.c
+++ b/drivers/platform/chrome/wilco_ec/telemetry.c
@@ -30,6 +30,7 @@
#include <linux/cdev.h>
#include <linux/device.h>
#include <linux/fs.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/wilco-ec.h>
#include <linux/platform_device.h>
@@ -409,12 +410,19 @@ static void telem_device_remove(struct platform_device *pdev)
put_device(&dev_data->dev);
}
+static const struct platform_device_id telem_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, telem_id);
+
static struct platform_driver telem_driver = {
.probe = telem_device_probe,
.remove_new = telem_device_remove,
.driver = {
.name = DRV_NAME,
},
+ .id_table = telem_id,
};
static int __init telem_module_init(void)
@@ -466,4 +474,3 @@ module_exit(telem_module_exit);
MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
MODULE_DESCRIPTION("Wilco EC telemetry driver");
MODULE_LICENSE("GPL");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 16/18] platform/chrome: wilco_ec: debugfs: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (14 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 15/18] platform/chrome: wilco_ec: telemetry: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:52 ` Krzysztof Kozlowski
2024-03-29 20:21 ` Benson Leung
2024-03-29 7:56 ` [PATCH 17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS Tzung-Bi Shih
` (5 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/wilco_ec/debugfs.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/wilco_ec/debugfs.c b/drivers/platform/chrome/wilco_ec/debugfs.c
index 93c11f81ca45..983f2fa44ba5 100644
--- a/drivers/platform/chrome/wilco_ec/debugfs.c
+++ b/drivers/platform/chrome/wilco_ec/debugfs.c
@@ -10,6 +10,7 @@
#include <linux/ctype.h>
#include <linux/debugfs.h>
#include <linux/fs.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/wilco-ec.h>
#include <linux/platform_device.h>
@@ -265,17 +266,23 @@ static void wilco_ec_debugfs_remove(struct platform_device *pdev)
debugfs_remove_recursive(debug_info->dir);
}
+static const struct platform_device_id wilco_ec_debugfs_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, wilco_ec_debugfs_id);
+
static struct platform_driver wilco_ec_debugfs_driver = {
.driver = {
.name = DRV_NAME,
},
.probe = wilco_ec_debugfs_probe,
.remove_new = wilco_ec_debugfs_remove,
+ .id_table = wilco_ec_debugfs_id,
};
module_platform_driver(wilco_ec_debugfs_driver);
-MODULE_ALIAS("platform:" DRV_NAME);
MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("Wilco EC debugfs driver");
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (15 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 16/18] platform/chrome: wilco_ec: debugfs: " Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:55 ` Krzysztof Kozlowski
2024-03-29 20:22 ` Benson Leung
2024-03-29 7:56 ` [PATCH 18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match Tzung-Bi Shih
` (4 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
There is no platform driver in the file. Remove the redundant
MODULE_ALIAS().
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/wilco_ec/event.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/platform/chrome/wilco_ec/event.c b/drivers/platform/chrome/wilco_ec/event.c
index 13291fb4214e..49043c644572 100644
--- a/drivers/platform/chrome/wilco_ec/event.c
+++ b/drivers/platform/chrome/wilco_ec/event.c
@@ -575,4 +575,3 @@ module_exit(event_module_exit);
MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
MODULE_DESCRIPTION("Wilco EC ACPI event driver");
MODULE_LICENSE("GPL");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* [PATCH 18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (16 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS Tzung-Bi Shih
@ 2024-03-29 7:56 ` Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:23 ` Benson Leung
2024-04-01 2:30 ` [PATCH 00/18] platform/chrome: " patchwork-bot+chrome-platform
` (3 subsequent siblings)
21 siblings, 2 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-03-29 7:56 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni
Cc: tzungbi, chrome-platform, pmalani, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
Instead of using fallback driver name match, provide ID table[1] for the
primary match.
[1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
---
drivers/platform/chrome/wilco_ec/core.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/drivers/platform/chrome/wilco_ec/core.c b/drivers/platform/chrome/wilco_ec/core.c
index 9b59a1bed286..3e6b6cd81a9b 100644
--- a/drivers/platform/chrome/wilco_ec/core.c
+++ b/drivers/platform/chrome/wilco_ec/core.c
@@ -10,6 +10,7 @@
#include <linux/acpi.h>
#include <linux/device.h>
#include <linux/ioport.h>
+#include <linux/mod_devicetable.h>
#include <linux/module.h>
#include <linux/platform_data/wilco-ec.h>
#include <linux/platform_device.h>
@@ -150,6 +151,12 @@ static const struct acpi_device_id wilco_ec_acpi_device_ids[] = {
};
MODULE_DEVICE_TABLE(acpi, wilco_ec_acpi_device_ids);
+static const struct platform_device_id wilco_ec_id[] = {
+ { DRV_NAME, 0 },
+ {}
+};
+MODULE_DEVICE_TABLE(platform, wilco_ec_id);
+
static struct platform_driver wilco_ec_driver = {
.driver = {
.name = DRV_NAME,
@@ -157,6 +164,7 @@ static struct platform_driver wilco_ec_driver = {
},
.probe = wilco_ec_probe,
.remove_new = wilco_ec_remove,
+ .id_table = wilco_ec_id,
};
module_platform_driver(wilco_ec_driver);
@@ -165,4 +173,3 @@ MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
MODULE_AUTHOR("Duncan Laurie <dlaurie@chromium.org>");
MODULE_LICENSE("GPL v2");
MODULE_DESCRIPTION("ChromeOS Wilco Embedded Controller driver");
-MODULE_ALIAS("platform:" DRV_NAME);
--
2.44.0.478.gd926399ef9-goog
^ permalink raw reply related [flat|nested] 64+ messages in thread
* Re: [PATCH 01/18] media: platform: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 01/18] media: platform: cros-ec: " Tzung-Bi Shih
@ 2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 20:05 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 12:15 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 02/18] gpio: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 02/18] gpio: " Tzung-Bi Shih
@ 2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 20:06 ` Benson Leung
2024-04-04 11:32 ` Linus Walleij
2 siblings, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 12:15 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match. Also allow automatic module loading by adding
> MODULE_DEVICE_TABLE().
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 03/18] rtc: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 03/18] rtc: " Tzung-Bi Shih
@ 2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 15:00 ` (subset) " Alexandre Belloni
2024-03-29 20:07 ` Benson Leung
2 siblings, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 12:15 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 04/18] platform/chrome: cros_ec_sensorhub: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 04/18] platform/chrome: cros_ec_sensorhub: " Tzung-Bi Shih
@ 2024-03-29 12:16 ` Krzysztof Kozlowski
2024-03-29 20:08 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 12:16 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 05/18] power: supply: cros_usbpd: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 05/18] power: supply: cros_usbpd: " Tzung-Bi Shih
@ 2024-03-29 12:18 ` Krzysztof Kozlowski
2024-03-29 20:09 ` Benson Leung
2024-03-29 23:03 ` Prashant Malani
2 siblings, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 12:18 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
You could add everywhere:
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
but I also don't mind if that's skipped. Up to you folks.
Thanks for the changes.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 10/18] platform/chrome: cros_ec_sysfs: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 10/18] platform/chrome: cros_ec_sysfs: " Tzung-Bi Shih
@ 2024-03-29 12:37 ` Krzysztof Kozlowski
2024-03-29 20:15 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 12:37 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: (subset) [PATCH 03/18] rtc: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 03/18] rtc: " Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
@ 2024-03-29 15:00 ` Alexandre Belloni
2024-03-29 20:07 ` Benson Leung
2 siblings, 0 replies; 64+ messages in thread
From: Alexandre Belloni @ 2024-03-29 15:00 UTC (permalink / raw)
To: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
Tzung-Bi Shih
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc, krzk
On Fri, 29 Mar 2024 15:56:15 +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
>
Applied, thanks!
[03/18] rtc: cros-ec: provide ID table for avoiding fallback match
https://git.kernel.org/abelloni/c/61c86d14745d
Best regards,
--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 07/18] platform/chrome: cros_usbpd_notify: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 07/18] platform/chrome: cros_usbpd_notify: " Tzung-Bi Shih
@ 2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:13 ` Benson Leung
2024-03-29 22:06 ` Prashant Malani
2 siblings, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:50 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 08/18] platform/chrome: cros_ec_chardev: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 08/18] platform/chrome: cros_ec_chardev: " Tzung-Bi Shih
@ 2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:13 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:50 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
> drivers/platform/chrome/cros_ec_chardev.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 11/18] power: supply: cros_pchg: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 11/18] power: supply: cros_pchg: " Tzung-Bi Shih
@ 2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:16 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:50 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 06/18] platform/chrome: cros_usbpd_logger: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 06/18] platform/chrome: cros_usbpd_logger: " Tzung-Bi Shih
@ 2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:11 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:50 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 09/18] platform/chrome: cros_ec_debugfs: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 09/18] platform/chrome: cros_ec_debugfs: " Tzung-Bi Shih
@ 2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:14 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:51 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 12/18] platform/chrome: cros_ec_lightbar: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 12/18] platform/chrome: cros_ec_lightbar: " Tzung-Bi Shih
@ 2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:18 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:51 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match Tzung-Bi Shih
@ 2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:23 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:51 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 16/18] platform/chrome: wilco_ec: debugfs: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 16/18] platform/chrome: wilco_ec: debugfs: " Tzung-Bi Shih
@ 2024-03-29 15:52 ` Krzysztof Kozlowski
2024-03-29 20:21 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:52 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 15/18] platform/chrome: wilco_ec: telemetry: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 15/18] platform/chrome: wilco_ec: telemetry: " Tzung-Bi Shih
@ 2024-03-29 15:52 ` Krzysztof Kozlowski
2024-03-29 20:21 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:52 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: " Tzung-Bi Shih
@ 2024-03-29 15:53 ` Krzysztof Kozlowski
2024-04-01 3:11 ` Tzung-Bi Shih
2024-03-29 20:20 ` Benson Leung
1 sibling, 1 reply; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:53 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> Also shrink the name for fitting to [2].
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
> [2]: https://elixir.bootlin.com/linux/v6.8/source/include/linux/mod_devicetable.h#L608
>
I think this should be two commits, where the first one is a fix (Fixes
tag) for the size of name.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS
2024-03-29 7:56 ` [PATCH 17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS Tzung-Bi Shih
@ 2024-03-29 15:55 ` Krzysztof Kozlowski
2024-03-29 20:22 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:55 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> There is no platform driver in the file. Remove the redundant
> MODULE_ALIAS().
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
> drivers/platform/chrome/wilco_ec/event.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/wilco_ec/event.c b/drivers/platform/chrome/wilco_ec/event.c
> index 13291fb4214e..49043c644572 100644
> --- a/drivers/platform/chrome/wilco_ec/event.c
> +++ b/drivers/platform/chrome/wilco_ec/event.c
> @@ -575,4 +575,3 @@ module_exit(event_module_exit);
> MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
> MODULE_DESCRIPTION("Wilco EC ACPI event driver");
> MODULE_LICENSE("GPL");
> -MODULE_ALIAS("platform:" DRV_NAME);
In this case some sort of alias, maybe not necessarily platform, could
have sense. I see there ACPI ID table, so it is fine for me.
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 13/18] platform/chrome: cros_ec_vbc: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 13/18] platform/chrome: cros_ec_vbc: " Tzung-Bi Shih
@ 2024-03-29 15:55 ` Krzysztof Kozlowski
2024-03-29 20:19 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Krzysztof Kozlowski @ 2024-03-29 15:55 UTC (permalink / raw)
To: Tzung-Bi Shih, bleung, groeck, linus.walleij, brgl,
hverkuil-cisco, mchehab, sre, alexandre.belloni
Cc: chrome-platform, pmalani, linux-gpio, linux-media, linux-pm,
linux-rtc
On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
> ---
> drivers/platform/chrome/cros_ec_vbc.c | 9 ++++++++-
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 01/18] media: platform: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 01/18] media: platform: cros-ec: " Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
@ 2024-03-29 20:05 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:05 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1898 bytes --]
On Fri, Mar 29, 2024 at 03:56:13PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/media/cec/platform/cros-ec/cros-ec-cec.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> index 48ed2993d2f0..8fbbb4091455 100644
> --- a/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> +++ b/drivers/media/cec/platform/cros-ec/cros-ec-cec.c
> @@ -8,6 +8,7 @@
>
> #include <linux/kernel.h>
> #include <linux/module.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/platform_device.h>
> #include <linux/dmi.h>
> #include <linux/pci.h>
> @@ -573,6 +574,12 @@ static void cros_ec_cec_remove(struct platform_device *pdev)
> }
> }
>
> +static const struct platform_device_id cros_ec_cec_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_cec_id);
> +
> static struct platform_driver cros_ec_cec_driver = {
> .probe = cros_ec_cec_probe,
> .remove_new = cros_ec_cec_remove,
> @@ -580,6 +587,7 @@ static struct platform_driver cros_ec_cec_driver = {
> .name = DRV_NAME,
> .pm = &cros_ec_cec_pm_ops,
> },
> + .id_table = cros_ec_cec_id,
> };
>
> module_platform_driver(cros_ec_cec_driver);
> @@ -587,4 +595,3 @@ module_platform_driver(cros_ec_cec_driver);
> MODULE_DESCRIPTION("CEC driver for ChromeOS ECs");
> MODULE_AUTHOR("Neil Armstrong <narmstrong@baylibre.com>");
> MODULE_LICENSE("GPL");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 02/18] gpio: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 02/18] gpio: " Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
@ 2024-03-29 20:06 ` Benson Leung
2024-04-04 11:32 ` Linus Walleij
2 siblings, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:06 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1599 bytes --]
On Fri, Mar 29, 2024 at 03:56:14PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match. Also allow automatic module loading by adding
> MODULE_DEVICE_TABLE().
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/gpio/gpio-cros-ec.c | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/drivers/gpio/gpio-cros-ec.c b/drivers/gpio/gpio-cros-ec.c
> index 842e1c060414..0c09bb54dc0c 100644
> --- a/drivers/gpio/gpio-cros-ec.c
> +++ b/drivers/gpio/gpio-cros-ec.c
> @@ -12,6 +12,7 @@
> #include <linux/errno.h>
> #include <linux/gpio/driver.h>
> #include <linux/kernel.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_commands.h>
> #include <linux/platform_data/cros_ec_proto.h>
> @@ -197,11 +198,18 @@ static int cros_ec_gpio_probe(struct platform_device *pdev)
> return devm_gpiochip_add_data(dev, gc, cros_ec);
> }
>
> +static const struct platform_device_id cros_ec_gpio_id[] = {
> + { "cros-ec-gpio", 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_gpio_id);
> +
> static struct platform_driver cros_ec_gpio_driver = {
> .probe = cros_ec_gpio_probe,
> .driver = {
> .name = "cros-ec-gpio",
> },
> + .id_table = cros_ec_gpio_id,
> };
> module_platform_driver(cros_ec_gpio_driver);
>
> --
> 2.44.0.478.gd926399ef9-goog
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 03/18] rtc: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 03/18] rtc: " Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 15:00 ` (subset) " Alexandre Belloni
@ 2024-03-29 20:07 ` Benson Leung
2 siblings, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:07 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1899 bytes --]
On Fri, Mar 29, 2024 at 03:56:15PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/rtc/rtc-cros-ec.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/rtc/rtc-cros-ec.c b/drivers/rtc/rtc-cros-ec.c
> index 0cd397c04ff0..f57462c7b2c6 100644
> --- a/drivers/rtc/rtc-cros-ec.c
> +++ b/drivers/rtc/rtc-cros-ec.c
> @@ -5,6 +5,7 @@
> // Author: Stephen Barber <smbarber@chromium.org>
>
> #include <linux/kernel.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_commands.h>
> #include <linux/platform_data/cros_ec_proto.h>
> @@ -392,6 +393,12 @@ static void cros_ec_rtc_remove(struct platform_device *pdev)
> dev_err(dev, "failed to unregister notifier\n");
> }
>
> +static const struct platform_device_id cros_ec_rtc_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_rtc_id);
> +
> static struct platform_driver cros_ec_rtc_driver = {
> .probe = cros_ec_rtc_probe,
> .remove_new = cros_ec_rtc_remove,
> @@ -399,6 +406,7 @@ static struct platform_driver cros_ec_rtc_driver = {
> .name = DRV_NAME,
> .pm = &cros_ec_rtc_pm_ops,
> },
> + .id_table = cros_ec_rtc_id,
> };
>
> module_platform_driver(cros_ec_rtc_driver);
> @@ -406,4 +414,3 @@ module_platform_driver(cros_ec_rtc_driver);
> MODULE_DESCRIPTION("RTC driver for Chrome OS ECs");
> MODULE_AUTHOR("Stephen Barber <smbarber@chromium.org>");
> MODULE_LICENSE("GPL v2");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 04/18] platform/chrome: cros_ec_sensorhub: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 04/18] platform/chrome: cros_ec_sensorhub: " Tzung-Bi Shih
2024-03-29 12:16 ` Krzysztof Kozlowski
@ 2024-03-29 20:08 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:08 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1842 bytes --]
On Fri, Mar 29, 2024 at 03:56:16PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_ec_sensorhub.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_sensorhub.c b/drivers/platform/chrome/cros_ec_sensorhub.c
> index 31fb8bdaad5a..50cdae67fa32 100644
> --- a/drivers/platform/chrome/cros_ec_sensorhub.c
> +++ b/drivers/platform/chrome/cros_ec_sensorhub.c
> @@ -8,6 +8,7 @@
>
> #include <linux/init.h>
> #include <linux/device.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_commands.h>
> #include <linux/platform_data/cros_ec_proto.h>
> @@ -247,17 +248,23 @@ static SIMPLE_DEV_PM_OPS(cros_ec_sensorhub_pm_ops,
> cros_ec_sensorhub_suspend,
> cros_ec_sensorhub_resume);
>
> +static const struct platform_device_id cros_ec_sensorhub_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_sensorhub_id);
> +
> static struct platform_driver cros_ec_sensorhub_driver = {
> .driver = {
> .name = DRV_NAME,
> .pm = &cros_ec_sensorhub_pm_ops,
> },
> .probe = cros_ec_sensorhub_probe,
> + .id_table = cros_ec_sensorhub_id,
> };
>
> module_platform_driver(cros_ec_sensorhub_driver);
>
> -MODULE_ALIAS("platform:" DRV_NAME);
> MODULE_AUTHOR("Gwendal Grignou <gwendal@chromium.org>");
> MODULE_DESCRIPTION("ChromeOS EC MEMS Sensor Hub Driver");
> MODULE_LICENSE("GPL");
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 05/18] power: supply: cros_usbpd: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 05/18] power: supply: cros_usbpd: " Tzung-Bi Shih
2024-03-29 12:18 ` Krzysztof Kozlowski
@ 2024-03-29 20:09 ` Benson Leung
2024-03-29 23:03 ` Prashant Malani
2 siblings, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:09 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1843 bytes --]
On Fri, Mar 29, 2024 at 03:56:17PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/power/supply/cros_usbpd-charger.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/power/supply/cros_usbpd-charger.c b/drivers/power/supply/cros_usbpd-charger.c
> index b6c96376776a..8008e31c0c09 100644
> --- a/drivers/power/supply/cros_usbpd-charger.c
> +++ b/drivers/power/supply/cros_usbpd-charger.c
> @@ -5,6 +5,7 @@
> * Copyright (c) 2014 - 2018 Google, Inc
> */
>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_commands.h>
> #include <linux/platform_data/cros_ec_proto.h>
> @@ -711,16 +712,22 @@ static int cros_usbpd_charger_resume(struct device *dev)
> static SIMPLE_DEV_PM_OPS(cros_usbpd_charger_pm_ops, NULL,
> cros_usbpd_charger_resume);
>
> +static const struct platform_device_id cros_usbpd_charger_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_usbpd_charger_id);
> +
> static struct platform_driver cros_usbpd_charger_driver = {
> .driver = {
> .name = DRV_NAME,
> .pm = &cros_usbpd_charger_pm_ops,
> },
> - .probe = cros_usbpd_charger_probe
> + .probe = cros_usbpd_charger_probe,
> + .id_table = cros_usbpd_charger_id,
> };
>
> module_platform_driver(cros_usbpd_charger_driver);
>
> MODULE_LICENSE("GPL");
> MODULE_DESCRIPTION("ChromeOS EC USBPD charger");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 06/18] platform/chrome: cros_usbpd_logger: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 06/18] platform/chrome: cros_usbpd_logger: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
@ 2024-03-29 20:11 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:11 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1957 bytes --]
On Fri, Mar 29, 2024 at 03:56:18PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_usbpd_logger.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/cros_usbpd_logger.c b/drivers/platform/chrome/cros_usbpd_logger.c
> index f618757f8b32..930c2f47269f 100644
> --- a/drivers/platform/chrome/cros_usbpd_logger.c
> +++ b/drivers/platform/chrome/cros_usbpd_logger.c
> @@ -7,6 +7,7 @@
>
> #include <linux/ktime.h>
> #include <linux/math64.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_commands.h>
> #include <linux/platform_data/cros_ec_proto.h>
> @@ -249,6 +250,12 @@ static int __maybe_unused cros_usbpd_logger_suspend(struct device *dev)
> static SIMPLE_DEV_PM_OPS(cros_usbpd_logger_pm_ops, cros_usbpd_logger_suspend,
> cros_usbpd_logger_resume);
>
> +static const struct platform_device_id cros_usbpd_logger_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_usbpd_logger_id);
> +
> static struct platform_driver cros_usbpd_logger_driver = {
> .driver = {
> .name = DRV_NAME,
> @@ -256,10 +263,10 @@ static struct platform_driver cros_usbpd_logger_driver = {
> },
> .probe = cros_usbpd_logger_probe,
> .remove_new = cros_usbpd_logger_remove,
> + .id_table = cros_usbpd_logger_id,
> };
>
> module_platform_driver(cros_usbpd_logger_driver);
>
> MODULE_LICENSE("GPL v2");
> MODULE_DESCRIPTION("Logging driver for ChromeOS EC USBPD Charger.");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 07/18] platform/chrome: cros_usbpd_notify: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 07/18] platform/chrome: cros_usbpd_notify: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
@ 2024-03-29 20:13 ` Benson Leung
2024-03-29 22:06 ` Prashant Malani
2 siblings, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:13 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1884 bytes --]
On Fri, Mar 29, 2024 at 03:56:19PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_usbpd_notify.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/cros_usbpd_notify.c b/drivers/platform/chrome/cros_usbpd_notify.c
> index aacad022f21d..c83f81d86483 100644
> --- a/drivers/platform/chrome/cros_usbpd_notify.c
> +++ b/drivers/platform/chrome/cros_usbpd_notify.c
> @@ -6,6 +6,7 @@
> */
>
> #include <linux/acpi.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_proto.h>
> #include <linux/platform_data/cros_usbpd_notify.h>
> @@ -218,12 +219,19 @@ static void cros_usbpd_notify_remove_plat(struct platform_device *pdev)
> &pdnotify->nb);
> }
>
> +static const struct platform_device_id cros_usbpd_notify_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_usbpd_notify_id);
> +
> static struct platform_driver cros_usbpd_notify_plat_driver = {
> .driver = {
> .name = DRV_NAME,
> },
> .probe = cros_usbpd_notify_probe_plat,
> .remove_new = cros_usbpd_notify_remove_plat,
> + .id_table = cros_usbpd_notify_id,
> };
>
> static int __init cros_usbpd_notify_init(void)
> @@ -258,4 +266,3 @@ module_exit(cros_usbpd_notify_exit);
> MODULE_LICENSE("GPL");
> MODULE_DESCRIPTION("ChromeOS power delivery notifier device");
> MODULE_AUTHOR("Jon Flatley <jflat@chromium.org>");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 08/18] platform/chrome: cros_ec_chardev: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 08/18] platform/chrome: cros_ec_chardev: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
@ 2024-03-29 20:13 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:13 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1845 bytes --]
On Fri, Mar 29, 2024 at 03:56:20PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_ec_chardev.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_chardev.c b/drivers/platform/chrome/cros_ec_chardev.c
> index 81950bb2c6da..7f034ead7ae4 100644
> --- a/drivers/platform/chrome/cros_ec_chardev.c
> +++ b/drivers/platform/chrome/cros_ec_chardev.c
> @@ -14,6 +14,7 @@
> #include <linux/device.h>
> #include <linux/fs.h>
> #include <linux/miscdevice.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/notifier.h>
> #include <linux/platform_data/cros_ec_chardev.h>
> @@ -403,17 +404,23 @@ static void cros_ec_chardev_remove(struct platform_device *pdev)
> misc_deregister(&data->misc);
> }
>
> +static const struct platform_device_id cros_ec_chardev_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_chardev_id);
> +
> static struct platform_driver cros_ec_chardev_driver = {
> .driver = {
> .name = DRV_NAME,
> },
> .probe = cros_ec_chardev_probe,
> .remove_new = cros_ec_chardev_remove,
> + .id_table = cros_ec_chardev_id,
> };
>
> module_platform_driver(cros_ec_chardev_driver);
>
> -MODULE_ALIAS("platform:" DRV_NAME);
> MODULE_AUTHOR("Enric Balletbo i Serra <enric.balletbo@collabora.com>");
> MODULE_DESCRIPTION("ChromeOS EC Miscellaneous Character Driver");
> MODULE_LICENSE("GPL");
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 09/18] platform/chrome: cros_ec_debugfs: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 09/18] platform/chrome: cros_ec_debugfs: " Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
@ 2024-03-29 20:14 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:14 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1900 bytes --]
On Fri, Mar 29, 2024 at 03:56:21PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_ec_debugfs.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_debugfs.c b/drivers/platform/chrome/cros_ec_debugfs.c
> index 6bf6f0e7b597..e1d313246beb 100644
> --- a/drivers/platform/chrome/cros_ec_debugfs.c
> +++ b/drivers/platform/chrome/cros_ec_debugfs.c
> @@ -7,6 +7,7 @@
> #include <linux/debugfs.h>
> #include <linux/delay.h>
> #include <linux/fs.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/mutex.h>
> #include <linux/platform_data/cros_ec_commands.h>
> @@ -564,6 +565,12 @@ static int __maybe_unused cros_ec_debugfs_resume(struct device *dev)
> static SIMPLE_DEV_PM_OPS(cros_ec_debugfs_pm_ops,
> cros_ec_debugfs_suspend, cros_ec_debugfs_resume);
>
> +static const struct platform_device_id cros_ec_debugfs_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_debugfs_id);
> +
> static struct platform_driver cros_ec_debugfs_driver = {
> .driver = {
> .name = DRV_NAME,
> @@ -572,10 +579,10 @@ static struct platform_driver cros_ec_debugfs_driver = {
> },
> .probe = cros_ec_debugfs_probe,
> .remove_new = cros_ec_debugfs_remove,
> + .id_table = cros_ec_debugfs_id,
> };
>
> module_platform_driver(cros_ec_debugfs_driver);
>
> MODULE_LICENSE("GPL");
> MODULE_DESCRIPTION("Debug logs for ChromeOS EC");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 10/18] platform/chrome: cros_ec_sysfs: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 10/18] platform/chrome: cros_ec_sysfs: " Tzung-Bi Shih
2024-03-29 12:37 ` Krzysztof Kozlowski
@ 2024-03-29 20:15 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:15 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1786 bytes --]
On Fri, Mar 29, 2024 at 03:56:22PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_ec_sysfs.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_sysfs.c b/drivers/platform/chrome/cros_ec_sysfs.c
> index 93e67ab4af06..9c944146ee50 100644
> --- a/drivers/platform/chrome/cros_ec_sysfs.c
> +++ b/drivers/platform/chrome/cros_ec_sysfs.c
> @@ -8,6 +8,7 @@
> #include <linux/device.h>
> #include <linux/fs.h>
> #include <linux/kobject.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_commands.h>
> #include <linux/platform_data/cros_ec_proto.h>
> @@ -347,16 +348,22 @@ static void cros_ec_sysfs_remove(struct platform_device *pd)
> sysfs_remove_group(&ec_dev->class_dev.kobj, &cros_ec_attr_group);
> }
>
> +static const struct platform_device_id cros_ec_sysfs_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_sysfs_id);
> +
> static struct platform_driver cros_ec_sysfs_driver = {
> .driver = {
> .name = DRV_NAME,
> },
> .probe = cros_ec_sysfs_probe,
> .remove_new = cros_ec_sysfs_remove,
> + .id_table = cros_ec_sysfs_id,
> };
>
> module_platform_driver(cros_ec_sysfs_driver);
>
> MODULE_LICENSE("GPL");
> MODULE_DESCRIPTION("Expose the ChromeOS EC through sysfs");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 11/18] power: supply: cros_pchg: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 11/18] power: supply: cros_pchg: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
@ 2024-03-29 20:16 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:16 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1761 bytes --]
On Fri, Mar 29, 2024 at 03:56:23PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/power/supply/cros_peripheral_charger.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/power/supply/cros_peripheral_charger.c b/drivers/power/supply/cros_peripheral_charger.c
> index a204f2355be4..d406f2a78449 100644
> --- a/drivers/power/supply/cros_peripheral_charger.c
> +++ b/drivers/power/supply/cros_peripheral_charger.c
> @@ -5,6 +5,7 @@
> * Copyright 2020 Google LLC.
> */
>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/notifier.h>
> #include <linux/platform_data/cros_ec_commands.h>
> @@ -367,16 +368,22 @@ static int __maybe_unused cros_pchg_resume(struct device *dev)
>
> static SIMPLE_DEV_PM_OPS(cros_pchg_pm_ops, NULL, cros_pchg_resume);
>
> +static const struct platform_device_id cros_pchg_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_pchg_id);
> +
> static struct platform_driver cros_pchg_driver = {
> .driver = {
> .name = DRV_NAME,
> .pm = &cros_pchg_pm_ops,
> },
> - .probe = cros_pchg_probe
> + .probe = cros_pchg_probe,
> + .id_table = cros_pchg_id,
> };
>
> module_platform_driver(cros_pchg_driver);
>
> MODULE_LICENSE("GPL");
> MODULE_DESCRIPTION("ChromeOS EC peripheral device charger");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 12/18] platform/chrome: cros_ec_lightbar: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 12/18] platform/chrome: cros_ec_lightbar: " Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
@ 2024-03-29 20:18 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:18 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1967 bytes --]
On Fri, Mar 29, 2024 at 03:56:24PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_ec_lightbar.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_lightbar.c b/drivers/platform/chrome/cros_ec_lightbar.c
> index 6677cc6c4984..1e69f61115a4 100644
> --- a/drivers/platform/chrome/cros_ec_lightbar.c
> +++ b/drivers/platform/chrome/cros_ec_lightbar.c
> @@ -9,6 +9,7 @@
> #include <linux/fs.h>
> #include <linux/kobject.h>
> #include <linux/kstrtox.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_commands.h>
> #include <linux/platform_data/cros_ec_proto.h>
> @@ -594,6 +595,12 @@ static int __maybe_unused cros_ec_lightbar_suspend(struct device *dev)
> static SIMPLE_DEV_PM_OPS(cros_ec_lightbar_pm_ops,
> cros_ec_lightbar_suspend, cros_ec_lightbar_resume);
>
> +static const struct platform_device_id cros_ec_lightbar_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_lightbar_id);
> +
> static struct platform_driver cros_ec_lightbar_driver = {
> .driver = {
> .name = DRV_NAME,
> @@ -602,10 +609,10 @@ static struct platform_driver cros_ec_lightbar_driver = {
> },
> .probe = cros_ec_lightbar_probe,
> .remove_new = cros_ec_lightbar_remove,
> + .id_table = cros_ec_lightbar_id,
> };
>
> module_platform_driver(cros_ec_lightbar_driver);
>
> MODULE_LICENSE("GPL");
> MODULE_DESCRIPTION("Expose the Chromebook Pixel's lightbar to userspace");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 13/18] platform/chrome: cros_ec_vbc: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 13/18] platform/chrome: cros_ec_vbc: " Tzung-Bi Shih
2024-03-29 15:55 ` Krzysztof Kozlowski
@ 2024-03-29 20:19 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:19 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1715 bytes --]
On Fri, Mar 29, 2024 at 03:56:25PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_ec_vbc.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/cros_ec_vbc.c b/drivers/platform/chrome/cros_ec_vbc.c
> index 274ea0c64b33..787a19db4911 100644
> --- a/drivers/platform/chrome/cros_ec_vbc.c
> +++ b/drivers/platform/chrome/cros_ec_vbc.c
> @@ -6,6 +6,7 @@
>
> #include <linux/of.h>
> #include <linux/platform_device.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/cros_ec_commands.h>
> #include <linux/platform_data/cros_ec_proto.h>
> @@ -133,16 +134,22 @@ static void cros_ec_vbc_remove(struct platform_device *pd)
> &cros_ec_vbc_attr_group);
> }
>
> +static const struct platform_device_id cros_ec_vbc_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, cros_ec_vbc_id);
> +
> static struct platform_driver cros_ec_vbc_driver = {
> .driver = {
> .name = DRV_NAME,
> },
> .probe = cros_ec_vbc_probe,
> .remove_new = cros_ec_vbc_remove,
> + .id_table = cros_ec_vbc_id,
> };
>
> module_platform_driver(cros_ec_vbc_driver);
>
> MODULE_LICENSE("GPL");
> MODULE_DESCRIPTION("Expose the vboot context nvram to userspace");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: " Tzung-Bi Shih
2024-03-29 15:53 ` Krzysztof Kozlowski
@ 2024-03-29 20:20 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:20 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 2136 bytes --]
On Fri, Mar 29, 2024 at 03:56:26PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> Also shrink the name for fitting to [2].
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
> [2]: https://elixir.bootlin.com/linux/v6.8/source/include/linux/mod_devicetable.h#L608
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/cros_kbd_led_backlight.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/platform/chrome/cros_kbd_led_backlight.c b/drivers/platform/chrome/cros_kbd_led_backlight.c
> index 793fd3f1015d..b83e4f328620 100644
> --- a/drivers/platform/chrome/cros_kbd_led_backlight.c
> +++ b/drivers/platform/chrome/cros_kbd_led_backlight.c
> @@ -9,6 +9,7 @@
> #include <linux/init.h>
> #include <linux/kernel.h>
> #include <linux/leds.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/of.h>
> #include <linux/platform_data/cros_ec_commands.h>
> @@ -247,17 +248,23 @@ static const struct of_device_id keyboard_led_of_match[] = {
> MODULE_DEVICE_TABLE(of, keyboard_led_of_match);
> #endif
>
> +static const struct platform_device_id keyboard_led_id[] = {
> + { "cros-keyboard-leds", 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, keyboard_led_id);
> +
> static struct platform_driver keyboard_led_driver = {
> .driver = {
> - .name = "chromeos-keyboard-leds",
> + .name = "cros-keyboard-leds",
> .acpi_match_table = ACPI_PTR(keyboard_led_acpi_match),
> .of_match_table = of_match_ptr(keyboard_led_of_match),
> },
> .probe = keyboard_led_probe,
> + .id_table = keyboard_led_id,
> };
> module_platform_driver(keyboard_led_driver);
>
> MODULE_AUTHOR("Simon Que <sque@chromium.org>");
> MODULE_DESCRIPTION("ChromeOS Keyboard backlight LED Driver");
> MODULE_LICENSE("GPL");
> -MODULE_ALIAS("platform:chromeos-keyboard-leds");
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 15/18] platform/chrome: wilco_ec: telemetry: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 15/18] platform/chrome: wilco_ec: telemetry: " Tzung-Bi Shih
2024-03-29 15:52 ` Krzysztof Kozlowski
@ 2024-03-29 20:21 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:21 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1811 bytes --]
On Fri, Mar 29, 2024 at 03:56:27PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/wilco_ec/telemetry.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/wilco_ec/telemetry.c b/drivers/platform/chrome/wilco_ec/telemetry.c
> index b7c616f3d179..21d4cbbb009a 100644
> --- a/drivers/platform/chrome/wilco_ec/telemetry.c
> +++ b/drivers/platform/chrome/wilco_ec/telemetry.c
> @@ -30,6 +30,7 @@
> #include <linux/cdev.h>
> #include <linux/device.h>
> #include <linux/fs.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/wilco-ec.h>
> #include <linux/platform_device.h>
> @@ -409,12 +410,19 @@ static void telem_device_remove(struct platform_device *pdev)
> put_device(&dev_data->dev);
> }
>
> +static const struct platform_device_id telem_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, telem_id);
> +
> static struct platform_driver telem_driver = {
> .probe = telem_device_probe,
> .remove_new = telem_device_remove,
> .driver = {
> .name = DRV_NAME,
> },
> + .id_table = telem_id,
> };
>
> static int __init telem_module_init(void)
> @@ -466,4 +474,3 @@ module_exit(telem_module_exit);
> MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
> MODULE_DESCRIPTION("Wilco EC telemetry driver");
> MODULE_LICENSE("GPL");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 16/18] platform/chrome: wilco_ec: debugfs: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 16/18] platform/chrome: wilco_ec: debugfs: " Tzung-Bi Shih
2024-03-29 15:52 ` Krzysztof Kozlowski
@ 2024-03-29 20:21 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:21 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1826 bytes --]
On Fri, Mar 29, 2024 at 03:56:28PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/wilco_ec/debugfs.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/wilco_ec/debugfs.c b/drivers/platform/chrome/wilco_ec/debugfs.c
> index 93c11f81ca45..983f2fa44ba5 100644
> --- a/drivers/platform/chrome/wilco_ec/debugfs.c
> +++ b/drivers/platform/chrome/wilco_ec/debugfs.c
> @@ -10,6 +10,7 @@
> #include <linux/ctype.h>
> #include <linux/debugfs.h>
> #include <linux/fs.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/wilco-ec.h>
> #include <linux/platform_device.h>
> @@ -265,17 +266,23 @@ static void wilco_ec_debugfs_remove(struct platform_device *pdev)
> debugfs_remove_recursive(debug_info->dir);
> }
>
> +static const struct platform_device_id wilco_ec_debugfs_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, wilco_ec_debugfs_id);
> +
> static struct platform_driver wilco_ec_debugfs_driver = {
> .driver = {
> .name = DRV_NAME,
> },
> .probe = wilco_ec_debugfs_probe,
> .remove_new = wilco_ec_debugfs_remove,
> + .id_table = wilco_ec_debugfs_id,
> };
>
> module_platform_driver(wilco_ec_debugfs_driver);
>
> -MODULE_ALIAS("platform:" DRV_NAME);
> MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
> MODULE_LICENSE("GPL v2");
> MODULE_DESCRIPTION("Wilco EC debugfs driver");
> --
> 2.44.0.478.gd926399ef9-goog
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS
2024-03-29 7:56 ` [PATCH 17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS Tzung-Bi Shih
2024-03-29 15:55 ` Krzysztof Kozlowski
@ 2024-03-29 20:22 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:22 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 875 bytes --]
On Fri, Mar 29, 2024 at 03:56:29PM +0800, Tzung-Bi Shih wrote:
> There is no platform driver in the file. Remove the redundant
> MODULE_ALIAS().
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/wilco_ec/event.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/wilco_ec/event.c b/drivers/platform/chrome/wilco_ec/event.c
> index 13291fb4214e..49043c644572 100644
> --- a/drivers/platform/chrome/wilco_ec/event.c
> +++ b/drivers/platform/chrome/wilco_ec/event.c
> @@ -575,4 +575,3 @@ module_exit(event_module_exit);
> MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
> MODULE_DESCRIPTION("Wilco EC ACPI event driver");
> MODULE_LICENSE("GPL");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
@ 2024-03-29 20:23 ` Benson Leung
1 sibling, 0 replies; 64+ messages in thread
From: Benson Leung @ 2024-03-29 20:23 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
[-- Attachment #1: Type: text/plain, Size: 1934 bytes --]
On Fri, Mar 29, 2024 at 03:56:30PM +0800, Tzung-Bi Shih wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Benson Leung <bleung@chromium.org>
> ---
> drivers/platform/chrome/wilco_ec/core.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/chrome/wilco_ec/core.c b/drivers/platform/chrome/wilco_ec/core.c
> index 9b59a1bed286..3e6b6cd81a9b 100644
> --- a/drivers/platform/chrome/wilco_ec/core.c
> +++ b/drivers/platform/chrome/wilco_ec/core.c
> @@ -10,6 +10,7 @@
> #include <linux/acpi.h>
> #include <linux/device.h>
> #include <linux/ioport.h>
> +#include <linux/mod_devicetable.h>
> #include <linux/module.h>
> #include <linux/platform_data/wilco-ec.h>
> #include <linux/platform_device.h>
> @@ -150,6 +151,12 @@ static const struct acpi_device_id wilco_ec_acpi_device_ids[] = {
> };
> MODULE_DEVICE_TABLE(acpi, wilco_ec_acpi_device_ids);
>
> +static const struct platform_device_id wilco_ec_id[] = {
> + { DRV_NAME, 0 },
> + {}
> +};
> +MODULE_DEVICE_TABLE(platform, wilco_ec_id);
> +
> static struct platform_driver wilco_ec_driver = {
> .driver = {
> .name = DRV_NAME,
> @@ -157,6 +164,7 @@ static struct platform_driver wilco_ec_driver = {
> },
> .probe = wilco_ec_probe,
> .remove_new = wilco_ec_remove,
> + .id_table = wilco_ec_id,
> };
>
> module_platform_driver(wilco_ec_driver);
> @@ -165,4 +173,3 @@ MODULE_AUTHOR("Nick Crews <ncrews@chromium.org>");
> MODULE_AUTHOR("Duncan Laurie <dlaurie@chromium.org>");
> MODULE_LICENSE("GPL v2");
> MODULE_DESCRIPTION("ChromeOS Wilco Embedded Controller driver");
> -MODULE_ALIAS("platform:" DRV_NAME);
> --
> 2.44.0.478.gd926399ef9-goog
>
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 07/18] platform/chrome: cros_usbpd_notify: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 07/18] platform/chrome: cros_usbpd_notify: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:13 ` Benson Leung
@ 2024-03-29 22:06 ` Prashant Malani
2 siblings, 0 replies; 64+ messages in thread
From: Prashant Malani @ 2024-03-29 22:06 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
On Fri, Mar 29, 2024 at 12:57 AM Tzung-Bi Shih <tzungbi@kernel.org> wrote:
>
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Acked-by: Prashant Malani <pmalani@chromium.org>
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 05/18] power: supply: cros_usbpd: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 05/18] power: supply: cros_usbpd: " Tzung-Bi Shih
2024-03-29 12:18 ` Krzysztof Kozlowski
2024-03-29 20:09 ` Benson Leung
@ 2024-03-29 23:03 ` Prashant Malani
2 siblings, 0 replies; 64+ messages in thread
From: Prashant Malani @ 2024-03-29 23:03 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, linux-gpio, linux-media,
linux-pm, linux-rtc, krzk
On Fri, Mar 29, 2024 at 12:57 AM Tzung-Bi Shih <tzungbi@kernel.org> wrote:
>
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match.
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Reviewed-by: Prashant Malani <pmalani@chromium.org>
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (17 preceding siblings ...)
2024-03-29 7:56 ` [PATCH 18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match Tzung-Bi Shih
@ 2024-04-01 2:30 ` patchwork-bot+chrome-platform
2024-04-01 2:30 ` patchwork-bot+chrome-platform
` (2 subsequent siblings)
21 siblings, 0 replies; 64+ messages in thread
From: patchwork-bot+chrome-platform @ 2024-04-01 2:30 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
Hello:
This series was applied to chrome-platform/linux.git (for-kernelci)
by Tzung-Bi Shih <tzungbi@kernel.org>:
On Fri, 29 Mar 2024 15:56:12 +0800 you wrote:
> Inspired by [1]. Turn all MODULE_ALIAS() in ChromeOS EC platform drivers into
> proper platform_device_id table and MODULE_DEVICE_TABLE().
>
> The series is basically looking for drivers from:
> - `struct mfd_cell` in drivers/mfd/cros_ec_dev.c.
> - grep -R MODULE_ALIAS drivers/platform/chrome/.
>
> [...]
Here is the summary with links:
- [01/18] media: platform: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [02/18] gpio: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [03/18] rtc: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [04/18] platform/chrome: cros_ec_sensorhub: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/12e981d4d643
- [05/18] power: supply: cros_usbpd: provide ID table for avoiding fallback match
(no matching commit)
- [06/18] platform/chrome: cros_usbpd_logger: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/2b895835bd56
- [07/18] platform/chrome: cros_usbpd_notify: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/522b9910fa25
- [08/18] platform/chrome: cros_ec_chardev: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/e4d063f262df
- [09/18] platform/chrome: cros_ec_debugfs: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/e26205b5728f
- [10/18] platform/chrome: cros_ec_sysfs: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/6e92e7f792eb
- [11/18] power: supply: cros_pchg: provide ID table for avoiding fallback match
(no matching commit)
- [12/18] platform/chrome: cros_ec_lightbar: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/4f6b823b5caa
- [13/18] platform/chrome: cros_ec_vbc: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/883030dae2d2
- [14/18] platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding fallback match
(no matching commit)
- [15/18] platform/chrome: wilco_ec: telemetry: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/b17dd111645b
- [16/18] platform/chrome: wilco_ec: debugfs: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/c0649ca22fe8
- [17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS
https://git.kernel.org/chrome-platform/c/fc87ba8e4525
- [18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/d8db2c6d7b0e
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (18 preceding siblings ...)
2024-04-01 2:30 ` [PATCH 00/18] platform/chrome: " patchwork-bot+chrome-platform
@ 2024-04-01 2:30 ` patchwork-bot+chrome-platform
2024-04-03 7:20 ` patchwork-bot+chrome-platform
2024-04-03 7:20 ` patchwork-bot+chrome-platform
21 siblings, 0 replies; 64+ messages in thread
From: patchwork-bot+chrome-platform @ 2024-04-01 2:30 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
Hello:
This series was applied to chrome-platform/linux.git (for-next)
by Tzung-Bi Shih <tzungbi@kernel.org>:
On Fri, 29 Mar 2024 15:56:12 +0800 you wrote:
> Inspired by [1]. Turn all MODULE_ALIAS() in ChromeOS EC platform drivers into
> proper platform_device_id table and MODULE_DEVICE_TABLE().
>
> The series is basically looking for drivers from:
> - `struct mfd_cell` in drivers/mfd/cros_ec_dev.c.
> - grep -R MODULE_ALIAS drivers/platform/chrome/.
>
> [...]
Here is the summary with links:
- [01/18] media: platform: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [02/18] gpio: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [03/18] rtc: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [04/18] platform/chrome: cros_ec_sensorhub: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/12e981d4d643
- [05/18] power: supply: cros_usbpd: provide ID table for avoiding fallback match
(no matching commit)
- [06/18] platform/chrome: cros_usbpd_logger: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/2b895835bd56
- [07/18] platform/chrome: cros_usbpd_notify: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/522b9910fa25
- [08/18] platform/chrome: cros_ec_chardev: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/e4d063f262df
- [09/18] platform/chrome: cros_ec_debugfs: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/e26205b5728f
- [10/18] platform/chrome: cros_ec_sysfs: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/6e92e7f792eb
- [11/18] power: supply: cros_pchg: provide ID table for avoiding fallback match
(no matching commit)
- [12/18] platform/chrome: cros_ec_lightbar: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/4f6b823b5caa
- [13/18] platform/chrome: cros_ec_vbc: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/883030dae2d2
- [14/18] platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding fallback match
(no matching commit)
- [15/18] platform/chrome: wilco_ec: telemetry: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/b17dd111645b
- [16/18] platform/chrome: wilco_ec: debugfs: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/c0649ca22fe8
- [17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS
https://git.kernel.org/chrome-platform/c/fc87ba8e4525
- [18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/d8db2c6d7b0e
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding fallback match
2024-03-29 15:53 ` Krzysztof Kozlowski
@ 2024-04-01 3:11 ` Tzung-Bi Shih
0 siblings, 0 replies; 64+ messages in thread
From: Tzung-Bi Shih @ 2024-04-01 3:11 UTC (permalink / raw)
To: Krzysztof Kozlowski
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc
On Fri, Mar 29, 2024 at 04:53:48PM +0100, Krzysztof Kozlowski wrote:
> On 29/03/2024 08:56, Tzung-Bi Shih wrote:
> > Instead of using fallback driver name match, provide ID table[1] for the
> > primary match.
> >
> > Also shrink the name for fitting to [2].
> >
> > [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
> > [2]: https://elixir.bootlin.com/linux/v6.8/source/include/linux/mod_devicetable.h#L608
> >
>
> I think this should be two commits, where the first one is a fix (Fixes
> tag) for the size of name.
Splitted to 2 commits in v2[3]. It doesn't include a Fixes tag as the compiler
complaints the oversize only if putting the original driver name to struct
platform_device_id.
[3]: https://patchwork.kernel.org/project/chrome-platform/patch/20240401030052.2887845-6-tzungbi@kernel.org/
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (19 preceding siblings ...)
2024-04-01 2:30 ` patchwork-bot+chrome-platform
@ 2024-04-03 7:20 ` patchwork-bot+chrome-platform
2024-04-03 7:20 ` patchwork-bot+chrome-platform
21 siblings, 0 replies; 64+ messages in thread
From: patchwork-bot+chrome-platform @ 2024-04-03 7:20 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
Hello:
This series was applied to chrome-platform/linux.git (for-kernelci)
by Tzung-Bi Shih <tzungbi@kernel.org>:
On Fri, 29 Mar 2024 15:56:12 +0800 you wrote:
> Inspired by [1]. Turn all MODULE_ALIAS() in ChromeOS EC platform drivers into
> proper platform_device_id table and MODULE_DEVICE_TABLE().
>
> The series is basically looking for drivers from:
> - `struct mfd_cell` in drivers/mfd/cros_ec_dev.c.
> - grep -R MODULE_ALIAS drivers/platform/chrome/.
>
> [...]
Here is the summary with links:
- [01/18] media: platform: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [02/18] gpio: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [03/18] rtc: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [04/18] platform/chrome: cros_ec_sensorhub: provide ID table for avoiding fallback match
(no matching commit)
- [05/18] power: supply: cros_usbpd: provide ID table for avoiding fallback match
(no matching commit)
- [06/18] platform/chrome: cros_usbpd_logger: provide ID table for avoiding fallback match
(no matching commit)
- [07/18] platform/chrome: cros_usbpd_notify: provide ID table for avoiding fallback match
(no matching commit)
- [08/18] platform/chrome: cros_ec_chardev: provide ID table for avoiding fallback match
(no matching commit)
- [09/18] platform/chrome: cros_ec_debugfs: provide ID table for avoiding fallback match
(no matching commit)
- [10/18] platform/chrome: cros_ec_sysfs: provide ID table for avoiding fallback match
(no matching commit)
- [11/18] power: supply: cros_pchg: provide ID table for avoiding fallback match
(no matching commit)
- [12/18] platform/chrome: cros_ec_lightbar: provide ID table for avoiding fallback match
(no matching commit)
- [13/18] platform/chrome: cros_ec_vbc: provide ID table for avoiding fallback match
(no matching commit)
- [14/18] platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/d91ca83599cd
- [15/18] platform/chrome: wilco_ec: telemetry: provide ID table for avoiding fallback match
(no matching commit)
- [16/18] platform/chrome: wilco_ec: debugfs: provide ID table for avoiding fallback match
(no matching commit)
- [17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS
(no matching commit)
- [18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match
(no matching commit)
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
` (20 preceding siblings ...)
2024-04-03 7:20 ` patchwork-bot+chrome-platform
@ 2024-04-03 7:20 ` patchwork-bot+chrome-platform
21 siblings, 0 replies; 64+ messages in thread
From: patchwork-bot+chrome-platform @ 2024-04-03 7:20 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, linus.walleij, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
Hello:
This series was applied to chrome-platform/linux.git (for-next)
by Tzung-Bi Shih <tzungbi@kernel.org>:
On Fri, 29 Mar 2024 15:56:12 +0800 you wrote:
> Inspired by [1]. Turn all MODULE_ALIAS() in ChromeOS EC platform drivers into
> proper platform_device_id table and MODULE_DEVICE_TABLE().
>
> The series is basically looking for drivers from:
> - `struct mfd_cell` in drivers/mfd/cros_ec_dev.c.
> - grep -R MODULE_ALIAS drivers/platform/chrome/.
>
> [...]
Here is the summary with links:
- [01/18] media: platform: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [02/18] gpio: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [03/18] rtc: cros-ec: provide ID table for avoiding fallback match
(no matching commit)
- [04/18] platform/chrome: cros_ec_sensorhub: provide ID table for avoiding fallback match
(no matching commit)
- [05/18] power: supply: cros_usbpd: provide ID table for avoiding fallback match
(no matching commit)
- [06/18] platform/chrome: cros_usbpd_logger: provide ID table for avoiding fallback match
(no matching commit)
- [07/18] platform/chrome: cros_usbpd_notify: provide ID table for avoiding fallback match
(no matching commit)
- [08/18] platform/chrome: cros_ec_chardev: provide ID table for avoiding fallback match
(no matching commit)
- [09/18] platform/chrome: cros_ec_debugfs: provide ID table for avoiding fallback match
(no matching commit)
- [10/18] platform/chrome: cros_ec_sysfs: provide ID table for avoiding fallback match
(no matching commit)
- [11/18] power: supply: cros_pchg: provide ID table for avoiding fallback match
(no matching commit)
- [12/18] platform/chrome: cros_ec_lightbar: provide ID table for avoiding fallback match
(no matching commit)
- [13/18] platform/chrome: cros_ec_vbc: provide ID table for avoiding fallback match
(no matching commit)
- [14/18] platform/chrome: cros_kbd_led_backlight: provide ID table for avoiding fallback match
https://git.kernel.org/chrome-platform/c/d91ca83599cd
- [15/18] platform/chrome: wilco_ec: telemetry: provide ID table for avoiding fallback match
(no matching commit)
- [16/18] platform/chrome: wilco_ec: debugfs: provide ID table for avoiding fallback match
(no matching commit)
- [17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS
(no matching commit)
- [18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match
(no matching commit)
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 64+ messages in thread
* Re: [PATCH 02/18] gpio: cros-ec: provide ID table for avoiding fallback match
2024-03-29 7:56 ` [PATCH 02/18] gpio: " Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 20:06 ` Benson Leung
@ 2024-04-04 11:32 ` Linus Walleij
2 siblings, 0 replies; 64+ messages in thread
From: Linus Walleij @ 2024-04-04 11:32 UTC (permalink / raw)
To: Tzung-Bi Shih
Cc: bleung, groeck, brgl, hverkuil-cisco, mchehab, sre,
alexandre.belloni, chrome-platform, pmalani, linux-gpio,
linux-media, linux-pm, linux-rtc, krzk
On Fri, Mar 29, 2024 at 8:57 AM Tzung-Bi Shih <tzungbi@kernel.org> wrote:
> Instead of using fallback driver name match, provide ID table[1] for the
> primary match. Also allow automatic module loading by adding
> MODULE_DEVICE_TABLE().
>
> [1]: https://elixir.bootlin.com/linux/v6.8/source/drivers/base/platform.c#L1353
>
> Signed-off-by: Tzung-Bi Shih <tzungbi@kernel.org>
Acked-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 64+ messages in thread
end of thread, other threads:[~2024-04-04 11:32 UTC | newest]
Thread overview: 64+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-29 7:56 [PATCH 00/18] platform/chrome: provide ID table for avoiding fallback match Tzung-Bi Shih
2024-03-29 7:56 ` [PATCH 01/18] media: platform: cros-ec: " Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 20:05 ` Benson Leung
2024-03-29 7:56 ` [PATCH 02/18] gpio: " Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 20:06 ` Benson Leung
2024-04-04 11:32 ` Linus Walleij
2024-03-29 7:56 ` [PATCH 03/18] rtc: " Tzung-Bi Shih
2024-03-29 12:15 ` Krzysztof Kozlowski
2024-03-29 15:00 ` (subset) " Alexandre Belloni
2024-03-29 20:07 ` Benson Leung
2024-03-29 7:56 ` [PATCH 04/18] platform/chrome: cros_ec_sensorhub: " Tzung-Bi Shih
2024-03-29 12:16 ` Krzysztof Kozlowski
2024-03-29 20:08 ` Benson Leung
2024-03-29 7:56 ` [PATCH 05/18] power: supply: cros_usbpd: " Tzung-Bi Shih
2024-03-29 12:18 ` Krzysztof Kozlowski
2024-03-29 20:09 ` Benson Leung
2024-03-29 23:03 ` Prashant Malani
2024-03-29 7:56 ` [PATCH 06/18] platform/chrome: cros_usbpd_logger: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:11 ` Benson Leung
2024-03-29 7:56 ` [PATCH 07/18] platform/chrome: cros_usbpd_notify: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:13 ` Benson Leung
2024-03-29 22:06 ` Prashant Malani
2024-03-29 7:56 ` [PATCH 08/18] platform/chrome: cros_ec_chardev: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:13 ` Benson Leung
2024-03-29 7:56 ` [PATCH 09/18] platform/chrome: cros_ec_debugfs: " Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:14 ` Benson Leung
2024-03-29 7:56 ` [PATCH 10/18] platform/chrome: cros_ec_sysfs: " Tzung-Bi Shih
2024-03-29 12:37 ` Krzysztof Kozlowski
2024-03-29 20:15 ` Benson Leung
2024-03-29 7:56 ` [PATCH 11/18] power: supply: cros_pchg: " Tzung-Bi Shih
2024-03-29 15:50 ` Krzysztof Kozlowski
2024-03-29 20:16 ` Benson Leung
2024-03-29 7:56 ` [PATCH 12/18] platform/chrome: cros_ec_lightbar: " Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:18 ` Benson Leung
2024-03-29 7:56 ` [PATCH 13/18] platform/chrome: cros_ec_vbc: " Tzung-Bi Shih
2024-03-29 15:55 ` Krzysztof Kozlowski
2024-03-29 20:19 ` Benson Leung
2024-03-29 7:56 ` [PATCH 14/18] platform/chrome: cros_kbd_led_backlight: " Tzung-Bi Shih
2024-03-29 15:53 ` Krzysztof Kozlowski
2024-04-01 3:11 ` Tzung-Bi Shih
2024-03-29 20:20 ` Benson Leung
2024-03-29 7:56 ` [PATCH 15/18] platform/chrome: wilco_ec: telemetry: " Tzung-Bi Shih
2024-03-29 15:52 ` Krzysztof Kozlowski
2024-03-29 20:21 ` Benson Leung
2024-03-29 7:56 ` [PATCH 16/18] platform/chrome: wilco_ec: debugfs: " Tzung-Bi Shih
2024-03-29 15:52 ` Krzysztof Kozlowski
2024-03-29 20:21 ` Benson Leung
2024-03-29 7:56 ` [PATCH 17/18] platform/chrome: wilco_ec: event: remove redundant MODULE_ALIAS Tzung-Bi Shih
2024-03-29 15:55 ` Krzysztof Kozlowski
2024-03-29 20:22 ` Benson Leung
2024-03-29 7:56 ` [PATCH 18/18] platform/chrome/wilco_ec: core: provide ID table for avoiding fallback match Tzung-Bi Shih
2024-03-29 15:51 ` Krzysztof Kozlowski
2024-03-29 20:23 ` Benson Leung
2024-04-01 2:30 ` [PATCH 00/18] platform/chrome: " patchwork-bot+chrome-platform
2024-04-01 2:30 ` patchwork-bot+chrome-platform
2024-04-03 7:20 ` patchwork-bot+chrome-platform
2024-04-03 7:20 ` patchwork-bot+chrome-platform
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.