Re: [PATCH V4] thermal: imx: fix for dependency on cpu-freq

2018-11-20 Thread Viresh Kumar
On 21-11-18, 05:28, Anson Huang wrote: > Since there is similar case of DEFER PROBE for the case of > imx_init_from_nvmem_cells > check, should I create another patch of same fix for it in V5 patch set? Send that as a separate patch please. Thanks. -- viresh

Re: [PATCH V4] thermal: imx: fix for dependency on cpu-freq

2018-11-20 Thread Viresh Kumar
On 21-11-18, 05:28, Anson Huang wrote: > Since there is similar case of DEFER PROBE for the case of > imx_init_from_nvmem_cells > check, should I create another patch of same fix for it in V5 patch set? Send that as a separate patch please. Thanks. -- viresh

RE: [PATCH V4] thermal: imx: fix for dependency on cpu-freq

2018-11-20 Thread Anson Huang
kernel.org; > l.st...@pengutronix.de; dl-linux-imx > Subject: Re: [PATCH V4] thermal: imx: fix for dependency on cpu-freq > > On 21-11-18, 05:08, Anson Huang wrote: > > The thermal driver is a standalone driver for monitoring SoC > > temperature by enabling ther

RE: [PATCH V4] thermal: imx: fix for dependency on cpu-freq

2018-11-20 Thread Anson Huang
kernel.org; > l.st...@pengutronix.de; dl-linux-imx > Subject: Re: [PATCH V4] thermal: imx: fix for dependency on cpu-freq > > On 21-11-18, 05:08, Anson Huang wrote: > > The thermal driver is a standalone driver for monitoring SoC > > temperature by enabling ther

Re: [PATCH V4] thermal: imx: fix for dependency on cpu-freq

2018-11-20 Thread Viresh Kumar
On 21-11-18, 05:08, Anson Huang wrote: > +static void imx_thermal_unregister_legacy_cooling(struct imx_thermal_data > *data) > +{ > + cpufreq_cooling_unregister(data->cdev); > + cpufreq_cpu_put(data->policy); > +} > +#else > +static inline int imx_thermal_register_legacy_cooling(struct >

Re: [PATCH V4] thermal: imx: fix for dependency on cpu-freq

2018-11-20 Thread Viresh Kumar
On 21-11-18, 05:08, Anson Huang wrote: > +static void imx_thermal_unregister_legacy_cooling(struct imx_thermal_data > *data) > +{ > + cpufreq_cooling_unregister(data->cdev); > + cpufreq_cpu_put(data->policy); > +} > +#else > +static inline int imx_thermal_register_legacy_cooling(struct >

Re: [PATCH V4] thermal: imx: fix for dependency on cpu-freq

2018-11-20 Thread Viresh Kumar
On 21-11-18, 05:08, Anson Huang wrote: > The thermal driver is a standalone driver for monitoring SoC temperature > by enabling thermal sensor, so it can be enabled even when CONFIG_CPU_FREQ > is NOT set. So remove the dependency with CPU_THERMAL. > > Introduce dummy function of legacy cooling

Re: [PATCH V4] thermal: imx: fix for dependency on cpu-freq

2018-11-20 Thread Viresh Kumar
On 21-11-18, 05:08, Anson Huang wrote: > The thermal driver is a standalone driver for monitoring SoC temperature > by enabling thermal sensor, so it can be enabled even when CONFIG_CPU_FREQ > is NOT set. So remove the dependency with CPU_THERMAL. > > Introduce dummy function of legacy cooling