diff mbox series

[v6,4/7] thermal: tsens: Rename tsens-8996 to tsens-v2 for reuse

Message ID 910ed3e440a5612f4113318c66ae2395f53abaae.1531135999.git.amit.kucheria@linaro.org
State Superseded
Headers show
Series None | expand

Commit Message

Amit Kucheria July 9, 2018, 11:43 a.m. UTC
The TSENS block inside the 8996 is internally classified as version 2 of
the IP. Several other SoC families use this block and can share this code.

We rename get_temp() to reflect that it can be used across the v2 family.

Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>

---
 drivers/thermal/qcom/Makefile                     |  2 +-
 drivers/thermal/qcom/{tsens-8996.c => tsens-v2.c} | 26 ++++++++---------------
 2 files changed, 10 insertions(+), 18 deletions(-)
 rename drivers/thermal/qcom/{tsens-8996.c => tsens-v2.c} (66%)

-- 
2.7.4

Comments

Bjorn Andersson July 11, 2018, 4:39 p.m. UTC | #1
On Mon 09 Jul 04:43 PDT 2018, Amit Kucheria wrote:

> The TSENS block inside the 8996 is internally classified as version 2 of

> the IP. Several other SoC families use this block and can share this code.

> 

> We rename get_temp() to reflect that it can be used across the v2 family.

> 

> Signed-off-by: Amit Kucheria <amit.kucheria@linaro.org>


Reviewed-by: Bjorn Andersson <bjorn.andersson@linaro.org>


Regards,
Bjorn

> ---

>  drivers/thermal/qcom/Makefile                     |  2 +-

>  drivers/thermal/qcom/{tsens-8996.c => tsens-v2.c} | 26 ++++++++---------------

>  2 files changed, 10 insertions(+), 18 deletions(-)

>  rename drivers/thermal/qcom/{tsens-8996.c => tsens-v2.c} (66%)

> 

> diff --git a/drivers/thermal/qcom/Makefile b/drivers/thermal/qcom/Makefile

> index 2cc2193..a821929 100644

> --- a/drivers/thermal/qcom/Makefile

> +++ b/drivers/thermal/qcom/Makefile

> @@ -1,2 +1,2 @@

>  obj-$(CONFIG_QCOM_TSENS)	+= qcom_tsens.o

> -qcom_tsens-y			+= tsens.o tsens-common.o tsens-8916.o tsens-8974.o tsens-8960.o tsens-8996.o

> +qcom_tsens-y			+= tsens.o tsens-common.o tsens-8916.o tsens-8974.o tsens-8960.o tsens-v2.o

> diff --git a/drivers/thermal/qcom/tsens-8996.c b/drivers/thermal/qcom/tsens-v2.c

> similarity index 66%

> rename from drivers/thermal/qcom/tsens-8996.c

> rename to drivers/thermal/qcom/tsens-v2.c

> index 60765b1..34ba6c7 100644

> --- a/drivers/thermal/qcom/tsens-8996.c

> +++ b/drivers/thermal/qcom/tsens-v2.c

> @@ -1,27 +1,18 @@

> +// SPDX-License-Identifier: GPL-2.0

>  /*

>   * Copyright (c) 2015, The Linux Foundation. All rights reserved.

> - *

> - * This program is free software; you can redistribute it and/or modify

> - * it under the terms of the GNU General Public License version 2 and

> - * only version 2 as published by the Free Software Foundation.

> - *

> - * This program is distributed in the hope that it will be useful,

> - * but WITHOUT ANY WARRANTY; without even the implied warranty of

> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

> - * GNU General Public License for more details.

> - *

> + * Copyright (c) 2018, Linaro Limited

>   */

>  

> -#include <linux/platform_device.h>

>  #include <linux/regmap.h>

>  #include "tsens.h"

>  

> -#define STATUS_OFFSET	0x10a0

> -#define LAST_TEMP_MASK	0xfff

