All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] dt-bindings: Add trivial Apple t6000/t6001 SoC bindings
@ 2021-12-09  5:09 ` Hector Martin
  0 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:09 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

Hi Rob et al,

This series adds trivial DT binding changes (compatibles and some misc
changes) to support Apple's t6000/t6001 SoCs and the devices that use
them. These changes are for devices that need no driver changes to
support these new SoCs. Drivers that need changes (notably AICv2) have
the binding change submitted as part of that series.

The only nontrivial patch is #3 for the PCIe binding, which grew a port.
I tried to set it up so it'll fail validation if you try to have too
many ports on t8103.

I'll be happy to merge this through the Asahi-SoC tree once reviewed :)

Hector Martin (4):
  dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16"
    compatibles
  dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible
  dt-bindings: pci: apple,pcie: Add t6000 support
  dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl
    compatible

 .../devicetree/bindings/arm/apple.yaml        | 21 ++++++++++++++
 .../devicetree/bindings/i2c/apple,i2c.yaml    |  4 ++-
 .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
 .../bindings/pinctrl/apple,pinctrl.yaml       |  4 ++-
 4 files changed, 48 insertions(+), 9 deletions(-)

-- 
2.33.0


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

* [PATCH 0/4] dt-bindings: Add trivial Apple t6000/t6001 SoC bindings
@ 2021-12-09  5:09 ` Hector Martin
  0 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:09 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

Hi Rob et al,

This series adds trivial DT binding changes (compatibles and some misc
changes) to support Apple's t6000/t6001 SoCs and the devices that use
them. These changes are for devices that need no driver changes to
support these new SoCs. Drivers that need changes (notably AICv2) have
the binding change submitted as part of that series.

The only nontrivial patch is #3 for the PCIe binding, which grew a port.
I tried to set it up so it'll fail validation if you try to have too
many ports on t8103.

I'll be happy to merge this through the Asahi-SoC tree once reviewed :)

Hector Martin (4):
  dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16"
    compatibles
  dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible
  dt-bindings: pci: apple,pcie: Add t6000 support
  dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl
    compatible

 .../devicetree/bindings/arm/apple.yaml        | 21 ++++++++++++++
 .../devicetree/bindings/i2c/apple,i2c.yaml    |  4 ++-
 .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
 .../bindings/pinctrl/apple,pinctrl.yaml       |  4 ++-
 4 files changed, 48 insertions(+), 9 deletions(-)

-- 
2.33.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 1/4] dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16" compatibles
  2021-12-09  5:09 ` Hector Martin
@ 2021-12-09  5:09   ` Hector Martin
  -1 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:09 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

This adds the initial apple,t6000 platforms:

- apple,j314s - MacBook Pro (14-inch, M1 Pro, 2021)
- apple,j316s - MacBook Pro (16-inch, M1 Pro, 2021)

And the initial apple,t6001 platforms:

- apple,j314c - MacBook Pro (14-inch, M1 Max, 2021)
- apple,j316c - MacBook Pro (16-inch, M1 Max, 2021)

Signed-off-by: Hector Martin <marcan@marcan.st>
---
 .../devicetree/bindings/arm/apple.yaml        | 21 +++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/apple.yaml b/Documentation/devicetree/bindings/arm/apple.yaml
index b23c8dc5a27d..8d93e8a6cc18 100644
--- a/Documentation/devicetree/bindings/arm/apple.yaml
+++ b/Documentation/devicetree/bindings/arm/apple.yaml
@@ -19,6 +19,13 @@ description: |
   - MacBook Air (M1, 2020)
   - iMac (24-inch, M1, 2021)
 
+  And devices based on the "M1 Pro" and "M1 Max" SoCs:
+
+  - MacBook Pro (14-inch, M1 Pro, 2021)
+  - MacBook Pro (14-inch, M1 Max, 2021)
+  - MacBook Pro (16-inch, M1 Pro, 2021)
+  - MacBook Pro (16-inch, M1 Max, 2021)
+
   The compatible property should follow this format:
 
   compatible = "apple,<targettype>", "apple,<socid>", "apple,arm-platform";
@@ -60,6 +67,20 @@ properties:
               - apple,j457 # iMac (24-inch, 2x USB-C, M1, 2021)
           - const: apple,t8103
           - const: apple,arm-platform
+      - description: Apple M1 Pro SoC based platforms
+        items:
+          - enum:
+              - apple,j314s # MacBook Pro (14-inch, M1 Pro, 2021)
+              - apple,j316s # MacBook Pro (16-inch, M1 Pro, 2021)
+          - const: apple,t6000
+          - const: apple,arm-platform
+      - description: Apple M1 Max SoC based platforms
+        items:
+          - enum:
+              - apple,j314c # MacBook Pro (14-inch, M1 Max, 2021)
+              - apple,j316c # MacBook Pro (16-inch, M1 Max, 2021)
+          - const: apple,t6001
+          - const: apple,arm-platform
 
 additionalProperties: true
 
-- 
2.33.0


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

