Re: [PATCH 1/2] dt-bindings: pinctl: amlogic,pinctrl-a4: Add compatible string for A5

From: Xianwei Zhao
Date: Thu Mar 27 2025 - 21:27:16 EST


Hi Conor,
Thanks for your advice.

On 2025/3/28 00:37, Conor Dooley wrote:

ForwardedMessage.eml

Subject:
Re: [PATCH 1/2] dt-bindings: pinctl: amlogic,pinctrl-a4: Add compatible string for A5
From:
Conor Dooley <conor@xxxxxxxxxx>
Date:
2025/3/28 00:37

To:
xianwei.zhao@xxxxxxxxxxx
CC:
Linus Walleij <linus.walleij@xxxxxxxxxx>, Rob Herring <robh@xxxxxxxxxx>, Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>, Conor Dooley <conor+dt@xxxxxxxxxx>, Neil Armstrong <neil.armstrong@xxxxxxxxxx>, Kevin Hilman <khilman@xxxxxxxxxxxx>, Jerome Brunet <jbrunet@xxxxxxxxxxxx>, Martin Blumenstingl <martin.blumenstingl@xxxxxxxxxxxxxx>, linux-amlogic@xxxxxxxxxxxxxxxxxxx, linux-gpio@xxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx


On Thu, Mar 27, 2025 at 05:26:35PM +0800, Xianwei Zhao via B4 Relay wrote:
From: Xianwei Zhao<xianwei.zhao@xxxxxxxxxxx>

Amlogic A5 SoCs uses the same pintrl controller as A4 SoCs. There is
no need for an extra compatible line in the driver, but add A5
compatible line for documentation.

Signed-off-by: Xianwei Zhao<xianwei.zhao@xxxxxxxxxxx>
---
.../devicetree/bindings/pinctrl/amlogic,pinctrl-a4.yaml | 9 +++++++--
1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/amlogic,pinctrl-a4.yaml b/Documentation/devicetree/bindings/pinctrl/amlogic,pinctrl-a4.yaml
index 8eb50cad61d5..a94f4dfb9a7f 100644
--- a/Documentation/devicetree/bindings/pinctrl/amlogic,pinctrl-a4.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/amlogic,pinctrl-a4.yaml
@@ -14,8 +14,13 @@ allOf:
properties:
compatible:
- const: amlogic,pinctrl-a4
-
+ oneOf:
+ - enum:
+ - amlogic,pinctrl-a4
This one here only has a single entry, so could just be
- const: amlogic,pinctrl-a4.


Will do.

With that,
Acked-by: Conor Dooley<conor.dooley@xxxxxxxxxxxxx>

+ - items:
+ - enum:
+ - amlogic,pinctrl-a5
+ - const: amlogic,pinctrl-a4
"#address-cells":
const: 2

--
2.37.1