2023-06-09 13:54:27

by Alexander Stein

[permalink] [raw]
Subject: [PATCH 0/3] TQMa93xxLA support

Hi,

this series adds initial support for TQMa93xxLA soldered on MBa93xxLA.
As the name indicates this is an i.MX93 based SBC board. The initial support
includes typical interfaces like UART, eMMC, I2C, Ethernet.
Support for USB, LVDS and PMIC is still pending, which require additional
patches.
Patch 1 adds the compatible to dt bindings
Patch 2 adds .dts[i] files for module and sbc board support
Patch 3 adds more i.MX93 drivers to arm64 defconfig

Best regards,
Alexander

Alexander Stein (2):
arm64: dts: freescale: add initial device tree for MBa93xxLA SBC board
arm64: defconfig: Enable i.MX93 devices

Markus Niebel (1):
dt: bindings: add bindings for TQMa93xxLA SOM

.../devicetree/bindings/arm/fsl.yaml | 19 +
arch/arm64/boot/dts/freescale/Makefile | 1 +
.../freescale/imx93-tqma9352-mba93xxla.dts | 629 ++++++++++++++++++
.../boot/dts/freescale/imx93-tqma9352.dtsi | 213 ++++++
arch/arm64/configs/defconfig | 4 +
5 files changed, 866 insertions(+)
create mode 100644 arch/arm64/boot/dts/freescale/imx93-tqma9352-mba93xxla.dts
create mode 100644 arch/arm64/boot/dts/freescale/imx93-tqma9352.dtsi

--
2.34.1



2023-06-09 14:04:43

by Alexander Stein

[permalink] [raw]
Subject: [PATCH 1/3] dt: bindings: add bindings for TQMa93xxLA SOM

From: Markus Niebel <[email protected]>

TQMa93xxLA is a SOM variant in the TQ-Systems GmbH TQMa93xx series using
NXP i.MX93 CPU on an LGA type board.
MBa93xxCA is a starterkit base board for TQMa93xxLA on an adapter board.

Signed-off-by: Markus Niebel <[email protected]>
Signed-off-by: Alexander Stein <[email protected]>
---
.../devicetree/bindings/arm/fsl.yaml | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)

diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
index 2510eaa8906dd..8048c7f6a299d 100644
--- a/Documentation/devicetree/bindings/arm/fsl.yaml
+++ b/Documentation/devicetree/bindings/arm/fsl.yaml
@@ -1219,6 +1219,25 @@ properties:
- fsl,imxrt1170-evk # i.MXRT1170 EVK Board
- const: fsl,imxrt1170

+ - description:
+ TQMa93xxLA and TQMa93xxCA are two series of feature compatible SOM
+ using NXP i.MX93 SOC in 11x11 mm package.
+ TQMa93xxLA is designed to be soldered on different carrier boards.
+ TQMa93xxCA is a compatible variant using board to board connectors.
+ All SOM and CPU variants use the same device tree hence only one
+ compatible is needed. Bootloader disables all features not present
+ in the assembled SOC.
+ MBa93xxCA mainboard can be used as starterkit for the SOM
+ soldered on an adapter board or for the connector variant
+ MBa93xxLA mainboard is a single board computer using the solderable
+ SOM variant
+ items:
+ - enum:
+ - tq,imx93-tqma9352-mba93xxca # TQ-Systems GmbH i.MX93 TQMa93xxCA/LA SOM on MBa93xxCA
+ - tq,imx93-tqma9352-mba93xxla # TQ-Systems GmbH i.MX93 TQMa93xxLA SOM on MBa93xxLA SBC
+ - const: tq,imx93-tqma9352 # TQ-Systems GmbH i.MX93 TQMa93xxCA/LA SOM
+ - const: fsl,imx93
+
- description:
Freescale Vybrid Platform Device Tree Bindings

--
2.34.1


2023-06-09 16:12:32

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt: bindings: add bindings for TQMa93xxLA SOM

