diff options
author | Viresh Kumar <viresh.kumar@linaro.org> | 2016-04-20 11:48:37 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@google.com> | 2016-04-23 09:37:21 +0900 |
commit | 7b62b61c752a4700ecf11d63a7ec40aeb3cee66c (patch) | |
tree | fd61c48dd760bf5fe4b3ecccb71eaef78dbe3709 /drivers/staging/greybus/arche-platform.c | |
parent | f2bf63a365425fa7df56c4e1dbe5a6bb29680324 (diff) |
greybus: arche-ctrl: Don't expose driver internals to arche-platform driver
We have chosen the *ugly* way of registering two platform drivers from
the module_init() of only one of them, so that we can avoid having two
separate modules for them.
But we should still be doing this in a sane way. There is no need to
expose internals of arche-ctrl to arche-platform, like PM-ops, probe,
resume, id-table, etc. Just expose an init and a exit callback.
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Reviewed-by: Vaibhav Hiremath <vaibhav.hiremath@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
Diffstat (limited to 'drivers/staging/greybus/arche-platform.c')
-rw-r--r-- | drivers/staging/greybus/arche-platform.c | 19 |
1 files changed, 2 insertions, 17 deletions
diff --git a/drivers/staging/greybus/arche-platform.c b/drivers/staging/greybus/arche-platform.c index 3ff4f69685bd..d1083cfbc081 100644 --- a/drivers/staging/greybus/arche-platform.c +++ b/drivers/staging/greybus/arche-platform.c @@ -580,11 +580,6 @@ static struct of_device_id arche_platform_of_match[] = { { }, }; -static struct of_device_id arche_apb_ctrl_of_match[] = { - { .compatible = "usbffff,2", }, - { }, -}; - static struct of_device_id arche_combined_id[] = { { .compatible = "google,arche-platform", }, /* Use PID/VID of SVC device */ { .compatible = "usbffff,2", }, @@ -602,16 +597,6 @@ static struct platform_driver arche_platform_device_driver = { } }; -static struct platform_driver arche_apb_ctrl_device_driver = { - .probe = arche_apb_ctrl_probe, - .remove = arche_apb_ctrl_remove, - .driver = { - .name = "arche-apb-ctrl", - .pm = &arche_apb_ctrl_pm_ops, - .of_match_table = arche_apb_ctrl_of_match, - } -}; - static int __init arche_init(void) { int retval; @@ -620,7 +605,7 @@ static int __init arche_init(void) if (retval) return retval; - retval = platform_driver_register(&arche_apb_ctrl_device_driver); + retval = arche_apb_init(); if (retval) platform_driver_unregister(&arche_platform_device_driver); @@ -630,7 +615,7 @@ module_init(arche_init); static void __exit arche_exit(void) { - platform_driver_unregister(&arche_apb_ctrl_device_driver); + arche_apb_exit(); platform_driver_unregister(&arche_platform_device_driver); } module_exit(arche_exit); |