* [PATCH 1/4] dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16" compatibles
@ 2021-12-09  5:09   ` Hector Martin
  0 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:09 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

This adds the initial apple,t6000 platforms:

- apple,j314s - MacBook Pro (14-inch, M1 Pro, 2021)
- apple,j316s - MacBook Pro (16-inch, M1 Pro, 2021)

And the initial apple,t6001 platforms:

- apple,j314c - MacBook Pro (14-inch, M1 Max, 2021)
- apple,j316c - MacBook Pro (16-inch, M1 Max, 2021)

Signed-off-by: Hector Martin <marcan@marcan.st>
---
 .../devicetree/bindings/arm/apple.yaml        | 21 +++++++++++++++++++
 1 file changed, 21 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/apple.yaml b/Documentation/devicetree/bindings/arm/apple.yaml
index b23c8dc5a27d..8d93e8a6cc18 100644
--- a/Documentation/devicetree/bindings/arm/apple.yaml
+++ b/Documentation/devicetree/bindings/arm/apple.yaml
@@ -19,6 +19,13 @@ description: |
   - MacBook Air (M1, 2020)
   - iMac (24-inch, M1, 2021)
 
+  And devices based on the "M1 Pro" and "M1 Max" SoCs:
+
+  - MacBook Pro (14-inch, M1 Pro, 2021)
+  - MacBook Pro (14-inch, M1 Max, 2021)
+  - MacBook Pro (16-inch, M1 Pro, 2021)
+  - MacBook Pro (16-inch, M1 Max, 2021)
+
   The compatible property should follow this format:
 
   compatible = "apple,<targettype>", "apple,<socid>", "apple,arm-platform";
@@ -60,6 +67,20 @@ properties:
               - apple,j457 # iMac (24-inch, 2x USB-C, M1, 2021)
           - const: apple,t8103
           - const: apple,arm-platform
+      - description: Apple M1 Pro SoC based platforms
+        items:
+          - enum:
+              - apple,j314s # MacBook Pro (14-inch, M1 Pro, 2021)
+              - apple,j316s # MacBook Pro (16-inch, M1 Pro, 2021)
+          - const: apple,t6000
+          - const: apple,arm-platform
+      - description: Apple M1 Max SoC based platforms
+        items:
+          - enum:
+              - apple,j314c # MacBook Pro (14-inch, M1 Max, 2021)
+              - apple,j316c # MacBook Pro (16-inch, M1 Max, 2021)
+          - const: apple,t6001
+          - const: apple,arm-platform
 
 additionalProperties: true
 
-- 
2.33.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 2/4] dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible
  2021-12-09  5:09 ` Hector Martin
@ 2021-12-09  5:09   ` Hector Martin
  -1 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:09 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

This block is compatible with t8103, so just add the new per-SoC
compatible under apple,i2c.

Signed-off-by: Hector Martin <marcan@marcan.st>
---
 Documentation/devicetree/bindings/i2c/apple,i2c.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/i2c/apple,i2c.yaml b/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
index 47dd6c107a6d..4ac61fec90e2 100644
--- a/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
+++ b/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
@@ -21,7 +21,9 @@ allOf:
 properties:
   compatible:
     items:
-      - const: apple,t8103-i2c
+      - enum:
+          - apple,t8103-i2c
+          - apple,t6000-i2c
       - const: apple,i2c
 
   reg:
-- 
2.33.0


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

* [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible
@ 2021-12-09  5:09   ` Hector Martin
  0 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:09 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

This block is compatible with t8103, so just add the new per-SoC
compatible under apple,i2c.

Signed-off-by: Hector Martin <marcan@marcan.st>
---
 Documentation/devicetree/bindings/i2c/apple,i2c.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/i2c/apple,i2c.yaml b/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
index 47dd6c107a6d..4ac61fec90e2 100644
--- a/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
+++ b/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
@@ -21,7 +21,9 @@ allOf:
 properties:
   compatible:
     items:
-      - const: apple,t8103-i2c
+      - enum:
+          - apple,t8103-i2c
+          - apple,t6000-i2c
       - const: apple,i2c
 
   reg:
-- 
2.33.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 3/4] dt-bindings: pci: apple,pcie: Add t6000 support
  2021-12-09  5:09 ` Hector Martin
@ 2021-12-09  5:10   ` Hector Martin
  -1 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:10 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

This new SoC is compatible with the existing driver, but the block
supports 4 downstream ports, so we need to adjust the binding to
allow that.

Signed-off-by: Hector Martin <marcan@marcan.st>
---
 .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
 1 file changed, 21 insertions(+), 7 deletions(-)

diff --git a/Documentation/devicetree/bindings/pci/apple,pcie.yaml b/Documentation/devicetree/bindings/pci/apple,pcie.yaml
index ef1d424ec299..7f01e15fc81c 100644
--- a/Documentation/devicetree/bindings/pci/apple,pcie.yaml
+++ b/Documentation/devicetree/bindings/pci/apple,pcie.yaml
@@ -28,19 +28,17 @@ description: |
   distributed over the root ports as the OS sees fit by programming
   the PCIe controller's port registers.
 
-allOf:
-  - $ref: /schemas/pci/pci-bus.yaml#
-  - $ref: /schemas/interrupt-controller/msi-controller.yaml#
-
 properties:
   compatible:
     items:
-      - const: apple,t8103-pcie
+      - enum:
+          - apple,t8103-pcie
+          - apple,t6000-pcie
       - const: apple,pcie
 
   reg:
     minItems: 3
-    maxItems: 5
+    maxItems: 6
 
   reg-names:
     minItems: 3
@@ -50,6 +48,7 @@ properties:
       - const: port0
       - const: port1
       - const: port2
+      - const: port3
 
   ranges:
     minItems: 2
@@ -59,7 +58,7 @@ properties:
     description:
       Interrupt specifiers, one for each root port.
     minItems: 1
-    maxItems: 3
+    maxItems: 4
 
   msi-parent: true
 
@@ -81,6 +80,21 @@ required:
 
 unevaluatedProperties: false
 
+allOf:
+  - $ref: /schemas/pci/pci-bus.yaml#
+  - $ref: /schemas/interrupt-controller/msi-controller.yaml#
+  - if:
+      properties:
+        compatible:
+          contains:
+            const: apple,t8103-pcie
+    then:
+      properties:
+        reg:
+          maxItems: 5
+        interrupts:
+          maxItems: 3
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/apple-aic.h>
-- 
2.33.0


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

