mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2025-01-06 13:16:22 +00:00
greybus: arche-ctrl: move device table to its right location
[ Upstream commit6a0b8c0da8
] 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:7b62b61c75
("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>
This commit is contained in:
parent
cb3cc5e92a
commit
b54f3b1682
@ -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,
|
||||
|
@ -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,
|
||||
|
Loading…
Reference in New Issue
Block a user