All of lore.kernel.org
 help / color / mirror / Atom feed
* Subject: [PATCH 3/6] pinctrl: qcom: add pinctrl and gpio drivers for SDM845
@ 2021-08-31  8:44 Дмитрий Санковский
  2021-09-05 13:35 ` Ramon Fried
  0 siblings, 1 reply; 2+ messages in thread
From: Дмитрий Санковский @ 2021-08-31  8:44 UTC (permalink / raw)
  To: u-boot, Ramon Fried

From 520bc565a5a6f62c59f87bbd15a194ee61c103af Mon Sep 17 00:00:00 2001
From: Dzmitry Sankouski <dsankouski@gmail.com>
Date: Sun, 29 Aug 2021 21:53:40 +0300
Subject: [PATCH 3/6] pinctrl: qcom: add pinctrl and gpio drivers for SDM845
 SoC

Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
Cc: Ramon Fried <rfried.dev@gmail.com>
---
 arch/arm/mach-snapdragon/pinctrl-sdm845.c     | 44 +++++++++++++++++++
 arch/arm/mach-snapdragon/pinctrl-snapdragon.c |  1 +
 arch/arm/mach-snapdragon/pinctrl-snapdragon.h |  1 +
 drivers/gpio/msm_gpio.c                       |  1 +
 drivers/gpio/pm8916_gpio.c                    |  8 ++--
 5 files changed, 52 insertions(+), 3 deletions(-)
 create mode 100644 arch/arm/mach-snapdragon/pinctrl-sdm845.c

diff --git a/arch/arm/mach-snapdragon/pinctrl-sdm845.c
b/arch/arm/mach-snapdragon/pinctrl-sdm845.c
new file mode 100644
index 0000000000..6d66582aa6
--- /dev/null
+++ b/arch/arm/mach-snapdragon/pinctrl-sdm845.c
@@ -0,0 +1,44 @@
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ * Qualcomm SDM845 pinctrl
+ *
+ * (C) Copyright 2021 Dzmitry Sankouski <dsankouski@gmail.com>
+ *
+ */
+
+#include "pinctrl-snapdragon.h"
+#include <common.h>
+
+#define MAX_PIN_NAME_LEN 32
+static char pin_name[MAX_PIN_NAME_LEN];
+
+static const struct pinctrl_function msm_pinctrl_functions[] = {
+ {"qup9", 1},
+ {"gpio", 0},
+};
+
+static const char *sdm845_get_function_name(struct udevice *dev,
+     unsigned int selector)
+{
+ return msm_pinctrl_functions[selector].name;
+}
+
+static const char *sdm845_get_pin_name(struct udevice *dev,
+ unsigned int selector)
+{
+ snprintf(pin_name, MAX_PIN_NAME_LEN, "GPIO_%u", selector);
+ return pin_name;
+}
+
+static unsigned int sdm845_get_function_mux(unsigned int selector)
+{
+ return msm_pinctrl_functions[selector].val;
+}
+
+struct msm_pinctrl_data sdm845_data = {
+ .pin_count = 150,
+ .functions_count = ARRAY_SIZE(msm_pinctrl_functions),
+ .get_function_name = sdm845_get_function_name,
+ .get_function_mux = sdm845_get_function_mux,
+ .get_pin_name = sdm845_get_pin_name,
+};
diff --git a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
index e6b87c3573..c0ed943036 100644
--- a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
+++ b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
@@ -116,6 +116,7 @@ static struct pinctrl_ops msm_pinctrl_ops = {
 static const struct udevice_id msm_pinctrl_ids[] = {
  { .compatible = "qcom,tlmm-apq8016", .data = (ulong)&apq8016_data },
  { .compatible = "qcom,tlmm-apq8096", .data = (ulong)&apq8096_data },
+ { .compatible = "qcom,tlmm-sdm845", .data = (ulong)&sdm845_data },
  { }
 };

diff --git a/arch/arm/mach-snapdragon/pinctrl-snapdragon.h
b/arch/arm/mach-snapdragon/pinctrl-snapdragon.h
index 61d466f4d8..ea524312a0 100644
--- a/arch/arm/mach-snapdragon/pinctrl-snapdragon.h
+++ b/arch/arm/mach-snapdragon/pinctrl-snapdragon.h
@@ -27,5 +27,6 @@ struct pinctrl_function {

 extern struct msm_pinctrl_data apq8016_data;
 extern struct msm_pinctrl_data apq8096_data;
+extern struct msm_pinctrl_data sdm845_data;

 #endif
diff --git a/drivers/gpio/msm_gpio.c b/drivers/gpio/msm_gpio.c
index e1ff84c1c0..a3c3cd7824 100644
--- a/drivers/gpio/msm_gpio.c
+++ b/drivers/gpio/msm_gpio.c
@@ -120,6 +120,7 @@ static const struct udevice_id msm_gpio_ids[] = {
  { .compatible = "qcom,msm8916-pinctrl" },
  { .compatible = "qcom,apq8016-pinctrl" },
  { .compatible = "qcom,ipq4019-pinctrl" },
+ { .compatible = "qcom,sdm845-pinctrl" },
  { }
 };

diff --git a/drivers/gpio/pm8916_gpio.c b/drivers/gpio/pm8916_gpio.c
index 40b0f2578b..7ad95784a8 100644
--- a/drivers/gpio/pm8916_gpio.c
+++ b/drivers/gpio/pm8916_gpio.c
@@ -202,6 +202,7 @@ static int pm8916_gpio_of_to_plat(struct udevice *dev)
 static const struct udevice_id pm8916_gpio_ids[] = {
  { .compatible = "qcom,pm8916-gpio" },
  { .compatible = "qcom,pm8994-gpio" }, /* 22 GPIO's */
+ { .compatible = "qcom,pm8998-gpio" },
  { }
 };

@@ -266,7 +267,7 @@ static int pm8941_pwrkey_probe(struct udevice *dev)
  return log_msg_ret("bad type", -ENXIO);

  reg = pmic_reg_read(dev->parent, priv->pid + REG_SUBTYPE);
- if (reg != 0x1)
+ if ((reg & 0x5) == 0)
  return log_msg_ret("bad subtype", -ENXIO);

  return 0;
@@ -287,11 +288,12 @@ static int pm8941_pwrkey_of_to_plat(struct udevice
*dev)
 static const struct udevice_id pm8941_pwrkey_ids[] = {
  { .compatible = "qcom,pm8916-pwrkey" },
  { .compatible = "qcom,pm8994-pwrkey" },
+ { .compatible = "qcom,pm8998-pwrkey" },
  { }
 };

-U_BOOT_DRIVER(pwrkey_pm8941) = {
- .name = "pwrkey_pm8916",
+U_BOOT_DRIVER(pwrkey_pm89xx) = {
+ .name = "pwrkey_pm89xx",
  .id = UCLASS_GPIO,
  .of_match = pm8941_pwrkey_ids,
  .of_to_plat = pm8941_pwrkey_of_to_plat,
-- 
2.20.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: Subject: [PATCH 3/6] pinctrl: qcom: add pinctrl and gpio drivers for SDM845
  2021-08-31  8:44 Subject: [PATCH 3/6] pinctrl: qcom: add pinctrl and gpio drivers for SDM845 Дмитрий Санковский
@ 2021-09-05 13:35 ` Ramon Fried
  0 siblings, 0 replies; 2+ messages in thread