On 09/06/2023 15:29, Alexander Stein wrote:
> From: Markus Niebel <[email protected]>
>
> TQMa93xxLA is a SOM variant in the TQ-Systems GmbH TQMa93xx series using
> NXP i.MX93 CPU on an LGA type board.
> MBa93xxCA is a starterkit base board for TQMa93xxLA on an adapter board.

Please use subject prefixes matching the subsystem. You can get them for
example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory
your patch is touching.

It is dt-bindings, not something else.

A nit, subject: drop second/last, redundant "bindings for". The
"dt-bindings" prefix is already stating that these are bindings. One
bindings is enough.


>
> Signed-off-by: Markus Niebel <[email protected]>
> Signed-off-by: Alexander Stein <[email protected]>
> ---
> .../devicetree/bindings/arm/fsl.yaml | 19 +++++++++++++++++++
> 1 file changed, 19 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.yaml
> index 2510eaa8906dd..8048c7f6a299d 100644
> --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> @@ -1219,6 +1219,25 @@ properties:
> - fsl,imxrt1170-evk # i.MXRT1170 EVK Board
> - const: fsl,imxrt1170
>
> + - description:
> + TQMa93xxLA and TQMa93xxCA are two series of feature compatible SOM
> + using NXP i.MX93 SOC in 11x11 mm package.
> + TQMa93xxLA is designed to be soldered on different carrier boards.
> + TQMa93xxCA is a compatible variant using board to board connectors.
> + All SOM and CPU variants use the same device tree hence only one
> + compatible is needed. Bootloader disables all features not present
> + in the assembled SOC.
> + MBa93xxCA mainboard can be used as starterkit for the SOM
> + soldered on an adapter board or for the connector variant
> + MBa93xxLA mainboard is a single board computer using the solderable
> + SOM variant
> + items:
> + - enum:
> + - tq,imx93-tqma9352-mba93xxca # TQ-Systems GmbH i.MX93 TQMa93xxCA/LA SOM on MBa93xxCA
> + - tq,imx93-tqma9352-mba93xxla # TQ-Systems GmbH i.MX93 TQMa93xxLA SOM on MBa93xxLA SBC

xx sound like wildcard which you should not use. If you open the product
page, it also suggests that.

Best regards,
Krzysztof


2023-06-15 14:56:08

by Alexander Stein

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt: bindings: add bindings for TQMa93xxLA SOM

Hi,

Am Freitag, 9. Juni 2023, 17:58:17 CEST schrieb Krzysztof Kozlowski:
> On 09/06/2023 15:29, Alexander Stein wrote:
> > From: Markus Niebel <[email protected]>
> >
> > TQMa93xxLA is a SOM variant in the TQ-Systems GmbH TQMa93xx series using
> > NXP i.MX93 CPU on an LGA type board.
> > MBa93xxCA is a starterkit base board for TQMa93xxLA on an adapter board.
>
> Please use subject prefixes matching the subsystem. You can get them for
> example with `git log --oneline -- DIRECTORY_OR_FILE` on the directory
> your patch is touching.
>
> It is dt-bindings, not something else.
>
> A nit, subject: drop second/last, redundant "bindings for". The
> "dt-bindings" prefix is already stating that these are bindings. One
> bindings is enough.

Thanks. I'll update that.