* [PATCH 3/4] dt-bindings: pci: apple,pcie: Add t6000 support
@ 2021-12-09  5:10   ` Hector Martin
  0 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:10 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

This new SoC is compatible with the existing driver, but the block
supports 4 downstream ports, so we need to adjust the binding to
allow that.

Signed-off-by: Hector Martin <marcan@marcan.st>
---
 .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
 1 file changed, 21 insertions(+), 7 deletions(-)

diff --git a/Documentation/devicetree/bindings/pci/apple,pcie.yaml b/Documentation/devicetree/bindings/pci/apple,pcie.yaml
index ef1d424ec299..7f01e15fc81c 100644
--- a/Documentation/devicetree/bindings/pci/apple,pcie.yaml
+++ b/Documentation/devicetree/bindings/pci/apple,pcie.yaml
@@ -28,19 +28,17 @@ description: |
   distributed over the root ports as the OS sees fit by programming
   the PCIe controller's port registers.
 
-allOf:
-  - $ref: /schemas/pci/pci-bus.yaml#
-  - $ref: /schemas/interrupt-controller/msi-controller.yaml#
-
 properties:
   compatible:
     items:
-      - const: apple,t8103-pcie
+      - enum:
+          - apple,t8103-pcie
+          - apple,t6000-pcie
       - const: apple,pcie
 
   reg:
     minItems: 3
-    maxItems: 5
+    maxItems: 6
 
   reg-names:
     minItems: 3
@@ -50,6 +48,7 @@ properties:
       - const: port0
       - const: port1
       - const: port2
+      - const: port3
 
   ranges:
     minItems: 2
@@ -59,7 +58,7 @@ properties:
     description:
       Interrupt specifiers, one for each root port.
     minItems: 1
-    maxItems: 3
+    maxItems: 4
 
   msi-parent: true
 
@@ -81,6 +80,21 @@ required:
 
 unevaluatedProperties: false
 
+allOf:
+  - $ref: /schemas/pci/pci-bus.yaml#
+  - $ref: /schemas/interrupt-controller/msi-controller.yaml#
+  - if:
+      properties:
+        compatible:
+          contains:
+            const: apple,t8103-pcie
+    then:
+      properties:
+        reg:
+          maxItems: 5
+        interrupts:
+          maxItems: 3
+
 examples:
   - |
     #include <dt-bindings/interrupt-controller/apple-aic.h>
-- 
2.33.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* [PATCH 4/4] dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl compatible
  2021-12-09  5:09 ` Hector Martin
@ 2021-12-09  5:10   ` Hector Martin
  -1 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:10 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

This new SoC uses the same pinctrl hardware, so just add a new per-SoC
compatible.

Signed-off-by: Hector Martin <marcan@marcan.st>
---
 Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
index 6ca59974664e..d3b11351ca45 100644
--- a/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
@@ -17,7 +17,9 @@ description: |
 properties:
   compatible:
     items:
-      - const: apple,t8103-pinctrl
+      - enum:
+          - apple,t8103-pinctrl
+          - apple,t6000-pinctrl
       - const: apple,pinctrl
 
   reg:
-- 
2.33.0


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

* [PATCH 4/4] dt-bindings: pinctrl: apple, pinctrl: Add apple, t6000-pinctrl compatible
@ 2021-12-09  5:10   ` Hector Martin
  0 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-09  5:10 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Hector Martin, Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier,
	linux-arm-kernel, devicetree, linux-kernel

This new SoC uses the same pinctrl hardware, so just add a new per-SoC
compatible.

Signed-off-by: Hector Martin <marcan@marcan.st>
---
 Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
index 6ca59974664e..d3b11351ca45 100644
--- a/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
+++ b/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
@@ -17,7 +17,9 @@ description: |
 properties:
   compatible:
     items:
-      - const: apple,t8103-pinctrl
+      - enum:
+          - apple,t8103-pinctrl
+          - apple,t6000-pinctrl
       - const: apple,pinctrl
 
   reg:
-- 
2.33.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 1/4] dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16" compatibles
  2021-12-09  5:09   ` Hector Martin
@ 2021-12-09 14:19     ` Mark Kettenis
  -1 siblings, 0 replies; 30+ messages in thread
From: Mark Kettenis @ 2021-12-09 14:19 UTC (permalink / raw)
  To: Hector Martin
  Cc: sven, robh+dt, marcan, alyssa, kettenis, maz, linux-arm-kernel,
	devicetree, linux-kernel

> From: Hector Martin <marcan@marcan.st>
> Date: Thu,  9 Dec 2021 14:09:58 +0900
> 
> This adds the initial apple,t6000 platforms:
> 
> - apple,j314s - MacBook Pro (14-inch, M1 Pro, 2021)
> - apple,j316s - MacBook Pro (16-inch, M1 Pro, 2021)
> 
> And the initial apple,t6001 platforms:
> 
> - apple,j314c - MacBook Pro (14-inch, M1 Max, 2021)
> - apple,j316c - MacBook Pro (16-inch, M1 Max, 2021)
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  .../devicetree/bindings/arm/apple.yaml        | 21 +++++++++++++++++++
>  1 file changed, 21 insertions(+)

Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

> diff --git a/Documentation/devicetree/bindings/arm/apple.yaml b/Documentation/devicetree/bindings/arm/apple.yaml
> index b23c8dc5a27d..8d93e8a6cc18 100644
> --- a/Documentation/devicetree/bindings/arm/apple.yaml
> +++ b/Documentation/devicetree/bindings/arm/apple.yaml
> @@ -19,6 +19,13 @@ description: |
>    - MacBook Air (M1, 2020)
>    - iMac (24-inch, M1, 2021)
>  
> +  And devices based on the "M1 Pro" and "M1 Max" SoCs:
> +
> +  - MacBook Pro (14-inch, M1 Pro, 2021)
> +  - MacBook Pro (14-inch, M1 Max, 2021)
> +  - MacBook Pro (16-inch, M1 Pro, 2021)
> +  - MacBook Pro (16-inch, M1 Max, 2021)
> +
>    The compatible property should follow this format:
>  
>    compatible = "apple,<targettype>", "apple,<socid>", "apple,arm-platform";
> @@ -60,6 +67,20 @@ properties:
>                - apple,j457 # iMac (24-inch, 2x USB-C, M1, 2021)
>            - const: apple,t8103
>            - const: apple,arm-platform
> +      - description: Apple M1 Pro SoC based platforms
> +        items:
> +          - enum:
> +              - apple,j314s # MacBook Pro (14-inch, M1 Pro, 2021)
> +              - apple,j316s # MacBook Pro (16-inch, M1 Pro, 2021)
> +          - const: apple,t6000
> +          - const: apple,arm-platform
> +      - description: Apple M1 Max SoC based platforms
> +        items:
> +          - enum:
> +              - apple,j314c # MacBook Pro (14-inch, M1 Max, 2021)
> +              - apple,j316c # MacBook Pro (16-inch, M1 Max, 2021)
> +          - const: apple,t6001
> +          - const: apple,arm-platform
>  
>  additionalProperties: true
>  
> -- 
> 2.33.0

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

