mbox series

[0/6] ARM: dts: meson: chip temperature (hwmon) support

Message ID 20190112130649.23838-1-martin.blumenstingl@googlemail.com (mailing list archive)
Headers show
Series ARM: dts: meson: chip temperature (hwmon) support | expand

Message

Martin Blumenstingl Jan. 12, 2019, 1:06 p.m. UTC
Now that the Meson SAR ADC driver can get the chip temperature on
Meson8, Meson8b and Meson8m2 (see [0]) we can add the missing bits to
the .dts{,i} and enable iio-hwmon for all boards where this could be
tested successfully.

This was successfully tested on multiple boards:
- Meson8b Odroid-C1 (me)
- Meson8b EC-100 (me)
- Meson8m2 M8S (not upstream yet, me)
- Meson8m2 MXIII-Plus (an earlier version of this series, Oleg Ivanov)

compile-time dependencies:
- none
runtime dependencies:
- my other series "meson-saradc: temperature sensor support for
  Meson8b/Meson8m2" from [0]
- multi_v7_defconfig already enables CONFIG_SENSORS_IIO_HWMON, so
  there's no patch which selects that.


[0] https://patchwork.kernel.org/cover/10744059/


Martin Blumenstingl (6):
  ARM: dts: meson8m2: use the Meson8m2 specific SAR ADC compatible
  ARM: dts: meson8: add the temperature calibration data for the SAR ADC
  ARM: dts: meson8b: add the temperature calibration data for the SAR
    ADC
  ARM: dts: meson8b: ec100: add iio-hwmon for the chip temperature
  ARM: dts: meson8b: odroidc1: add iio-hwmon for the chip temperature
  ARM: dts: meson8m2: mxiii-plus: add iio-hwmon for the chip temperature

 arch/arm/boot/dts/meson8.dtsi             | 8 ++++++++
 arch/arm/boot/dts/meson8b-ec100.dts       | 5 +++++
 arch/arm/boot/dts/meson8b-odroidc1.dts    | 5 +++++
 arch/arm/boot/dts/meson8b.dtsi            | 8 ++++++++
 arch/arm/boot/dts/meson8m2-mxiii-plus.dts | 5 +++++
 arch/arm/boot/dts/meson8m2.dtsi           | 4 ++++
 6 files changed, 35 insertions(+)

Comments

Kevin Hilman Jan. 17, 2019, 8:26 p.m. UTC | #1
Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:

> Now that the Meson SAR ADC driver can get the chip temperature on
> Meson8, Meson8b and Meson8m2 (see [0]) we can add the missing bits to
> the .dts{,i} and enable iio-hwmon for all boards where this could be
> tested successfully.
>
> This was successfully tested on multiple boards:
> - Meson8b Odroid-C1 (me)
> - Meson8b EC-100 (me)
> - Meson8m2 M8S (not upstream yet, me)
> - Meson8m2 MXIII-Plus (an earlier version of this series, Oleg Ivanov)
>
> compile-time dependencies:
> - none

This seems to depend on your HHI series, which I held off on last cycle
and you said you were planning to resend.  

Kevin
Martin Blumenstingl Jan. 17, 2019, 8:29 p.m. UTC | #2
Hi Kevin,

On Thu, Jan 17, 2019 at 9:26 PM Kevin Hilman <khilman@baylibre.com> wrote:
>
> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:
>
> > Now that the Meson SAR ADC driver can get the chip temperature on
> > Meson8, Meson8b and Meson8m2 (see [0]) we can add the missing bits to
> > the .dts{,i} and enable iio-hwmon for all boards where this could be
> > tested successfully.
> >
> > This was successfully tested on multiple boards:
> > - Meson8b Odroid-C1 (me)
> > - Meson8b EC-100 (me)
> > - Meson8m2 M8S (not upstream yet, me)
> > - Meson8m2 MXIII-Plus (an earlier version of this series, Oleg Ivanov)
> >
> > compile-time dependencies:
> > - none
>
> This seems to depend on your HHI series, which I held off on last cycle
> and you said you were planning to resend.
ouch, indeed

is it OK for you if I re-send this series with the HHI patch included
here - or do you want to have it separately?


Regards
Martin
Kevin Hilman Jan. 17, 2019, 9:04 p.m. UTC | #3
Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:

> Hi Kevin,
>
> On Thu, Jan 17, 2019 at 9:26 PM Kevin Hilman <khilman@baylibre.com> wrote:
>>
>> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes:
>>
>> > Now that the Meson SAR ADC driver can get the chip temperature on
>> > Meson8, Meson8b and Meson8m2 (see [0]) we can add the missing bits to
>> > the .dts{,i} and enable iio-hwmon for all boards where this could be
>> > tested successfully.
>> >
>> > This was successfully tested on multiple boards:
>> > - Meson8b Odroid-C1 (me)
>> > - Meson8b EC-100 (me)
>> > - Meson8m2 M8S (not upstream yet, me)
>> > - Meson8m2 MXIII-Plus (an earlier version of this series, Oleg Ivanov)
>> >
>> > compile-time dependencies:
>> > - none
>>
>> This seems to depend on your HHI series, which I held off on last cycle
>> and you said you were planning to resend.
> ouch, indeed
>
> is it OK for you if I re-send this series with the HHI patch included
> here - or do you want to have it separately?

Feel free to include it.

Thanks,

Kevin