On 6/18/2017 10:04 PM, Rob Herring wrote:
On Tue, Jun 13, 2017 at 02:16:03PM +0800, [email protected] wrote:
From: Fenglin Wu <[email protected]>

GPIO LV (low voltage)/MV (medium voltage) subtypes have different
features and register mappings than 4CH/8CH subtypes. Add support
for LV and MV subtypes.

Signed-off-by: Fenglin Wu <[email protected]>
---
  .../devicetree/bindings/pinctrl/qcom,pmic-gpio.txt |  28 ++-
  drivers/pinctrl/qcom/pinctrl-spmi-gpio.c           | 269 ++++++++++++++++++---
  include/dt-bindings/pinctrl/qcom,pmic-gpio.h       |   9 +
  3 files changed, 264 insertions(+), 42 deletions(-)

[...]

diff --git a/include/dt-bindings/pinctrl/qcom,pmic-gpio.h 
b/include/dt-bindings/pinctrl/qcom,pmic-gpio.h
index d33f17c..85d8809 100644
--- a/include/dt-bindings/pinctrl/qcom,pmic-gpio.h
+++ b/include/dt-bindings/pinctrl/qcom,pmic-gpio.h
@@ -93,15 +93,24 @@
  #define PM8994_GPIO_S4                        2
  #define PM8994_GPIO_L12                       3
+/* ATEST MUX selection for analog-pass-through mode */
+#define PMIC_GPIO_AOUT_ATEST1          0
+#define PMIC_GPIO_AOUT_ATEST2          1
+#define PMIC_GPIO_AOUT_ATEST3          2
+#define PMIC_GPIO_AOUT_ATEST4          3
+
  /* To be used with "function" */
  #define PMIC_GPIO_FUNC_NORMAL         "normal"
  #define PMIC_GPIO_FUNC_PAIRED         "paired"
  #define PMIC_GPIO_FUNC_FUNC1          "func1"
  #define PMIC_GPIO_FUNC_FUNC2          "func2"
+#define PMIC_GPIO_FUNC_FUNC3           "func3"
+#define PMIC_GPIO_FUNC_FUNC4           "func4"
  #define PMIC_GPIO_FUNC_DTEST1         "dtest1"
  #define PMIC_GPIO_FUNC_DTEST2         "dtest2"
  #define PMIC_GPIO_FUNC_DTEST3         "dtest3"
  #define PMIC_GPIO_FUNC_DTEST4         "dtest4"
+#define PMIC_GPIO_FUNC_ANALOG          "analog"

defines for strings? That's really pointless. I'd prefer you drop using
them than add more.

Thanks for the suggestion, I will remove these string definitions in next patch.
Does other part look good? I would post a new patch if no other comments.

Rob
--
To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


--
Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,\n a Linux Foundation Collaborative Project.

Reply via email to