* Re: [PATCH 1/4] dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16" compatibles
@ 2021-12-09 14:19     ` Mark Kettenis
  0 siblings, 0 replies; 30+ messages in thread
From: Mark Kettenis @ 2021-12-09 14:19 UTC (permalink / raw)
  To: Hector Martin
  Cc: sven, robh+dt, marcan, alyssa, kettenis, maz, linux-arm-kernel,
	devicetree, linux-kernel

> From: Hector Martin <marcan@marcan.st>
> Date: Thu,  9 Dec 2021 14:09:58 +0900
> 
> This adds the initial apple,t6000 platforms:
> 
> - apple,j314s - MacBook Pro (14-inch, M1 Pro, 2021)
> - apple,j316s - MacBook Pro (16-inch, M1 Pro, 2021)
> 
> And the initial apple,t6001 platforms:
> 
> - apple,j314c - MacBook Pro (14-inch, M1 Max, 2021)
> - apple,j316c - MacBook Pro (16-inch, M1 Max, 2021)
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  .../devicetree/bindings/arm/apple.yaml        | 21 +++++++++++++++++++
>  1 file changed, 21 insertions(+)

Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

> diff --git a/Documentation/devicetree/bindings/arm/apple.yaml b/Documentation/devicetree/bindings/arm/apple.yaml
> index b23c8dc5a27d..8d93e8a6cc18 100644
> --- a/Documentation/devicetree/bindings/arm/apple.yaml
> +++ b/Documentation/devicetree/bindings/arm/apple.yaml
> @@ -19,6 +19,13 @@ description: |
>    - MacBook Air (M1, 2020)
>    - iMac (24-inch, M1, 2021)
>  
> +  And devices based on the "M1 Pro" and "M1 Max" SoCs:
> +
> +  - MacBook Pro (14-inch, M1 Pro, 2021)
> +  - MacBook Pro (14-inch, M1 Max, 2021)
> +  - MacBook Pro (16-inch, M1 Pro, 2021)
> +  - MacBook Pro (16-inch, M1 Max, 2021)
> +
>    The compatible property should follow this format:
>  
>    compatible = "apple,<targettype>", "apple,<socid>", "apple,arm-platform";
> @@ -60,6 +67,20 @@ properties:
>                - apple,j457 # iMac (24-inch, 2x USB-C, M1, 2021)
>            - const: apple,t8103
>            - const: apple,arm-platform
> +      - description: Apple M1 Pro SoC based platforms
> +        items:
> +          - enum:
> +              - apple,j314s # MacBook Pro (14-inch, M1 Pro, 2021)
> +              - apple,j316s # MacBook Pro (16-inch, M1 Pro, 2021)
> +          - const: apple,t6000
> +          - const: apple,arm-platform
> +      - description: Apple M1 Max SoC based platforms
> +        items:
> +          - enum:
> +              - apple,j314c # MacBook Pro (14-inch, M1 Max, 2021)
> +              - apple,j316c # MacBook Pro (16-inch, M1 Max, 2021)
> +          - const: apple,t6001
> +          - const: apple,arm-platform
>  
>  additionalProperties: true
>  
> -- 
> 2.33.0

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible
  2021-12-09  5:09   ` [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible Hector Martin
@ 2021-12-09 14:20     ` Mark Kettenis
  -1 siblings, 0 replies; 30+ messages in thread
From: Mark Kettenis @ 2021-12-09 14:20 UTC (permalink / raw)
  To: Hector Martin
  Cc: sven, robh+dt, marcan, alyssa, kettenis, maz, linux-arm-kernel,
	devicetree, linux-kernel

> From: Hector Martin <marcan@marcan.st>
> Date: Thu,  9 Dec 2021 14:09:59 +0900
> 
> This block is compatible with t8103, so just add the new per-SoC
> compatible under apple,i2c.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/i2c/apple,i2c.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

> diff --git a/Documentation/devicetree/bindings/i2c/apple,i2c.yaml b/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
> index 47dd6c107a6d..4ac61fec90e2 100644
> --- a/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
> +++ b/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
> @@ -21,7 +21,9 @@ allOf:
>  properties:
>    compatible:
>      items:
> -      - const: apple,t8103-i2c
> +      - enum:
> +          - apple,t8103-i2c
> +          - apple,t6000-i2c
>        - const: apple,i2c
>  
>    reg:
> -- 
> 2.33.0

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

* Re: [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible
@ 2021-12-09 14:20     ` Mark Kettenis
  0 siblings, 0 replies; 30+ messages in thread
From: Mark Kettenis @ 2021-12-09 14:20 UTC (permalink / raw)
  To: Hector Martin
  Cc: sven, robh+dt, marcan, alyssa, kettenis, maz, linux-arm-kernel,
	devicetree, linux-kernel

> From: Hector Martin <marcan@marcan.st>
> Date: Thu,  9 Dec 2021 14:09:59 +0900
> 
> This block is compatible with t8103, so just add the new per-SoC
> compatible under apple,i2c.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/i2c/apple,i2c.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

> diff --git a/Documentation/devicetree/bindings/i2c/apple,i2c.yaml b/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
> index 47dd6c107a6d..4ac61fec90e2 100644
> --- a/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
> +++ b/Documentation/devicetree/bindings/i2c/apple,i2c.yaml
> @@ -21,7 +21,9 @@ allOf:
>  properties:
>    compatible:
>      items:
> -      - const: apple,t8103-i2c
> +      - enum:
> +          - apple,t8103-i2c
> +          - apple,t6000-i2c
>        - const: apple,i2c
>  
>    reg:
> -- 
> 2.33.0

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 3/4] dt-bindings: pci: apple,pcie: Add t6000 support
  2021-12-09  5:10   ` Hector Martin
@ 2021-12-09 14:25     ` Mark Kettenis
  -1 siblings, 0 replies; 30+ messages in thread
