On Tue, Feb 21, 2017 at 08:16:58PM +0800, Dawei Chien wrote:
> If thermal bank with 4 sensors, thermal driver should read TEMP_MSR3.
>
> However, currently thermal driver would not read TEMP_MSR3 since mt8173
> thermal driver only use 3 sensors on each thermal bank at the same time,
> so this patch would not effect temperature.
> Only if mt mt8173 thermal driver use 4 sensors on any thermal bank, would
> read third sensor two times, and lose fourth sensor of vale.
>
> cc: stable@xxxxxxxxxxxxxxx
> Fixes: b7cf0053738c ("thermal: Add Mediatek thermal driver for mt2701.")
> Change-Id: Ief4f4986d637cf69c2b15cc013d1b93d77c6b615
> Reviewed-by: Matthias Brugger <matthias.bgg@xxxxxxxxx>
> Signed-off-by: Dawei Chien <dawei.chien@xxxxxxxxxxxx>
> ---
> drivers/thermal/mtk_thermal.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/thermal/mtk_thermal.c b/drivers/thermal/mtk_thermal.c
> index 34169c3..c124151 100644
> --- a/drivers/thermal/mtk_thermal.c
> +++ b/drivers/thermal/mtk_thermal.c
> @@ -191,7 +191,7 @@ struct mtk_thermal {
> };
>
> const int mt8173_msr[MT8173_NUM_SENSORS_PER_ZONE] = {
> - TEMP_MSR0, TEMP_MSR1, TEMP_MSR2, TEMP_MSR2
> + TEMP_MSR0, TEMP_MSR1, TEMP_MSR2, TEMP_MSR3
> };
>
> const int mt8173_adcpnp[MT8173_NUM_SENSORS_PER_ZONE] = {
> --
> 1.9.1
<formletter>
This is not the correct way to submit patches for inclusion in the
stable kernel tree. Please read Documentation/stable_kernel_rules.txt
for how to do this properly.
</formletter>