> +#define STATUS_OFFSET		0xa0

> +#define LAST_TEMP_MASK		0xfff

>  #define STATUS_VALID_BIT	BIT(21)

>  #define CODE_SIGN_BIT		BIT(11)

>  

> -static int get_temp_8996(struct tsens_device *tmdev, int id, int *temp)

> +static int get_temp_tsens_v2(struct tsens_device *tmdev, int id, int *temp)

>  {

>  	struct tsens_sensor *s = &tmdev->sensor[id];

>  	u32 code;

> @@ -73,12 +64,13 @@ static int get_temp_8996(struct tsens_device *tmdev, int id, int *temp)

>  	return 0;

>  }

>  

> -static const struct tsens_ops ops_8996 = {

> +static const struct tsens_ops ops_generic_v2 = {

>  	.init		= init_common,

> -	.get_temp	= get_temp_8996,

> +	.get_temp	= get_temp_tsens_v2,

>  };

>  

>  const struct tsens_data data_8996 = {

>  	.num_sensors	= 13,

> -	.ops		= &ops_8996,

> +	.ops		= &ops_generic_v2,

>  };

> +

> -- 

> 2.7.4

>
diff mbox series

Patch

diff --git a/drivers/thermal/qcom/Makefile b/drivers/thermal/qcom/Makefile
index 2cc2193..a821929 100644
--- a/drivers/thermal/qcom/Makefile
+++ b/drivers/thermal/qcom/Makefile
@@ -1,2 +1,2 @@ 
 obj-$(CONFIG_QCOM_TSENS)	+= qcom_tsens.o
-qcom_tsens-y			+= tsens.o tsens-common.o tsens-8916.o tsens-8974.o tsens-8960.o tsens-8996.o
+qcom_tsens-y			+= tsens.o tsens-common.o tsens-8916.o tsens-8974.o tsens-8960.o tsens-v2.o
diff --git a/drivers/thermal/qcom/tsens-8996.c b/drivers/thermal/qcom/tsens-v2.c
similarity index 66%
rename from drivers/thermal/qcom/tsens-8996.c
rename to drivers/thermal/qcom/tsens-v2.c
index 60765b1..34ba6c7 100644
--- a/drivers/thermal/qcom/tsens-8996.c
+++ b/drivers/thermal/qcom/tsens-v2.c
@@ -1,27 +1,18 @@ 
+// SPDX-License-Identifier: GPL-2.0
 /*
  * Copyright (c) 2015, The Linux Foundation. All rights reserved.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 and
- * only version 2 as published by the Free Software Foundation.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
+ * Copyright (c) 2018, Linaro Limited
  */
 
-#include <linux/platform_device.h>
 #include <linux/regmap.h>
 #include "tsens.h"
 
-#define STATUS_OFFSET	0x10a0
-#define LAST_TEMP_MASK	0xfff
+#define STATUS_OFFSET		0xa0
+#define LAST_TEMP_MASK		0xfff
 #define STATUS_VALID_BIT	BIT(21)
 #define CODE_SIGN_BIT		BIT(11)
 
-static int get_temp_8996(struct tsens_device *tmdev, int id, int *temp)
+static int get_temp_tsens_v2(struct tsens_device *tmdev, int id, int *temp)
 {
 	struct tsens_sensor *s = &tmdev->sensor[id];
 	u32 code;
@@ -73,12 +64,13 @@  static int get_temp_8996(struct tsens_device *tmdev, int id, int *temp)
 	return 0;
 }
 
-static const struct tsens_ops ops_8996 = {
+static const struct tsens_ops ops_generic_v2 = {
 	.init		= init_common,
-	.get_temp	= get_temp_8996,
+	.get_temp	= get_temp_tsens_v2,
 };
 
 const struct tsens_data data_8996 = {
 	.num_sensors	= 13,
-	.ops		= &ops_8996,
+	.ops		= &ops_generic_v2,
 };
+