All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: tegra: fix naming in GPIO DT binding header
@ 2016-04-05 19:25 ` Stephen Warren
  0 siblings, 0 replies; 10+ messages in thread
From: Stephen Warren @ 2016-04-05 19:25 UTC (permalink / raw)
  To: Thierry Reding, Alexandre Courbot
  Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	Stephen Warren

From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

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.

Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
 include/dt-bindings/gpio/tegra-gpio.h | 68 +++++++++++++++++------------------
 1 file changed, 34 insertions(+), 34 deletions(-)

diff --git a/include/dt-bindings/gpio/tegra-gpio.h b/include/dt-bindings/gpio/tegra-gpio.h
index 197dc28b676e..a1c09e88e80b 100644
--- a/include/dt-bindings/gpio/tegra-gpio.h
+++ b/include/dt-bindings/gpio/tegra-gpio.h
@@ -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
-- 
2.8.1

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

* [PATCH] ARM: tegra: fix naming in GPIO DT binding header
@ 2016-04-05 19:25 ` Stephen Warren
  0 siblings, 0 replies; 10+ messages in thread
From: Stephen Warren @ 2016-04-05 19:25 UTC (permalink / raw)
  To: linux-arm-kernel

From: Stephen Warren <swarren@nvidia.com>

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.

Signed-off-by: Stephen Warren <swarren@nvidia.com>
---
 include/dt-bindings/gpio/tegra-gpio.h | 68 +++++++++++++++++------------------
 1 file changed, 34 insertions(+), 34 deletions(-)

diff --git a/include/dt-bindings/gpio/tegra-gpio.h b/include/dt-bindings/gpio/tegra-gpio.h
index 197dc28b676e..a1c09e88e80b 100644
--- a/include/dt-bindings/gpio/tegra-gpio.h
+++ b/include/dt-bindings/gpio/tegra-gpio.h
@@ -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
-- 
2.8.1

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

* Re: [PATCH] ARM: tegra: fix naming in GPIO DT binding header
  2016-04-05 19:25 ` Stephen Warren
@ 2016-04-06 17:28     ` Thierry Reding
  -1 siblings, 0 replies; 10+ messages in thread
From: Thierry Reding @ 2016-04-06 17:28 UTC (permalink / raw)
  To: Stephen Warren
  Cc: Alexandre Courbot, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	Stephen Warren