From: Ramon Fried @ 2021-09-05 13:35 UTC (permalink / raw)
  To: Дмитрий
	Санковский
  Cc: U-Boot Mailing List

On Tue, Aug 31, 2021 at 11:44 AM Дмитрий Санковский
<dsankouski@gmail.com> wrote:
>
> From 520bc565a5a6f62c59f87bbd15a194ee61c103af Mon Sep 17 00:00:00 2001
> From: Dzmitry Sankouski <dsankouski@gmail.com>
> Date: Sun, 29 Aug 2021 21:53:40 +0300
> Subject: [PATCH 3/6] pinctrl: qcom: add pinctrl and gpio drivers for SDM845
>  SoC
>
> Signed-off-by: Dzmitry Sankouski <dsankouski@gmail.com>
> Cc: Ramon Fried <rfried.dev@gmail.com>
> ---
>  arch/arm/mach-snapdragon/pinctrl-sdm845.c     | 44 +++++++++++++++++++
>  arch/arm/mach-snapdragon/pinctrl-snapdragon.c |  1 +
>  arch/arm/mach-snapdragon/pinctrl-snapdragon.h |  1 +
>  drivers/gpio/msm_gpio.c                       |  1 +
>  drivers/gpio/pm8916_gpio.c                    |  8 ++--
>  5 files changed, 52 insertions(+), 3 deletions(-)
>  create mode 100644 arch/arm/mach-snapdragon/pinctrl-sdm845.c
>
> diff --git a/arch/arm/mach-snapdragon/pinctrl-sdm845.c b/arch/arm/mach-snapdragon/pinctrl-sdm845.c
> new file mode 100644
> index 0000000000..6d66582aa6
> --- /dev/null
> +++ b/arch/arm/mach-snapdragon/pinctrl-sdm845.c
> @@ -0,0 +1,44 @@
> +// SPDX-License-Identifier: GPL-2.0+
> +/*
> + * Qualcomm SDM845 pinctrl
> + *
> + * (C) Copyright 2021 Dzmitry Sankouski <dsankouski@gmail.com>
> + *
> + */
> +
> +#include "pinctrl-snapdragon.h"
> +#include <common.h>
> +
> +#define MAX_PIN_NAME_LEN 32
> +static char pin_name[MAX_PIN_NAME_LEN];
> +
> +static const struct pinctrl_function msm_pinctrl_functions[] = {
> + {"qup9", 1},
> + {"gpio", 0},
> +};
> +
> +static const char *sdm845_get_function_name(struct udevice *dev,
> +     unsigned int selector)
> +{
> + return msm_pinctrl_functions[selector].name;
> +}
> +
> +static const char *sdm845_get_pin_name(struct udevice *dev,
> + unsigned int selector)
> +{
> + snprintf(pin_name, MAX_PIN_NAME_LEN, "GPIO_%u", selector);
> + return pin_name;
> +}
> +
> +static unsigned int sdm845_get_function_mux(unsigned int selector)
> +{
> + return msm_pinctrl_functions[selector].val;
> +}
> +
> +struct msm_pinctrl_data sdm845_data = {
> + .pin_count = 150,
> + .functions_count = ARRAY_SIZE(msm_pinctrl_functions),
> + .get_function_name = sdm845_get_function_name,
> + .get_function_mux = sdm845_get_function_mux,
> + .get_pin_name = sdm845_get_pin_name,
> +};
> diff --git a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
> index e6b87c3573..c0ed943036 100644
> --- a/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
> +++ b/arch/arm/mach-snapdragon/pinctrl-snapdragon.c
> @@ -116,6 +116,7 @@ static struct pinctrl_ops msm_pinctrl_ops = {
>  static const struct udevice_id msm_pinctrl_ids[] = {
>   { .compatible = "qcom,tlmm-apq8016", .data = (ulong)&apq8016_data },
>   { .compatible = "qcom,tlmm-apq8096", .data = (ulong)&apq8096_data },
> + { .compatible = "qcom,tlmm-sdm845", .data = (ulong)&sdm845_data },
>   { }
>  };
>
> diff --git a/arch/arm/mach-snapdragon/pinctrl-snapdragon.h b/arch/arm/mach-snapdragon/pinctrl-snapdragon.h
> index 61d466f4d8..ea524312a0 100644
> --- a/arch/arm/mach-snapdragon/pinctrl-snapdragon.h
> +++ b/arch/arm/mach-snapdragon/pinctrl-snapdragon.h
> @@ -27,5 +27,6 @@ struct pinctrl_function {
>
>  extern struct msm_pinctrl_data apq8016_data;
>  extern struct msm_pinctrl_data apq8096_data;
> +extern struct msm_pinctrl_data sdm845_data;
>
>  #endif
> diff --git a/drivers/gpio/msm_gpio.c b/drivers/gpio/msm_gpio.c
> index e1ff84c1c0..a3c3cd7824 100644
> --- a/drivers/gpio/msm_gpio.c
> +++ b/drivers/gpio/msm_gpio.c
> @@ -120,6 +120,7 @@ static const struct udevice_id msm_gpio_ids[] = {
>   { .compatible = "qcom,msm8916-pinctrl" },
>   { .compatible = "qcom,apq8016-pinctrl" },
>   { .compatible = "qcom,ipq4019-pinctrl" },
> + { .compatible = "qcom,sdm845-pinctrl" },
>   { }
>  };
>
> diff --git a/drivers/gpio/pm8916_gpio.c b/drivers/gpio/pm8916_gpio.c
> index 40b0f2578b..7ad95784a8 100644
> --- a/drivers/gpio/pm8916_gpio.c
> +++ b/drivers/gpio/pm8916_gpio.c
> @@ -202,6 +202,7 @@ static int pm8916_gpio_of_to_plat(struct udevice *dev)
>  static const struct udevice_id pm8916_gpio_ids[] = {
>   { .compatible = "qcom,pm8916-gpio" },
>   { .compatible = "qcom,pm8994-gpio" }, /* 22 GPIO's */
> + { .compatible = "qcom,pm8998-gpio" },
>   { }
>  };
>
> @@ -266,7 +267,7 @@ static int pm8941_pwrkey_probe(struct udevice *dev)
>   return log_msg_ret("bad type", -ENXIO);
>
>   reg = pmic_reg_read(dev->parent, priv->pid + REG_SUBTYPE);
> - if (reg != 0x1)
> + if ((reg & 0x5) == 0)
>   return log_msg_ret("bad subtype", -ENXIO);
>
>   return 0;
> @@ -287,11 +288,12 @@ static int pm8941_pwrkey_of_to_plat(struct udevice *dev)
>  static const struct udevice_id pm8941_pwrkey_ids[] = {
>   { .compatible = "qcom,pm8916-pwrkey" },
>   { .compatible = "qcom,pm8994-pwrkey" },
> + { .compatible = "qcom,pm8998-pwrkey" },
>   { }
>  };
>
> -U_BOOT_DRIVER(pwrkey_pm8941) = {
> - .name = "pwrkey_pm8916",
> +U_BOOT_DRIVER(pwrkey_pm89xx) = {
> + .name = "pwrkey_pm89xx",
>   .id = UCLASS_GPIO,
>   .of_match = pm8941_pwrkey_ids,
>   .of_to_plat = pm8941_pwrkey_of_to_plat,
> --
> 2.20.1
>
Reviewed-by: Ramon Fried <rfried.dev@gmail.com>

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-09-05 13:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-31  8:44 Subject: [PATCH 3/6] pinctrl: qcom: add pinctrl and gpio drivers for SDM845 Дмитрий Санковский
2021-09-05 13:35 ` Ramon Fried

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.