Linux-ARM-Kernel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs
@ 2025-09-15  1:52 Nick Chan
  2025-09-15  1:52 ` [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible Nick Chan
                   ` (3 more replies)
  0 siblings, 4 replies; 8+ messages in thread
From: Nick Chan @ 2025-09-15  1:52 UTC (permalink / raw)
  To: Janne Grunau, Neal Gompa, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Sasha Finkelstein, Sven Peter
  Cc: asahi, linux-arm-kernel, linux-kernel, devicetree, Nick Chan

Hi,

This series adds dt-bindings and nodes for Apple A11 and T2 SoCs,
and the existing driver appears to be compatible. Drivers for the attached
Dialog DA2422 and DA2449 PMICs will be in a future patch series.

The dt-binding patch depends on other spmi dt-binding patches that are
already applied in apple-soc/drivers-6.18, so it is best for it to be
applied through there.

Signed-off-by: Nick Chan <towinchenmi@gmail.com>
---
Changes in v4:
- Keep the empty lines as is in the spmi dt-bindings file
- Keep DT includes in alphabetical order
- Drop tags for dt-bindings since they are sigificantly different from v2
- Link to v3: https://lore.kernel.org/r/20250829-t8015-spmi-v3-0-58b15ee2c825@gmail.com

Changes in v3:
- After discussion with the DT maintainers we agreed not to extend the
  generic compatible[1]. Change the series to the agreed-upon style.
- Now depends on a patch[2] adding the t602x spmi compatible.
- Link to v2: https://lore.kernel.org/r/20250811-t8015-spmi-v2-0-a291acf0d469@gmail.com

[1]: https://lore.kernel.org/asahi/12ab93b7-1fc2-4ce0-926e-c8141cfe81bf@kernel.org/
[2]: https://lore.kernel.org/asahi/20250828-dt-apple-t6020-v1-22-507ba4c4b98e@jannau.net/

Changes in v2:
- Rebased on top of v6.17-rc1
- Collect Rob and Sven's tags
- Link to v1: https://lore.kernel.org/r/20250609-t8015-spmi-v1-0-b3c55ad01491@gmail.com

---
Nick Chan (3):
      dt-bindings: spmi: Add Apple A11 and T2 compatible
      arm64: dts: apple: t8012: Add SPMI node
      arm64: dts: apple: t8015: Add SPMI node

 Documentation/devicetree/bindings/spmi/apple,spmi.yaml | 5 ++++-
 arch/arm64/boot/dts/apple/t8012.dtsi                   | 8 ++++++++
 arch/arm64/boot/dts/apple/t8015.dtsi                   | 8 ++++++++
 3 files changed, 20 insertions(+), 1 deletion(-)
---
base-commit: 989ca853ddeefad448bdfb771113faf9cc480936
change-id: 20250527-t8015-spmi-838bb49d34ae

Best regards,
-- 
Nick Chan <towinchenmi@gmail.com>



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

