* [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE @ 2023-05-24 15:34 Zhipeng Wang 2023-05-24 10:45 ` Viresh Kumar ` (2 more replies) 0 siblings, 3 replies; 9+ messages in thread From: Zhipeng Wang @ 2023-05-24 15:34 UTC (permalink / raw) To: rafael, viresh.kumar; +Cc: linux-pm, linux-kernel Add MODULE_LICENSE to support building as module. Signed-off-by: Zhipeng Wang <zhipeng.wang_1@nxp.com> --- drivers/cpufreq/cpufreq-dt-platdev.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c index 338cf6cc6596..54529aa16d53 100644 --- a/drivers/cpufreq/cpufreq-dt-platdev.c +++ b/drivers/cpufreq/cpufreq-dt-platdev.c @@ -214,3 +214,4 @@ static int __init cpufreq_dt_platdev_init(void) sizeof(struct cpufreq_dt_platform_data))); } core_initcall(cpufreq_dt_platdev_init); +MODULE_LICENSE("GPL"); -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE 2023-05-24 15:34 [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE Zhipeng Wang @ 2023-05-24 10:45 ` Viresh Kumar 2023-05-24 11:32 ` [EXT] " Zhipeng Wang 2023-05-24 15:34 ` [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module Zhipeng Wang 2023-06-02 4:12 ` [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE Viresh Kumar 2 siblings, 1 reply; 9+ messages in thread From: Viresh Kumar @ 2023-05-24 10:45 UTC (permalink / raw) To: Zhipeng Wang; +Cc: rafael, linux-pm, linux-kernel On 24-05-23, 15:34, Zhipeng Wang wrote: > Add MODULE_LICENSE to support building as module. > > Signed-off-by: Zhipeng Wang <zhipeng.wang_1@nxp.com> > --- > drivers/cpufreq/cpufreq-dt-platdev.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c > index 338cf6cc6596..54529aa16d53 100644 > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > @@ -214,3 +214,4 @@ static int __init cpufreq_dt_platdev_init(void) > sizeof(struct cpufreq_dt_platform_data))); > } > core_initcall(cpufreq_dt_platdev_init); > +MODULE_LICENSE("GPL"); Any specific reason why you need this as a module ? Just curious. -- viresh ^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [EXT] Re: [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE 2023-05-24 10:45 ` Viresh Kumar @ 2023-05-24 11:32 ` Zhipeng Wang 0 siblings, 0 replies; 9+ messages in thread From: Zhipeng Wang @ 2023-05-24 11:32 UTC (permalink / raw) To: Viresh Kumar; +Cc: rafael, linux-pm, linux-kernel I am working on enabling support for Google's Generic Kernel Image (GKI) project on our boards (mek_8qxp and 8qm). The cpufreq-dt driver already supports modularity, and the cpufreq-dt-platdev is also required. From my understanding, cpufreq-dt-platdev is not necessarily required to be built in, as it does not affect the basic boot process. Therefore, I would like to make it support building as a module. BRs, Zhipeng > -----Original Message----- > From: Viresh Kumar <viresh.kumar@linaro.org> > Sent: 2023年5月24日 18:45 > To: Zhipeng Wang <zhipeng.wang_1@nxp.com> > Cc: rafael@kernel.org; linux-pm@vger.kernel.org; linux-kernel@vger.kernel.org > Subject: [EXT] Re: [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE > > Caution: This is an external email. Please take care when clicking links or > opening attachments. When in doubt, report the message using the 'Report > this email' button > > > On 24-05-23, 15:34, Zhipeng Wang wrote: > > Add MODULE_LICENSE to support building as module. > > > > Signed-off-by: Zhipeng Wang <zhipeng.wang_1@nxp.com> > > --- > > drivers/cpufreq/cpufreq-dt-platdev.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c > > b/drivers/cpufreq/cpufreq-dt-platdev.c > > index 338cf6cc6596..54529aa16d53 100644 > > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > > @@ -214,3 +214,4 @@ static int __init cpufreq_dt_platdev_init(void) > > sizeof(struct > > cpufreq_dt_platform_data))); } > > core_initcall(cpufreq_dt_platdev_init); > > +MODULE_LICENSE("GPL"); > > Any specific reason why you need this as a module ? Just curious. > > -- > viresh ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module 2023-05-24 15:34 [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE Zhipeng Wang 2023-05-24 10:45 ` Viresh Kumar @ 2023-05-24 15:34 ` Zhipeng Wang 2023-08-29 18:00 ` Aurelien Jarno 2023-06-02 4:12 ` [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE Viresh Kumar 2 siblings, 1 reply; 9+ messages in thread From: Zhipeng Wang @ 2023-05-24 15:34 UTC (permalink / raw) To: rafael, viresh.kumar; +Cc: linux-pm, linux-kernel Make the cpufreq platdev driver as tristate so that it can be built as loadable module. Signed-off-by: Zhipeng Wang <zhipeng.wang_1@nxp.com> --- drivers/cpufreq/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig index 2c839bd2b051..dda3a78bfd5c 100644 --- a/drivers/cpufreq/Kconfig +++ b/drivers/cpufreq/Kconfig @@ -218,7 +218,7 @@ config CPUFREQ_DT If in doubt, say N. config CPUFREQ_DT_PLATDEV - bool + tristate "Generic DT based cpufreq platdev driver" help This adds a generic DT based cpufreq platdev driver for frequency management. This creates a 'cpufreq-dt' platform device, on the -- 2.25.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module 2023-05-24 15:34 ` [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module Zhipeng Wang @ 2023-08-29 18:00 ` Aurelien Jarno 2023-08-30 3:09 ` [EXT] " Zhipeng Wang 0 siblings, 1 reply; 9+ messages in thread From: Aurelien Jarno @ 2023-08-29 18:00 UTC (permalink / raw) To: Zhipeng Wang; +Cc: rafael, viresh.kumar, linux-pm, linux-kernel Hi, On 2023-05-24 15:34, Zhipeng Wang wrote: > Make the cpufreq platdev driver as tristate so that it can be built > as loadable module. > > Signed-off-by: Zhipeng Wang <zhipeng.wang_1@nxp.com> > --- > drivers/cpufreq/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig > index 2c839bd2b051..dda3a78bfd5c 100644 > --- a/drivers/cpufreq/Kconfig > +++ b/drivers/cpufreq/Kconfig > @@ -218,7 +218,7 @@ config CPUFREQ_DT > If in doubt, say N. > > config CPUFREQ_DT_PLATDEV > - bool > + tristate "Generic DT based cpufreq platdev driver" > help > This adds a generic DT based cpufreq platdev driver for frequency > management. This creates a 'cpufreq-dt' platform device, on the It seems that this new module is missing support for autoloading, causing issues with CPUFREQ_DT=m. Before that change, CPUFREQ_DT_PLATDEV ended up selected as =y. Now, CPUFREQ_DT_PLATDEV ends up selected as =m. As it is not autoloaded, it means that cpufreq-dt is not working anymore unless manually running "modprobe cpufreq-dt-platdev". Could you therefore please add support for autoloading? Thanks, Aurelien -- Aurelien Jarno GPG: 4096R/1DDD8C9B aurelien@aurel32.net http://aurel32.net ^ permalink raw reply [flat|nested] 9+ messages in thread
* RE: [EXT] Re: [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module 2023-08-29 18:00 ` Aurelien Jarno @ 2023-08-30 3:09 ` Zhipeng Wang 2023-08-30 5:08 ` Aurelien Jarno 0 siblings, 1 reply; 9+ messages in thread From: Zhipeng Wang @ 2023-08-30 3:09 UTC (permalink / raw) To: Aurelien Jarno; +Cc: rafael, viresh.kumar, linux-pm, linux-kernel Hi Aurelien, I'm not sure why we need to make cpufreq-dt-platdev autoloading, does it make sense? As I understand it we need to actively modprobe cpufreq-dt-platdev, why not? BRs, Zhipeng > -----Original Message----- > From: Aurelien Jarno <aurelien@aurel32.net> > Sent: 2023年8月30日 2:01 > To: Zhipeng Wang <zhipeng.wang_1@nxp.com> > Cc: rafael@kernel.org; viresh.kumar@linaro.org; linux-pm@vger.kernel.org; > linux-kernel@vger.kernel.org > Subject: [EXT] Re: [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq > platdev driver as module > > Caution: This is an external email. Please take care when clicking links or opening > attachments. When in doubt, report the message using the 'Report this email' > button > > > Hi, > > On 2023-05-24 15:34, Zhipeng Wang wrote: > > Make the cpufreq platdev driver as tristate so that it can be built as > > loadable module. > > > > Signed-off-by: Zhipeng Wang <zhipeng.wang_1@nxp.com> > > --- > > drivers/cpufreq/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig index > > 2c839bd2b051..dda3a78bfd5c 100644 > > --- a/drivers/cpufreq/Kconfig > > +++ b/drivers/cpufreq/Kconfig > > @@ -218,7 +218,7 @@ config CPUFREQ_DT > > If in doubt, say N. > > > > config CPUFREQ_DT_PLATDEV > > - bool > > + tristate "Generic DT based cpufreq platdev driver" > > help > > This adds a generic DT based cpufreq platdev driver for frequency > > management. This creates a 'cpufreq-dt' platform device, on > > the > > It seems that this new module is missing support for autoloading, causing issues > with CPUFREQ_DT=m. > > Before that change, CPUFREQ_DT_PLATDEV ended up selected as =y. Now, > CPUFREQ_DT_PLATDEV ends up selected as =m. As it is not autoloaded, it means > that cpufreq-dt is not working anymore unless manually running "modprobe > cpufreq-dt-platdev". > > Could you therefore please add support for autoloading? > > Thanks, > Aurelien > > -- > Aurelien Jarno GPG: 4096R/1DDD8C9B > aurelien@aurel32.net > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Faurel32. > net%2F&data=05%7C01%7Czhipeng.wang_1%40nxp.com%7C84e6deca09fb4 > e86857608dba8b9df5f%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0 > %7C638289288492430681%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLj > AwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C% > 7C%7C&sdata=02iXE8HQXHNXRiA7SMtl%2B5ldGiVDF%2F7GtO%2F0DtAP0EQ > %3D&reserved=0 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EXT] Re: [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module 2023-08-30 3:09 ` [EXT] " Zhipeng Wang @ 2023-08-30 5:08 ` Aurelien Jarno 2023-08-31 10:50 ` Viresh Kumar 0 siblings, 1 reply; 9+ messages in thread From: Aurelien Jarno @ 2023-08-30 5:08 UTC (permalink / raw) To: Zhipeng Wang; +Cc: rafael, viresh.kumar, linux-pm, linux-kernel Hi, On 2023-08-30 03:09, Zhipeng Wang wrote: > Hi Aurelien, > > I'm not sure why we need to make cpufreq-dt-platdev autoloading, does it make sense? As I understand it we need to actively modprobe cpufreq-dt-platdev, why not? This is needed to recover the old behavior, where if a CPU supports cpufreq-dt, it get loaded automatically. Now users has to modprobe cpufreq-dt-platdev explicitly to get cpufreq-dt to work. This seems like a regression to me. Of course one can explicitly make CPUFREQ_DT_PLATDEV=y, but the default is to build cpufreq-dt-platdev as module when cpufreq-dt is built as module. Regards Aurelien > > -----Original Message----- > > From: Aurelien Jarno <aurelien@aurel32.net> > > Sent: 2023年8月30日 2:01 > > To: Zhipeng Wang <zhipeng.wang_1@nxp.com> > > Cc: rafael@kernel.org; viresh.kumar@linaro.org; linux-pm@vger.kernel.org; > > linux-kernel@vger.kernel.org > > Subject: [EXT] Re: [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq > > platdev driver as module > > > > Caution: This is an external email. Please take care when clicking links or opening > > attachments. When in doubt, report the message using the 'Report this email' > > button > > > > > > Hi, > > > > On 2023-05-24 15:34, Zhipeng Wang wrote: > > > Make the cpufreq platdev driver as tristate so that it can be built as > > > loadable module. > > > > > > Signed-off-by: Zhipeng Wang <zhipeng.wang_1@nxp.com> > > > --- > > > drivers/cpufreq/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig index > > > 2c839bd2b051..dda3a78bfd5c 100644 > > > --- a/drivers/cpufreq/Kconfig > > > +++ b/drivers/cpufreq/Kconfig > > > @@ -218,7 +218,7 @@ config CPUFREQ_DT > > > If in doubt, say N. > > > > > > config CPUFREQ_DT_PLATDEV > > > - bool > > > + tristate "Generic DT based cpufreq platdev driver" > > > help > > > This adds a generic DT based cpufreq platdev driver for frequency > > > management. This creates a 'cpufreq-dt' platform device, on > > > the > > > > It seems that this new module is missing support for autoloading, causing issues > > with CPUFREQ_DT=m. > > > > Before that change, CPUFREQ_DT_PLATDEV ended up selected as =y. Now, > > CPUFREQ_DT_PLATDEV ends up selected as =m. As it is not autoloaded, it means > > that cpufreq-dt is not working anymore unless manually running "modprobe > > cpufreq-dt-platdev". > > > > Could you therefore please add support for autoloading? > > > > Thanks, > > Aurelien > > > > -- > > Aurelien Jarno GPG: 4096R/1DDD8C9B > > aurelien@aurel32.net > > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Faurel32. > > net%2F&data=05%7C01%7Czhipeng.wang_1%40nxp.com%7C84e6deca09fb4 > > e86857608dba8b9df5f%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0 > > %7C638289288492430681%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLj > > AwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C% > > 7C%7C&sdata=02iXE8HQXHNXRiA7SMtl%2B5ldGiVDF%2F7GtO%2F0DtAP0EQ > > %3D&reserved=0 -- Aurelien Jarno GPG: 4096R/1DDD8C9B aurelien@aurel32.net http://aurel32.net ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [EXT] Re: [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module 2023-08-30 5:08 ` Aurelien Jarno @ 2023-08-31 10:50 ` Viresh Kumar 0 siblings, 0 replies; 9+ messages in thread From: Viresh Kumar @ 2023-08-31 10:50 UTC (permalink / raw) To: Zhipeng Wang, rafael, linux-pm, linux-kernel On 30-08-23, 07:08, Aurelien Jarno wrote: > Hi, > > On 2023-08-30 03:09, Zhipeng Wang wrote: > > Hi Aurelien, > > > > I'm not sure why we need to make cpufreq-dt-platdev autoloading, does it make sense? As I understand it we need to actively modprobe cpufreq-dt-platdev, why not? > > This is needed to recover the old behavior, where if a CPU supports > cpufreq-dt, it get loaded automatically. Now users has to modprobe > cpufreq-dt-platdev explicitly to get cpufreq-dt to work. This seems like > a regression to me. > > Of course one can explicitly make CPUFREQ_DT_PLATDEV=y, but the default > is to build cpufreq-dt-platdev as module when cpufreq-dt is built as > module. Feel free to send me a patch to make this auto-load. -- viresh ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE 2023-05-24 15:34 [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE Zhipeng Wang 2023-05-24 10:45 ` Viresh Kumar 2023-05-24 15:34 ` [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module Zhipeng Wang @ 2023-06-02 4:12 ` Viresh Kumar 2 siblings, 0 replies; 9+ messages in thread From: Viresh Kumar @ 2023-06-02 4:12 UTC (permalink / raw) To: Zhipeng Wang; +Cc: rafael, linux-pm, linux-kernel On 24-05-23, 15:34, Zhipeng Wang wrote: > Add MODULE_LICENSE to support building as module. > > Signed-off-by: Zhipeng Wang <zhipeng.wang_1@nxp.com> > --- > drivers/cpufreq/cpufreq-dt-platdev.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c > index 338cf6cc6596..54529aa16d53 100644 > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > @@ -214,3 +214,4 @@ static int __init cpufreq_dt_platdev_init(void) > sizeof(struct cpufreq_dt_platform_data))); > } > core_initcall(cpufreq_dt_platdev_init); > +MODULE_LICENSE("GPL"); Merged both patches and applied. Thanks. -- viresh ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2023-08-31 10:51 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-05-24 15:34 [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE Zhipeng Wang 2023-05-24 10:45 ` Viresh Kumar 2023-05-24 11:32 ` [EXT] " Zhipeng Wang 2023-05-24 15:34 ` [PATCH 2/2] cpufreq: dt-platdev: Support building cpufreq platdev driver as module Zhipeng Wang 2023-08-29 18:00 ` Aurelien Jarno 2023-08-30 3:09 ` [EXT] " Zhipeng Wang 2023-08-30 5:08 ` Aurelien Jarno 2023-08-31 10:50 ` Viresh Kumar 2023-06-02 4:12 ` [PATCH 1/2] cpufreq: dt-platdev: Add MODULE_LICENSE Viresh Kumar
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).