| Message ID | 20211209055049.99205-2-marcan@marcan.st |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | Apple mailbox fixup: switch to generic compatibles | expand |
| Context | Check | Description |
|---|---|---|
| robh/checkpatch | success | |
| robh/dtbs-check | success | |
| robh/dt-meta-schema | success |
> From: Hector Martin <marcan@marcan.st> > Date: Thu, 9 Dec 2021 14:50:48 +0900 > > Much as we've done with other blocks, let's introduce generic > compatibles so drivers can bind to those and still work with future > SoCs, as long as the hardware remains the same. Also go ahead and add > compatibles for the new t600x SoCs (we group those as t6000). > > Note that no DTs instantiate devices with this binding yet. I think this makes sense. There is no OpenBSD driver for this yet and my U-Boot driver has not been submitted upstream yet. So I think there are no real backwards compatibility issues. Reviewed-by: Mark Kettenis <kettenis@openbsd.org> > > Signed-off-by: Hector Martin <marcan@marcan.st> > --- > .../devicetree/bindings/mailbox/apple,mailbox.yaml | 12 +++++++++--- > 1 file changed, 9 insertions(+), 3 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mailbox/apple,mailbox.yaml b/Documentation/devicetree/bindings/mailbox/apple,mailbox.yaml > index 2c1704b34e7a..58007c789671 100644 > --- a/Documentation/devicetree/bindings/mailbox/apple,mailbox.yaml > +++ b/Documentation/devicetree/bindings/mailbox/apple,mailbox.yaml > @@ -27,14 +27,20 @@ properties: > for example for the display controller, the system management > controller and the NVMe coprocessor. > items: > - - const: apple,t8103-asc-mailbox > + - enum: > + - apple,t8103-asc-mailbox > + - apple,t6000-asc-mailbox > + - const: apple,asc-mailbox > > - description: > M3 mailboxes are an older variant with a slightly different MMIO > interface still found on the M1. It is used for the Thunderbolt > co-processors. > items: > - - const: apple,t8103-m3-mailbox > + - enum: > + - apple,t8103-m3-mailbox > + - apple,t6000-m3-mailbox > + - const: apple,m3-mailbox > > reg: > maxItems: 1 > @@ -68,7 +74,7 @@ additionalProperties: false > examples: > - | > mailbox@77408000 { > - compatible = "apple,t8103-asc-mailbox"; > + compatible = "apple,t8103-asc-mailbox", "apple,asc-mailbox"; > reg = <0x77408000 0x4000>; > interrupts = <1 583 4>, <1 584 4>, <1 585 4>, <1 586 4>; > interrupt-names = "send-empty", "send-not-empty", > -- > 2.33.0 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel >
On Thu, 09 Dec 2021 14:50:48 +0900, Hector Martin wrote: > Much as we've done with other blocks, let's introduce generic > compatibles so drivers can bind to those and still work with future > SoCs, as long as the hardware remains the same. Also go ahead and add > compatibles for the new t600x SoCs (we group those as t6000). > > Note that no DTs instantiate devices with this binding yet. > > Signed-off-by: Hector Martin <marcan@marcan.st> > --- > .../devicetree/bindings/mailbox/apple,mailbox.yaml | 12 +++++++++--- > 1 file changed, 9 insertions(+), 3 deletions(-) > Reviewed-by: Rob Herring <robh@kernel.org>
On Thu, Dec 9, 2021, at 06:50, Hector Martin wrote: > Much as we've done with other blocks, let's introduce generic > compatibles so drivers can bind to those and still work with future > SoCs, as long as the hardware remains the same. Also go ahead and add > compatibles for the new t600x SoCs (we group those as t6000). > > Note that no DTs instantiate devices with this binding yet. > > Signed-off-by: Hector Martin <marcan@marcan.st> > --- Reviewed-by: Sven Peter <sven@svenpeter.dev>
On Wed, Dec 15, 2021, at 18:13, Sven Peter wrote: > On Thu, Dec 9, 2021, at 06:50, Hector Martin wrote: >> Much as we've done with other blocks, let's introduce generic >> compatibles so drivers can bind to those and still work with future >> SoCs, as long as the hardware remains the same. Also go ahead and add >> compatibles for the new t600x SoCs (we group those as t6000). >> >> Note that no DTs instantiate devices with this binding yet. >> >> Signed-off-by: Hector Martin <marcan@marcan.st> >> --- > Reviewed-by: Sven Peter <sven@svenpeter.dev> er... wait... that was meant for v2, sorry *sigh*. Sven
diff --git a/Documentation/devicetree/bindings/mailbox/apple,mailbox.yaml b/Documentation/devicetree/bindings/mailbox/apple,mailbox.yaml index 2c1704b34e7a..58007c789671 100644 --- a/Documentation/devicetree/bindings/mailbox/apple,mailbox.yaml +++ b/Documentation/devicetree/bindings/mailbox/apple,mailbox.yaml @@ -27,14 +27,20 @@ properties: for example for the display controller, the system management controller and the NVMe coprocessor. items: - - const: apple,t8103-asc-mailbox + - enum: + - apple,t8103-asc-mailbox + - apple,t6000-asc-mailbox + - const: apple,asc-mailbox - description: M3 mailboxes are an older variant with a slightly different MMIO interface still found on the M1. It is used for the Thunderbolt co-processors. items: - - const: apple,t8103-m3-mailbox + - enum: + - apple,t8103-m3-mailbox + - apple,t6000-m3-mailbox + - const: apple,m3-mailbox reg: maxItems: 1 @@ -68,7 +74,7 @@ additionalProperties: false examples: - | mailbox@77408000 { - compatible = "apple,t8103-asc-mailbox"; + compatible = "apple,t8103-asc-mailbox", "apple,asc-mailbox"; reg = <0x77408000 0x4000>; interrupts = <1 583 4>, <1 584 4>, <1 585 4>, <1 586 4>; interrupt-names = "send-empty", "send-not-empty",
Much as we've done with other blocks, let's introduce generic compatibles so drivers can bind to those and still work with future SoCs, as long as the hardware remains the same. Also go ahead and add compatibles for the new t600x SoCs (we group those as t6000). Note that no DTs instantiate devices with this binding yet. Signed-off-by: Hector Martin <marcan@marcan.st> --- .../devicetree/bindings/mailbox/apple,mailbox.yaml | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-)