From: Mark Kettenis @ 2021-12-09 14:25 UTC (permalink / raw)
  To: Hector Martin
  Cc: sven, robh+dt, marcan, alyssa, kettenis, maz, linux-arm-kernel,
	devicetree, linux-kernel

> From: Hector Martin <marcan@marcan.st>
> Date: Thu,  9 Dec 2021 14:10:00 +0900
> 
> This new SoC is compatible with the existing driver, but the block
> supports 4 downstream ports, so we need to adjust the binding to
> allow that.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
>  1 file changed, 21 insertions(+), 7 deletions(-)

Not 100% certain if we really want to constrain things on a per-SoC
basis this way.  But it matches my understanding of the hardware.

Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

> diff --git a/Documentation/devicetree/bindings/pci/apple,pcie.yaml b/Documentation/devicetree/bindings/pci/apple,pcie.yaml
> index ef1d424ec299..7f01e15fc81c 100644
> --- a/Documentation/devicetree/bindings/pci/apple,pcie.yaml
> +++ b/Documentation/devicetree/bindings/pci/apple,pcie.yaml
> @@ -28,19 +28,17 @@ description: |
>    distributed over the root ports as the OS sees fit by programming
>    the PCIe controller's port registers.
>  
> -allOf:
> -  - $ref: /schemas/pci/pci-bus.yaml#
> -  - $ref: /schemas/interrupt-controller/msi-controller.yaml#
> -
>  properties:
>    compatible:
>      items:
> -      - const: apple,t8103-pcie
> +      - enum:
> +          - apple,t8103-pcie
> +          - apple,t6000-pcie
>        - const: apple,pcie
>  
>    reg:
>      minItems: 3
> -    maxItems: 5
> +    maxItems: 6
>  
>    reg-names:
>      minItems: 3
> @@ -50,6 +48,7 @@ properties:
>        - const: port0
>        - const: port1
>        - const: port2
> +      - const: port3
>  
>    ranges:
>      minItems: 2
> @@ -59,7 +58,7 @@ properties:
>      description:
>        Interrupt specifiers, one for each root port.
>      minItems: 1
> -    maxItems: 3
> +    maxItems: 4
>  
>    msi-parent: true
>  
> @@ -81,6 +80,21 @@ required:
>  
>  unevaluatedProperties: false
>  
> +allOf:
> +  - $ref: /schemas/pci/pci-bus.yaml#
> +  - $ref: /schemas/interrupt-controller/msi-controller.yaml#
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            const: apple,t8103-pcie
> +    then:
> +      properties:
> +        reg:
> +          maxItems: 5
> +        interrupts:
> +          maxItems: 3
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/apple-aic.h>
> -- 
> 2.33.0

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

* Re: [PATCH 3/4] dt-bindings: pci: apple,pcie: Add t6000 support
@ 2021-12-09 14:25     ` Mark Kettenis
  0 siblings, 0 replies; 30+ messages in thread
From: Mark Kettenis @ 2021-12-09 14:25 UTC (permalink / raw)
  To: Hector Martin
  Cc: sven, robh+dt, marcan, alyssa, kettenis, maz, linux-arm-kernel,
	devicetree, linux-kernel

> From: Hector Martin <marcan@marcan.st>
> Date: Thu,  9 Dec 2021 14:10:00 +0900
> 
> This new SoC is compatible with the existing driver, but the block
> supports 4 downstream ports, so we need to adjust the binding to
> allow that.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
>  1 file changed, 21 insertions(+), 7 deletions(-)

Not 100% certain if we really want to constrain things on a per-SoC
basis this way.  But it matches my understanding of the hardware.

Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

> diff --git a/Documentation/devicetree/bindings/pci/apple,pcie.yaml b/Documentation/devicetree/bindings/pci/apple,pcie.yaml
> index ef1d424ec299..7f01e15fc81c 100644
> --- a/Documentation/devicetree/bindings/pci/apple,pcie.yaml
> +++ b/Documentation/devicetree/bindings/pci/apple,pcie.yaml
> @@ -28,19 +28,17 @@ description: |
>    distributed over the root ports as the OS sees fit by programming
>    the PCIe controller's port registers.
>  
> -allOf:
> -  - $ref: /schemas/pci/pci-bus.yaml#
> -  - $ref: /schemas/interrupt-controller/msi-controller.yaml#
> -
>  properties:
>    compatible:
>      items:
> -      - const: apple,t8103-pcie
> +      - enum:
> +          - apple,t8103-pcie
> +          - apple,t6000-pcie
>        - const: apple,pcie
>  
>    reg:
>      minItems: 3
> -    maxItems: 5
> +    maxItems: 6
>  
>    reg-names:
>      minItems: 3
> @@ -50,6 +48,7 @@ properties:
>        - const: port0
>        - const: port1
>        - const: port2
> +      - const: port3
>  
>    ranges:
>      minItems: 2
> @@ -59,7 +58,7 @@ properties:
>      description:
>        Interrupt specifiers, one for each root port.
>      minItems: 1
> -    maxItems: 3
> +    maxItems: 4
>  
>    msi-parent: true
>  
> @@ -81,6 +80,21 @@ required:
>  
>  unevaluatedProperties: false
>  
> +allOf:
> +  - $ref: /schemas/pci/pci-bus.yaml#
> +  - $ref: /schemas/interrupt-controller/msi-controller.yaml#
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            const: apple,t8103-pcie
> +    then:
> +      properties:
> +        reg:
> +          maxItems: 5
> +        interrupts:
> +          maxItems: 3
> +
>  examples:
>    - |
>      #include <dt-bindings/interrupt-controller/apple-aic.h>
> -- 
> 2.33.0

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 4/4] dt-bindings: pinctrl: apple, pinctrl: Add apple, t6000-pinctrl compatible
  2021-12-09  5:10   ` [PATCH 4/4] dt-bindings: pinctrl: apple, pinctrl: Add apple, t6000-pinctrl compatible Hector Martin
