[v2,0/3] thermal: tsens: bugfixes, cleanups

Message

Amit Kucheria July 26, 2018, 10:33 a.m. UTC

Here is another series of tsens-related bugfixes and cleanups to prepare
for IRQ support, among other things. It applies on top of another series[1]
that did some code consolidation to add support for sdm845.

[1] https://lore.kernel.org/lkml/[e-mail]/T/#t

Changes since v1:
- Fixup a couple of typos
- Add review tag

Amit Kucheria (3):
  thermal: tsens: Rename variable
  thermal: tsens: switch from of_iomap() to devm_ioremap_resource()
  thermal: tsens: Fix negative temperature reporting

 drivers/thermal/qcom/tsens-common.c | 19 ++++++++++---------
 drivers/thermal/qcom/tsens-v2.c     | 23 ++++++++++-------------
 2 files changed, 20 insertions(+), 22 deletions(-)

Comments

Eduardo Valentin July 26, 2018, 9:03 p.m. UTC | #1

Hey Amit,


On Thu, Jul 26, 2018 at 04:03:07PM +0530, Amit Kucheria wrote:
> Here is another series of tsens-related bugfixes and cleanups to prepare
> for IRQ support, among other things. It applies on top of another series[1]
> that did some code consolidation to add support for sdm845.
> 
> [1] https://lore.kernel.org/lkml/[e-mail]/T/#t


Given it is already rc7 time, even if this is fixes series, I will only consider this for the merge window.

> 
> Changes since v1:
> - Fixup a couple of typos
> - Add review tag
> 
> Amit Kucheria (3):
>   thermal: tsens: Rename variable
>   thermal: tsens: switch from of_iomap() to devm_ioremap_resource()
>   thermal: tsens: Fix negative temperature reporting
> 
>  drivers/thermal/qcom/tsens-common.c | 19 ++++++++++---------
>  drivers/thermal/qcom/tsens-v2.c     | 23 ++++++++++-------------
>  2 files changed, 20 insertions(+), 22 deletions(-)
> 
> -- 
> 2.17.GIT
>

Amit Kucheria July 26, 2018, 10:12 p.m. UTC | #2

On Fri, Jul 27, 2018 at 2:38 AM Eduardo Valentin <[e-mail]> wrote:
>
> Hey Amit,
>
>
> On Thu, Jul 26, 2018 at 04:03:07PM +0530, Amit Kucheria wrote:
> > Here is another series of tsens-related bugfixes and cleanups to prepare
> > for IRQ support, among other things. It applies on top of another series[1]
> > that did some code consolidation to add support for sdm845.
> >
> > [1] https://lore.kernel.org/lkml/[e-mail]/T/#t
>
>
> Given it is already rc7 time, even if this is fixes series, I will only consider this for the merge window.

Sure, I wasn't expecting to land this series in 4.19. I'm hoping that
[1] can be landed though.

Regards,
Amit