* [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible
  2025-09-15  1:52 [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs Nick Chan
@ 2025-09-15  1:52 ` Nick Chan
  2025-09-15 13:53   ` Rob Herring (Arm)
  2025-09-15 13:54   ` Rob Herring (Arm)
  2025-09-15  1:52 ` [PATCH v4 2/3] arm64: dts: apple: t8012: Add SPMI node Nick Chan
                   ` (2 subsequent siblings)
  3 siblings, 2 replies; 8+ messages in thread
From: Nick Chan @ 2025-09-15  1:52 UTC (permalink / raw)
  To: Janne Grunau, Neal Gompa, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Sasha Finkelstein, Sven Peter
  Cc: asahi, linux-arm-kernel, linux-kernel, devicetree, Nick Chan

The SPMI bus found on Apple A11 and T2 SoCs are compatible with the
existing driver for t8103's spmi so add their compatibles.

Signed-off-by: Nick Chan <towinchenmi@gmail.com>
---
 Documentation/devicetree/bindings/spmi/apple,spmi.yaml | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/spmi/apple,spmi.yaml b/Documentation/devicetree/bindings/spmi/apple,spmi.yaml
index dbf09ad0ecdecab82074344aa6a840a820bc448a..ba524f1eb7049dc5ba7c5f83f734d3f2f7e905b3 100644
--- a/Documentation/devicetree/bindings/spmi/apple,spmi.yaml
+++ b/Documentation/devicetree/bindings/spmi/apple,spmi.yaml
@@ -18,7 +18,10 @@ properties:
   compatible:
     oneOf:
       - items:
-          - const: apple,t6020-spmi
+          - enum:
+              - apple,t6020-spmi
+              - apple,t8012-spmi
+              - apple,t8015-spmi
           - const: apple,t8103-spmi
       - items:
           - enum:

-- 
2.51.0



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

* [PATCH v4 2/3] arm64: dts: apple: t8012: Add SPMI node
  2025-09-15  1:52 [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs Nick Chan
  2025-09-15  1:52 ` [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible Nick Chan
@ 2025-09-15  1:52 ` Nick Chan
  2025-09-15  1:52 ` [PATCH v4 3/3] arm64: dts: apple: t8015: " Nick Chan
  2025-09-18 19:15 ` [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs Sven Peter
  3 siblings, 0 replies; 8+ messages in thread
From: Nick Chan @ 2025-09-15  1:52 UTC (permalink / raw)
  To: Janne Grunau, Neal Gompa, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Sasha Finkelstein, Sven Peter
  Cc: asahi, linux-arm-kernel, linux-kernel, devicetree, Nick Chan

Add SPMI node for Apple T2 SoC.

Signed-off-by: Nick Chan <towinchenmi@gmail.com>
---
 arch/arm64/boot/dts/apple/t8012.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/apple/t8012.dtsi b/arch/arm64/boot/dts/apple/t8012.dtsi
index a259e5735d938cfa5b29cee6c754c7a3c0aaae08..e7923814169bd4060706945561910ed1d5c2e0c8 100644
--- a/arch/arm64/boot/dts/apple/t8012.dtsi
+++ b/arch/arm64/boot/dts/apple/t8012.dtsi
@@ -11,6 +11,7 @@
 #include <dt-bindings/interrupt-controller/apple-aic.h>
 #include <dt-bindings/interrupt-controller/irq.h>
 #include <dt-bindings/pinctrl/apple.h>
+#include <dt-bindings/spmi/spmi.h>
 
 / {
 	interrupt-parent = <&aic>;
@@ -220,6 +221,13 @@ pinctrl_aop: pinctrl@2100f0000 {
 				     <AIC_IRQ 137 IRQ_TYPE_LEVEL_HIGH>;
 		};
 
+		spmi: spmi@211180700 {
+			compatible = "apple,t8012-spmi", "apple,t8103-spmi";
+			reg = <0x2 0x11180700 0x0 0x100>;
+			#address-cells = <2>;
+			#size-cells = <0>;
+		};
+
 		pinctrl_nub: pinctrl@2111f0000 {
 			compatible = "apple,t8010-pinctrl", "apple,pinctrl";
 			reg = <0x2 0x111f0000 0x0 0x1000>;

-- 
2.51.0



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

* [PATCH v4 3/3] arm64: dts: apple: t8015: Add SPMI node
  2025-09-15  1:52 [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs Nick Chan
  2025-09-15  1:52 ` [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible Nick Chan
  2025-09-15  1:52 ` [PATCH v4 2/3] arm64: dts: apple: t8012: Add SPMI node Nick Chan
@ 2025-09-15  1:52 ` Nick Chan
  2025-09-18 19:15 ` [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs Sven Peter
  3 siblings, 0 replies; 8+ messages in thread
From: Nick Chan @ 2025-09-15  1:52 UTC (permalink / raw)
  To: Janne Grunau, Neal Gompa, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Sasha Finkelstein, Sven Peter
  Cc: asahi, linux-arm-kernel, linux-kernel, devicetree, Nick Chan

Add SPMI node for Apple A11 SoC.

Signed-off-by: Nick Chan <towinchenmi@gmail.com>
---
 arch/arm64/boot/dts/apple/t8015.dtsi | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/arch/arm64/boot/dts/apple/t8015.dtsi b/arch/arm64/boot/dts/apple/t8015.dtsi
index 12acf8fc8bc6bcde6b11773cadd97e9ee115f510..4266667b23c1c851540a4a68411c50234f228071 100644
--- a/arch/arm64/boot/dts/apple/t8015.dtsi
+++ b/arch/arm64/boot/dts/apple/t8015.dtsi
@@ -11,6 +11,7 @@
 #include <dt-bindings/interrupt-controller/apple-aic.h>
 #include <dt-bindings/interrupt-controller/irq.h>
 #include <dt-bindings/pinctrl/apple.h>
+#include <dt-bindings/spmi/spmi.h>
 
 / {
 	interrupt-parent = <&aic>;
@@ -344,6 +345,13 @@ pinctrl_aop: pinctrl@2340f0000 {
 				     <AIC_IRQ 141 IRQ_TYPE_LEVEL_HIGH>;
 		};
 
+		spmi: spmi@235180700 {
+			compatible = "apple,t8015-spmi", "apple,t8103-spmi";
+			reg = <0x2 0x35180700 0x0 0x100>;
+			#address-cells = <2>;
+			#size-cells = <0>;
+		};
+
 		pinctrl_nub: pinctrl@2351f0000 {
 			compatible = "apple,t8015-pinctrl", "apple,pinctrl";
 			reg = <0x2 0x351f0000 0x0 0x4000>;

-- 
2.51.0



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

* Re: [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible
  2025-09-15  1:52 ` [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible Nick Chan
@ 2025-09-15 13:53   ` Rob Herring (Arm)
  2025-09-15 23:18     ` Nick Chan
  2025-09-15 13:54   ` Rob Herring (Arm)
  1 sibling, 1 reply; 8+ messages in thread
From: Rob Herring (Arm) @ 2025-09-15 13:53 UTC (permalink / raw)
  To: Nick Chan
  Cc: Krzysztof Kozlowski, Sasha Finkelstein, linux-arm-kernel,
	Conor Dooley, Sven Peter, linux-kernel, Janne Grunau,
	Stephen Boyd, Neal Gompa, devicetree, asahi


On Mon, 15 Sep 2025 09:52:24 +0800, Nick Chan wrote:
> The SPMI bus found on Apple A11 and T2 SoCs are compatible with the
> existing driver for t8103's spmi so add their compatibles.
> 
> Signed-off-by: Nick Chan <towinchenmi@gmail.com>
> ---
>  Documentation/devicetree/bindings/spmi/apple,spmi.yaml | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 


Please add Acked-by/Reviewed-by tags when posting new versions. However,
there's no need to repost patches *only* to add the tags. The upstream
maintainer will do that for acks received on the version they apply.

If a tag was not added on purpose, please state why and what changed.

Missing tags:

Acked-by: Rob Herring (Arm) <robh@kernel.org>





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

* Re: [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible
  2025-09-15  1:52 ` [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible Nick Chan
  2025-09-15 13:53   ` Rob Herring (Arm)
@ 2025-09-15 13:54   ` Rob Herring (Arm)
  1 sibling, 0 replies; 8+ messages in thread
From: Rob Herring (Arm) @ 2025-09-15 13:54 UTC (permalink / raw)
  To: Nick Chan
  Cc: Conor Dooley, Stephen Boyd, asahi, linux-arm-kernel, Neal Gompa,
	devicetree, Janne Grunau, Sasha Finkelstein, Krzysztof Kozlowski,
	Sven Peter, linux-kernel


On Mon, 15 Sep 2025 09:52:24 +0800, Nick Chan wrote:
> The SPMI bus found on Apple A11 and T2 SoCs are compatible with the
> existing driver for t8103's spmi so add their compatibles.
> 
> Signed-off-by: Nick Chan <towinchenmi@gmail.com>
> ---
>  Documentation/devicetree/bindings/spmi/apple,spmi.yaml | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
> 

Acked-by: Rob Herring (Arm) <robh@kernel.org>



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

* Re: [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible
  2025-09-15 13:53   ` Rob Herring (Arm)
@ 2025-09-15 23:18     ` Nick Chan
  0 siblings, 0 replies; 8+ messages in thread
From: Nick Chan @ 2025-09-15 23:18 UTC (permalink / raw)
  To: Rob Herring (Arm)
  Cc: Krzysztof Kozlowski, Sasha Finkelstein, linux-arm-kernel,
	Conor Dooley, Sven Peter, linux-kernel, Janne Grunau,
	Stephen Boyd, Neal Gompa, devicetree, asahi


Rob Herring (Arm) 於 2025/9/15 晚上9:53 寫道:
> On Mon, 15 Sep 2025 09:52:24 +0800, Nick Chan wrote:
>> The SPMI bus found on Apple A11 and T2 SoCs are compatible with the
>> existing driver for t8103's spmi so add their compatibles.
>>
>> Signed-off-by: Nick Chan <towinchenmi@gmail.com>
>> ---
>>  Documentation/devicetree/bindings/spmi/apple,spmi.yaml | 5 ++++-
>>  1 file changed, 4 insertions(+), 1 deletion(-)
>>
>
> Please add Acked-by/Reviewed-by tags when posting new versions. However,
> there's no need to repost patches *only* to add the tags. The upstream
> maintainer will do that for acks received on the version they apply.
>
> If a tag was not added on purpose, please state why and what changed.

As stated in cover letter, the tags are dropped in v4 because the dt-binding patch has changed
significantly from v2. (really should be dropped in v3 though but that did not happen). And this
is actually a different patch from v3, the only difference is a newline is no longer removed.


>
> Missing tags:
>
> Acked-by: Rob Herring (Arm) <robh@kernel.org>
>
>
>
>
Best,
Nick Chan


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

* Re: [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs
  2025-09-15  1:52 [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs Nick Chan
                   ` (2 preceding siblings ...)
  2025-09-15  1:52 ` [PATCH v4 3/3] arm64: dts: apple: t8015: " Nick Chan
@ 2025-09-18 19:15 ` Sven Peter
  3 siblings, 0 replies; 8+ messages in thread
From: Sven Peter @ 2025-09-18 19:15 UTC (permalink / raw)
  To: Janne Grunau, Neal Gompa, Stephen Boyd, Rob Herring,
	Krzysztof Kozlowski, Conor Dooley, Sasha Finkelstein, Nick Chan
  Cc: Sven Peter, asahi, linux-arm-kernel, linux-kernel, devicetree

On Mon, 15 Sep 2025 09:52:23 +0800, Nick Chan wrote:
> This series adds dt-bindings and nodes for Apple A11 and T2 SoCs,
> and the existing driver appears to be compatible. Drivers for the attached
> Dialog DA2422 and DA2449 PMICs will be in a future patch series.
> 
> The dt-binding patch depends on other spmi dt-binding patches that are
> already applied in apple-soc/drivers-6.18, so it is best for it to be
> applied through there.
> 
> [...]

Applied to git@github.com:AsahiLinux/linux.git (asahi-soc/for-next), thanks!

[1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible
      https://github.com/AsahiLinux/linux/commit/4b167146ec05
[2/3] arm64: dts: apple: t8012: Add SPMI node
      https://github.com/AsahiLinux/linux/commit/8f6e6934e33e
[3/3] arm64: dts: apple: t8015: Add SPMI node
      https://github.com/AsahiLinux/linux/commit/70fa521f4d55

Best regards,
-- 
Sven Peter <sven@kernel.org>



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

end of thread, other threads:[~2025-09-18 19:16 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-09-15  1:52 [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs Nick Chan
2025-09-15  1:52 ` [PATCH v4 1/3] dt-bindings: spmi: Add Apple A11 and T2 compatible Nick Chan
2025-09-15 13:53   ` Rob Herring (Arm)
2025-09-15 23:18     ` Nick Chan
2025-09-15 13:54   ` Rob Herring (Arm)
2025-09-15  1:52 ` [PATCH v4 2/3] arm64: dts: apple: t8012: Add SPMI node Nick Chan
2025-09-15  1:52 ` [PATCH v4 3/3] arm64: dts: apple: t8015: " Nick Chan
2025-09-18 19:15 ` [PATCH v4 0/3] SPMI dt-bindings and nodes for Apple A11 and T2 SoCs Sven Peter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox