[PATCH 6/6] pinctrl: qcom: rename msm -> qcom

Caleb Connolly caleb.connolly at linaro.org
Tue Oct 31 15:22:25 CET 2023


The "MSM" naming hasn't been correct for quite a while now, in line
with Linux lets rename all these msm_* functions to qcom_* as well as
ensure namespacing is consistent across the pinctrl and GPIO drivers.

Signed-off-by: Caleb Connolly <caleb.connolly at linaro.org>
---
 configs/dragonboard410c_defconfig        |  2 +-
 configs/dragonboard845c_defconfig        |  2 +-
 configs/qcs404evb_defconfig              |  2 +-
 configs/starqltechn_defconfig            |  2 +-
 drivers/gpio/Kconfig                     |  6 +-
 drivers/gpio/Makefile                    |  2 +-
 drivers/gpio/{msm_gpio.c => qcom_gpio.c} | 72 ++++++++++++------------
 drivers/pinctrl/qcom/pinctrl-apq8016.c   | 22 ++++----
 drivers/pinctrl/qcom/pinctrl-apq8096.c   | 22 ++++----
 drivers/pinctrl/qcom/pinctrl-ipq4019.c   | 18 +++---
 drivers/pinctrl/qcom/pinctrl-qcom.c      | 96 ++++++++++++++++----------------
 drivers/pinctrl/qcom/pinctrl-qcom.h      |  8 +--
 drivers/pinctrl/qcom/pinctrl-qcs404.c    | 22 ++++----
 drivers/pinctrl/qcom/pinctrl-sdm845.c    | 19 +++----
 include/qcom-gpio.h                      |  6 +-
 15 files changed, 149 insertions(+), 152 deletions(-)

diff --git a/configs/dragonboard410c_defconfig b/configs/dragonboard410c_defconfig
index b338326e34c0..11c631799ccf 100644
--- a/configs/dragonboard410c_defconfig
+++ b/configs/dragonboard410c_defconfig
@@ -43,7 +43,7 @@ CONFIG_USB_FUNCTION_FASTBOOT=y
 CONFIG_FASTBOOT_BUF_ADDR=0x91000000
 CONFIG_FASTBOOT_FLASH=y
 CONFIG_FASTBOOT_FLASH_MMC_DEV=0
-CONFIG_MSM_GPIO=y
+CONFIG_QCOM_GPIO=y
 CONFIG_QCOM_PMIC_GPIO=y
 CONFIG_LED=y
 CONFIG_LED_GPIO=y
diff --git a/configs/dragonboard845c_defconfig b/configs/dragonboard845c_defconfig
index a69d82761a8d..afbdc15d88b2 100644
--- a/configs/dragonboard845c_defconfig
+++ b/configs/dragonboard845c_defconfig
@@ -19,7 +19,7 @@ CONFIG_SYS_CBSIZE=512
 CONFIG_CMD_GPIO=y
 # CONFIG_NET is not set
 CONFIG_CLK=y
-CONFIG_MSM_GPIO=y
+CONFIG_QCOM_GPIO=y
 CONFIG_QCOM_PMIC_GPIO=y
 CONFIG_PINCTRL=y
 CONFIG_DM_PMIC=y
diff --git a/configs/qcs404evb_defconfig b/configs/qcs404evb_defconfig
index 9e72f64f7849..545ee25b0e7d 100644
--- a/configs/qcs404evb_defconfig
+++ b/configs/qcs404evb_defconfig
@@ -31,7 +31,7 @@ CONFIG_CMD_FAT=y
 CONFIG_CMD_FS_GENERIC=y
 # CONFIG_NET is not set
 CONFIG_CLK=y
-CONFIG_MSM_GPIO=y
+CONFIG_QCOM_GPIO=y
 CONFIG_QCOM_PMIC_GPIO=y
 CONFIG_MISC=y
 CONFIG_MMC_HS400_SUPPORT=y
diff --git a/configs/starqltechn_defconfig b/configs/starqltechn_defconfig
index 5b85ce5fe96f..9962211d8ab7 100644
--- a/configs/starqltechn_defconfig
+++ b/configs/starqltechn_defconfig
@@ -23,7 +23,7 @@ CONFIG_CMD_BMP=y
 # CONFIG_NET is not set
 CONFIG_BUTTON=y
 CONFIG_CLK=y
-CONFIG_MSM_GPIO=y
+CONFIG_QCOM_GPIO=y
 CONFIG_QCOM_PMIC_GPIO=y
 CONFIG_DM_KEYBOARD=y
 CONFIG_BUTTON_KEYBOARD=y
diff --git a/drivers/gpio/Kconfig b/drivers/gpio/Kconfig
index 74baa98d3c15..2e57eedd466f 100644
--- a/drivers/gpio/Kconfig
+++ b/drivers/gpio/Kconfig
@@ -259,7 +259,7 @@ config MSCC_SGPIO
           SIO controller is to connect control signals from SFP
           modules and to act as an LED controller.
 
-config MSM_GPIO
+config QCOM_GPIO
 	bool "Qualcomm GPIO driver"
 	depends on DM_GPIO
 	help