[-- Attachment #1: Type: text/plain, Size: 1055 bytes --]

On Tue, Apr 05, 2016 at 01:25:07PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> 
> 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.
> 
> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
>  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:

Acked-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

* [PATCH] ARM: tegra: fix naming in GPIO DT binding header
@ 2016-04-06 17:28     ` Thierry Reding
  0 siblings, 0 replies; 10+ messages in thread
From: Thierry Reding @ 2016-04-06 17:28 UTC (permalink / raw)
  To: linux-arm-kernel

On Tue, Apr 05, 2016 at 01:25:07PM -0600, Stephen Warren wrote:
> From: Stephen Warren <swarren@nvidia.com>
> 
> 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.
> 
> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> ---
>  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:

Acked-by: Thierry Reding <treding@nvidia.com>
-------------- 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>

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

* Re: [PATCH] ARM: tegra: fix naming in GPIO DT binding header
  2016-04-06 17:28     ` Thierry Reding
@ 2016-04-06 17:32         ` Stephen Warren
  -1 siblings, 0 replies; 10+ messages in thread
From: Stephen Warren @ 2016-04-06 17:32 UTC (permalink / raw)
  To: Thierry Reding, Linus Walleij
  Cc: Alexandre Courbot, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	Stephen Warren

On 04/06/2016 11:28 AM, Thierry Reding wrote:
> On Tue, Apr 05, 2016 at 01:25:07PM -0600, Stephen Warren wrote:
>> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>
>> 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.
>>
>> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>> ---
>>   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:
>
> Acked-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

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.

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

* [PATCH] ARM: tegra: fix naming in GPIO DT binding header
@ 2016-04-06 17:32         ` Stephen Warren
  0 siblings, 0 replies; 10+ messages in thread
From: Stephen Warren @ 2016-04-06 17:32 UTC (permalink / raw)
  To: linux-arm-kernel

On 04/06/2016 11:28 AM, Thierry Reding wrote:
> On Tue, Apr 05, 2016 at 01:25:07PM -0600, Stephen Warren wrote:
>> From: Stephen Warren <swarren@nvidia.com>
>>
>> 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.
>>
>> Signed-off-by: Stephen Warren <swarren@nvidia.com>
>> ---
>>   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:
>
> Acked-by: Thierry Reding <treding@nvidia.com>

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.

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

* Re: [PATCH] ARM: tegra: fix naming in GPIO DT binding header
  2016-04-06 17:32         ` Stephen Warren
@ 2016-04-11  7:04             ` Linus Walleij
  -1 siblings, 0 replies; 10+ messages in thread
From: Linus Walleij @ 2016-04-11  7:04 UTC (permalink / raw)
  To: Stephen Warren
  Cc: Thierry Reding, Alexandre Courbot,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	Stephen Warren

On Wed, Apr 6, 2016 at 7:32 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> On 04/06/2016 11:28 AM, Thierry Reding wrote:
>>
>> On Tue, Apr 05, 2016 at 01:25:07PM -0600, Stephen Warren wrote:
>>>
>>> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>>
>>> 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.
>>>
>>> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>> ---
>>>   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:
>>
>> Acked-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>
> 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.
Acked-by: Linus Walleij <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>

Yours,
Linus Walleij

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

* [PATCH] ARM: tegra: fix naming in GPIO DT binding header
@ 2016-04-11  7:04             ` Linus Walleij
  0 siblings, 0 replies; 10+ messages in thread
From: Linus Walleij @ 2016-04-11  7:04 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, Apr 6, 2016 at 7:32 PM, Stephen Warren <swarren@wwwdotorg.org> wrote:
> On 04/06/2016 11:28 AM, Thierry Reding wrote:
>>
>> On Tue, Apr 05, 2016 at 01:25:07PM -0600, Stephen Warren wrote:
>>>
>>> From: Stephen Warren <swarren@nvidia.com>
>>>
>>> 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.
>>>
>>> Signed-off-by: Stephen Warren <swarren@nvidia.com>
>>> ---
>>>   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:
>>
>> Acked-by: Thierry Reding <treding@nvidia.com>
>
> 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.
Acked-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij

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

* Re: [PATCH] ARM: tegra: fix naming in GPIO DT binding header
  2016-04-11  7:04             ` Linus Walleij
@ 2016-04-22 11:24                 ` Thierry Reding
  -1 siblings, 0 replies; 10+ messages in thread
From: Thierry Reding @ 2016-04-22 11:24 UTC (permalink / raw)
  To: Linus Walleij
  Cc: Stephen Warren, Alexandre Courbot,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	Stephen Warren

[-- Attachment #1: Type: text/plain, Size: 1934 bytes --]

On Mon, Apr 11, 2016 at 09:04:09AM +0200, Linus Walleij wrote:
> On Wed, Apr 6, 2016 at 7:32 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> > On 04/06/2016 11:28 AM, Thierry Reding wrote:
> >>
> >> On Tue, Apr 05, 2016 at 01:25:07PM -0600, Stephen Warren wrote:
> >>>
> >>> From: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >>>
> >>> 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.
> >>>
> >>> Signed-off-by: Stephen Warren <swarren-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >>> ---
> >>>   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:
> >>
> >> Acked-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> >
> > 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.
> Acked-by: Linus Walleij <linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>

Applied, thanks.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

* [PATCH] ARM: tegra: fix naming in GPIO DT binding header
@ 2016-04-22 11:24                 ` Thierry Reding
  0 siblings, 0 replies; 10+ messages in thread
From: Thierry Reding @ 2016-04-22 11:24 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Apr 11, 2016 at 09:04:09AM +0200, Linus Walleij wrote:
> On Wed, Apr 6, 2016 at 7:32 PM, Stephen Warren <swarren@wwwdotorg.org> wrote:
> > On 04/06/2016 11:28 AM, Thierry Reding wrote:
> >>
> >> On Tue, Apr 05, 2016 at 01:25:07PM -0600, Stephen Warren wrote:
> >>>
> >>> From: Stephen Warren <swarren@nvidia.com>
> >>>
> >>> 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.
> >>>
> >>> Signed-off-by: Stephen Warren <swarren@nvidia.com>
> >>> ---
> >>>   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:
> >>
> >> Acked-by: Thierry Reding <treding@nvidia.com>
> >
> > 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.
> Acked-by: Linus Walleij <linus.walleij@linaro.org>

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>

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

end of thread, other threads:[~2016-04-22 11:24 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-05 19:25 [PATCH] ARM: tegra: fix naming in GPIO DT binding header Stephen Warren
2016-04-05 19:25 ` Stephen Warren
     [not found] ` <1459884307-24043-1-git-send-email-swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2016-04-06 17:28   ` Thierry Reding
2016-04-06 17:28     ` Thierry Reding
     [not found]     ` <20160406172845.GC28843-EkSeR96xj6Pcmrwk2tT4+A@public.gmane.org>
2016-04-06 17:32       ` Stephen Warren
2016-04-06 17:32         ` Stephen Warren
     [not found]         ` <5705481D.3030700-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2016-04-11  7:04           ` Linus Walleij
2016-04-11  7:04             ` Linus Walleij
     [not found]             ` <CACRpkdZxeV07_rb2xL+HRGp-ay1B-U=_tEbPCYqvM+ZYD-wpYQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-04-22 11:24               ` Thierry Reding
2016-04-22 11:24                 ` Thierry Reding

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.