| Message ID | 20190107183439.50626-4-lkundrak@v3.sk (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | Add support for OLPC XO 1.75 Embedded Controller | expand |
On Mon, 2019-01-07 at 19:34 +0100, Lubomir Rintel wrote: > It doesn't make sense to always have this built-in, e.g. on ARM > multiplatform kernels. > > A better way to address the problem the original commit aimed to solve is > to fix Kconfig. That is what the next commit in the series does. > > This reverts commit f48d1496b8537d75776478c6942dd87f34d7f270. The kBuild but figured out that this is not such a great idea. I'm going to follow-up with a new patch set; sorry for the noise. > > Signed-off-by: Lubomir Rintel <lkundrak@v3.sk> > > --- > Changes since v1: > - Sort it later in the patch set > > drivers/platform/olpc/olpc-ec.c | 8 +++++--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/drivers/platform/olpc/olpc-ec.c b/drivers/platform/olpc/olpc-ec.c > index 981955dce926..35a21c66cd0d 100644 > --- a/drivers/platform/olpc/olpc-ec.c > +++ b/drivers/platform/olpc/olpc-ec.c > @@ -1,8 +1,6 @@ > /* > * Generic driver for the OLPC Embedded Controller. > * > - * Author: Andres Salomon <dilinger@queued.net> > - * > * Copyright (C) 2011-2012 One Laptop per Child Foundation. > * > * Licensed under the GPL v2 or later. > @@ -14,7 +12,7 @@ > #include <linux/platform_device.h> > #include <linux/slab.h> > #include <linux/workqueue.h> > -#include <linux/init.h> > +#include <linux/module.h> > #include <linux/list.h> > #include <linux/olpc-ec.h> > > @@ -327,4 +325,8 @@ static int __init olpc_ec_init_module(void) > { > return platform_driver_register(&olpc_ec_plat_driver); > } > + > arch_initcall(olpc_ec_init_module); > + > +MODULE_AUTHOR("Andres Salomon <dilinger@queued.net>"); > +MODULE_LICENSE("GPL");
diff --git a/drivers/platform/olpc/olpc-ec.c b/drivers/platform/olpc/olpc-ec.c index 981955dce926..35a21c66cd0d 100644 --- a/drivers/platform/olpc/olpc-ec.c +++ b/drivers/platform/olpc/olpc-ec.c @@ -1,8 +1,6 @@ /* * Generic driver for the OLPC Embedded Controller. * - * Author: Andres Salomon <dilinger@queued.net> - * * Copyright (C) 2011-2012 One Laptop per Child Foundation. * * Licensed under the GPL v2 or later. @@ -14,7 +12,7 @@ #include <linux/platform_device.h> #include <linux/slab.h> #include <linux/workqueue.h> -#include <linux/init.h> +#include <linux/module.h> #include <linux/list.h> #include <linux/olpc-ec.h> @@ -327,4 +325,8 @@ static int __init olpc_ec_init_module(void) { return platform_driver_register(&olpc_ec_plat_driver); } + arch_initcall(olpc_ec_init_module); + +MODULE_AUTHOR("Andres Salomon <dilinger@queued.net>"); +MODULE_LICENSE("GPL");
It doesn't make sense to always have this built-in, e.g. on ARM multiplatform kernels. A better way to address the problem the original commit aimed to solve is to fix Kconfig. That is what the next commit in the series does. This reverts commit f48d1496b8537d75776478c6942dd87f34d7f270. Signed-off-by: Lubomir Rintel <lkundrak@v3.sk> --- Changes since v1: - Sort it later in the patch set drivers/platform/olpc/olpc-ec.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-)