@@ -268,9 +268,7 @@ config MSM_GPIO
 	  gpio has it's own set of registers.
 	  Only simple GPIO operations are supported (get/set, change of
 	  direction and checking pin function).
-	  Supported devices:
-	  - APQ8016
-	  - MSM8916
+	  See PINCTRL_QCOM for a list of supported platforms.
 
 config MXC_GPIO
 	bool "Freescale/NXP MXC GPIO driver"
diff --git a/drivers/gpio/Makefile b/drivers/gpio/Makefile
index c8b3fd78141a..b5ebcf75dbfa 100644
--- a/drivers/gpio/Makefile
+++ b/drivers/gpio/Makefile
@@ -58,7 +58,7 @@ obj-$(CONFIG_IMX_RGPIO2P)	+= imx_rgpio2p.o
 obj-$(CONFIG_PIC32_GPIO)	+= pic32_gpio.o
 obj-$(CONFIG_OCTEON_GPIO)	+= octeon_gpio.o
 obj-$(CONFIG_MVEBU_GPIO)	+= mvebu_gpio.o
-obj-$(CONFIG_MSM_GPIO)		+= msm_gpio.o
+obj-$(CONFIG_QCOM_GPIO)		+= qcom_gpio.o
 obj-$(CONFIG_$(SPL_)PCF8575_GPIO)	+= pcf8575_gpio.o
 obj-$(CONFIG_$(SPL_TPL_)QCOM_PMIC_GPIO)	+= qcom_pmic_gpio.o
 obj-$(CONFIG_MT7620_GPIO)	+= mt7620_gpio.o
diff --git a/drivers/gpio/msm_gpio.c b/drivers/gpio/qcom_gpio.c
similarity index 50%
rename from drivers/gpio/msm_gpio.c
rename to drivers/gpio/qcom_gpio.c
index 7d01fecf46f2..a0b183eb0784 100644
--- a/drivers/gpio/msm_gpio.c
+++ b/drivers/gpio/qcom_gpio.c
@@ -24,22 +24,22 @@ DECLARE_GLOBAL_DATA_PTR;
 #define GPIO_IN          0
 #define GPIO_OUT         1
 