@ 2021-12-09 14:25     ` Mark Kettenis
  -1 siblings, 0 replies; 30+ messages in thread
From: Mark Kettenis @ 2021-12-09 14:25 UTC (permalink / raw)
  To: Hector Martin
  Cc: sven, robh+dt, marcan, alyssa, kettenis, maz, linux-arm-kernel,
	devicetree, linux-kernel

> From: Hector Martin <marcan@marcan.st>
> Date: Thu,  9 Dec 2021 14:10:01 +0900
> 
> This new SoC uses the same pinctrl hardware, so just add a new per-SoC
> compatible.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

> diff --git a/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
> index 6ca59974664e..d3b11351ca45 100644
> --- a/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
> @@ -17,7 +17,9 @@ description: |
>  properties:
>    compatible:
>      items:
> -      - const: apple,t8103-pinctrl
> +      - enum:
> +          - apple,t8103-pinctrl
> +          - apple,t6000-pinctrl
>        - const: apple,pinctrl
>  
>    reg:
> -- 
> 2.33.0
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

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

* Re: [PATCH 4/4] dt-bindings: pinctrl: apple, pinctrl: Add apple, t6000-pinctrl compatible
@ 2021-12-09 14:25     ` Mark Kettenis
  0 siblings, 0 replies; 30+ messages in thread
From: Mark Kettenis @ 2021-12-09 14:25 UTC (permalink / raw)
  To: Hector Martin
  Cc: sven, robh+dt, marcan, alyssa, kettenis, maz, linux-arm-kernel,
	devicetree, linux-kernel

> From: Hector Martin <marcan@marcan.st>
> Date: Thu,  9 Dec 2021 14:10:01 +0900
> 
> This new SoC uses the same pinctrl hardware, so just add a new per-SoC
> compatible.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>

Reviewed-by: Mark Kettenis <kettenis@openbsd.org>

> diff --git a/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml b/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
> index 6ca59974664e..d3b11351ca45 100644
> --- a/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
> +++ b/Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml
> @@ -17,7 +17,9 @@ description: |
>  properties:
>    compatible:
>      items:
> -      - const: apple,t8103-pinctrl
> +      - enum:
> +          - apple,t8103-pinctrl
> +          - apple,t6000-pinctrl
>        - const: apple,pinctrl
>  
>    reg:
> -- 
> 2.33.0
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 2/4] dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible
  2021-12-09  5:09   ` [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible Hector Martin
@ 2021-12-09 16:34     ` Sven Peter
  -1 siblings, 0 replies; 30+ messages in thread
From: Sven Peter @ 2021-12-09 16:34 UTC (permalink / raw)
  To: Hector Martin, Rob Herring
  Cc: Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier, linux-arm-kernel,
	devicetree, linux-kernel



On Thu, Dec 9, 2021, at 06:09, Hector Martin wrote:
> This block is compatible with t8103, so just add the new per-SoC
> compatible under apple,i2c.
>
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/i2c/apple,i2c.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Reviewed-by: Sven Peter <sven@svenpeter.dev>

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

* Re: [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible
@ 2021-12-09 16:34     ` Sven Peter
  0 siblings, 0 replies; 30+ messages in thread
From: Sven Peter @ 2021-12-09 16:34 UTC (permalink / raw)
  To: Hector Martin, Rob Herring
  Cc: Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier, linux-arm-kernel,
	devicetree, linux-kernel



On Thu, Dec 9, 2021, at 06:09, Hector Martin wrote:
> This block is compatible with t8103, so just add the new per-SoC
> compatible under apple,i2c.
>
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/i2c/apple,i2c.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Reviewed-by: Sven Peter <sven@svenpeter.dev>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 1/4] dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16" compatibles
  2021-12-09  5:09   ` Hector Martin
@ 2021-12-09 17:12     ` Rob Herring
  -1 siblings, 0 replies; 30+ messages in thread
From: Rob Herring @ 2021-12-09 17:12 UTC (permalink / raw)
  To: Hector Martin
  Cc: devicetree, Marc Zyngier, Alyssa Rosenzweig, linux-arm-kernel,
	Rob Herring, linux-kernel, Mark Kettenis, Sven Peter

On Thu, 09 Dec 2021 14:09:58 +0900, Hector Martin wrote:
> This adds the initial apple,t6000 platforms:
> 
> - apple,j314s - MacBook Pro (14-inch, M1 Pro, 2021)
> - apple,j316s - MacBook Pro (16-inch, M1 Pro, 2021)
> 
> And the initial apple,t6001 platforms:
> 
> - apple,j314c - MacBook Pro (14-inch, M1 Max, 2021)
> - apple,j316c - MacBook Pro (16-inch, M1 Max, 2021)
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  .../devicetree/bindings/arm/apple.yaml        | 21 +++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 

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

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

* Re: [PATCH 1/4] dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16" compatibles
@ 2021-12-09 17:12     ` Rob Herring
  0 siblings, 0 replies; 30+ messages in thread
From: Rob Herring @ 2021-12-09 17:12 UTC (permalink / raw)
  To: Hector Martin
  Cc: devicetree, Marc Zyngier, Alyssa Rosenzweig, linux-arm-kernel,
	Rob Herring, linux-kernel, Mark Kettenis, Sven Peter

On Thu, 09 Dec 2021 14:09:58 +0900, Hector Martin wrote:
> This adds the initial apple,t6000 platforms:
> 
> - apple,j314s - MacBook Pro (14-inch, M1 Pro, 2021)
> - apple,j316s - MacBook Pro (16-inch, M1 Pro, 2021)
> 
> And the initial apple,t6001 platforms:
> 
> - apple,j314c - MacBook Pro (14-inch, M1 Max, 2021)
> - apple,j316c - MacBook Pro (16-inch, M1 Max, 2021)
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  .../devicetree/bindings/arm/apple.yaml        | 21 +++++++++++++++++++
>  1 file changed, 21 insertions(+)
> 

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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 2/4] dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible
  2021-12-09  5:09   ` [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible Hector Martin
@ 2021-12-09 17:13     ` Rob Herring
  -1 siblings, 0 replies; 30+ messages in thread
