Re: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-24 Thread Rob Herring
On Tue, Oct 16, 2018 at 6:21 AM Daniel Lezcano wrote: > > On 16/10/2018 05:01, Andy Tang wrote: > > Hi Daniel, > > > > Please see my reply inline. > > > >> -Original Message- > > [ ... ] > > >> I'm proposing to move struct qoriq_tmu_site_regs inside the struct > >> qoriq_sensor. > >> > >>

Re: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-24 Thread Daniel Lezcano
l.org; Rob Herring >> Subject: Re: [PATCH] thermal: qoriq: add multiple sensors support >> >>>>>> The current code is reading the DT in order to get the sensor id >>>>>> and initialize it. IOW, the DT gives the sensors to use. >>>>>>

RE: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-23 Thread Andy Tang
Hi Daniel, > -Original Message- > From: Daniel Lezcano > Sent: 2018年10月16日 19:21 > To: Andy Tang ; rui.zh...@intel.com > Cc: edubez...@gmail.com; linux...@vger.kernel.org; > linux-kernel@vger.kernel.org; Rob Herring > Subject: Re: [PATCH] thermal: qoriq: add mul

Re: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-16 Thread Daniel Lezcano
On 16/10/2018 05:01, Andy Tang wrote: > Hi Daniel, > > Please see my reply inline. > >> -Original Message- [ ... ] >> I'm proposing to move struct qoriq_tmu_site_regs inside the struct >> qoriq_sensor. >> >> >> We end up with: >> >> struct qoriq_sensor { >>struct thermal_zone_device

RE: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-15 Thread Andy Tang
Hi Daniel, Please see my reply inline. > -Original Message- > From: Daniel Lezcano > Sent: 2018年10月15日 16:56 > To: Andy Tang ; rui.zh...@intel.com > Cc: edubez...@gmail.com; linux...@vger.kernel.org; > linux-kernel@vger.kernel.org > Subject: Re: [PATCH] thermal

Re: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-15 Thread Daniel Lezcano
ernel.org; >> linux-kernel@vger.kernel.org >> Subject: Re: [PATCH] thermal: qoriq: add multiple sensors support >> >> >> Hi Yuantian, >> >> >> On 27/09/2018 04:42, andy.t...@nxp.com wrote: >>> From: Yuantian Tang >>> >>> Ther

RE: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-14 Thread Andy Tang
Thanks Daniel, Please see my reply inline. > -Original Message- > From: Daniel Lezcano > Sent: 2018年10月14日 4:43 > To: Andy Tang ; rui.zh...@intel.com > Cc: edubez...@gmail.com; linux...@vger.kernel.org; > linux-kernel@vger.kernel.org > Subject: Re: [PATCH] thermal

Re: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-13 Thread Daniel Lezcano
Hi Yuantian, On 27/09/2018 04:42, andy.t...@nxp.com wrote: > From: Yuantian Tang > > There is only one sensor supported in current driver. > Multiple sensors are existing on Layscape socs. To support them, > covert this driver to support multiple sensors. s/covert/convert/ What about the fo

Re: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-11 Thread Daniel Lezcano
.@intel.com >> Cc: edubez...@gmail.com; daniel.lezc...@linaro.org; >> linux...@vger.kernel.org; linux-kernel@vger.kernel.org; Andy Tang >> >> Subject: [PATCH] thermal: qoriq: add multiple sensors support >> >> From: Yuantian Tang >> >> There is only one

RE: [PATCH] thermal: qoriq: add multiple sensors support

2018-10-11 Thread Andy Tang
nel.org; Andy Tang > > Subject: [PATCH] thermal: qoriq: add multiple sensors support > > From: Yuantian Tang > > There is only one sensor supported in current driver. > Multiple sensors are existing on Layscape socs. To support them, covert > this driver to support multiple

[PATCH] thermal: qoriq: add multiple sensors support

2018-09-26 Thread andy . tang
From: Yuantian Tang There is only one sensor supported in current driver. Multiple sensors are existing on Layscape socs. To support them, covert this driver to support multiple sensors. Signed-off-by: Tang Yuantian --- drivers/thermal/qoriq_thermal.c | 117 +++