thermal: armada: rename the initialization routine
authorMiquel Raynal <miquel.raynal@bootlin.com>
Mon, 16 Jul 2018 14:41:47 +0000 (16:41 +0200)
committerEduardo Valentin <edubezval@gmail.com>
Fri, 27 Jul 2018 21:43:05 +0000 (14:43 -0700)
Calling a hook ->init_sensor() while what is initialized is the IP
itself and not the sensors is misleading. Rename the hook ->init() to
avoid any confusion in later work bringing multi-sensors support.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Signed-off-by: Eduardo Valentin <edubezval@gmail.com>
drivers/thermal/armada_thermal.c

index 6cc53eef651d8dfad19a1ca27d38d503d6556d97..2f2150f9639fa054716e4df9e2f21df40440c048 100644 (file)
@@ -73,9 +73,9 @@ struct armada_thermal_priv {
 };
 
 struct armada_thermal_data {
-       /* Initialize the sensor */
-       void (*init_sensor)(struct platform_device *pdev,
-                           struct armada_thermal_priv *);
+       /* Initialize the thermal IC */
+       void (*init)(struct platform_device *pdev,
+                    struct armada_thermal_priv *priv);
 
        /* Test for a valid sensor value (optional) */
        bool (*is_valid)(struct armada_thermal_priv *);
@@ -94,8 +94,8 @@ struct armada_thermal_data {
        bool needs_control0;
 };
 
-static void armadaxp_init_sensor(struct platform_device *pdev,
-                                struct armada_thermal_priv *priv)
+static void armadaxp_init(struct platform_device *pdev,
+                         struct armada_thermal_priv *priv)
 {
        u32 reg;
 
@@ -117,8 +117,8 @@ static void armadaxp_init_sensor(struct platform_device *pdev,
        writel(reg, priv->status);
 }
 
-static void armada370_init_sensor(struct platform_device *pdev,
-                                 struct armada_thermal_priv *priv)
+static void armada370_init(struct platform_device *pdev,
+                          struct armada_thermal_priv *priv)
 {
        u32 reg;
 
@@ -136,8 +136,8 @@ static void armada370_init_sensor(struct platform_device *pdev,
        msleep(10);
 }
 
-static void armada375_init_sensor(struct platform_device *pdev,
-                                 struct armada_thermal_priv *priv)
+static void armada375_init(struct platform_device *pdev,
+                          struct armada_thermal_priv *priv)
 {
        u32 reg;
 
@@ -164,8 +164,8 @@ static void armada_wait_sensor_validity(struct armada_thermal_priv *priv)
                                   STATUS_POLL_TIMEOUT_US);
 }
 
-static void armada380_init_sensor(struct platform_device *pdev,
-                                 struct armada_thermal_priv *priv)
+static void armada380_init(struct platform_device *pdev,
+                          struct armada_thermal_priv *priv)
 {
        u32 reg = readl_relaxed(priv->control1);
 
@@ -186,8 +186,8 @@ static void armada380_init_sensor(struct platform_device *pdev,
        armada_wait_sensor_validity(priv);
 }
 
-static void armada_ap806_init_sensor(struct platform_device *pdev,
-                                    struct armada_thermal_priv *priv)
+static void armada_ap806_init(struct platform_device *pdev,
+                             struct armada_thermal_priv *priv)
 {
        u32 reg;
 
@@ -247,7 +247,7 @@ static struct thermal_zone_device_ops ops = {
 };
 
 static const struct armada_thermal_data armadaxp_data = {
-       .init_sensor = armadaxp_init_sensor,
+       .init = armadaxp_init,
        .temp_shift = 10,
        .temp_mask = 0x1ff,
        .coef_b = 3153000000ULL,
@@ -257,7 +257,7 @@ static const struct armada_thermal_data armadaxp_data = {
 
 static const struct armada_thermal_data armada370_data = {
        .is_valid = armada_is_valid,
-       .init_sensor = armada370_init_sensor,
+       .init = armada370_init,
        .is_valid_bit = BIT(9),
        .temp_shift = 10,
        .temp_mask = 0x1ff,
@@ -268,7 +268,7 @@ static const struct armada_thermal_data armada370_data = {
 
 static const struct armada_thermal_data armada375_data = {
        .is_valid = armada_is_valid,
-       .init_sensor = armada375_init_sensor,
+       .init = armada375_init,
        .is_valid_bit = BIT(10),
        .temp_shift = 0,
        .temp_mask = 0x1ff,
@@ -280,7 +280,7 @@ static const struct armada_thermal_data armada375_data = {
 
 static const struct armada_thermal_data armada380_data = {
        .is_valid = armada_is_valid,
-       .init_sensor = armada380_init_sensor,
+       .init = armada380_init,
        .is_valid_bit = BIT(10),
        .temp_shift = 0,
        .temp_mask = 0x3ff,
@@ -292,7 +292,7 @@ static const struct armada_thermal_data armada380_data = {
 
 static const struct armada_thermal_data armada_ap806_data = {
        .is_valid = armada_is_valid,
-       .init_sensor = armada_ap806_init_sensor,
+       .init = armada_ap806_init,
        .is_valid_bit = BIT(16),
        .temp_shift = 0,
        .temp_mask = 0x3ff,
@@ -306,7 +306,7 @@ static const struct armada_thermal_data armada_ap806_data = {
 
 static const struct armada_thermal_data armada_cp110_data = {
        .is_valid = armada_is_valid,
-       .init_sensor = armada380_init_sensor,
+       .init = armada380_init,
        .is_valid_bit = BIT(10),
        .temp_shift = 0,
        .temp_mask = 0x3ff,
@@ -429,7 +429,7 @@ static int armada_thermal_probe(struct platform_device *pdev)
                priv->control1 = control + CONTROL1_OFFSET;
        }
 
-       priv->data->init_sensor(pdev, priv);
+       priv->data->init(pdev, priv);
 
        thermal = thermal_zone_device_register(priv->zone_name, 0, 0, priv,
                                               &ops, NULL, 0, 0);