ARM: tegra: fix naming in GPIO DT binding header
Subsystems (3)
PATCH 1/1 ARM: tegra: fix naming in GPIO DT binding header
From: Stephen Warren <[email protected]>
According to the Tegra TRM, GPIOs are aggregated into /ports/ of 8 GPIOs, not into /banks/. Fix <dt-bindings/gpio/tegra-gpio.h> to correctly reflect this naming convention. While this seems like silly churn, it will become slightly more important once we introduce the GPIO binding for upcoming Tegra chips.
--- include/dt-bindings/gpio/tegra-gpio.h | 68 +++++++++++++++++------------------ 1 file changed, 34 insertions(+), 34 deletions(-)
include/dt-bindings/gpio/tegra-gpio.h +34 -34
@@ -12,40 +12,40 @@#include <dt-bindings/gpio/gpio.h> -#define TEGRA_GPIO_BANK_ID_A 0 -#define TEGRA_GPIO_BANK_ID_B 1 -#define TEGRA_GPIO_BANK_ID_C 2 -#define TEGRA_GPIO_BANK_ID_D 3 -#define TEGRA_GPIO_BANK_ID_E 4 -#define TEGRA_GPIO_BANK_ID_F 5 -#define TEGRA_GPIO_BANK_ID_G 6 -#define TEGRA_GPIO_BANK_ID_H 7 -#define TEGRA_GPIO_BANK_ID_I 8 -#define TEGRA_GPIO_BANK_ID_J 9 -#define TEGRA_GPIO_BANK_ID_K 10 -#define TEGRA_GPIO_BANK_ID_L 11 -#define TEGRA_GPIO_BANK_ID_M 12 -#define TEGRA_GPIO_BANK_ID_N 13 -#define TEGRA_GPIO_BANK_ID_O 14 -#define TEGRA_GPIO_BANK_ID_P 15 -#define TEGRA_GPIO_BANK_ID_Q 16 -#define TEGRA_GPIO_BANK_ID_R 17 -#define TEGRA_GPIO_BANK_ID_S 18 -#define TEGRA_GPIO_BANK_ID_T 19 -#define TEGRA_GPIO_BANK_ID_U 20 -#define TEGRA_GPIO_BANK_ID_V 21 -#define TEGRA_GPIO_BANK_ID_W 22 -#define TEGRA_GPIO_BANK_ID_X 23 -#define TEGRA_GPIO_BANK_ID_Y 24 -#define TEGRA_GPIO_BANK_ID_Z 25 -#define TEGRA_GPIO_BANK_ID_AA 26 -#define TEGRA_GPIO_BANK_ID_BB 27 -#define TEGRA_GPIO_BANK_ID_CC 28 -#define TEGRA_GPIO_BANK_ID_DD 29 -#define TEGRA_GPIO_BANK_ID_EE 30 -#define TEGRA_GPIO_BANK_ID_FF 31 +#define TEGRA_GPIO_PORT_A 0 +#define TEGRA_GPIO_PORT_B 1 +#define TEGRA_GPIO_PORT_C 2 +#define TEGRA_GPIO_PORT_D 3 +#define TEGRA_GPIO_PORT_E 4 +#define TEGRA_GPIO_PORT_F 5 +#define TEGRA_GPIO_PORT_G 6 +#define TEGRA_GPIO_PORT_H 7 +#define TEGRA_GPIO_PORT_I 8 +#define TEGRA_GPIO_PORT_J 9 +#define TEGRA_GPIO_PORT_K 10 +#define TEGRA_GPIO_PORT_L 11 +#define TEGRA_GPIO_PORT_M 12 +#define TEGRA_GPIO_PORT_N 13 +#define TEGRA_GPIO_PORT_O 14 +#define TEGRA_GPIO_PORT_P 15 +#define TEGRA_GPIO_PORT_Q 16 +#define TEGRA_GPIO_PORT_R 17 +#define TEGRA_GPIO_PORT_S 18 +#define TEGRA_GPIO_PORT_T 19 +#define TEGRA_GPIO_PORT_U 20 +#define TEGRA_GPIO_PORT_V 21 +#define TEGRA_GPIO_PORT_W 22 +#define TEGRA_GPIO_PORT_X 23 +#define TEGRA_GPIO_PORT_Y 24 +#define TEGRA_GPIO_PORT_Z 25 +#define TEGRA_GPIO_PORT_AA 26 +#define TEGRA_GPIO_PORT_BB 27 +#define TEGRA_GPIO_PORT_CC 28 +#define TEGRA_GPIO_PORT_DD 29 +#define TEGRA_GPIO_PORT_EE 30 +#define TEGRA_GPIO_PORT_FF 31 -#define TEGRA_GPIO (bank, offset) \ - ((TEGRA_GPIO_BANK_ID_##bank * 8) + offset) +#define TEGRA_GPIO (port, offset) \ + ((TEGRA_GPIO_PORT_##port * 8) + offset)#endif
Signature · 1 line
2.8.1
PATCH 1/1 ARM: tegra: fix naming in GPIO DT binding header
Quoted 8 lines · Stephen Warren
From: Stephen Warren <[email protected]>
According to the Tegra TRM, GPIOs are aggregated into /ports/ of 8 GPIOs, not into /banks/. Fix <
dt-bindings/gpio/tegra-gpio.h> to correctly reflect this naming convention. While this seems like silly churn, it will become slightly more important once we introduce the GPIO binding for upcoming Tegra chips.
Quoted 4 lines
Signed-off-by Stephen Warren <[email protected]>--- include/dt-bindings/gpio/tegra-gpio.h | 68 +++++++++++++++++------------------ 1 file changed, 34 insertions(+), 34 deletions(-)
It's not clear to me where this should be applied. This is technically part of the GPIO controller bindings, in which case it'd need to go via the GPIO tree. I'm fine with taking it through the Tegra tree, too, but in case you agree that it should go through the GPIO tree:
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160406/132c303f/attachment.sig>
PATCH 1/1 ARM: tegra: fix naming in GPIO DT binding header
Quoted 14 lines · Thierry Reding
Quoted 8 lines · Stephen Warren
From: Stephen Warren <[email protected]>
According to the Tegra TRM, GPIOs are aggregated into /ports/ of 8 GPIOs, not into /banks/. Fix <
dt-bindings/gpio/tegra-gpio.h> to correctly reflect this naming convention. While this seems like silly churn, it will become slightly more important once we introduce the GPIO binding for upcoming Tegra chips.Quoted 4 lines
Signed-off-by Stephen Warren <[email protected]>--- include/dt-bindings/gpio/tegra-gpio.h | 68 +++++++++++++++++------------------ 1 file changed, 34 insertions(+), 34 deletions(-)
Quoted 6 lines
It's not clear to me where this should be applied. This is technically part of the GPIO controller bindings, in which case it'd need to go via the GPIO tree. I'm fine with taking it through the Tegra tree, too, but in case you agree that it should go through the GPIO tree:
Acked-by Thierry Reding <[email protected]>
I typically consider bindings part of the SoC code-base they related to, so I'd imagine this going through the Tegra tree. I didn't Cc LinusW on the patch because of that thinking and oversight, but have done so now just in case he feels strongly.
PATCH 1/1 ARM: tegra: fix naming in GPIO DT binding header
Quoted 25 lines · Stephen Warren
Quoted 24 lines · Thierry Reding
Quoted 10 lines · Stephen Warren
From: Stephen Warren <[email protected]>
According to the Tegra TRM, GPIOs are aggregated into /ports/ of 8 GPIOs, not into /banks/. Fix <
dt-bindings/gpio/tegra-gpio.h> to correctly reflect
this naming convention. While this seems like silly churn, it will become slightly more important once we introduce the GPIO binding for upcoming Tegra chips.Quoted 5 lines
Signed-off-by Stephen Warren <[email protected]>--- include/dt-bindings/gpio/tegra-gpio.h | 68 +++++++++++++++++------------------ 1 file changed, 34 insertions(+), 34 deletions(-)It's not clear to me where this should be applied. This is technically part of the GPIO controller bindings, in which case it'd need to go via the GPIO tree. I'm fine with taking it through the Tegra tree, too, but in case you agree that it should go through the GPIO tree:
Quoted 6 lines
Quoted 1 line
Acked-by Thierry Reding <[email protected]>I typically consider bindings part of the SoC code-base they related to, so I'd imagine this going through the Tegra tree. I didn't Cc LinusW on the patch because of that thinking and oversight, but have done so now just in case he feels strongly.
No strong opinion, only time I care is when we merge a new driver and it #includes <dt-bindings/...>.
Take it through the tegra tree.
Yours,
Linus Walleij
Thierry Reding
AppliedPATCH 1/1 ARM: tegra: fix naming in GPIO DT binding header
Quoted 33 lines · Linus Walleij
Quoted 25 lines · Stephen Warren
Quoted 24 lines · Thierry Reding
Quoted 10 lines · Stephen Warren
From: Stephen Warren <[email protected]>
According to the Tegra TRM, GPIOs are aggregated into /ports/ of 8 GPIOs, not into /banks/. Fix <
dt-bindings/gpio/tegra-gpio.h> to correctly reflect
this naming convention. While this seems like silly churn, it will become slightly more important once we introduce the GPIO binding for upcoming Tegra chips.Quoted 5 lines
Signed-off-by Stephen Warren <[email protected]>--- include/dt-bindings/gpio/tegra-gpio.h | 68 +++++++++++++++++------------------ 1 file changed, 34 insertions(+), 34 deletions(-)It's not clear to me where this should be applied. This is technically part of the GPIO controller bindings, in which case it'd need to go via the GPIO tree. I'm fine with taking it through the Tegra tree, too, but in case you agree that it should go through the GPIO tree:
Quoted 6 lines
Quoted 1 line
Acked-by Thierry Reding <[email protected]>I typically consider bindings part of the SoC code-base they related to, so I'd imagine this going through the Tegra tree. I didn't Cc LinusW on the patch because of that thinking and oversight, but have done so now just in case he feels strongly.
Quoted 5 lines
No strong opinion, only time I care is when we merge a new driver and it #includes <dt-bindings/...>.
Take it through the tegra tree.
Acked-by Linus Walleij <[email protected]>
Applied, thanks.
Thierry
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160422/0f5b4c7d/attachment.sig>