-struct msm_gpio_bank {
+struct qcom_gpio_priv {
 	phys_addr_t base;
-	const struct msm_pin_data *pin_data;
+	const struct qcom_pin_data *pin_data;
 };
 
 #define GPIO_CONFIG_REG(dev, x) \
-	(qcom_pin_offset(((struct msm_gpio_bank *)dev_get_priv(dev))->pin_data->pin_offsets, x))
+	(qcom_pin_offset(((struct qcom_gpio_priv *)dev_get_priv(dev))->pin_data->pin_offsets, x))
 
 #define GPIO_IN_OUT_REG(dev, x) \
 	(GPIO_CONFIG_REG(dev, x) + 0x4)
 
-static int msm_gpio_direction_input(struct udevice *dev, unsigned int gpio)
+static int qcom_gpio_direction_input(struct udevice *dev, unsigned int gpio)
 {
-	struct msm_gpio_bank *priv = dev_get_priv(dev);
+	struct qcom_gpio_priv *priv = dev_get_priv(dev);
 
-	if (msm_pinctrl_is_reserved(dev_get_parent(dev), gpio))
+	if (qcom_pinctrl_is_reserved(dev_get_parent(dev), gpio))
 		return 0;
 
 	/* Disable OE bit */
@@ -49,11 +49,11 @@ static int msm_gpio_direction_input(struct udevice *dev, unsigned int gpio)
 	return 0;
 }
 
-static int msm_gpio_set_value(struct udevice *dev, unsigned int gpio, int value)
+static int qcom_gpio_set_value(struct udevice *dev, unsigned int gpio, int value)
 {
-	struct msm_gpio_bank *priv = dev_get_priv(dev);
+	struct qcom_gpio_priv *priv = dev_get_priv(dev);
 
-	if (msm_pinctrl_is_reserved(dev_get_parent(dev), gpio))
+	if (qcom_pinctrl_is_reserved(dev_get_parent(dev), gpio))
 		return 0;
 
 	value = !!value;
@@ -63,12 +63,12 @@ static int msm_gpio_set_value(struct udevice *dev, unsigned int gpio, int value)
 	return 0;
 }
 
-static int msm_gpio_direction_output(struct udevice *dev, unsigned int gpio,
-				     int value)
+static int qcom_gpio_direction_output(struct udevice *dev, unsigned int gpio,
+				      int value)
 {
-	struct msm_gpio_bank *priv = dev_get_priv(dev);
+	struct qcom_gpio_priv *priv = dev_get_priv(dev);
 
-	if (msm_pinctrl_is_reserved(dev_get_parent(dev), gpio))
+	if (qcom_pinctrl_is_reserved(dev_get_parent(dev), gpio))
 		return 0;
 
 	value = !!value;
@@ -81,21 +81,21 @@ static int msm_gpio_direction_output(struct udevice *dev, unsigned int gpio,
 	return 0;
 }
 
-static int msm_gpio_get_value(struct udevice *dev, unsigned int gpio)
+static int qcom_gpio_get_value(struct udevice *dev, unsigned int gpio)
 {
-	struct msm_gpio_bank *priv = dev_get_priv(dev);
+	struct qcom_gpio_priv *priv = dev_get_priv(dev);
 
-	if (msm_pinctrl_is_reserved(dev_get_parent(dev), gpio))
+	if (qcom_pinctrl_is_reserved(dev_get_parent(dev), gpio))
 		return 0;
 
 	return !!(readl(priv->base + GPIO_IN_OUT_REG(dev, gpio)) >> GPIO_IN);
 }
 
-static int msm_gpio_get_function(struct udevice *dev, unsigned int gpio)
+static int qcom_gpio_get_function(struct udevice *dev, unsigned int gpio)
 {
-	struct msm_gpio_bank *priv = dev_get_priv(dev);
+	struct qcom_gpio_priv *priv = dev_get_priv(dev);
 
-	if (msm_pinctrl_is_reserved(dev_get_parent(dev), gpio))
+	if (qcom_pinctrl_is_reserved(dev_get_parent(dev), gpio))
 		return GPIOF_UNKNOWN;
 
 	if (readl(priv->base + GPIO_CONFIG_REG(dev, gpio)) & GPIO_OE_ENABLE)
@@ -104,28 +104,28 @@ static int msm_gpio_get_function(struct udevice *dev, unsigned int gpio)
 	return GPIOF_INPUT;
 }
 
-static const struct dm_gpio_ops gpio_msm_ops = {
-	.direction_input	= msm_gpio_direction_input,
-	.direction_output	= msm_gpio_direction_output,
-	.get_value		= msm_gpio_get_value,
-	.set_value		= msm_gpio_set_value,
-	.get_function		= msm_gpio_get_function,
+static const struct dm_gpio_ops qcom_gpio_ops = {
+	.direction_input	= qcom_gpio_direction_input,
+	.direction_output	= qcom_gpio_direction_output,
+	.get_value		= qcom_gpio_get_value,
+	.set_value		= qcom_gpio_set_value,
+	.get_function		= qcom_gpio_get_function,
 };
 
-static int msm_gpio_probe(struct udevice *dev)
+static int qcom_gpio_probe(struct udevice *dev)
 {
-	struct msm_gpio_bank *priv = dev_get_priv(dev);
+	struct qcom_gpio_priv *priv = dev_get_priv(dev);
 
 	priv->base = dev_read_addr(dev);
-	priv->pin_data = (struct msm_pin_data *)dev_get_driver_data(dev);
+	priv->pin_data = (struct qcom_pin_data *)dev_get_driver_data(dev);
 
 	return priv->base == FDT_ADDR_T_NONE ? -EINVAL : 0;
 }
 
-static int msm_gpio_of_to_plat(struct udevice *dev)
+static int qcom_gpio_of_to_plat(struct udevice *dev)
 {
 	struct gpio_dev_priv *uc_priv = dev_get_uclass_priv(dev);
-	const struct msm_pin_data *pin_data = (struct msm_pin_data *)dev_get_driver_data(dev);
+	const struct qcom_pin_data *pin_data = (struct qcom_pin_data *)dev_get_driver_data(dev);
 
 	/* Get the pin count from the pinctrl driver */
 	uc_priv->gpio_count = pin_data->pin_count;
@@ -137,12 +137,12 @@ static int msm_gpio_of_to_plat(struct udevice *dev)
 	return 0;
 }
 
-U_BOOT_DRIVER(gpio_msm) = {
-	.name	= "gpio_msm",
+U_BOOT_DRIVER(qcom_gpio) = {
+	.name	= "qcom_gpio",
 	.id	= UCLASS_GPIO,
-	.of_to_plat = msm_gpio_of_to_plat,
-	.probe	= msm_gpio_probe,
-	.ops	= &gpio_msm_ops,
+	.of_to_plat = qcom_gpio_of_to_plat,
+	.probe	= qcom_gpio_probe,
+	.ops	= &qcom_gpio_ops,
 	.flags	= DM_UC_FLAG_SEQ_ALIAS,
-	.priv_auto	= sizeof(struct msm_gpio_bank),
+	.priv_auto	= sizeof(struct qcom_gpio_priv),
 };
diff --git a/drivers/pinctrl/qcom/pinctrl-apq8016.c b/drivers/pinctrl/qcom/pinctrl-apq8016.c
index 8149ffd83cc4..652f8a51be60 100644
--- a/drivers/pinctrl/qcom/pinctrl-apq8016.c
+++ b/drivers/pinctrl/qcom/pinctrl-apq8016.c
@@ -13,7 +13,7 @@
 
 #define MAX_PIN_NAME_LEN 32
 static char pin_name[MAX_PIN_NAME_LEN] __section(".data");
-static const char * const msm_pinctrl_pins[] = {
+static const char * const qcom_pinctrl_pins[] = {
 	"SDC1_CLK",
 	"SDC1_CMD",
 	"SDC1_DATA",
@@ -28,14 +28,14 @@ static const char * const msm_pinctrl_pins[] = {
 	"QDSD_DATA3",
 };
 
-static const struct pinctrl_function msm_pinctrl_functions[] = {
+static const struct pinctrl_function qcom_pinctrl_functions[] = {
 	{"blsp1_uart", 2},
 };
 
 static const char *apq8016_get_function_name(struct udevice *dev,
 					     unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].name;
+	return qcom_pinctrl_functions[selector].name;
 }
 
 static const char *apq8016_get_pin_name(struct udevice *dev,
@@ -45,24 +45,24 @@ static const char *apq8016_get_pin_name(struct udevice *dev,
 		snprintf(pin_name, MAX_PIN_NAME_LEN, "GPIO_%u", selector);
 		return pin_name;
 	} else {
-		return msm_pinctrl_pins[selector - 122];
+		return qcom_pinctrl_pins[selector - 122];
 	}
 }
 
 static unsigned int apq8016_get_function_mux(unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].val;
+	return qcom_pinctrl_functions[selector].val;
 }
 
-static const struct msm_pinctrl_data apq8016_data = {
+static const struct qcom_pinctrl_data apq8016_data = {
 	.pin_data = { .pin_count = 133, },
-	.functions_count = ARRAY_SIZE(msm_pinctrl_functions),
+	.functions_count = ARRAY_SIZE(qcom_pinctrl_functions),
 	.get_function_name = apq8016_get_function_name,
 	.get_function_mux = apq8016_get_function_mux,
 	.get_pin_name = apq8016_get_pin_name,
 };
 
-static const struct udevice_id msm_pinctrl_ids[] = {
+static const struct udevice_id qcom_pinctrl_ids[] = {
 	{ .compatible = "qcom,msm8916-pinctrl", .data = (ulong)&apq8016_data },
 	{ /* Sentinal */ }
 };
@@ -70,7 +70,7 @@ static const struct udevice_id msm_pinctrl_ids[] = {
 U_BOOT_DRIVER(pinctrl_apq8016) = {
 	.name		= "pinctrl_apq8016",
 	.id		= UCLASS_NOP,
-	.of_match	= msm_pinctrl_ids,
-	.ops		= &msm_pinctrl_ops,
-	.bind		= msm_pinctrl_bind,
+	.of_match	= qcom_pinctrl_ids,
+	.ops		= &qcom_pinctrl_ops,
+	.bind		= qcom_pinctrl_bind,
 };
diff --git a/drivers/pinctrl/qcom/pinctrl-apq8096.c b/drivers/pinctrl/qcom/pinctrl-apq8096.c
index d64ab1ff7bee..d0c1424dff7a 100644
--- a/drivers/pinctrl/qcom/pinctrl-apq8096.c
+++ b/drivers/pinctrl/qcom/pinctrl-apq8096.c
@@ -13,7 +13,7 @@
 
 #define MAX_PIN_NAME_LEN 32
 static char pin_name[MAX_PIN_NAME_LEN] __section(".data");
-static const char * const msm_pinctrl_pins[] = {
+static const char * const qcom_pinctrl_pins[] = {
 	"SDC1_CLK",
 	"SDC1_CMD",
 	"SDC1_DATA",
@@ -23,14 +23,14 @@ static const char * const msm_pinctrl_pins[] = {
 	"SDC1_RCLK",
 };
 
-static const struct pinctrl_function msm_pinctrl_functions[] = {
+static const struct pinctrl_function qcom_pinctrl_functions[] = {
 	{"blsp_uart8", 2},
 };
 
 static const char *apq8096_get_function_name(struct udevice *dev,
 					     unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].name;
+	return qcom_pinctrl_functions[selector].name;
 }
 
 static const char *apq8096_get_pin_name(struct udevice *dev,
@@ -40,24 +40,24 @@ static const char *apq8096_get_pin_name(struct udevice *dev,
 		snprintf(pin_name, MAX_PIN_NAME_LEN, "GPIO_%u", selector);
 		return pin_name;
 	} else {
-		return msm_pinctrl_pins[selector - 150];
+		return qcom_pinctrl_pins[selector - 150];
 	}
 }
 
 static unsigned int apq8096_get_function_mux(unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].val;
+	return qcom_pinctrl_functions[selector].val;
 }
 
-static const struct msm_pinctrl_data apq8096_data = {
+static const struct qcom_pinctrl_data apq8096_data = {
 	.pin_data = { .pin_count = 157, },
-	.functions_count = ARRAY_SIZE(msm_pinctrl_functions),
+	.functions_count = ARRAY_SIZE(qcom_pinctrl_functions),
 	.get_function_name = apq8096_get_function_name,
 	.get_function_mux = apq8096_get_function_mux,
 	.get_pin_name = apq8096_get_pin_name,
 };
 
-static const struct udevice_id msm_pinctrl_ids[] = {
+static const struct udevice_id qcom_pinctrl_ids[] = {
 	{ .compatible = "qcom,msm8996-pinctrl", .data = (ulong)&apq8096_data },
 	{ /* Sentinal */ }
 };
@@ -65,7 +65,7 @@ static const struct udevice_id msm_pinctrl_ids[] = {
 U_BOOT_DRIVER(pinctrl_apq8096) = {
 	.name		= "pinctrl_apq8096",
 	.id		= UCLASS_NOP,
-	.of_match	= msm_pinctrl_ids,
-	.ops		= &msm_pinctrl_ops,
-	.bind		= msm_pinctrl_bind,
+	.of_match	= qcom_pinctrl_ids,
+	.ops		= &qcom_pinctrl_ops,
+	.bind		= qcom_pinctrl_bind,
 };
diff --git a/drivers/pinctrl/qcom/pinctrl-ipq4019.c b/drivers/pinctrl/qcom/pinctrl-ipq4019.c
index 2d99f99e1e45..2081d6d86f82 100644
--- a/drivers/pinctrl/qcom/pinctrl-ipq4019.c
+++ b/drivers/pinctrl/qcom/pinctrl-ipq4019.c
@@ -14,7 +14,7 @@
 
 #define MAX_PIN_NAME_LEN 32
 static char pin_name[MAX_PIN_NAME_LEN] __section(".data");
-static const struct pinctrl_function msm_pinctrl_functions[] = {
+static const struct pinctrl_function qcom_pinctrl_functions[] = {
 	{"gpio", 0},
 	{"blsp_uart0_0", 1}, /* Only for GPIO:16,17 */
 	{"blsp_uart0_1", 2}, /* Only for GPIO:60,61 */
@@ -30,7 +30,7 @@ static const struct pinctrl_function msm_pinctrl_functions[] = {
 static const char *ipq4019_get_function_name(struct udevice *dev,
 					     unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].name;
+	return qcom_pinctrl_functions[selector].name;
 }
 
 static const char *ipq4019_get_pin_name(struct udevice *dev,
@@ -42,18 +42,18 @@ static const char *ipq4019_get_pin_name(struct udevice *dev,
 
 static unsigned int ipq4019_get_function_mux(unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].val;
+	return qcom_pinctrl_functions[selector].val;
 }
 
-static const struct msm_pinctrl_data ipq4019_data = {
+static const struct qcom_pinctrl_data ipq4019_data = {
 	.pin_data = { .pin_count = 100, },
-	.functions_count = ARRAY_SIZE(msm_pinctrl_functions),
+	.functions_count = ARRAY_SIZE(qcom_pinctrl_functions),
 	.get_function_name = ipq4019_get_function_name,
 	.get_function_mux = ipq4019_get_function_mux,
 	.get_pin_name = ipq4019_get_pin_name,
 };
 
-static const struct udevice_id msm_pinctrl_ids[] = {
+static const struct udevice_id qcom_pinctrl_ids[] = {
 	{ .compatible = "qcom,ipq4019-pinctrl", .data = (ulong)&ipq4019_data },
 	{ /* Sentinal */ }
 };
@@ -61,7 +61,7 @@ static const struct udevice_id msm_pinctrl_ids[] = {
 U_BOOT_DRIVER(pinctrl_ipq4019) = {
 	.name		= "pinctrl_ipq4019",
 	.id		= UCLASS_NOP,
-	.of_match	= msm_pinctrl_ids,
-	.ops		= &msm_pinctrl_ops,
-	.bind		= msm_pinctrl_bind,
+	.of_match	= qcom_pinctrl_ids,
+	.ops		= &qcom_pinctrl_ops,
+	.bind		= qcom_pinctrl_bind,
 };
diff --git a/drivers/pinctrl/qcom/pinctrl-qcom.c b/drivers/pinctrl/qcom/pinctrl-qcom.c
index 92b35c198788..4f5ef5c14dd4 100644
--- a/drivers/pinctrl/qcom/pinctrl-qcom.c
+++ b/drivers/pinctrl/qcom/pinctrl-qcom.c
@@ -20,12 +20,12 @@
 
 #include "pinctrl-qcom.h"
 
-#define MSM_PINCTRL_MAX_RESERVED_RANGES 32
+#define qcom_PINCTRL_MAX_RESERVED_RANGES 32
 
-struct msm_pinctrl_priv {
+struct qcom_pinctrl_priv {
 	phys_addr_t base;
-	struct msm_pinctrl_data *data;
-	u32 reserved_ranges[MSM_PINCTRL_MAX_RESERVED_RANGES * 2];
+	struct qcom_pinctrl_data *data;
+	u32 reserved_ranges[qcom_PINCTRL_MAX_RESERVED_RANGES * 2];
 	int reserved_ranges_count;
 };
 
@@ -37,37 +37,37 @@ struct msm_pinctrl_priv {
 #define TLMM_DRV_STRENGTH_MASK GENMASK(8, 6)
 #define TLMM_GPIO_DISABLE BIT(9)
 
-static const struct pinconf_param msm_conf_params[] = {
+static const struct pinconf_param qcom_conf_params[] = {
 	{ "drive-strength", PIN_CONFIG_DRIVE_STRENGTH, 2 },
 	{ "bias-disable", PIN_CONFIG_BIAS_DISABLE, 0 },
 	{ "bias-pull-up", PIN_CONFIG_BIAS_PULL_UP, 3 },
 };
 
-static int msm_get_functions_count(struct udevice *dev)
+static int qcom_get_functions_count(struct udevice *dev)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 
 	return priv->data->functions_count;
 }
 
-static int msm_get_pins_count(struct udevice *dev)
+static int qcom_get_pins_count(struct udevice *dev)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 
 	return priv->data->pin_data.pin_count;
 }
 
-static const char *msm_get_function_name(struct udevice *dev,
-					 unsigned int selector)
+static const char *qcom_get_function_name(struct udevice *dev,
+					  unsigned int selector)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 
 	return priv->data->get_function_name(dev, selector);
 }
 
-static int msm_pinctrl_parse_ranges(struct udevice *dev)
+static int qcom_pinctrl_parse_ranges(struct udevice *dev)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 	int count;
 
 	if (ofnode_read_prop(dev_ofnode(dev), "gpio-reserved-ranges",
@@ -79,9 +79,9 @@ static int msm_pinctrl_parse_ranges(struct udevice *dev)
 		/* Size is in bytes, but we're indexing by ints */
 		count /= 4;
 
-		if (count > MSM_PINCTRL_MAX_RESERVED_RANGES) {
+		if (count > qcom_PINCTRL_MAX_RESERVED_RANGES) {
 			dev_err(dev, "gpio-reserved-ranges must be less than %d (got %d)\n",
-				MSM_PINCTRL_MAX_RESERVED_RANGES, count);
+				qcom_PINCTRL_MAX_RESERVED_RANGES, count);
 			return -EINVAL;
 		}
 
@@ -98,15 +98,15 @@ static int msm_pinctrl_parse_ranges(struct udevice *dev)
 	return 0;
 }
 
-static int msm_pinctrl_probe(struct udevice *dev)
+static int qcom_pinctrl_probe(struct udevice *dev)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 	int ret;
 
 	priv->base = dev_read_addr(dev);
-	priv->data = (struct msm_pinctrl_data *)dev_get_driver_data(dev);
+	priv->data = (struct qcom_pinctrl_data *)dev_get_driver_data(dev);
 
-	ret = msm_pinctrl_parse_ranges(dev);
+	ret = qcom_pinctrl_parse_ranges(dev);
 	if (ret) {
 		printf("Couldn't parse reserved GPIO ranges!\n");
 		return ret;
@@ -115,19 +115,19 @@ static int msm_pinctrl_probe(struct udevice *dev)
 	return priv->base == FDT_ADDR_T_NONE ? -EINVAL : 0;
 }
 
-static const char *msm_get_pin_name(struct udevice *dev, unsigned int selector)
+static const char *qcom_get_pin_name(struct udevice *dev, unsigned int selector)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 
 	return priv->data->get_pin_name(dev, selector);
 }
 
-static int msm_pinmux_set(struct udevice *dev, unsigned int pin_selector,
-			  unsigned int func_selector)
+static int qcom_pinmux_set(struct udevice *dev, unsigned int pin_selector,
+			   unsigned int func_selector)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 
-	if (msm_pinctrl_is_reserved(dev, pin_selector))
+	if (qcom_pinctrl_is_reserved(dev, pin_selector))
 		return 0;
 
 	clrsetbits_le32(priv->base + GPIO_CONFIG_REG(priv, pin_selector),
@@ -136,12 +136,12 @@ static int msm_pinmux_set(struct udevice *dev, unsigned int pin_selector,
 	return 0;
 }
 
-static int msm_pinconf_set(struct udevice *dev, unsigned int pin_selector,
-			   unsigned int param, unsigned int argument)
+static int qcom_pinconf_set(struct udevice *dev, unsigned int pin_selector,
+			    unsigned int param, unsigned int argument)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 
-	if (msm_pinctrl_is_reserved(dev, pin_selector))
+	if (qcom_pinctrl_is_reserved(dev, pin_selector))
 		return 0;
 
 	switch (param) {
@@ -165,22 +165,22 @@ static int msm_pinconf_set(struct udevice *dev, unsigned int pin_selector,
 	return 0;
 }
 
-struct pinctrl_ops msm_pinctrl_ops = {
-	.get_pins_count = msm_get_pins_count,
-	.get_pin_name = msm_get_pin_name,
+struct pinctrl_ops qcom_pinctrl_ops = {
+	.get_pins_count = qcom_get_pins_count,
+	.get_pin_name = qcom_get_pin_name,
 	.set_state = pinctrl_generic_set_state,
-	.pinmux_set = msm_pinmux_set,
-	.pinconf_num_params = ARRAY_SIZE(msm_conf_params),
-	.pinconf_params = msm_conf_params,
-	.pinconf_set = msm_pinconf_set,
-	.get_functions_count = msm_get_functions_count,
-	.get_function_name = msm_get_function_name,
+	.pinmux_set = qcom_pinmux_set,
+	.pinconf_num_params = ARRAY_SIZE(qcom_conf_params),
+	.pinconf_params = qcom_conf_params,
+	.pinconf_set = qcom_pinconf_set,
+	.get_functions_count = qcom_get_functions_count,
+	.get_function_name = qcom_get_function_name,
 };
 
-int msm_pinctrl_bind(struct udevice *dev)
+int qcom_pinctrl_bind(struct udevice *dev)
 {
 	ofnode node = dev_ofnode(dev);
-	struct msm_pinctrl_data *data = (struct msm_pinctrl_data *)dev_get_driver_data(dev);
+	struct qcom_pinctrl_data *data = (struct qcom_pinctrl_data *)dev_get_driver_data(dev);
 	struct driver *drv;
 	struct udevice *pinctrl_dev;
 	const char *name;
@@ -204,9 +204,9 @@ int msm_pinctrl_bind(struct udevice *dev)
 	if (!name)
 		return -EINVAL;
 
-	drv = lists_driver_lookup_name("gpio_msm");
+	drv = lists_driver_lookup_name("qcom_gpio");
 	if (!drv) {
-		printf("Can't find gpio_msm driver\n");
+		printf("Can't find qcom_gpio driver\n");
 		return -ENODEV;
 	}
 
@@ -224,14 +224,14 @@ int msm_pinctrl_bind(struct udevice *dev)
 U_BOOT_DRIVER(qcom_pinctrl) = {
 	.name		= "qcom_pinctrl",
 	.id		= UCLASS_PINCTRL,
-	.priv_auto	= sizeof(struct msm_pinctrl_priv),
-	.ops		= &msm_pinctrl_ops,
-	.probe		= msm_pinctrl_probe,
+	.priv_auto	= sizeof(struct qcom_pinctrl_priv),
+	.ops		= &qcom_pinctrl_ops,
+	.probe		= qcom_pinctrl_probe,
 };
 
-bool msm_pinctrl_is_reserved(struct udevice *dev, unsigned int pin)
+bool qcom_pinctrl_is_reserved(struct udevice *dev, unsigned int pin)
 {
-	struct msm_pinctrl_priv *priv = dev_get_priv(dev);
+	struct qcom_pinctrl_priv *priv = dev_get_priv(dev);
 	unsigned int i, start;
 
 	for (i = 0; i < priv->reserved_ranges_count; i += 2) {
diff --git a/drivers/pinctrl/qcom/pinctrl-qcom.h b/drivers/pinctrl/qcom/pinctrl-qcom.h
index efb6a44b1409..2aa8c80c696f 100644
--- a/drivers/pinctrl/qcom/pinctrl-qcom.h
+++ b/drivers/pinctrl/qcom/pinctrl-qcom.h
@@ -13,8 +13,8 @@
 
 struct udevice;
 
-struct msm_pinctrl_data {
-	struct msm_pin_data pin_data;
+struct qcom_pinctrl_data {
+	struct qcom_pin_data pin_data;
 	int functions_count;
 	const char *(*get_function_name)(struct udevice *dev,
 					 unsigned int selector);
@@ -28,8 +28,8 @@ struct pinctrl_function {
 	int val;
 };
 
-extern struct pinctrl_ops msm_pinctrl_ops;
+extern struct pinctrl_ops qcom_pinctrl_ops;
 
-int msm_pinctrl_bind(struct udevice *dev);
+int qcom_pinctrl_bind(struct udevice *dev);
 
 #endif
diff --git a/drivers/pinctrl/qcom/pinctrl-qcs404.c b/drivers/pinctrl/qcom/pinctrl-qcs404.c
index ac00afa2a1f4..410cdbb23925 100644
--- a/drivers/pinctrl/qcom/pinctrl-qcs404.c
+++ b/drivers/pinctrl/qcom/pinctrl-qcs404.c
@@ -12,7 +12,7 @@
 
 #define MAX_PIN_NAME_LEN 32
 static char pin_name[MAX_PIN_NAME_LEN] __section(".data");
-static const char * const msm_pinctrl_pins[] = {
+static const char * const qcom_pinctrl_pins[] = {
 	"SDC1_RCLK",
 	"SDC1_CLK",
 	"SDC1_CMD",
@@ -22,7 +22,7 @@ static const char * const msm_pinctrl_pins[] = {
 	"SDC2_DATA",
 };
 
-static const struct pinctrl_function msm_pinctrl_functions[] = {
+static const struct pinctrl_function qcom_pinctrl_functions[] = {
 	{"blsp_uart2", 1},
 	{"rgmii_int", 1},
 	{"rgmii_ck", 1},
@@ -42,7 +42,7 @@ static const struct pinctrl_function msm_pinctrl_functions[] = {
 static const char *qcs404_get_function_name(struct udevice *dev,
 					    unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].name;
+	return qcom_pinctrl_functions[selector].name;
 }
 
 static const char *qcs404_get_pin_name(struct udevice *dev,
@@ -52,24 +52,24 @@ static const char *qcs404_get_pin_name(struct udevice *dev,
 		snprintf(pin_name, MAX_PIN_NAME_LEN, "GPIO_%u", selector);
 		return pin_name;
 	} else {
-		return msm_pinctrl_pins[selector - 120];
+		return qcom_pinctrl_pins[selector - 120];
 	}
 }
 
 static unsigned int qcs404_get_function_mux(unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].val;
+	return qcom_pinctrl_functions[selector].val;
 }
 
-static struct msm_pinctrl_data qcs404_data = {
+static struct qcom_pinctrl_data qcs404_data = {
 	.pin_data = { .pin_count = 126, },
-	.functions_count = ARRAY_SIZE(msm_pinctrl_functions),
+	.functions_count = ARRAY_SIZE(qcom_pinctrl_functions),
 	.get_function_name = qcs404_get_function_name,
 	.get_function_mux = qcs404_get_function_mux,
 	.get_pin_name = qcs404_get_pin_name,
 };
 
-static const struct udevice_id msm_pinctrl_ids[] = {
+static const struct udevice_id qcom_pinctrl_ids[] = {
 	{ .compatible = "qcom,qcs404-pinctrl", .data = (ulong)&qcs404_data },
 	{ /* Sentinal */ }
 };
@@ -77,7 +77,7 @@ static const struct udevice_id msm_pinctrl_ids[] = {
 U_BOOT_DRIVER(pinctrl_qcs404) = {
 	.name		= "pinctrl_qcs404",
 	.id		= UCLASS_NOP,
-	.of_match	= msm_pinctrl_ids,
-	.ops		= &msm_pinctrl_ops,
-	.bind		= msm_pinctrl_bind,
+	.of_match	= qcom_pinctrl_ids,
+	.ops		= &qcom_pinctrl_ops,
+	.bind		= qcom_pinctrl_bind,
 };
diff --git a/drivers/pinctrl/qcom/pinctrl-sdm845.c b/drivers/pinctrl/qcom/pinctrl-sdm845.c
index 9f0f4085ce2d..08cca602e94a 100644
--- a/drivers/pinctrl/qcom/pinctrl-sdm845.c
+++ b/drivers/pinctrl/qcom/pinctrl-sdm845.c
@@ -3,7 +3,6 @@
  * Qualcomm SDM845 pinctrl
  *
  * (C) Copyright 2021 Dzmitry Sankouski <dsankouski at gmail.com>
- * (C) Copyright 2023 Linaro Ltd.
  *
  */
 
@@ -19,7 +18,7 @@
 #define MAX_PIN_NAME_LEN 32
 static char pin_name[MAX_PIN_NAME_LEN] __section(".data");
 
-static const struct pinctrl_function msm_pinctrl_functions[] = {
+static const struct pinctrl_function qcom_pinctrl_functions[] = {
 	{"qup9", 1},
 	{"gpio", 0},
 };
@@ -60,7 +59,7 @@ static const unsigned int sdm845_pin_offsets[] = {
 static const char *sdm845_get_function_name(struct udevice *dev,
 					     unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].name;
+	return qcom_pinctrl_functions[selector].name;
 }
 
 static const char *sdm845_get_pin_name(struct udevice *dev,
@@ -72,21 +71,21 @@ static const char *sdm845_get_pin_name(struct udevice *dev,
 
 static unsigned int sdm845_get_function_mux(unsigned int selector)
 {
-	return msm_pinctrl_functions[selector].val;
+	return qcom_pinctrl_functions[selector].val;
 }
 
-static struct msm_pinctrl_data sdm845_data = {
+static struct qcom_pinctrl_data sdm845_data = {
 	.pin_data = {
 		.pin_offsets = sdm845_pin_offsets,
 		.pin_count = ARRAY_SIZE(sdm845_pin_offsets),
 	},
-	.functions_count = ARRAY_SIZE(msm_pinctrl_functions),
+	.functions_count = ARRAY_SIZE(qcom_pinctrl_functions),
 	.get_function_name = sdm845_get_function_name,
 	.get_function_mux = sdm845_get_function_mux,
 	.get_pin_name = sdm845_get_pin_name,
 };
 
-static const struct udevice_id msm_pinctrl_ids[] = {
+static const struct udevice_id qcom_pinctrl_ids[] = {
 	{ .compatible = "qcom,sdm845-pinctrl", .data = (ulong)&sdm845_data },
 	{ /* Sentinal */ }
 };
@@ -94,7 +93,7 @@ static const struct udevice_id msm_pinctrl_ids[] = {
 U_BOOT_DRIVER(pinctrl_sdm845) = {
 	.name		= "pinctrl_sdm845",
 	.id		= UCLASS_NOP,
-	.of_match	= msm_pinctrl_ids,
-	.ops		= &msm_pinctrl_ops,
-	.bind		= msm_pinctrl_bind,
+	.of_match	= qcom_pinctrl_ids,
+	.ops		= &qcom_pinctrl_ops,
+	.bind		= qcom_pinctrl_bind,
 };
diff --git a/include/qcom-gpio.h b/include/qcom-gpio.h
index 490a1de55f89..758a6ad9234f 100644
--- a/include/qcom-gpio.h
+++ b/include/qcom-gpio.h
@@ -10,7 +10,7 @@
 #include <asm/types.h>
 #include <stdbool.h>
 
-struct msm_pin_data {
+struct qcom_pin_data {
 	int pin_count;
 	const unsigned int *pin_offsets;
 };
@@ -28,7 +28,7 @@ static inline u32 qcom_pin_offset(const unsigned int *offs, unsigned int selecto
 struct udevice;
 
 /**
- * msm_pinctrl_is_reserved() - Check if a pin lies in a reserved range
+ * qcom_pinctrl_is_reserved() - Check if a pin lies in a reserved range
  *
  * @dev: pinctrl device
  * @pin: Pin number
@@ -38,6 +38,6 @@ struct udevice;
  * Call using dev_get_parent() from the GPIO device, it is a child of
  * the pinctrl device.
  */
-bool msm_pinctrl_is_reserved(struct udevice *dev, unsigned int pin);
+bool qcom_pinctrl_is_reserved(struct udevice *dev, unsigned int pin);
 
 #endif /* _QCOM_GPIO_H_ */

-- 
2.42.0



More information about the U-Boot mailing list