* [PATCH v2] gpio: aspeed: port Linux dt-bindings header file
@ 2022-08-04 1:02 Dhananjay Phadke
2022-08-04 2:14 ` Billy Tsai
2022-08-04 2:35 ` Joel Stanley
0 siblings, 2 replies; 4+ messages in thread
From: Dhananjay Phadke @ 2022-08-04 1:02 UTC (permalink / raw)
To: Ryan Chen, Chia-Wei Wang, Aspeed BMC SW team, Joel Stanley,
Billy Tsai, u-boot
Cc: Dhananjay Phadke
Ported as is, makes it easier to add readable GPIO definitions
in DTS files.
Signed-off-by: Dhananjay Phadke <dphadke@linux.microsoft.com>
---
include/dt-bindings/gpio/aspeed-gpio.h | 49 ++++++++++++++++++++++++++
1 file changed, 49 insertions(+)
create mode 100644 include/dt-bindings/gpio/aspeed-gpio.h
diff --git a/include/dt-bindings/gpio/aspeed-gpio.h b/include/dt-bindings/gpio/aspeed-gpio.h
new file mode 100644
index 0000000000..adab2f56c2
--- /dev/null
+++ b/include/dt-bindings/gpio/aspeed-gpio.h
@@ -0,0 +1,49 @@
+/* SPDX-License-Identifier: GPL-2.0+ */
+/*
+ * This header provides constants for binding aspeed,*-gpio.
+ *
+ * The first cell in Aspeed's GPIO specifier is the GPIO ID. The macros below
+ * provide names for this.
+ *
+ * The second cell contains standard flag values specified in gpio.h.
+ */
+
+#ifndef _DT_BINDINGS_GPIO_ASPEED_GPIO_H
+#define _DT_BINDINGS_GPIO_ASPEED_GPIO_H
+
+#include <dt-bindings/gpio/gpio.h>
+
+#define ASPEED_GPIO_PORT_A 0
+#define ASPEED_GPIO_PORT_B 1
+#define ASPEED_GPIO_PORT_C 2
+#define ASPEED_GPIO_PORT_D 3
+#define ASPEED_GPIO_PORT_E 4
+#define ASPEED_GPIO_PORT_F 5
+#define ASPEED_GPIO_PORT_G 6
+#define ASPEED_GPIO_PORT_H 7
+#define ASPEED_GPIO_PORT_I 8
+#define ASPEED_GPIO_PORT_J 9
+#define ASPEED_GPIO_PORT_K 10
+#define ASPEED_GPIO_PORT_L 11
+#define ASPEED_GPIO_PORT_M 12
+#define ASPEED_GPIO_PORT_N 13
+#define ASPEED_GPIO_PORT_O 14
+#define ASPEED_GPIO_PORT_P 15
+#define ASPEED_GPIO_PORT_Q 16
+#define ASPEED_GPIO_PORT_R 17
+#define ASPEED_GPIO_PORT_S 18
+#define ASPEED_GPIO_PORT_T 19
+#define ASPEED_GPIO_PORT_U 20
+#define ASPEED_GPIO_PORT_V 21
+#define ASPEED_GPIO_PORT_W 22
+#define ASPEED_GPIO_PORT_X 23
+#define ASPEED_GPIO_PORT_Y 24
+#define ASPEED_GPIO_PORT_Z 25
+#define ASPEED_GPIO_PORT_AA 26
+#define ASPEED_GPIO_PORT_AB 27
+#define ASPEED_GPIO_PORT_AC 28
+
+#define ASPEED_GPIO(port, offset) \
+ ((ASPEED_GPIO_PORT_##port * 8) + (offset))
+
+#endif
--
2.25.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2] gpio: aspeed: port Linux dt-bindings header file
2022-08-04 1:02 [PATCH v2] gpio: aspeed: port Linux dt-bindings header file Dhananjay Phadke
@ 2022-08-04 2:14 ` Billy Tsai
2022-08-04 2:35 ` Joel Stanley
1 sibling, 0 replies; 4+ messages in thread
From: Billy Tsai @ 2022-08-04 2:14 UTC (permalink / raw)
To: Dhananjay Phadke, Ryan Chen, ChiaWei Wang, BMC-SW, Joel Stanley,
u-boot@lists.denx.de
Thanks.
Reviewed-by: Billy Tsai <billy_tsai@aspeedtech.com>
On 2022/8/4, 9:02 AM, "Dhananjay Phadke" <dphadke@linux.microsoft.com> wrote:
Ported as is, makes it easier to add readable GPIO definitions
in DTS files.
Signed-off-by: Dhananjay Phadke <dphadke@linux.microsoft.com>
---
include/dt-bindings/gpio/aspeed-gpio.h | 49 ++++++++++++++++++++++++++
1 file changed, 49 insertions(+)
create mode 100644 include/dt-bindings/gpio/aspeed-gpio.h
diff --git a/include/dt-bindings/gpio/aspeed-gpio.h b/include/dt-bindings/gpio/aspeed-gpio.h
new file mode 100644
index 0000000000..adab2f56c2
--- /dev/null
+++ b/include/dt-bindings/gpio/aspeed-gpio.h
@@ -0,0 +1,49 @@
+/* SPDX-License-Identifier: GPL-2.0+ */
+/*
+ * This header provides constants for binding aspeed,*-gpio.
+ *
+ * The first cell in Aspeed's GPIO specifier is the GPIO ID. The macros below
+ * provide names for this.
+ *
+ * The second cell contains standard flag values specified in gpio.h.
+ */
+
+#ifndef _DT_BINDINGS_GPIO_ASPEED_GPIO_H
+#define _DT_BINDINGS_GPIO_ASPEED_GPIO_H
+
+#include <dt-bindings/gpio/gpio.h>
+
+#define ASPEED_GPIO_PORT_A 0
+#define ASPEED_GPIO_PORT_B 1
+#define ASPEED_GPIO_PORT_C 2
+#define ASPEED_GPIO_PORT_D 3
+#define ASPEED_GPIO_PORT_E 4
+#define ASPEED_GPIO_PORT_F 5
+#define ASPEED_GPIO_PORT_G 6
+#define ASPEED_GPIO_PORT_H 7
+#define ASPEED_GPIO_PORT_I 8
+#define ASPEED_GPIO_PORT_J 9
+#define ASPEED_GPIO_PORT_K 10
+#define ASPEED_GPIO_PORT_L 11
+#define ASPEED_GPIO_PORT_M 12
+#define ASPEED_GPIO_PORT_N 13
+#define ASPEED_GPIO_PORT_O 14
+#define ASPEED_GPIO_PORT_P 15
+#define ASPEED_GPIO_PORT_Q 16
+#define ASPEED_GPIO_PORT_R 17
+#define ASPEED_GPIO_PORT_S 18
+#define ASPEED_GPIO_PORT_T 19
+#define ASPEED_GPIO_PORT_U 20
+#define ASPEED_GPIO_PORT_V 21
+#define ASPEED_GPIO_PORT_W 22
+#define ASPEED_GPIO_PORT_X 23
+#define ASPEED_GPIO_PORT_Y 24
+#define ASPEED_GPIO_PORT_Z 25
+#define ASPEED_GPIO_PORT_AA 26
+#define ASPEED_GPIO_PORT_AB 27
+#define ASPEED_GPIO_PORT_AC 28
+
+#define ASPEED_GPIO(port, offset) \
+ ((ASPEED_GPIO_PORT_##port * 8) + (offset))
+
+#endif
--
2.25.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] gpio: aspeed: port Linux dt-bindings header file
2022-08-04 1:02 [PATCH v2] gpio: aspeed: port Linux dt-bindings header file Dhananjay Phadke
2022-08-04 2:14 ` Billy Tsai
@ 2022-08-04 2:35 ` Joel Stanley
2022-08-04 15:43 ` Dhananjay Phadke
1 sibling, 1 reply; 4+ messages in thread
From: Joel Stanley @ 2022-08-04 2:35 UTC (permalink / raw)
To: Dhananjay Phadke
Cc: Ryan Chen, Chia-Wei Wang, Aspeed BMC SW team, Billy Tsai, u-boot
On Thu, 4 Aug 2022 at 01:02, Dhananjay Phadke
<dphadke@linux.microsoft.com> wrote:
>
> Ported as is, makes it easier to add readable GPIO definitions
> in DTS files.
>
> Signed-off-by: Dhananjay Phadke <dphadke@linux.microsoft.com>
> ---
> include/dt-bindings/gpio/aspeed-gpio.h | 49 ++++++++++++++++++++++++++
> 1 file changed, 49 insertions(+)
> create mode 100644 include/dt-bindings/gpio/aspeed-gpio.h
>
> diff --git a/include/dt-bindings/gpio/aspeed-gpio.h b/include/dt-bindings/gpio/aspeed-gpio.h
> new file mode 100644
> index 0000000000..adab2f56c2
> --- /dev/null
> +++ b/include/dt-bindings/gpio/aspeed-gpio.h
> @@ -0,0 +1,49 @@
> +/* SPDX-License-Identifier: GPL-2.0+ */
As you mention this is from the kernel, but you forgot to add the
copyright information for the file you copied. I suggest you add:
Copyright 2022 IBM Corp.
Cheers,
Joel
> +/*
> + * This header provides constants for binding aspeed,*-gpio.
> + *
> + * The first cell in Aspeed's GPIO specifier is the GPIO ID. The macros below
> + * provide names for this.
> + *
> + * The second cell contains standard flag values specified in gpio.h.
> + */
> +
> +#ifndef _DT_BINDINGS_GPIO_ASPEED_GPIO_H
> +#define _DT_BINDINGS_GPIO_ASPEED_GPIO_H
> +
> +#include <dt-bindings/gpio/gpio.h>
> +
> +#define ASPEED_GPIO_PORT_A 0
> +#define ASPEED_GPIO_PORT_B 1
> +#define ASPEED_GPIO_PORT_C 2
> +#define ASPEED_GPIO_PORT_D 3
> +#define ASPEED_GPIO_PORT_E 4
> +#define ASPEED_GPIO_PORT_F 5
> +#define ASPEED_GPIO_PORT_G 6
> +#define ASPEED_GPIO_PORT_H 7
> +#define ASPEED_GPIO_PORT_I 8
> +#define ASPEED_GPIO_PORT_J 9
> +#define ASPEED_GPIO_PORT_K 10
> +#define ASPEED_GPIO_PORT_L 11
> +#define ASPEED_GPIO_PORT_M 12
> +#define ASPEED_GPIO_PORT_N 13
> +#define ASPEED_GPIO_PORT_O 14
> +#define ASPEED_GPIO_PORT_P 15
> +#define ASPEED_GPIO_PORT_Q 16
> +#define ASPEED_GPIO_PORT_R 17
> +#define ASPEED_GPIO_PORT_S 18
> +#define ASPEED_GPIO_PORT_T 19
> +#define ASPEED_GPIO_PORT_U 20
> +#define ASPEED_GPIO_PORT_V 21
> +#define ASPEED_GPIO_PORT_W 22
> +#define ASPEED_GPIO_PORT_X 23
> +#define ASPEED_GPIO_PORT_Y 24
> +#define ASPEED_GPIO_PORT_Z 25
> +#define ASPEED_GPIO_PORT_AA 26
> +#define ASPEED_GPIO_PORT_AB 27
> +#define ASPEED_GPIO_PORT_AC 28
> +
> +#define ASPEED_GPIO(port, offset) \
> + ((ASPEED_GPIO_PORT_##port * 8) + (offset))
> +
> +#endif
> --
> 2.25.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2] gpio: aspeed: port Linux dt-bindings header file
2022-08-04 2:35 ` Joel Stanley
@ 2022-08-04 15:43 ` Dhananjay Phadke
0 siblings, 0 replies; 4+ messages in thread
From: Dhananjay Phadke @ 2022-08-04 15:43 UTC (permalink / raw)
To: Joel Stanley
Cc: Ryan Chen, Chia-Wei Wang, Aspeed BMC SW team, Billy Tsai, u-boot
On 8/3/2022 7:35 PM, Joel Stanley wrote:
> On Thu, 4 Aug 2022 at 01:02, Dhananjay Phadke
> <dphadke@linux.microsoft.com> wrote:
>>
>> Ported as is, makes it easier to add readable GPIO definitions
>> in DTS files.
>>
>> Signed-off-by: Dhananjay Phadke <dphadke@linux.microsoft.com>
>> ---
>> include/dt-bindings/gpio/aspeed-gpio.h | 49 ++++++++++++++++++++++++++
>> 1 file changed, 49 insertions(+)
>> create mode 100644 include/dt-bindings/gpio/aspeed-gpio.h
>>
>> diff --git a/include/dt-bindings/gpio/aspeed-gpio.h b/include/dt-bindings/gpio/aspeed-gpio.h
>> new file mode 100644
>> index 0000000000..adab2f56c2
>> --- /dev/null
>> +++ b/include/dt-bindings/gpio/aspeed-gpio.h
>> @@ -0,0 +1,49 @@
>> +/* SPDX-License-Identifier: GPL-2.0+ */
>
> As you mention this is from the kernel, but you forgot to add the
> copyright information for the file you copied. I suggest you add:
>
> Copyright 2022 IBM Corp.
It would have been nicer to assert copyrights in kernel source itself,
so that it would have been inherited instead of inserting here.
For now, will send v3 with just that.
Thanks,
Dhananjay
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-08-04 15:43 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-08-04 1:02 [PATCH v2] gpio: aspeed: port Linux dt-bindings header file Dhananjay Phadke
2022-08-04 2:14 ` Billy Tsai
2022-08-04 2:35 ` Joel Stanley
2022-08-04 15:43 ` Dhananjay Phadke
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox