Re: [PATCH 1/2] thermal: sun8i: fix r40 ths number

[Vasily Khoruzhick]

On Mon, Jan 6, 2020 at 9:23 AM Yangtao Li <tiny.windzz@xxxxxxxxx> wrote:
>
> According to the spec, r40 has 2 thermal sensors.
> Sensor0 located in the CPU, another in the GPU.
>
> Signed-off-by: Yangtao Li <tiny.windzz@xxxxxxxxx>

You should also add "Fixes" tag here.

> Tested-by: Corentin Labbe <clabbe.montjoie@xxxxxxxxx>
> Tested-on: sun8i-r40-bananapi-m2-ultra
> ---
>  drivers/thermal/sun8i_thermal.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/thermal/sun8i_thermal.c b/drivers/thermal/sun8i_thermal.c
> index 23a5f4aa4be4..c5661d7c3e20 100644
> --- a/drivers/thermal/sun8i_thermal.c
> +++ b/drivers/thermal/sun8i_thermal.c
> @@ -565,7 +565,7 @@ static const struct ths_thermal_chip sun8i_h3_ths = {
>  };
>
>  static const struct ths_thermal_chip sun8i_r40_ths = {
> -       .sensor_num = 3,
> +       .sensor_num = 2,
>         .offset = 251086,
>         .scale = 1130,
>         .has_mod_clk = true,
> --
> 2.17.1
>