Add the missing MODULE_DEVICE_TABLE and remove the comma from the
separator on the end of the of_device_id array.
Signed-off-by: Charles Keepax <[email protected]>
---
drivers/regulator/lochnagar-regulator.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/regulator/lochnagar-regulator.c b/drivers/regulator/lochnagar-regulator.c
index 818d204105186..4962d13659aab 100644
--- a/drivers/regulator/lochnagar-regulator.c
+++ b/drivers/regulator/lochnagar-regulator.c
@@ -232,8 +232,9 @@ static const struct of_device_id lochnagar_of_match[] = {
.compatible = "cirrus,lochnagar2-vddcore",
.data = &lochnagar_regulators[LOCHNAGAR_VDDCORE],
},
- {},
+ {}
};
+MODULE_DEVICE_TABLE(of, lochnagar_of_match);
static int lochnagar_regulator_probe(struct platform_device *pdev)
{
--
2.11.0
The module alias has no effect as the driver can only be loaded through
device tree at this time.
Signed-off-by: Charles Keepax <[email protected]>
---
drivers/regulator/lochnagar-regulator.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/regulator/lochnagar-regulator.c b/drivers/regulator/lochnagar-regulator.c
index 4962d13659aab..41de3b715b4db 100644
--- a/drivers/regulator/lochnagar-regulator.c
+++ b/drivers/regulator/lochnagar-regulator.c
@@ -280,4 +280,3 @@ module_platform_driver(lochnagar_regulator_driver);
MODULE_AUTHOR("Charles Keepax <[email protected]>");
MODULE_DESCRIPTION("Regulator driver for Cirrus Logic Lochnagar Board");
MODULE_LICENSE("GPL v2");
-MODULE_ALIAS("platform:lochnagar-regulator");
--
2.11.0
On Fri, Jan 18, 2019 at 11:36:31AM +0000, Charles Keepax wrote:
> The module alias has no effect as the driver can only be loaded through
> device tree at this time.
It also does no harm...
On Fri, Jan 18, 2019 at 05:28:54PM +0000, Mark Brown wrote:
> On Fri, Jan 18, 2019 at 11:36:31AM +0000, Charles Keepax wrote:
> > The module alias has no effect as the driver can only be loaded through
> > device tree at this time.
>
> It also does no harm...
Indeed yes, I have no problem with leaving it.
Thanks,
Charles