From: Conor Dooley <conor@kernel.org>
To: Huqiang Qin <huqiang.qin@amlogic.com>
Cc: linus.walleij@linaro.org, robh+dt@kernel.org,
krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org,
neil.armstrong@linaro.org, khilman@baylibre.com,
jbrunet@baylibre.com, martin.blumenstingl@googlemail.com,
brgl@bgdev.pl, andy@kernel.org, linux-gpio@vger.kernel.org,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH V2 1/3] dt-bindings: pinctrl: Add compatibles for Amlogic T7 SoCs
Date: Thu, 21 Sep 2023 10:19:37 +0100 [thread overview]
Message-ID: <20230921-fa16f2f1f118091489708226@fedora> (raw)
In-Reply-To: <20230921083407.1167510-3-huqiang.qin@amlogic.com>
[-- Attachment #1: Type: text/plain, Size: 5972 bytes --]
On Thu, Sep 21, 2023 at 04:34:06PM +0800, Huqiang Qin wrote:
> Add a new compatible name for Amlogic T7 pin controller, and add
> a new dt-binding header file which document the detail pin names.
>
> Signed-off-by: Huqiang Qin <huqiang.qin@amlogic.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
I'd argue that "Add support for" would be a better $subject for the
changes here though given that there is only one compatible and you've
added a bunch of new definitions.
Thanks,
Conor.
> ---
>
> V1 -> V2: Rename amlogic-t7-gpio.h to amlogic,t7-periphs-pinctrl.h
>
> .../pinctrl/amlogic,meson-pinctrl-a1.yaml | 1 +
> .../gpio/amlogic,t7-periphs-pinctrl.h | 179 ++++++++++++++++++
> 2 files changed, 180 insertions(+)
> create mode 100644 include/dt-bindings/gpio/amlogic,t7-periphs-pinctrl.h
>
> diff --git a/Documentation/devicetree/bindings/pinctrl/amlogic,meson-pinctrl-a1.yaml b/Documentation/devicetree/bindings/pinctrl/amlogic,meson-pinctrl-a1.yaml
> index 4e7a456ea4cc..c7df4cd34197 100644
> --- a/Documentation/devicetree/bindings/pinctrl/amlogic,meson-pinctrl-a1.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/amlogic,meson-pinctrl-a1.yaml
> @@ -16,6 +16,7 @@ properties:
> compatible:
> enum:
> - amlogic,c3-periphs-pinctrl
> + - amlogic,t7-periphs-pinctrl
> - amlogic,meson-a1-periphs-pinctrl
> - amlogic,meson-s4-periphs-pinctrl
>
> diff --git a/include/dt-bindings/gpio/amlogic,t7-periphs-pinctrl.h b/include/dt-bindings/gpio/amlogic,t7-periphs-pinctrl.h
> new file mode 100644
> index 000000000000..4e16d31a71c9
> --- /dev/null
> +++ b/include/dt-bindings/gpio/amlogic,t7-periphs-pinctrl.h
> @@ -0,0 +1,179 @@
> +/* SPDX-License-Identifier: (GPL-2.0-only OR MIT) */
> +/*
> + * Copyright (c) 2023 Amlogic, Inc. All rights reserved.
> + * Author: Huqiang Qin <huqiang.qin@amlogic.com>
> + */
> +
> +#ifndef _DT_BINDINGS_AMLOGIC_T7_GPIO_H
> +#define _DT_BINDINGS_AMLOGIC_T7_GPIO_H
> +
> +#define GPIOB_0 0
> +#define GPIOB_1 1
> +#define GPIOB_2 2
> +#define GPIOB_3 3
> +#define GPIOB_4 4
> +#define GPIOB_5 5
> +#define GPIOB_6 6
> +#define GPIOB_7 7
> +#define GPIOB_8 8
> +#define GPIOB_9 9
> +#define GPIOB_10 10
> +#define GPIOB_11 11
> +#define GPIOB_12 12
> +
> +#define GPIOC_0 13
> +#define GPIOC_1 14
> +#define GPIOC_2 15
> +#define GPIOC_3 16
> +#define GPIOC_4 17
> +#define GPIOC_5 18
> +#define GPIOC_6 19
> +
> +#define GPIOX_0 20
> +#define GPIOX_1 21
> +#define GPIOX_2 22
> +#define GPIOX_3 23
> +#define GPIOX_4 24
> +#define GPIOX_5 25
> +#define GPIOX_6 26
> +#define GPIOX_7 27
> +#define GPIOX_8 28
> +#define GPIOX_9 29
> +#define GPIOX_10 30
> +#define GPIOX_11 31
> +#define GPIOX_12 32
> +#define GPIOX_13 33
> +#define GPIOX_14 34
> +#define GPIOX_15 35
> +#define GPIOX_16 36
> +#define GPIOX_17 37
> +#define GPIOX_18 38
> +#define GPIOX_19 39
> +
> +#define GPIOW_0 40
> +#define GPIOW_1 41
> +#define GPIOW_2 42
> +#define GPIOW_3 43
> +#define GPIOW_4 44
> +#define GPIOW_5 45
> +#define GPIOW_6 46
> +#define GPIOW_7 47
> +#define GPIOW_8 48
> +#define GPIOW_9 49
> +#define GPIOW_10 50
> +#define GPIOW_11 51
> +#define GPIOW_12 52
> +#define GPIOW_13 53
> +#define GPIOW_14 54
> +#define GPIOW_15 55
> +#define GPIOW_16 56
> +
> +#define GPIOD_0 57
> +#define GPIOD_1 58
> +#define GPIOD_2 59
> +#define GPIOD_3 60
> +#define GPIOD_4 61
> +#define GPIOD_5 62
> +#define GPIOD_6 63
> +#define GPIOD_7 64
> +#define GPIOD_8 65
> +#define GPIOD_9 66
> +#define GPIOD_10 67
> +#define GPIOD_11 68
> +#define GPIOD_12 69
> +
> +#define GPIOE_0 70
> +#define GPIOE_1 71
> +#define GPIOE_2 72
> +#define GPIOE_3 73
> +#define GPIOE_4 74
> +#define GPIOE_5 75
> +#define GPIOE_6 76
> +
> +#define GPIOZ_0 77
> +#define GPIOZ_1 78
> +#define GPIOZ_2 79
> +#define GPIOZ_3 80
> +#define GPIOZ_4 81
> +#define GPIOZ_5 82
> +#define GPIOZ_6 83
> +#define GPIOZ_7 84
> +#define GPIOZ_8 85
> +#define GPIOZ_9 86
> +#define GPIOZ_10 87
> +#define GPIOZ_11 88
> +#define GPIOZ_12 89
> +#define GPIOZ_13 90
> +
> +#define GPIOT_0 91
> +#define GPIOT_1 92
> +#define GPIOT_2 93
> +#define GPIOT_3 94
> +#define GPIOT_4 95
> +#define GPIOT_5 96
> +#define GPIOT_6 97
> +#define GPIOT_7 98
> +#define GPIOT_8 99
> +#define GPIOT_9 100
> +#define GPIOT_10 101
> +#define GPIOT_11 102
> +#define GPIOT_12 103
> +#define GPIOT_13 104
> +#define GPIOT_14 105
> +#define GPIOT_15 106
> +#define GPIOT_16 107
> +#define GPIOT_17 108
> +#define GPIOT_18 109
> +#define GPIOT_19 110
> +#define GPIOT_20 111
> +#define GPIOT_21 112
> +#define GPIOT_22 113
> +#define GPIOT_23 114
> +
> +#define GPIOM_0 115
> +#define GPIOM_1 116
> +#define GPIOM_2 117
> +#define GPIOM_3 118
> +#define GPIOM_4 119
> +#define GPIOM_5 120
> +#define GPIOM_6 121
> +#define GPIOM_7 122
> +#define GPIOM_8 123
> +#define GPIOM_9 124
> +#define GPIOM_10 125
> +#define GPIOM_11 126
> +#define GPIOM_12 127
> +#define GPIOM_13 128
> +
> +#define GPIOY_0 129
> +#define GPIOY_1 130
> +#define GPIOY_2 131
> +#define GPIOY_3 132
> +#define GPIOY_4 133
> +#define GPIOY_5 134
> +#define GPIOY_6 135
> +#define GPIOY_7 136
> +#define GPIOY_8 137
> +#define GPIOY_9 138
> +#define GPIOY_10 139
> +#define GPIOY_11 140
> +#define GPIOY_12 141
> +#define GPIOY_13 142
> +#define GPIOY_14 143
> +#define GPIOY_15 144
> +#define GPIOY_16 145
> +#define GPIOY_17 146
> +#define GPIOY_18 147
> +
> +#define GPIOH_0 148
> +#define GPIOH_1 149
> +#define GPIOH_2 150
> +#define GPIOH_3 151
> +#define GPIOH_4 152
> +#define GPIOH_5 153
> +#define GPIOH_6 154
> +#define GPIOH_7 155
> +
> +#define GPIO_TEST_N 156
> +
> +#endif /* _DT_BINDINGS_AMLOGIC_T7_GPIO_H */
> --
> 2.42.0
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
next prev parent reply other threads:[~2023-09-21 21:01 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-21 8:34 [PATCH V2 0/3] Add pinctrl driver support for Amlogic T7 SoCs Huqiang Qin
2023-09-21 8:34 ` [PATCH V2 1/3] dt-bindings: pinctrl: Add compatibles " Huqiang Qin
2023-09-21 9:19 ` Conor Dooley [this message]
2023-09-21 9:43 ` Huqiang Qin
2023-09-21 8:34 ` [PATCH V2 2/3] pinctrl: Add driver support " Huqiang Qin
2023-09-21 10:00 ` Andy Shevchenko
2023-09-21 8:34 ` [PATCH V2 3/3] arm64: dts: Add pinctrl node " Huqiang Qin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230921-fa16f2f1f118091489708226@fedora \
--to=conor@kernel.org \
--cc=andy@kernel.org \
--cc=brgl@bgdev.pl \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=huqiang.qin@amlogic.com \
--cc=jbrunet@baylibre.com \
--cc=khilman@baylibre.com \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=martin.blumenstingl@googlemail.com \
--cc=neil.armstrong@linaro.org \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).