Re: [PATCH] cpufreq: dt: fix oops on armada37xx

2020-06-21 Thread Viresh Kumar
On 20-06-20, 17:44, Ivan Kokshaysky wrote: > Commit 0c868627e617e43a295d8 (cpufreq: dt: Allow platform specific > intermediate callbacks) added two function pointers to the > struct cpufreq_dt_platform_data. However, armada37xx_cpufreq_driver_init() > has this struct (pdata) located on the stack an

Re: [PATCH] cpufreq: dt: fix oops on armada37xx

2020-06-20 Thread Ivan Kokshaysky
On Sat, Jun 20, 2020 at 07:54:38PM +0200, Andrew Lunn wrote: > On Sat, Jun 20, 2020 at 05:44:49PM +0100, Ivan Kokshaysky wrote: > > Commit 0c868627e617e43a295d8 (cpufreq: dt: Allow platform specific > > intermediate callbacks) added two function pointers to the > > struct cpufreq_dt_platform_data.

Re: [PATCH] cpufreq: dt: fix oops on armada37xx

2020-06-20 Thread Andrew Lunn
On Sat, Jun 20, 2020 at 05:44:49PM +0100, Ivan Kokshaysky wrote: > Commit 0c868627e617e43a295d8 (cpufreq: dt: Allow platform specific > intermediate callbacks) added two function pointers to the > struct cpufreq_dt_platform_data. However, armada37xx_cpufreq_driver_init() > has this struct (pdata) l

[PATCH] cpufreq: dt: fix oops on armada37xx

2020-06-20 Thread Ivan Kokshaysky
Commit 0c868627e617e43a295d8 (cpufreq: dt: Allow platform specific intermediate callbacks) added two function pointers to the struct cpufreq_dt_platform_data. However, armada37xx_cpufreq_driver_init() has this struct (pdata) located on the stack and uses only "suspend" and "resume" fields. So these