thermal: armada: Change sensors trim default value
authorMiquel Raynal <miquel.raynal@free-electrons.com>
Fri, 22 Dec 2017 16:14:11 +0000 (17:14 +0100)
committerEduardo Valentin <edubezval@gmail.com>
Mon, 1 Jan 2018 21:23:04 +0000 (13:23 -0800)
Errata #132698 highlights an error in the default value of Tc trim.
Set this parameter to b'011.

Suggested-by: David Sniatkiwicz <davidsn@marvell.com>
Signed-off-by: Miquel Raynal <miquel.raynal@free-electrons.com>
Reviewed-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
drivers/thermal/armada_thermal.c

index d426b23..e6025d2 100644 (file)
 #define CONTROL0_OFFSET                        0x0
 #define CONTROL1_OFFSET                        0x4
 
+/* Errata fields */
+#define CONTROL0_TSEN_TC_TRIM_MASK     0x7
+#define CONTROL0_TSEN_TC_TRIM_VAL      0x3
+
 /* TSEN refers to the temperature sensors within the AP */
 #define CONTROL0_TSEN_START            BIT(0)
 #define CONTROL0_TSEN_RESET            BIT(1)
@@ -161,6 +165,15 @@ static void armada380_init_sensor(struct platform_device *pdev,
        reg &= ~CONTROL1_EXT_TSEN_SW_RESET;
        writel(reg, priv->control1);
        msleep(10);
+
+       /* Set Tsen Tc Trim to correct default value (errata #132698) */
+       if (priv->control0) {
+               reg = readl_relaxed(priv->control0);
+               reg &= ~CONTROL0_TSEN_TC_TRIM_MASK;
+               reg |= CONTROL0_TSEN_TC_TRIM_VAL;
+               writel(reg, priv->control0);
+               msleep(10);
+       }
 }
 
 static void armada_ap806_init_sensor(struct platform_device *pdev,