* [PATCH] reset: hisilicon: Fix hi6220 module license
@ 2017-05-25 9:22 ` Andreas Färber
0 siblings, 0 replies; 14+ messages in thread
From: Andreas Färber @ 2017-05-25 9:22 UTC (permalink / raw)
To: linux-arm-kernel
The hi6220_reset driver fails to load:
[ 10.423640] hi6220_reset: module license 'unspecified' taints kernel.
[ 10.423644] Disabling lock debugging due to kernel taint
[ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0)
[ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0)
[ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0)
[ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0)
[ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0)
Add a MODULE_LICENSE() to fix this.
Fixes: 70b3590f639f ("reset: hi6220: fix modular build")
Cc: Arnd Bergmann <arnd@arndb.de>
Cc: Chen Feng <puck.chen@hisilicon.com>
Signed-off-by: Andreas F?rber <afaerber@suse.de>
---
drivers/reset/hisilicon/hi6220_reset.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/reset/hisilicon/hi6220_reset.c b/drivers/reset/hisilicon/hi6220_reset.c
index 35ce53edabf9..d5e5229308f2 100644
--- a/drivers/reset/hisilicon/hi6220_reset.c
+++ b/drivers/reset/hisilicon/hi6220_reset.c
@@ -155,3 +155,5 @@ static int __init hi6220_reset_init(void)
}
postcore_initcall(hi6220_reset_init);
+
+MODULE_LICENSE("GPL v2");
--
2.12.0
^ permalink raw reply related [flat|nested] 14+ messages in thread* [PATCH] reset: hisilicon: Fix hi6220 module license @ 2017-05-25 9:22 ` Andreas Färber 0 siblings, 0 replies; 14+ messages in thread From: Andreas Färber @ 2017-05-25 9:22 UTC (permalink / raw) To: linux-arm-kernel Cc: Andreas Färber, Arnd Bergmann, Chen Feng, Philipp Zabel, linux-kernel The hi6220_reset driver fails to load: [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. [ 10.423644] Disabling lock debugging due to kernel taint [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) Add a MODULE_LICENSE() to fix this. Fixes: 70b3590f639f ("reset: hi6220: fix modular build") Cc: Arnd Bergmann <arnd@arndb.de> Cc: Chen Feng <puck.chen@hisilicon.com> Signed-off-by: Andreas Färber <afaerber@suse.de> --- drivers/reset/hisilicon/hi6220_reset.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/reset/hisilicon/hi6220_reset.c b/drivers/reset/hisilicon/hi6220_reset.c index 35ce53edabf9..d5e5229308f2 100644 --- a/drivers/reset/hisilicon/hi6220_reset.c +++ b/drivers/reset/hisilicon/hi6220_reset.c @@ -155,3 +155,5 @@ static int __init hi6220_reset_init(void) } postcore_initcall(hi6220_reset_init); + +MODULE_LICENSE("GPL v2"); -- 2.12.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH] reset: hisilicon: Fix hi6220 module license 2017-05-25 9:22 ` Andreas Färber @ 2017-05-26 3:12 ` Chen Feng -1 siblings, 0 replies; 14+ messages in thread From: Chen Feng @ 2017-05-26 3:12 UTC (permalink / raw) To: linux-arm-kernel looks good to me. On 2017/5/25 17:22, Andreas F?rber wrote: > The hi6220_reset driver fails to load: > > [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. > [ 10.423644] Disabling lock debugging due to kernel taint > [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) > [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) > [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) > [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) > [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) > > Add a MODULE_LICENSE() to fix this. > > Fixes: 70b3590f639f ("reset: hi6220: fix modular build") > Cc: Arnd Bergmann <arnd@arndb.de> > Cc: Chen Feng <puck.chen@hisilicon.com> > Signed-off-by: Andreas F?rber <afaerber@suse.de> > --- > drivers/reset/hisilicon/hi6220_reset.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/reset/hisilicon/hi6220_reset.c b/drivers/reset/hisilicon/hi6220_reset.c > index 35ce53edabf9..d5e5229308f2 100644 > --- a/drivers/reset/hisilicon/hi6220_reset.c > +++ b/drivers/reset/hisilicon/hi6220_reset.c > @@ -155,3 +155,5 @@ static int __init hi6220_reset_init(void) > } > > postcore_initcall(hi6220_reset_init); > + > +MODULE_LICENSE("GPL v2"); ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] reset: hisilicon: Fix hi6220 module license @ 2017-05-26 3:12 ` Chen Feng 0 siblings, 0 replies; 14+ messages in thread From: Chen Feng @ 2017-05-26 3:12 UTC (permalink / raw) To: Andreas Färber, linux-arm-kernel Cc: Arnd Bergmann, Philipp Zabel, linux-kernel looks good to me. On 2017/5/25 17:22, Andreas Färber wrote: > The hi6220_reset driver fails to load: > > [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. > [ 10.423644] Disabling lock debugging due to kernel taint > [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) > [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) > [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) > [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) > [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) > > Add a MODULE_LICENSE() to fix this. > > Fixes: 70b3590f639f ("reset: hi6220: fix modular build") > Cc: Arnd Bergmann <arnd@arndb.de> > Cc: Chen Feng <puck.chen@hisilicon.com> > Signed-off-by: Andreas Färber <afaerber@suse.de> > --- > drivers/reset/hisilicon/hi6220_reset.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/reset/hisilicon/hi6220_reset.c b/drivers/reset/hisilicon/hi6220_reset.c > index 35ce53edabf9..d5e5229308f2 100644 > --- a/drivers/reset/hisilicon/hi6220_reset.c > +++ b/drivers/reset/hisilicon/hi6220_reset.c > @@ -155,3 +155,5 @@ static int __init hi6220_reset_init(void) > } > > postcore_initcall(hi6220_reset_init); > + > +MODULE_LICENSE("GPL v2"); ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] reset: hisilicon: Fix hi6220 module license 2017-05-25 9:22 ` Andreas Färber @ 2017-05-26 8:44 ` Arnd Bergmann -1 siblings, 0 replies; 14+ messages in thread From: Arnd Bergmann @ 2017-05-26 8:44 UTC (permalink / raw) To: linux-arm-kernel On Thu, May 25, 2017 at 11:22 AM, Andreas F?rber <afaerber@suse.de> wrote: > The hi6220_reset driver fails to load: > > [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. > [ 10.423644] Disabling lock debugging due to kernel taint > [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) > [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) > [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) > [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) > [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) > > Add a MODULE_LICENSE() to fix this. > > Fixes: 70b3590f639f ("reset: hi6220: fix modular build") > Cc: Arnd Bergmann <arnd@arndb.de> > Cc: Chen Feng <puck.chen@hisilicon.com> > Signed-off-by: Andreas F?rber <afaerber@suse.de> Acked-by: Arnd Bergmann <arnd@arndb.de> I wonder why I never ran into this in randconfig testing. ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] reset: hisilicon: Fix hi6220 module license @ 2017-05-26 8:44 ` Arnd Bergmann 0 siblings, 0 replies; 14+ messages in thread From: Arnd Bergmann @ 2017-05-26 8:44 UTC (permalink / raw) To: Andreas Färber Cc: Linux ARM, Chen Feng, Philipp Zabel, Linux Kernel Mailing List On Thu, May 25, 2017 at 11:22 AM, Andreas Färber <afaerber@suse.de> wrote: > The hi6220_reset driver fails to load: > > [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. > [ 10.423644] Disabling lock debugging due to kernel taint > [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) > [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) > [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) > [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) > [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) > > Add a MODULE_LICENSE() to fix this. > > Fixes: 70b3590f639f ("reset: hi6220: fix modular build") > Cc: Arnd Bergmann <arnd@arndb.de> > Cc: Chen Feng <puck.chen@hisilicon.com> > Signed-off-by: Andreas Färber <afaerber@suse.de> Acked-by: Arnd Bergmann <arnd@arndb.de> I wonder why I never ran into this in randconfig testing. ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] reset: hisilicon: Fix hi6220 module license 2017-05-26 8:44 ` Arnd Bergmann @ 2017-05-26 9:25 ` Andreas Färber -1 siblings, 0 replies; 14+ messages in thread From: Andreas Färber @ 2017-05-26 9:25 UTC (permalink / raw) To: linux-arm-kernel Am 26.05.2017 um 10:44 schrieb Arnd Bergmann: > On Thu, May 25, 2017 at 11:22 AM, Andreas F?rber <afaerber@suse.de> wrote: >> The hi6220_reset driver fails to load: >> >> [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. >> [ 10.423644] Disabling lock debugging due to kernel taint >> [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) >> [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) >> [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) >> [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) >> [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) >> >> Add a MODULE_LICENSE() to fix this. >> >> Fixes: 70b3590f639f ("reset: hi6220: fix modular build") >> Cc: Arnd Bergmann <arnd@arndb.de> >> Cc: Chen Feng <puck.chen@hisilicon.com> >> Signed-off-by: Andreas F?rber <afaerber@suse.de> > > Acked-by: Arnd Bergmann <arnd@arndb.de> > > I wonder why I never ran into this in randconfig testing. In the bigger picture I also found that PINCTRL_SINGLE needs to be built-in for the HiKey to proceed beyond earlycon. Usually when an option can be built as module, we build it as module. Further to that, I needed the following config for dracut so far: add_drivers+=" hi6220_reset hi6220-mailbox hi655x-pmic hi655x-regulator" I'm still not able to boot 4.11 from SD with our config: https://bugzilla.opensuse.org/show_bug.cgi?id=1040492 Any hints appreciated on what drivers may be missing in the initrd and why dracut is not picking them up on its own. And as pointed out at BUD17, it would be really great if Linaro would actually test modules for 96Boards - be it via Lava or manually once. Our configs are public on kernel.opensuse.org / GitHub kernel-source.git in case that aids with your testing - master is for linux.git tags, vanilla for linux.git master and linux-next for linux-next.git master. Investigating why the DragonBoard 410c gets stuck is still ongoing, too. For the Poplar I'm currently running into partition size issues trying to install additional kernels, so I couldn't verify beyond 4.11 yet. https://github.com/Linaro/poplar-tools/pull/3#issuecomment-304215367 It also seems to need CONFIG_PINCTRL_SINGLE=y though. For the HiKey 960 I'm lacking documentation how to get started bootloader-wise - by default it's uninterruptibly booting AOSP. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany GF: Felix Imend?rffer, Jane Smithard, Graham Norton HRB 21284 (AG N?rnberg) ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] reset: hisilicon: Fix hi6220 module license @ 2017-05-26 9:25 ` Andreas Färber 0 siblings, 0 replies; 14+ messages in thread From: Andreas Färber @ 2017-05-26 9:25 UTC (permalink / raw) To: Arnd Bergmann Cc: Linux ARM, Chen Feng, Philipp Zabel, Linux Kernel Mailing List, Bamvor Jian Zhang Am 26.05.2017 um 10:44 schrieb Arnd Bergmann: > On Thu, May 25, 2017 at 11:22 AM, Andreas Färber <afaerber@suse.de> wrote: >> The hi6220_reset driver fails to load: >> >> [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. >> [ 10.423644] Disabling lock debugging due to kernel taint >> [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) >> [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) >> [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) >> [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) >> [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) >> >> Add a MODULE_LICENSE() to fix this. >> >> Fixes: 70b3590f639f ("reset: hi6220: fix modular build") >> Cc: Arnd Bergmann <arnd@arndb.de> >> Cc: Chen Feng <puck.chen@hisilicon.com> >> Signed-off-by: Andreas Färber <afaerber@suse.de> > > Acked-by: Arnd Bergmann <arnd@arndb.de> > > I wonder why I never ran into this in randconfig testing. In the bigger picture I also found that PINCTRL_SINGLE needs to be built-in for the HiKey to proceed beyond earlycon. Usually when an option can be built as module, we build it as module. Further to that, I needed the following config for dracut so far: add_drivers+=" hi6220_reset hi6220-mailbox hi655x-pmic hi655x-regulator" I'm still not able to boot 4.11 from SD with our config: https://bugzilla.opensuse.org/show_bug.cgi?id=1040492 Any hints appreciated on what drivers may be missing in the initrd and why dracut is not picking them up on its own. And as pointed out at BUD17, it would be really great if Linaro would actually test modules for 96Boards - be it via Lava or manually once. Our configs are public on kernel.opensuse.org / GitHub kernel-source.git in case that aids with your testing - master is for linux.git tags, vanilla for linux.git master and linux-next for linux-next.git master. Investigating why the DragonBoard 410c gets stuck is still ongoing, too. For the Poplar I'm currently running into partition size issues trying to install additional kernels, so I couldn't verify beyond 4.11 yet. https://github.com/Linaro/poplar-tools/pull/3#issuecomment-304215367 It also seems to need CONFIG_PINCTRL_SINGLE=y though. For the HiKey 960 I'm lacking documentation how to get started bootloader-wise - by default it's uninterruptibly booting AOSP. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] reset: hisilicon: Fix hi6220 module license 2017-05-25 9:22 ` Andreas Färber @ 2017-05-29 9:46 ` Philipp Zabel -1 siblings, 0 replies; 14+ messages in thread From: Philipp Zabel @ 2017-05-29 9:46 UTC (permalink / raw) To: linux-arm-kernel Hi Andreas, On Thu, 2017-05-25 at 11:22 +0200, Andreas F?rber wrote: > The hi6220_reset driver fails to load: > > [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. > [ 10.423644] Disabling lock debugging due to kernel taint > [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) > [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) > [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) > [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) > [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) > > Add a MODULE_LICENSE() to fix this. > > Fixes: 70b3590f639f ("reset: hi6220: fix modular build") > Cc: Arnd Bergmann <arnd@arndb.de> > Cc: Chen Feng <puck.chen@hisilicon.com> > Signed-off-by: Andreas F?rber <afaerber@suse.de> > --- > drivers/reset/hisilicon/hi6220_reset.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/reset/hisilicon/hi6220_reset.c b/drivers/reset/hisilicon/hi6220_reset.c > index 35ce53edabf9..d5e5229308f2 100644 > --- a/drivers/reset/hisilicon/hi6220_reset.c > +++ b/drivers/reset/hisilicon/hi6220_reset.c > @@ -155,3 +155,5 @@ static int __init hi6220_reset_init(void) > } > > postcore_initcall(hi6220_reset_init); > + > +MODULE_LICENSE("GPL v2"); Thank you, I have the same patch queued already. regards Philipp ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] reset: hisilicon: Fix hi6220 module license @ 2017-05-29 9:46 ` Philipp Zabel 0 siblings, 0 replies; 14+ messages in thread From: Philipp Zabel @ 2017-05-29 9:46 UTC (permalink / raw) To: Andreas Färber Cc: linux-arm-kernel, Arnd Bergmann, Chen Feng, linux-kernel Hi Andreas, On Thu, 2017-05-25 at 11:22 +0200, Andreas Färber wrote: > The hi6220_reset driver fails to load: > > [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. > [ 10.423644] Disabling lock debugging due to kernel taint > [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) > [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) > [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) > [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) > [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) > > Add a MODULE_LICENSE() to fix this. > > Fixes: 70b3590f639f ("reset: hi6220: fix modular build") > Cc: Arnd Bergmann <arnd@arndb.de> > Cc: Chen Feng <puck.chen@hisilicon.com> > Signed-off-by: Andreas Färber <afaerber@suse.de> > --- > drivers/reset/hisilicon/hi6220_reset.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/reset/hisilicon/hi6220_reset.c b/drivers/reset/hisilicon/hi6220_reset.c > index 35ce53edabf9..d5e5229308f2 100644 > --- a/drivers/reset/hisilicon/hi6220_reset.c > +++ b/drivers/reset/hisilicon/hi6220_reset.c > @@ -155,3 +155,5 @@ static int __init hi6220_reset_init(void) > } > > postcore_initcall(hi6220_reset_init); > + > +MODULE_LICENSE("GPL v2"); Thank you, I have the same patch queued already. regards Philipp ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] reset: hisilicon: Fix hi6220 module license 2017-05-29 9:46 ` Philipp Zabel @ 2017-05-29 13:07 ` Andreas Färber -1 siblings, 0 replies; 14+ messages in thread From: Andreas Färber @ 2017-05-29 13:07 UTC (permalink / raw) To: linux-arm-kernel Hi Philipp, Am 29.05.2017 um 11:46 schrieb Philipp Zabel: > On Thu, 2017-05-25 at 11:22 +0200, Andreas F?rber wrote: >> The hi6220_reset driver fails to load: >> >> [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. >> [ 10.423644] Disabling lock debugging due to kernel taint >> [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) >> [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) >> [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) >> [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) >> [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) >> >> Add a MODULE_LICENSE() to fix this. >> >> Fixes: 70b3590f639f ("reset: hi6220: fix modular build") >> Cc: Arnd Bergmann <arnd@arndb.de> >> Cc: Chen Feng <puck.chen@hisilicon.com> >> Signed-off-by: Andreas F?rber <afaerber@suse.de> >> --- >> drivers/reset/hisilicon/hi6220_reset.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/reset/hisilicon/hi6220_reset.c b/drivers/reset/hisilicon/hi6220_reset.c >> index 35ce53edabf9..d5e5229308f2 100644 >> --- a/drivers/reset/hisilicon/hi6220_reset.c >> +++ b/drivers/reset/hisilicon/hi6220_reset.c >> @@ -155,3 +155,5 @@ static int __init hi6220_reset_init(void) >> } >> >> postcore_initcall(hi6220_reset_init); >> + >> +MODULE_LICENSE("GPL v2"); > > Thank you, I have the same patch queued already. Is your tree not integrated into linux-next? If so, please send Stephen Rothwell an email to have that fixed. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany GF: Felix Imend?rffer, Jane Smithard, Graham Norton HRB 21284 (AG N?rnberg) ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] reset: hisilicon: Fix hi6220 module license @ 2017-05-29 13:07 ` Andreas Färber 0 siblings, 0 replies; 14+ messages in thread From: Andreas Färber @ 2017-05-29 13:07 UTC (permalink / raw) To: Philipp Zabel; +Cc: linux-arm-kernel, Arnd Bergmann, Chen Feng, linux-kernel Hi Philipp, Am 29.05.2017 um 11:46 schrieb Philipp Zabel: > On Thu, 2017-05-25 at 11:22 +0200, Andreas Färber wrote: >> The hi6220_reset driver fails to load: >> >> [ 10.423640] hi6220_reset: module license 'unspecified' taints kernel. >> [ 10.423644] Disabling lock debugging due to kernel taint >> [ 10.423735] hi6220_reset: Unknown symbol regmap_write (err 0) >> [ 10.423747] hi6220_reset: Unknown symbol devm_kmalloc (err 0) >> [ 10.423759] hi6220_reset: Unknown symbol syscon_node_to_regmap (err 0) >> [ 10.423768] hi6220_reset: Unknown symbol reset_controller_register (err 0) >> [ 10.423784] hi6220_reset: Unknown symbol __platform_driver_register (err 0) >> >> Add a MODULE_LICENSE() to fix this. >> >> Fixes: 70b3590f639f ("reset: hi6220: fix modular build") >> Cc: Arnd Bergmann <arnd@arndb.de> >> Cc: Chen Feng <puck.chen@hisilicon.com> >> Signed-off-by: Andreas Färber <afaerber@suse.de> >> --- >> drivers/reset/hisilicon/hi6220_reset.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/reset/hisilicon/hi6220_reset.c b/drivers/reset/hisilicon/hi6220_reset.c >> index 35ce53edabf9..d5e5229308f2 100644 >> --- a/drivers/reset/hisilicon/hi6220_reset.c >> +++ b/drivers/reset/hisilicon/hi6220_reset.c >> @@ -155,3 +155,5 @@ static int __init hi6220_reset_init(void) >> } >> >> postcore_initcall(hi6220_reset_init); >> + >> +MODULE_LICENSE("GPL v2"); > > Thank you, I have the same patch queued already. Is your tree not integrated into linux-next? If so, please send Stephen Rothwell an email to have that fixed. Regards, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG Nürnberg) ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH] reset: hisilicon: Fix hi6220 module license 2017-05-29 13:07 ` Andreas Färber @ 2017-05-30 8:43 ` Philipp Zabel -1 siblings, 0 replies; 14+ messages in thread From: Philipp Zabel @ 2017-05-30 8:43 UTC (permalink / raw) To: linux-arm-kernel Hi Andreas, On Mon, 2017-05-29 at 15:07 +0200, Andreas F?rber wrote: [...] > > Thank you, I have the same patch queued already. > > Is your tree not integrated into linux-next? If so, please send Stephen > Rothwell an email to have that fixed. Fixed, sorry for the duplicated effort. regards Philipp ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] reset: hisilicon: Fix hi6220 module license @ 2017-05-30 8:43 ` Philipp Zabel 0 siblings, 0 replies; 14+ messages in thread From: Philipp Zabel @ 2017-05-30 8:43 UTC (permalink / raw) To: Andreas Färber Cc: linux-arm-kernel, Arnd Bergmann, Chen Feng, linux-kernel Hi Andreas, On Mon, 2017-05-29 at 15:07 +0200, Andreas Färber wrote: [...] > > Thank you, I have the same patch queued already. > > Is your tree not integrated into linux-next? If so, please send Stephen > Rothwell an email to have that fixed. Fixed, sorry for the duplicated effort. regards Philipp ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2017-05-30 8:43 UTC | newest] Thread overview: 14+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-05-25 9:22 [PATCH] reset: hisilicon: Fix hi6220 module license Andreas Färber 2017-05-25 9:22 ` Andreas Färber 2017-05-26 3:12 ` Chen Feng 2017-05-26 3:12 ` Chen Feng 2017-05-26 8:44 ` Arnd Bergmann 2017-05-26 8:44 ` Arnd Bergmann 2017-05-26 9:25 ` Andreas Färber 2017-05-26 9:25 ` Andreas Färber 2017-05-29 9:46 ` Philipp Zabel 2017-05-29 9:46 ` Philipp Zabel 2017-05-29 13:07 ` Andreas Färber 2017-05-29 13:07 ` Andreas Färber 2017-05-30 8:43 ` Philipp Zabel 2017-05-30 8:43 ` Philipp Zabel
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.