From: Rob Herring @ 2021-12-09 17:13 UTC (permalink / raw)
  To: Hector Martin
  Cc: Alyssa Rosenzweig, Mark Kettenis, devicetree, linux-arm-kernel,
	linux-kernel, Rob Herring, Marc Zyngier, Sven Peter

On Thu, 09 Dec 2021 14:09:59 +0900, Hector Martin wrote:
> This block is compatible with t8103, so just add the new per-SoC
> compatible under apple,i2c.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/i2c/apple,i2c.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

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

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

* Re: [PATCH 2/4] dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible
@ 2021-12-09 17:13     ` Rob Herring
  0 siblings, 0 replies; 30+ messages in thread
From: Rob Herring @ 2021-12-09 17:13 UTC (permalink / raw)
  To: Hector Martin
  Cc: Alyssa Rosenzweig, Mark Kettenis, devicetree, linux-arm-kernel,
	linux-kernel, Rob Herring, Marc Zyngier, Sven Peter

On Thu, 09 Dec 2021 14:09:59 +0900, Hector Martin wrote:
> This block is compatible with t8103, so just add the new per-SoC
> compatible under apple,i2c.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/i2c/apple,i2c.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 3/4] dt-bindings: pci: apple,pcie: Add t6000 support
  2021-12-09  5:10   ` Hector Martin
@ 2021-12-09 17:15     ` Rob Herring
  -1 siblings, 0 replies; 30+ messages in thread
From: Rob Herring @ 2021-12-09 17:15 UTC (permalink / raw)
  To: Hector Martin
  Cc: Rob Herring, Marc Zyngier, Mark Kettenis, linux-arm-kernel,
	devicetree, Sven Peter, linux-kernel, Alyssa Rosenzweig

On Thu, 09 Dec 2021 14:10:00 +0900, Hector Martin wrote:
> This new SoC is compatible with the existing driver, but the block
> supports 4 downstream ports, so we need to adjust the binding to
> allow that.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
>  1 file changed, 21 insertions(+), 7 deletions(-)
> 

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

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

* Re: [PATCH 3/4] dt-bindings: pci: apple,pcie: Add t6000 support
@ 2021-12-09 17:15     ` Rob Herring
  0 siblings, 0 replies; 30+ messages in thread
From: Rob Herring @ 2021-12-09 17:15 UTC (permalink / raw)
  To: Hector Martin
  Cc: Rob Herring, Marc Zyngier, Mark Kettenis, linux-arm-kernel,
	devicetree, Sven Peter, linux-kernel, Alyssa Rosenzweig

On Thu, 09 Dec 2021 14:10:00 +0900, Hector Martin wrote:
> This new SoC is compatible with the existing driver, but the block
> supports 4 downstream ports, so we need to adjust the binding to
> allow that.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
>  1 file changed, 21 insertions(+), 7 deletions(-)
> 

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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 4/4] dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl compatible
  2021-12-09  5:10   ` [PATCH 4/4] dt-bindings: pinctrl: apple, pinctrl: Add apple, t6000-pinctrl compatible Hector Martin
@ 2021-12-09 17:16     ` Rob Herring
  -1 siblings, 0 replies; 30+ messages in thread
From: Rob Herring @ 2021-12-09 17:16 UTC (permalink / raw)
  To: Hector Martin
  Cc: Sven Peter, devicetree, Alyssa Rosenzweig, linux-arm-kernel,
	linux-kernel, Mark Kettenis, Rob Herring, Marc Zyngier

On Thu, 09 Dec 2021 14:10:01 +0900, Hector Martin wrote:
> This new SoC uses the same pinctrl hardware, so just add a new per-SoC
> compatible.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

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

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

* Re: [PATCH 4/4] dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl compatible
@ 2021-12-09 17:16     ` Rob Herring
  0 siblings, 0 replies; 30+ messages in thread
From: Rob Herring @ 2021-12-09 17:16 UTC (permalink / raw)
  To: Hector Martin
  Cc: Sven Peter, devicetree, Alyssa Rosenzweig, linux-arm-kernel,
	linux-kernel, Mark Kettenis, Rob Herring, Marc Zyngier

On Thu, 09 Dec 2021 14:10:01 +0900, Hector Martin wrote:
> This new SoC uses the same pinctrl hardware, so just add a new per-SoC
> compatible.
> 
> Signed-off-by: Hector Martin <marcan@marcan.st>
> ---
>  Documentation/devicetree/bindings/pinctrl/apple,pinctrl.yaml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 

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

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

* Re: [PATCH 0/4] dt-bindings: Add trivial Apple t6000/t6001 SoC bindings
  2021-12-09  5:09 ` Hector Martin
