]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
greybus: arche-ctrl: move device table to its right location
authorArnd Bergmann <arnd@arndb.de>
Wed, 3 Apr 2024 08:06:35 +0000 (10:06 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 16 Jun 2024 11:39:35 +0000 (13:39 +0200)
[ Upstream commit 6a0b8c0da8d8d418cde6894a104cf74e6098ddfa ]

The arche-ctrl has two platform drivers and three of_device_id tables,
but one table is only used for the the module loader, while the other
two seem to be associated with their drivers.

This leads to a W=1 warning when the driver is built-in:

drivers/staging/greybus/arche-platform.c:623:34: error: 'arche_combined_id' defined but not used [-Werror=unused-const-variable=]
  623 | static const struct of_device_id arche_combined_id[] = {

Drop the extra table and register both tables that are actually
used as the ones for the module loader instead.

Fixes: 7b62b61c752a ("greybus: arche-ctrl: Don't expose driver internals to arche-platform driver")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Link: https://lore.kernel.org/r/20240403080702.3509288-18-arnd@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/staging/greybus/arche-apb-ctrl.c
drivers/staging/greybus/arche-platform.c

index bbf3ba744fc441b71e84cd38ec65bfadeb103b2c..c7383c6c6094d80992fddc18a775e99d003e5cfb 100644 (file)
@@ -468,6 +468,7 @@ static const struct of_device_id arche_apb_ctrl_of_match[] = {
        { .compatible = "usbffff,2", },
        { },
 };
+MODULE_DEVICE_TABLE(of, arche_apb_ctrl_of_match);
 
 static struct platform_driver arche_apb_ctrl_device_driver = {
        .probe          = arche_apb_ctrl_probe,
index e374dfc0c92fd69cd19b6812331a53143b7288e6..00beb8bb1b331b33c6674e6284ed788cfdeddc89 100644 (file)
@@ -620,14 +620,7 @@ static const struct of_device_id arche_platform_of_match[] = {
        { .compatible = "google,arche-platform", },
        { },
 };
-
-static const struct of_device_id arche_combined_id[] = {
-       /* Use PID/VID of SVC device */
-       { .compatible = "google,arche-platform", },
-       { .compatible = "usbffff,2", },
-       { },
-};
-MODULE_DEVICE_TABLE(of, arche_combined_id);
+MODULE_DEVICE_TABLE(of, arche_platform_of_match);
 
 static struct platform_driver arche_platform_device_driver = {
        .probe          = arche_platform_probe,