> > Signed-off-by: Markus Niebel <[email protected]>
> > Signed-off-by: Alexander Stein <[email protected]>
> > ---
> >
> > .../devicetree/bindings/arm/fsl.yaml | 19 +++++++++++++++++++
> > 1 file changed, 19 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml
> > b/Documentation/devicetree/bindings/arm/fsl.yaml index
> > 2510eaa8906dd..8048c7f6a299d 100644
> > --- a/Documentation/devicetree/bindings/arm/fsl.yaml
> > +++ b/Documentation/devicetree/bindings/arm/fsl.yaml
> >
> > @@ -1219,6 +1219,25 @@ properties:
> > - fsl,imxrt1170-evk # i.MXRT1170 EVK Board
> >
> > - const: fsl,imxrt1170
> >
> > + - description:
> > + TQMa93xxLA and TQMa93xxCA are two series of feature compatible
> > SOM + using NXP i.MX93 SOC in 11x11 mm package.
> > + TQMa93xxLA is designed to be soldered on different carrier
> > boards. + TQMa93xxCA is a compatible variant using board to
> > board connectors. + All SOM and CPU variants use the same device
> > tree hence only one + compatible is needed. Bootloader disables
> > all features not present + in the assembled SOC.
> > + MBa93xxCA mainboard can be used as starterkit for the SOM
> > + soldered on an adapter board or for the connector variant
> > + MBa93xxLA mainboard is a single board computer using the
> > solderable + SOM variant
> > + items:
> > + - enum:
> > + - tq,imx93-tqma9352-mba93xxca # TQ-Systems GmbH i.MX93
> > TQMa93xxCA/LA SOM on MBa93xxCA + -
> > tq,imx93-tqma9352-mba93xxla # TQ-Systems GmbH i.MX93 TQMa93xxLA SOM on
> > MBa93xxLA SBC
> xx sound like wildcard which you should not use. If you open the product
> page, it also suggests that.

xx is a wildcard only for modules TQMa93xx, depending on the soldered SoC, but
not for the mainboard MBa93xx.
You are right that there is the ordering number MBa9352LA-AA, so the
compatible would be "tq,imx93-tqma9352-mba9352la". But the actual values for
the wildcards is copied from the soldered module. So using a TQMa9351LA would
result in MBa9351LA-AA, although the mainboard is identical.

Best regards,
Alexander

> Best regards,
> Krzysztof


--
TQ-Systems GmbH | M?hlstra?e 2, Gut Delling | 82229 Seefeld, Germany
Amtsgericht M?nchen, HRB 105018
Gesch?ftsf?hrer: Detlef Schneider, R?diger Stahl, Stefan Schneider
http://www.tq-group.com/



2023-06-15 20:36:49

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 1/3] dt: bindings: add bindings for TQMa93xxLA SOM

On 15/06/2023 16:46, Alexander Stein wrote:
>>> + TQMa93xxLA and TQMa93xxCA are two series of feature compatible
>>> SOM + using NXP i.MX93 SOC in 11x11 mm package.
>>> + TQMa93xxLA is designed to be soldered on different carrier
>>> boards. + TQMa93xxCA is a compatible variant using board to
>>> board connectors. + All SOM and CPU variants use the same device
>>> tree hence only one + compatible is needed. Bootloader disables
>>> all features not present + in the assembled SOC.
>>> + MBa93xxCA mainboard can be used as starterkit for the SOM
>>> + soldered on an adapter board or for the connector variant
>>> + MBa93xxLA mainboard is a single board computer using the
>>> solderable + SOM variant
>>> + items:
>>> + - enum:
>>> + - tq,imx93-tqma9352-mba93xxca # TQ-Systems GmbH i.MX93
>>> TQMa93xxCA/LA SOM on MBa93xxCA + -
>>> tq,imx93-tqma9352-mba93xxla # TQ-Systems GmbH i.MX93 TQMa93xxLA SOM on
>>> MBa93xxLA SBC
>> xx sound like wildcard which you should not use. If you open the product
>> page, it also suggests that.
>
> xx is a wildcard only for modules TQMa93xx, depending on the soldered SoC, but
> not for the mainboard MBa93xx.
> You are right that there is the ordering number MBa9352LA-AA, so the
> compatible would be "tq,imx93-tqma9352-mba9352la". But the actual values for
> the wildcards is copied from the soldered module. So using a TQMa9351LA would
> result in MBa9351LA-AA, although the mainboard is identical.

OK

Best regards,
Krzysztof