@ 2021-12-12  4:04   ` Hector Martin
  -1 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-12  4:04 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier, linux-arm-kernel,
	devicetree, linux-kernel

On 09/12/2021 14.09, Hector Martin wrote:
> Hi Rob et al,
> 
> This series adds trivial DT binding changes (compatibles and some misc
> changes) to support Apple's t6000/t6001 SoCs and the devices that use
> them. These changes are for devices that need no driver changes to
> support these new SoCs. Drivers that need changes (notably AICv2) have
> the binding change submitted as part of that series.
> 
> The only nontrivial patch is #3 for the PCIe binding, which grew a port.
> I tried to set it up so it'll fail validation if you try to have too
> many ports on t8103.
> 
> I'll be happy to merge this through the Asahi-SoC tree once reviewed :)
> 
> Hector Martin (4):
>    dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16"
>      compatibles
>    dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible
>    dt-bindings: pci: apple,pcie: Add t6000 support
>    dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl
>      compatible
> 
>   .../devicetree/bindings/arm/apple.yaml        | 21 ++++++++++++++
>   .../devicetree/bindings/i2c/apple,i2c.yaml    |  4 ++-
>   .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
>   .../bindings/pinctrl/apple,pinctrl.yaml       |  4 ++-
>   4 files changed, 48 insertions(+), 9 deletions(-)
> 

Thanks for the reviews! This is now queued in asahi-soc/dt.

-- 
Hector Martin (marcan@marcan.st)
Public Key: https://mrcn.st/pub

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

* Re: [PATCH 0/4] dt-bindings: Add trivial Apple t6000/t6001 SoC bindings
@ 2021-12-12  4:04   ` Hector Martin
  0 siblings, 0 replies; 30+ messages in thread
From: Hector Martin @ 2021-12-12  4:04 UTC (permalink / raw)
  To: Sven Peter, Rob Herring
  Cc: Alyssa Rosenzweig, Mark Kettenis, Marc Zyngier, linux-arm-kernel,
	devicetree, linux-kernel

On 09/12/2021 14.09, Hector Martin wrote:
> Hi Rob et al,
> 
> This series adds trivial DT binding changes (compatibles and some misc
> changes) to support Apple's t6000/t6001 SoCs and the devices that use
> them. These changes are for devices that need no driver changes to
> support these new SoCs. Drivers that need changes (notably AICv2) have
> the binding change submitted as part of that series.
> 
> The only nontrivial patch is #3 for the PCIe binding, which grew a port.
> I tried to set it up so it'll fail validation if you try to have too
> many ports on t8103.
> 
> I'll be happy to merge this through the Asahi-SoC tree once reviewed :)
> 
> Hector Martin (4):
>    dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16"
>      compatibles
>    dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible
>    dt-bindings: pci: apple,pcie: Add t6000 support
>    dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl
>      compatible
> 
>   .../devicetree/bindings/arm/apple.yaml        | 21 ++++++++++++++
>   .../devicetree/bindings/i2c/apple,i2c.yaml    |  4 ++-
>   .../devicetree/bindings/pci/apple,pcie.yaml   | 28 ++++++++++++++-----
>   .../bindings/pinctrl/apple,pinctrl.yaml       |  4 ++-
>   4 files changed, 48 insertions(+), 9 deletions(-)
> 

Thanks for the reviews! This is now queued in asahi-soc/dt.

-- 
Hector Martin (marcan@marcan.st)
Public Key: https://mrcn.st/pub

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

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

end of thread, other threads:[~2021-12-12  4:06 UTC | newest]

Thread overview: 30+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-09  5:09 [PATCH 0/4] dt-bindings: Add trivial Apple t6000/t6001 SoC bindings Hector Martin
2021-12-09  5:09 ` Hector Martin
2021-12-09  5:09 ` [PATCH 1/4] dt-bindings: arm: apple: Add t6000/t6001 MacBook Pro 14/16" compatibles Hector Martin
2021-12-09  5:09   ` Hector Martin
2021-12-09 14:19   ` Mark Kettenis
2021-12-09 14:19     ` Mark Kettenis
2021-12-09 17:12   ` Rob Herring
2021-12-09 17:12     ` Rob Herring
2021-12-09  5:09 ` [PATCH 2/4] dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible Hector Martin
2021-12-09  5:09   ` [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible Hector Martin
2021-12-09 14:20   ` Mark Kettenis
2021-12-09 14:20     ` Mark Kettenis
2021-12-09 16:34   ` [PATCH 2/4] dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible Sven Peter
2021-12-09 16:34     ` [PATCH 2/4] dt-bindings: i2c: apple, i2c: Add apple, t6000-i2c compatible Sven Peter
2021-12-09 17:13   ` [PATCH 2/4] dt-bindings: i2c: apple,i2c: Add apple,t6000-i2c compatible Rob Herring
2021-12-09 17:13     ` Rob Herring
2021-12-09  5:10 ` [PATCH 3/4] dt-bindings: pci: apple,pcie: Add t6000 support Hector Martin
2021-12-09  5:10   ` Hector Martin
2021-12-09 14:25   ` Mark Kettenis
2021-12-09 14:25     ` Mark Kettenis
2021-12-09 17:15   ` Rob Herring
2021-12-09 17:15     ` Rob Herring
2021-12-09  5:10 ` [PATCH 4/4] dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl compatible Hector Martin
2021-12-09  5:10   ` [PATCH 4/4] dt-bindings: pinctrl: apple, pinctrl: Add apple, t6000-pinctrl compatible Hector Martin
2021-12-09 14:25   ` Mark Kettenis
2021-12-09 14:25     ` Mark Kettenis
2021-12-09 17:16   ` [PATCH 4/4] dt-bindings: pinctrl: apple,pinctrl: Add apple,t6000-pinctrl compatible Rob Herring
2021-12-09 17:16     ` Rob Herring
2021-12-12  4:04 ` [PATCH 0/4] dt-bindings: Add trivial Apple t6000/t6001 SoC bindings Hector Martin
2021-12-12  4:04   ` Hector Martin

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.