* [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform @ 2020-08-27 6:51 Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema Lokesh Vutla ` (5 more replies) 0 siblings, 6 replies; 23+ messages in thread From: Lokesh Vutla @ 2020-08-27 6:51 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Lokesh Vutla, Sekhar Nori, Linux ARM Mailing List This series adds initial support for latest new SoC, J7200, from Texas Instruments. The J7200 SoC is a part of the K3 Multicore SoC architecture platform. It is targeted for for automotive gateway, vehicle compute systems, Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. The SoC aims to meet the complex processing needs of modern embedded products. See J7200 Technical Reference Manual (SPRUIU1, June 2020) for further details: https://www.ti.com/lit/pdf/spruiu1 Changes since v1: - Swapped Patch 1 and 2 as suggested by Nishanth. - Added description for each SoC in yaml bindings. Testing: - ./scripts/checkpatch --strict - Few warningns about Line length exceeding 100 columns. But these are corresponding to comments - v8make dtbs_check - DT_SCHEMA_FLAGS="-u" DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" v8make dtbs_check - DT_SCHEMA_FLAGS="-u" DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" v8make dt_binding_check Lokesh Vutla (4): dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema dt-bindings: arm: ti: Add bindings for J7200 SoC arm64: dts: ti: Add support for J7200 SoC arm64: dts: ti: Add support for J7200 Common Processor Board .../devicetree/bindings/arm/ti/k3.txt | 26 --- .../devicetree/bindings/arm/ti/k3.yaml | 35 +++ MAINTAINERS | 2 +- arch/arm64/boot/dts/ti/Makefile | 3 +- .../dts/ti/k3-j7200-common-proc-board.dts | 64 ++++++ arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 199 ++++++++++++++++++ .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 84 ++++++++ arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi | 29 +++ arch/arm64/boot/dts/ti/k3-j7200.dtsi | 165 +++++++++++++++ 9 files changed, 579 insertions(+), 28 deletions(-) delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-main.dtsi create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi create mode 100644 arch/arm64/boot/dts/ti/k3-j7200.dtsi -- 2.28.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] 23+ messages in thread
* [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema 2020-08-27 6:51 [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Lokesh Vutla @ 2020-08-27 6:51 ` Lokesh Vutla 2020-08-28 0:41 ` Nishanth Menon 2020-09-04 7:15 ` Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 2/4] dt-bindings: arm: ti: Add bindings for J7200 SoC Lokesh Vutla ` (4 subsequent siblings) 5 siblings, 2 replies; 23+ messages in thread From: Lokesh Vutla @ 2020-08-27 6:51 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Lokesh Vutla, Sekhar Nori, Linux ARM Mailing List Convert TI K3 Board/SoC bindings to DT schema format. Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> --- .../devicetree/bindings/arm/ti/k3.txt | 26 ---------------- .../devicetree/bindings/arm/ti/k3.yaml | 31 +++++++++++++++++++ MAINTAINERS | 2 +- 3 files changed, 32 insertions(+), 27 deletions(-) delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml diff --git a/Documentation/devicetree/bindings/arm/ti/k3.txt b/Documentation/devicetree/bindings/arm/ti/k3.txt deleted file mode 100644 index 333e7256126a..000000000000 --- a/Documentation/devicetree/bindings/arm/ti/k3.txt +++ /dev/null @@ -1,26 +0,0 @@ -Texas Instruments K3 Multicore SoC architecture device tree bindings --------------------------------------------------------------------- - -Platforms based on Texas Instruments K3 Multicore SoC architecture -shall follow the following scheme: - -SoCs ----- - -Each device tree root node must specify which exact SoC in K3 Multicore SoC -architecture it uses, using one of the following compatible values: - -- AM654 - compatible = "ti,am654"; - -- J721E - compatible = "ti,j721e"; - -Boards ------- - -In addition, each device tree root node must specify which one or more -of the following board-specific compatible values: - -- AM654 EVM - compatible = "ti,am654-evm", "ti,am654"; diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml new file mode 100644 index 000000000000..c5e3e4aeda8e --- /dev/null +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml @@ -0,0 +1,31 @@ +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/arm/ti/k3.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Texas Instruments K3 Multicore SoC architecture device tree bindings + +maintainers: + - Nishanth Menon <nm@ti.com> + +description: | + Platforms based on Texas Instruments K3 Multicore SoC architecture + shall have the following properties. + +properties: + $nodename: + const: '/' + compatible: + oneOf: + + - description: K3 AM654 SoC + items: + - enum: + - ti,am654-evm + - const: ti,am654 + + - description: K3 J721E SoC + items: + - const: ti,j721e +... diff --git a/MAINTAINERS b/MAINTAINERS index 3b186ade3597..40d31bb7ecf4 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2636,7 +2636,7 @@ M: Tero Kristo <t-kristo@ti.com> M: Nishanth Menon <nm@ti.com> L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) S: Supported -F: Documentation/devicetree/bindings/arm/ti/k3.txt +F: Documentation/devicetree/bindings/arm/ti/k3.yaml F: arch/arm64/boot/dts/ti/Makefile F: arch/arm64/boot/dts/ti/k3-* F: include/dt-bindings/pinctrl/k3.h -- 2.28.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] 23+ messages in thread
* Re: [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema 2020-08-27 6:51 ` [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema Lokesh Vutla @ 2020-08-28 0:41 ` Nishanth Menon 2020-08-28 3:14 ` Lokesh Vutla 2020-09-04 7:15 ` Lokesh Vutla 1 sibling, 1 reply; 23+ messages in thread From: Nishanth Menon @ 2020-08-28 0:41 UTC (permalink / raw) To: Lokesh Vutla Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List On 12:21-20200827, Lokesh Vutla wrote: > Convert TI K3 Board/SoC bindings to DT schema format. > > Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> > --- > .../devicetree/bindings/arm/ti/k3.txt | 26 ---------------- > .../devicetree/bindings/arm/ti/k3.yaml | 31 +++++++++++++++++++ > MAINTAINERS | 2 +- > 3 files changed, 32 insertions(+), 27 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt > create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml Thanks for doing this, but I have a problem with dbs_check and dtbs W=2 build warnings on existing dts files that this exposes further.. Do you mind pulling this patch out of this j7200 series ? I would rather us cleanup the warnings a bit as well, and deal with yaml conversion seperate from j7200 bindings? We will need Rob's ack anyways, I would rather we look at things independently. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema 2020-08-28 0:41 ` Nishanth Menon @ 2020-08-28 3:14 ` Lokesh Vutla 2020-08-28 3:47 ` Suman Anna ` (2 more replies) 0 siblings, 3 replies; 23+ messages in thread From: Lokesh Vutla @ 2020-08-28 3:14 UTC (permalink / raw) To: Nishanth Menon Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List Hi Nishanth, On 28/08/20 6:11 am, Nishanth Menon wrote: > On 12:21-20200827, Lokesh Vutla wrote: >> Convert TI K3 Board/SoC bindings to DT schema format. >> >> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> >> --- >> .../devicetree/bindings/arm/ti/k3.txt | 26 ---------------- >> .../devicetree/bindings/arm/ti/k3.yaml | 31 +++++++++++++++++++ >> MAINTAINERS | 2 +- >> 3 files changed, 32 insertions(+), 27 deletions(-) >> delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt >> create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml > > > Thanks for doing this, but I have a problem with dbs_check and dtbs > W=2 build warnings on existing dts files that this exposes further.. IMHO, that should not block the conversion to yaml bindings. May I know the problem you are seeing? > Do you mind pulling this patch out of this j7200 series ? I would > rather us cleanup the warnings a bit as well, and deal with yaml > conversion seperate from j7200 bindings? Ill wait for Rob's view on this. He already asked to convert it during J721e binding update. Not sure we would like to delay any further. Thanks and regards, Lokesh > > We will need Rob's ack anyways, I would rather we look at things > independently. > _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema 2020-08-28 3:14 ` Lokesh Vutla @ 2020-08-28 3:47 ` Suman Anna 2020-08-28 13:07 ` Nishanth Menon 2020-09-04 11:55 ` Nishanth Menon 2 siblings, 0 replies; 23+ messages in thread From: Suman Anna @ 2020-08-28 3:47 UTC (permalink / raw) To: Lokesh Vutla, Nishanth Menon Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List Hi Nishanth, On 8/27/20 10:14 PM, Lokesh Vutla wrote: > Hi Nishanth, > > On 28/08/20 6:11 am, Nishanth Menon wrote: >> On 12:21-20200827, Lokesh Vutla wrote: >>> Convert TI K3 Board/SoC bindings to DT schema format. >>> >>> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> >>> --- >>> .../devicetree/bindings/arm/ti/k3.txt | 26 ---------------- >>> .../devicetree/bindings/arm/ti/k3.yaml | 31 +++++++++++++++++++ >>> MAINTAINERS | 2 +- >>> 3 files changed, 32 insertions(+), 27 deletions(-) >>> delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt >>> create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml >> >> >> Thanks for doing this, but I have a problem with dbs_check and dtbs >> W=2 build warnings on existing dts files that this exposes further.. > > IMHO, that should not block the conversion to yaml bindings. May I know the > problem you are seeing? Agree with Lokesh here. This should neither increase or decrease the existing warnings present. FWIW, both dt_bindings_check and dtbs_check with the DT_SCHEMA_FILES set to k3.yaml is clean. And the J7200 dts files are clean even without any DT_SCHEMA_FILES specified. It is going to take sometime before the existing dtbs_check warnings are cleaned up on all the K3 dts files, and that has to do with the pre-existing bindings files in text and no yaml equivalent for them. regards Suman > >> Do you mind pulling this patch out of this j7200 series ? I would >> rather us cleanup the warnings a bit as well, and deal with yaml >> conversion seperate from j7200 bindings? > > Ill wait for Rob's view on this. He already asked to convert it during J721e > binding update. Not sure we would like to delay any further. > > Thanks and regards, > Lokesh > >> >> We will need Rob's ack anyways, I would rather we look at things >> independently. >> _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema 2020-08-28 3:14 ` Lokesh Vutla 2020-08-28 3:47 ` Suman Anna @ 2020-08-28 13:07 ` Nishanth Menon 2020-09-04 11:55 ` Nishanth Menon 2 siblings, 0 replies; 23+ messages in thread From: Nishanth Menon @ 2020-08-28 13:07 UTC (permalink / raw) To: Lokesh Vutla Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List On 08:44-20200828, Lokesh Vutla wrote: > Hi Nishanth, > > On 28/08/20 6:11 am, Nishanth Menon wrote: > > On 12:21-20200827, Lokesh Vutla wrote: > >> Convert TI K3 Board/SoC bindings to DT schema format. > >> > >> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> > >> --- > >> .../devicetree/bindings/arm/ti/k3.txt | 26 ---------------- > >> .../devicetree/bindings/arm/ti/k3.yaml | 31 +++++++++++++++++++ > >> MAINTAINERS | 2 +- > >> 3 files changed, 32 insertions(+), 27 deletions(-) > >> delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt > >> create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml > > > > > > Thanks for doing this, but I have a problem with dbs_check and dtbs > > W=2 build warnings on existing dts files that this exposes further.. > > IMHO, that should not block the conversion to yaml bindings. May I know the > problem you are seeing? Things are starting to literally bitrot and I saw Rob's[1] attempt to try and clean up the cruft of warnings we had introduced. we should fix all that up before introducing new platforms. I am going to see how much cleanup I can do today, but will help if more folks pitch in. > > > Do you mind pulling this patch out of this j7200 series ? I would > > rather us cleanup the warnings a bit as well, and deal with yaml > > conversion seperate from j7200 bindings? > > Ill wait for Rob's view on this. He already asked to convert it during J721e > binding update. Not sure we would like to delay any further. > Sure. [1] https://lore.kernel.org/linux-arm-kernel/CAL_JsqLqVdyKkVKJP0EG7s7m4A=r-+DjY+X4kVs9boFfPoHsfw@mail.gmail.com/#r -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema 2020-08-28 3:14 ` Lokesh Vutla 2020-08-28 3:47 ` Suman Anna 2020-08-28 13:07 ` Nishanth Menon @ 2020-09-04 11:55 ` Nishanth Menon 2 siblings, 0 replies; 23+ messages in thread From: Nishanth Menon @ 2020-09-04 11:55 UTC (permalink / raw) To: Lokesh Vutla Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List On 08:44-20200828, Lokesh Vutla wrote: > Hi Nishanth, > > On 28/08/20 6:11 am, Nishanth Menon wrote: > > On 12:21-20200827, Lokesh Vutla wrote: > >> Convert TI K3 Board/SoC bindings to DT schema format. > >> > >> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> > >> --- > >> .../devicetree/bindings/arm/ti/k3.txt | 26 ---------------- > >> .../devicetree/bindings/arm/ti/k3.yaml | 31 +++++++++++++++++++ > >> MAINTAINERS | 2 +- > >> 3 files changed, 32 insertions(+), 27 deletions(-) > >> delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt > >> create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml > > > > > > Thanks for doing this, but I have a problem with dbs_check and dtbs > > W=2 build warnings on existing dts files that this exposes further.. > > IMHO, that should not block the conversion to yaml bindings. May I know the > problem you are seeing? > > > Do you mind pulling this patch out of this j7200 series ? I would > > rather us cleanup the warnings a bit as well, and deal with yaml > > conversion seperate from j7200 bindings? > I just wanted to follow up on this thread as well.. [1] attempts stage 1 of cleanup, and we can follow up cleanup in parallel. So if Rob acks the binding, we can carry on based on the window it fits in. [1] https://lore.kernel.org/linux-arm-kernel/20200903130015.21361-1-nm@ti.com/ -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema 2020-08-27 6:51 ` [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema Lokesh Vutla 2020-08-28 0:41 ` Nishanth Menon @ 2020-09-04 7:15 ` Lokesh Vutla 1 sibling, 0 replies; 23+ messages in thread From: Lokesh Vutla @ 2020-09-04 7:15 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Linux ARM Mailing List Hi Rob, On 27/08/20 12:21 pm, Lokesh Vutla wrote: > Convert TI K3 Board/SoC bindings to DT schema format. > > Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> Do you have any comments on this patch? Thanks and regards, Lokesh > --- > .../devicetree/bindings/arm/ti/k3.txt | 26 ---------------- > .../devicetree/bindings/arm/ti/k3.yaml | 31 +++++++++++++++++++ > MAINTAINERS | 2 +- > 3 files changed, 32 insertions(+), 27 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt > create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml > > diff --git a/Documentation/devicetree/bindings/arm/ti/k3.txt b/Documentation/devicetree/bindings/arm/ti/k3.txt > deleted file mode 100644 > index 333e7256126a..000000000000 > --- a/Documentation/devicetree/bindings/arm/ti/k3.txt > +++ /dev/null > @@ -1,26 +0,0 @@ > -Texas Instruments K3 Multicore SoC architecture device tree bindings > --------------------------------------------------------------------- > - > -Platforms based on Texas Instruments K3 Multicore SoC architecture > -shall follow the following scheme: > - > -SoCs > ----- > - > -Each device tree root node must specify which exact SoC in K3 Multicore SoC > -architecture it uses, using one of the following compatible values: > - > -- AM654 > - compatible = "ti,am654"; > - > -- J721E > - compatible = "ti,j721e"; > - > -Boards > ------- > - > -In addition, each device tree root node must specify which one or more > -of the following board-specific compatible values: > - > -- AM654 EVM > - compatible = "ti,am654-evm", "ti,am654"; > diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml > new file mode 100644 > index 000000000000..c5e3e4aeda8e > --- /dev/null > +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml > @@ -0,0 +1,31 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/arm/ti/k3.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Texas Instruments K3 Multicore SoC architecture device tree bindings > + > +maintainers: > + - Nishanth Menon <nm@ti.com> > + > +description: | > + Platforms based on Texas Instruments K3 Multicore SoC architecture > + shall have the following properties. > + > +properties: > + $nodename: > + const: '/' > + compatible: > + oneOf: > + > + - description: K3 AM654 SoC > + items: > + - enum: > + - ti,am654-evm > + - const: ti,am654 > + > + - description: K3 J721E SoC > + items: > + - const: ti,j721e > +... > diff --git a/MAINTAINERS b/MAINTAINERS > index 3b186ade3597..40d31bb7ecf4 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -2636,7 +2636,7 @@ M: Tero Kristo <t-kristo@ti.com> > M: Nishanth Menon <nm@ti.com> > L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers) > S: Supported > -F: Documentation/devicetree/bindings/arm/ti/k3.txt > +F: Documentation/devicetree/bindings/arm/ti/k3.yaml > F: arch/arm64/boot/dts/ti/Makefile > F: arch/arm64/boot/dts/ti/k3-* > F: include/dt-bindings/pinctrl/k3.h > _______________________________________________ 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] 23+ messages in thread
* [PATCH v2 2/4] dt-bindings: arm: ti: Add bindings for J7200 SoC 2020-08-27 6:51 [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema Lokesh Vutla @ 2020-08-27 6:51 ` Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 3/4] arm64: dts: ti: Add support " Lokesh Vutla ` (3 subsequent siblings) 5 siblings, 0 replies; 23+ messages in thread From: Lokesh Vutla @ 2020-08-27 6:51 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Lokesh Vutla, Sekhar Nori, Linux ARM Mailing List The J7200 SoC is a part of the K3 Multicore SoC architecture platform. It is targeted for automotive gateway, vehicle compute systems, Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. The SoC aims to meet the complex processing needs of modern embedded products. Some highlights of this SoC are: * Dual Cortex-A72s in a single cluster, two clusters of lockstep capable dual Cortex-R5F MCUs and a Centralized Device Management and Security Controller (DMSC). * Configurable L3 Cache and IO-coherent architecture with high data throughput capable distributed DMA architecture under NAVSS. * Integrated Ethernet switch supporting up to a total of 4 external ports in addition to legacy Ethernet switch of up to 2 ports. * Upto 1 PCIe-GEN3 controller, 1 USB3.0 Dual-role device subsystems, 20 MCANs, 3 McASP, eMMC and SD, OSPI/HyperBus memory controller, I3C and I2C, eCAP/eQEP, eHRPWM among other peripherals. * One hardware accelerator block containing AES/DES/SHA/MD5 called SA2UL management. See J7200 Technical Reference Manual (SPRUIU1, June 2020) for further details: https://www.ti.com/lit/pdf/spruiu1 Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> --- Documentation/devicetree/bindings/arm/ti/k3.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml index c5e3e4aeda8e..829751209543 100644 --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml @@ -28,4 +28,8 @@ properties: - description: K3 J721E SoC items: - const: ti,j721e + + - description: K3 J7200 SoC + items: + - const: ti,j7200 ... -- 2.28.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] 23+ messages in thread
* [PATCH v2 3/4] arm64: dts: ti: Add support for J7200 SoC 2020-08-27 6:51 [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 2/4] dt-bindings: arm: ti: Add bindings for J7200 SoC Lokesh Vutla @ 2020-08-27 6:51 ` Lokesh Vutla 2020-08-27 17:04 ` Suman Anna 2020-08-31 9:13 ` Peter Ujfalusi 2020-08-27 6:51 ` [PATCH v2 4/4] arm64: dts: ti: Add support for J7200 Common Processor Board Lokesh Vutla ` (2 subsequent siblings) 5 siblings, 2 replies; 23+ messages in thread From: Lokesh Vutla @ 2020-08-27 6:51 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Lokesh Vutla, Sekhar Nori, Linux ARM Mailing List The J7200 SoC is a part of the K3 Multicore SoC architecture platform. It is targeted for automotive gateway, vehicle compute systems, Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. The SoC aims to meet the complex processing needs of modern embedded products. Some highlights of this SoC are: * Dual Cortex-A72s in a single cluster, two clusters of lockstep capable dual Cortex-R5F MCUs and a Centralized Device Management and Security Controller (DMSC). * Configurable L3 Cache and IO-coherent architecture with high data throughput capable distributed DMA architecture under NAVSS. * Integrated Ethernet switch supporting up to a total of 4 external ports in addition to legacy Ethernet switch of up to 2 ports. * Upto 1 PCIe-GEN3 controller, 1 USB3.0 Dual-role device subsystems, 20 MCANs, 3 McASP, eMMC and SD, OSPI/HyperBus memory controller, I3C and I2C, eCAP/eQEP, eHRPWM among other peripherals. * One hardware accelerator block containing AES/DES/SHA/MD5 called SA2UL management. See J7200 Technical Reference Manual (SPRUIU1, June 2020) for further details: https://www.ti.com/lit/pdf/spruiu1 Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> --- arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 199 ++++++++++++++++++ .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 84 ++++++++ arch/arm64/boot/dts/ti/k3-j7200.dtsi | 165 +++++++++++++++ 3 files changed, 448 insertions(+) create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-main.dtsi create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi create mode 100644 arch/arm64/boot/dts/ti/k3-j7200.dtsi diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi new file mode 100644 index 000000000000..70c8f7e941fb --- /dev/null +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi @@ -0,0 +1,199 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Device Tree Source for J7200 SoC Family Main Domain peripherals + * + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ + */ + +&cbass_main { + msmc_ram: sram@70000000 { + compatible = "mmio-sram"; + reg = <0x0 0x70000000 0x0 0x100000>; + #address-cells = <1>; + #size-cells = <1>; + ranges = <0x0 0x0 0x70000000 0x100000>; + + atf-sram@0 { + reg = <0x0 0x20000>; + }; + }; + + gic500: interrupt-controller@1800000 { + compatible = "arm,gic-v3"; + #address-cells = <2>; + #size-cells = <2>; + ranges; + #interrupt-cells = <3>; + interrupt-controller; + reg = <0x00 0x01800000 0x00 0x10000>, /* GICD */ + <0x00 0x01900000 0x00 0x100000>; /* GICR */ + + /* vcpumntirq: virtual CPU interface maintenance interrupt */ + interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>; + + gic_its: msi-controller@1820000 { + compatible = "arm,gic-v3-its"; + reg = <0x00 0x01820000 0x00 0x10000>; + socionext,synquacer-pre-its = <0x1000000 0x400000>; + msi-controller; + #msi-cells = <1>; + }; + }; + + main_navss: navss@30000000 { + compatible = "simple-mfd"; + #address-cells = <2>; + #size-cells = <2>; + ranges = <0x00 0x30000000 0x00 0x30000000 0x00 0x0c400000>; + + secure_proxy_main: mailbox@32c00000 { + compatible = "ti,am654-secure-proxy"; + #mbox-cells = <1>; + reg-names = "target_data", "rt", "scfg"; + reg = <0x00 0x32c00000 0x00 0x100000>, + <0x00 0x32400000 0x00 0x100000>, + <0x00 0x32800000 0x00 0x100000>; + interrupt-names = "rx_011"; + interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>; + }; + }; + + main_pmx0: pinmux@11c000 { + compatible = "pinctrl-single"; + /* Proxy 0 addressing */ + reg = <0x0 0x11c000 0x0 0x2b4>; + #pinctrl-cells = <1>; + pinctrl-single,register-width = <32>; + pinctrl-single,function-mask = <0xffffffff>; + }; + + main_uart0: serial@2800000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02800000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 192 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 146 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 146 2>; + clock-names = "fclk"; + }; + + main_uart1: serial@2810000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02810000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 193 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 278 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 278 2>; + clock-names = "fclk"; + }; + + main_uart2: serial@2820000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02820000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 194 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 279 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 279 2>; + clock-names = "fclk"; + }; + + main_uart3: serial@2830000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02830000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 195 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 280 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 280 2>; + clock-names = "fclk"; + }; + + main_uart4: serial@2840000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02840000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 196 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 281 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 281 2>; + clock-names = "fclk"; + }; + + main_uart5: serial@2850000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02850000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 197 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 282 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 282 2>; + clock-names = "fclk"; + }; + + main_uart6: serial@2860000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02860000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 198 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 283 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 283 2>; + clock-names = "fclk"; + }; + + main_uart7: serial@2870000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02870000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 199 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 284 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 284 2>; + clock-names = "fclk"; + }; + + main_uart8: serial@2880000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02880000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 248 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 285 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 285 2>; + clock-names = "fclk"; + }; + + main_uart9: serial@2890000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x02890000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 249 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 286 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 286 2>; + clock-names = "fclk"; + }; +}; diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi new file mode 100644 index 000000000000..670e4c7cd9fe --- /dev/null +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi @@ -0,0 +1,84 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Device Tree Source for J7200 SoC Family MCU/WAKEUP Domain peripherals + * + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ + */ + +&cbass_mcu_wakeup { + dmsc: dmsc@44083000 { + compatible = "ti,k2g-sci"; + ti,host-id = <12>; + + mbox-names = "rx", "tx"; + + mboxes= <&secure_proxy_main 11>, + <&secure_proxy_main 13>; + + reg-names = "debug_messages"; + reg = <0x00 0x44083000 0x0 0x1000>; + + k3_pds: power-controller { + compatible = "ti,sci-pm-domain"; + #power-domain-cells = <2>; + }; + + k3_clks: clocks { + compatible = "ti,k2g-sci-clk"; + #clock-cells = <2>; + }; + + k3_reset: reset-controller { + compatible = "ti,sci-reset"; + #reset-cells = <2>; + }; + }; + + chipid@43000014 { + compatible = "ti,am654-chipid"; + reg = <0x0 0x43000014 0x0 0x4>; + }; + + wkup_pmx0: pinmux@4301c000 { + compatible = "pinctrl-single"; + /* Proxy 0 addressing */ + reg = <0x00 0x4301c000 0x00 0x178>; + #pinctrl-cells = <1>; + pinctrl-single,register-width = <32>; + pinctrl-single,function-mask = <0xffffffff>; + }; + + mcu_ram: sram@41c00000 { + compatible = "mmio-sram"; + reg = <0x00 0x41c00000 0x00 0x100000>; + ranges = <0x0 0x00 0x41c00000 0x100000>; + #address-cells = <1>; + #size-cells = <1>; + }; + + wkup_uart0: serial@42300000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x42300000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 897 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <48000000>; + current-speed = <115200>; + power-domains = <&k3_pds 287 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 287 2>; + clock-names = "fclk"; + }; + + mcu_uart0: serial@40a00000 { + compatible = "ti,j721e-uart", "ti,am654-uart"; + reg = <0x00 0x40a00000 0x00 0x100>; + reg-shift = <2>; + reg-io-width = <4>; + interrupts = <GIC_SPI 846 IRQ_TYPE_LEVEL_HIGH>; + clock-frequency = <96000000>; + current-speed = <115200>; + power-domains = <&k3_pds 149 TI_SCI_PD_EXCLUSIVE>; + clocks = <&k3_clks 149 2>; + clock-names = "fclk"; + }; +}; diff --git a/arch/arm64/boot/dts/ti/k3-j7200.dtsi b/arch/arm64/boot/dts/ti/k3-j7200.dtsi new file mode 100644 index 000000000000..aadf707f25f5 --- /dev/null +++ b/arch/arm64/boot/dts/ti/k3-j7200.dtsi @@ -0,0 +1,165 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Device Tree Source for J7200 SoC Family + * + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ + */ + +#include <dt-bindings/interrupt-controller/irq.h> +#include <dt-bindings/interrupt-controller/arm-gic.h> +#include <dt-bindings/pinctrl/k3.h> +#include <dt-bindings/soc/ti,sci_pm_domain.h> + +/ { + model = "Texas Instruments K3 J7200 SoC"; + compatible = "ti,j7200"; + interrupt-parent = <&gic500>; + #address-cells = <2>; + #size-cells = <2>; + + aliases { + serial0 = &wkup_uart0; + serial1 = &mcu_uart0; + serial2 = &main_uart0; + serial3 = &main_uart1; + serial4 = &main_uart2; + serial5 = &main_uart3; + serial6 = &main_uart4; + serial7 = &main_uart5; + serial8 = &main_uart6; + serial9 = &main_uart7; + serial10 = &main_uart8; + serial11 = &main_uart9; + }; + + cpus { + #address-cells = <1>; + #size-cells = <0>; + cpu-map { + cluster0: cluster0 { + core0 { + cpu = <&cpu0>; + }; + + core1 { + cpu = <&cpu1>; + }; + }; + + }; + + cpu0: cpu@0 { + compatible = "arm,cortex-a72"; + reg = <0x000>; + device_type = "cpu"; + enable-method = "psci"; + i-cache-size = <0xC000>; + i-cache-line-size = <64>; + i-cache-sets = <256>; + d-cache-size = <0x8000>; + d-cache-line-size = <64>; + d-cache-sets = <128>; + next-level-cache = <&L2_0>; + }; + + cpu1: cpu@1 { + compatible = "arm,cortex-a72"; + reg = <0x001>; + device_type = "cpu"; + enable-method = "psci"; + i-cache-size = <0xC000>; + i-cache-line-size = <64>; + i-cache-sets = <256>; + d-cache-size = <0x8000>; + d-cache-line-size = <64>; + d-cache-sets = <128>; + next-level-cache = <&L2_0>; + }; + }; + + L2_0: l2-cache0 { + compatible = "cache"; + cache-level = <2>; + cache-size = <0x100000>; + cache-line-size = <64>; + cache-sets = <2048>; + next-level-cache = <&msmc_l3>; + }; + + msmc_l3: l3-cache0 { + compatible = "cache"; + cache-level = <3>; + }; + + firmware { + optee { + compatible = "linaro,optee-tz"; + method = "smc"; + }; + + psci: psci { + compatible = "arm,psci-1.0"; + method = "smc"; + }; + }; + + a72_timer0: timer-cl0-cpu0 { + compatible = "arm,armv8-timer"; + interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>, /* cntpsirq */ + <GIC_PPI 14 IRQ_TYPE_LEVEL_LOW>, /* cntpnsirq */ + <GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>, /* cntvirq */ + <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>; /* cnthpirq */ + }; + + pmu: pmu { + compatible = "arm,armv8-pmuv3"; + interrupts = <GIC_PPI 7 IRQ_TYPE_LEVEL_HIGH>; + }; + + cbass_main: bus@100000 { + compatible = "simple-bus"; + #address-cells = <2>; + #size-cells = <2>; + ranges = <0x00 0x00100000 0x00 0x00100000 0x00 0x00020000>, /* ctrl mmr */ + <0x00 0x00600000 0x00 0x00600000 0x00 0x00031100>, /* GPIO */ + <0x00 0x00a40000 0x00 0x00a40000 0x00 0x00000800>, /* timesync router */ + <0x00 0x01000000 0x00 0x01000000 0x00 0x0d000000>, /* Most peripherals */ + <0x00 0x30000000 0x00 0x30000000 0x00 0x0c400000>, /* MAIN NAVSS */ + <0x00 0x70000000 0x00 0x70000000 0x00 0x00800000>, /* MSMC RAM */ + <0x41 0x00000000 0x41 0x00000000 0x01 0x00000000>, /* PCIe1 DAT */ + + /* MCUSS_WKUP Range */ + <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>, + <0x00 0x40200000 0x00 0x40200000 0x00 0x00998400>, + <0x00 0x40f00000 0x00 0x40f00000 0x00 0x00020000>, + <0x00 0x41000000 0x00 0x41000000 0x00 0x00020000>, + <0x00 0x41400000 0x00 0x41400000 0x00 0x00020000>, + <0x00 0x41c00000 0x00 0x41c00000 0x00 0x00100000>, + <0x00 0x42040000 0x00 0x42040000 0x00 0x03ac2400>, + <0x00 0x45100000 0x00 0x45100000 0x00 0x00c24000>, + <0x00 0x46000000 0x00 0x46000000 0x00 0x00200000>, + <0x00 0x47000000 0x00 0x47000000 0x00 0x00068400>, + <0x00 0x50000000 0x00 0x50000000 0x00 0x10000000>; + + cbass_mcu_wakeup: bus@28380000 { + compatible = "simple-bus"; + #address-cells = <2>; + #size-cells = <2>; + ranges = <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>, /* MCU NAVSS*/ + <0x00 0x40200000 0x00 0x40200000 0x00 0x00998400>, /* First peripheral window */ + <0x00 0x40f00000 0x00 0x40f00000 0x00 0x00020000>, /* CTRL_MMR0 */ + <0x00 0x41000000 0x00 0x41000000 0x00 0x00020000>, /* MCU R5F Core0 */ + <0x00 0x41400000 0x00 0x41400000 0x00 0x00020000>, /* MCU R5F Core1 */ + <0x00 0x41c00000 0x00 0x41c00000 0x00 0x00100000>, /* MCU SRAM */ + <0x00 0x42040000 0x00 0x42040000 0x00 0x03ac2400>, /* WKUP peripheral window */ + <0x00 0x45100000 0x00 0x45100000 0x00 0x00c24000>, /* MMRs, remaining NAVSS */ + <0x00 0x46000000 0x00 0x46000000 0x00 0x00200000>, /* CPSW */ + <0x00 0x47000000 0x00 0x47000000 0x00 0x00068400>, /* OSPI register space */ + <0x00 0x50000000 0x00 0x50000000 0x00 0x10000000>; /* FSS OSPI0/1 data region 0 */ + }; + }; +}; + +/* Now include the peripherals for each bus segments */ +#include "k3-j7200-main.dtsi" +#include "k3-j7200-mcu-wakeup.dtsi" -- 2.28.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] 23+ messages in thread
* Re: [PATCH v2 3/4] arm64: dts: ti: Add support for J7200 SoC 2020-08-27 6:51 ` [PATCH v2 3/4] arm64: dts: ti: Add support " Lokesh Vutla @ 2020-08-27 17:04 ` Suman Anna 2020-09-08 11:47 ` Nishanth Menon 2020-08-31 9:13 ` Peter Ujfalusi 1 sibling, 1 reply; 23+ messages in thread From: Suman Anna @ 2020-08-27 17:04 UTC (permalink / raw) To: Lokesh Vutla, Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Linux ARM Mailing List Hi Lokesh, On 8/27/20 1:51 AM, Lokesh Vutla wrote: > The J7200 SoC is a part of the K3 Multicore SoC architecture platform. > It is targeted for automotive gateway, vehicle compute systems, > Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. > The SoC aims to meet the complex processing needs of modern embedded > products. > > Some highlights of this SoC are: > * Dual Cortex-A72s in a single cluster, two clusters of lockstep > capable dual Cortex-R5F MCUs and a Centralized Device Management and > Security Controller (DMSC). > * Configurable L3 Cache and IO-coherent architecture with high data > throughput capable distributed DMA architecture under NAVSS. > * Integrated Ethernet switch supporting up to a total of 4 external ports > in addition to legacy Ethernet switch of up to 2 ports. > * Upto 1 PCIe-GEN3 controller, 1 USB3.0 Dual-role device subsystems, > 20 MCANs, 3 McASP, eMMC and SD, OSPI/HyperBus memory controller, I3C > and I2C, eCAP/eQEP, eHRPWM among other peripherals. > * One hardware accelerator block containing AES/DES/SHA/MD5 called SA2UL > management. > > See J7200 Technical Reference Manual (SPRUIU1, June 2020) > for further details: https://www.ti.com/lit/pdf/spruiu1 > > Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> > --- > arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 199 ++++++++++++++++++ > .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 84 ++++++++ > arch/arm64/boot/dts/ti/k3-j7200.dtsi | 165 +++++++++++++++ > 3 files changed, 448 insertions(+) > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200.dtsi > > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > new file mode 100644 > index 000000000000..70c8f7e941fb > --- /dev/null > +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > @@ -0,0 +1,199 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Device Tree Source for J7200 SoC Family Main Domain peripherals > + * > + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ > + */ > + > +&cbass_main { > + msmc_ram: sram@70000000 { > + compatible = "mmio-sram"; > + reg = <0x0 0x70000000 0x0 0x100000>; nit, I prefer that we use a consistent style across all nodes. Most of the places we are using 0x00 on the first cells of address and size. > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x0 0x0 0x70000000 0x100000>; > + > + atf-sram@0 { > + reg = <0x0 0x20000>; > + }; > + }; > + > + gic500: interrupt-controller@1800000 { > + compatible = "arm,gic-v3"; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + #interrupt-cells = <3>; > + interrupt-controller; > + reg = <0x00 0x01800000 0x00 0x10000>, /* GICD */ > + <0x00 0x01900000 0x00 0x100000>; /* GICR */ > + > + /* vcpumntirq: virtual CPU interface maintenance interrupt */ > + interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>; > + > + gic_its: msi-controller@1820000 { > + compatible = "arm,gic-v3-its"; > + reg = <0x00 0x01820000 0x00 0x10000>; > + socionext,synquacer-pre-its = <0x1000000 0x400000>; > + msi-controller; > + #msi-cells = <1>; > + }; > + }; > + > + main_navss: navss@30000000 { > + compatible = "simple-mfd"; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges = <0x00 0x30000000 0x00 0x30000000 0x00 0x0c400000>; > + > + secure_proxy_main: mailbox@32c00000 { > + compatible = "ti,am654-secure-proxy"; > + #mbox-cells = <1>; > + reg-names = "target_data", "rt", "scfg"; > + reg = <0x00 0x32c00000 0x00 0x100000>, > + <0x00 0x32400000 0x00 0x100000>, > + <0x00 0x32800000 0x00 0x100000>; > + interrupt-names = "rx_011"; > + interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>; > + }; > + }; > + > + main_pmx0: pinmux@11c000 { > + compatible = "pinctrl-single"; > + /* Proxy 0 addressing */ > + reg = <0x0 0x11c000 0x0 0x2b4>; This is the other node that uses a different style compared to all other nodes. Otherwise, Reviewed-by: Suman Anna <s-anna@ti.com> regards Suman > + #pinctrl-cells = <1>; > + pinctrl-single,register-width = <32>; > + pinctrl-single,function-mask = <0xffffffff>; > + }; > + > + main_uart0: serial@2800000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02800000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 192 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 146 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 146 2>; > + clock-names = "fclk"; > + }; > + > + main_uart1: serial@2810000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02810000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 193 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 278 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 278 2>; > + clock-names = "fclk"; > + }; > + > + main_uart2: serial@2820000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02820000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 194 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 279 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 279 2>; > + clock-names = "fclk"; > + }; > + > + main_uart3: serial@2830000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02830000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 195 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 280 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 280 2>; > + clock-names = "fclk"; > + }; > + > + main_uart4: serial@2840000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02840000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 196 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 281 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 281 2>; > + clock-names = "fclk"; > + }; > + > + main_uart5: serial@2850000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02850000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 197 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 282 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 282 2>; > + clock-names = "fclk"; > + }; > + > + main_uart6: serial@2860000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02860000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 198 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 283 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 283 2>; > + clock-names = "fclk"; > + }; > + > + main_uart7: serial@2870000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02870000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 199 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 284 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 284 2>; > + clock-names = "fclk"; > + }; > + > + main_uart8: serial@2880000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02880000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 248 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 285 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 285 2>; > + clock-names = "fclk"; > + }; > + > + main_uart9: serial@2890000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x02890000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 249 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 286 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 286 2>; > + clock-names = "fclk"; > + }; > +}; > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi > new file mode 100644 > index 000000000000..670e4c7cd9fe > --- /dev/null > +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi > @@ -0,0 +1,84 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Device Tree Source for J7200 SoC Family MCU/WAKEUP Domain peripherals > + * > + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ > + */ > + > +&cbass_mcu_wakeup { > + dmsc: dmsc@44083000 { > + compatible = "ti,k2g-sci"; > + ti,host-id = <12>; > + > + mbox-names = "rx", "tx"; > + > + mboxes= <&secure_proxy_main 11>, > + <&secure_proxy_main 13>; > + > + reg-names = "debug_messages"; > + reg = <0x00 0x44083000 0x0 0x1000>; > + > + k3_pds: power-controller { > + compatible = "ti,sci-pm-domain"; > + #power-domain-cells = <2>; > + }; > + > + k3_clks: clocks { > + compatible = "ti,k2g-sci-clk"; > + #clock-cells = <2>; > + }; > + > + k3_reset: reset-controller { > + compatible = "ti,sci-reset"; > + #reset-cells = <2>; > + }; > + }; > + > + chipid@43000014 { > + compatible = "ti,am654-chipid"; > + reg = <0x0 0x43000014 0x0 0x4>; > + }; > + > + wkup_pmx0: pinmux@4301c000 { > + compatible = "pinctrl-single"; > + /* Proxy 0 addressing */ > + reg = <0x00 0x4301c000 0x00 0x178>; > + #pinctrl-cells = <1>; > + pinctrl-single,register-width = <32>; > + pinctrl-single,function-mask = <0xffffffff>; > + }; > + > + mcu_ram: sram@41c00000 { > + compatible = "mmio-sram"; > + reg = <0x00 0x41c00000 0x00 0x100000>; > + ranges = <0x0 0x00 0x41c00000 0x100000>; > + #address-cells = <1>; > + #size-cells = <1>; > + }; > + > + wkup_uart0: serial@42300000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x42300000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 897 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <48000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 287 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 287 2>; > + clock-names = "fclk"; > + }; > + > + mcu_uart0: serial@40a00000 { > + compatible = "ti,j721e-uart", "ti,am654-uart"; > + reg = <0x00 0x40a00000 0x00 0x100>; > + reg-shift = <2>; > + reg-io-width = <4>; > + interrupts = <GIC_SPI 846 IRQ_TYPE_LEVEL_HIGH>; > + clock-frequency = <96000000>; > + current-speed = <115200>; > + power-domains = <&k3_pds 149 TI_SCI_PD_EXCLUSIVE>; > + clocks = <&k3_clks 149 2>; > + clock-names = "fclk"; > + }; > +}; > diff --git a/arch/arm64/boot/dts/ti/k3-j7200.dtsi b/arch/arm64/boot/dts/ti/k3-j7200.dtsi > new file mode 100644 > index 000000000000..aadf707f25f5 > --- /dev/null > +++ b/arch/arm64/boot/dts/ti/k3-j7200.dtsi > @@ -0,0 +1,165 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Device Tree Source for J7200 SoC Family > + * > + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ > + */ > + > +#include <dt-bindings/interrupt-controller/irq.h> > +#include <dt-bindings/interrupt-controller/arm-gic.h> > +#include <dt-bindings/pinctrl/k3.h> > +#include <dt-bindings/soc/ti,sci_pm_domain.h> > + > +/ { > + model = "Texas Instruments K3 J7200 SoC"; > + compatible = "ti,j7200"; > + interrupt-parent = <&gic500>; > + #address-cells = <2>; > + #size-cells = <2>; > + > + aliases { > + serial0 = &wkup_uart0; > + serial1 = &mcu_uart0; > + serial2 = &main_uart0; > + serial3 = &main_uart1; > + serial4 = &main_uart2; > + serial5 = &main_uart3; > + serial6 = &main_uart4; > + serial7 = &main_uart5; > + serial8 = &main_uart6; > + serial9 = &main_uart7; > + serial10 = &main_uart8; > + serial11 = &main_uart9; > + }; > + > + cpus { > + #address-cells = <1>; > + #size-cells = <0>; > + cpu-map { > + cluster0: cluster0 { > + core0 { > + cpu = <&cpu0>; > + }; > + > + core1 { > + cpu = <&cpu1>; > + }; > + }; > + > + }; > + > + cpu0: cpu@0 { > + compatible = "arm,cortex-a72"; > + reg = <0x000>; > + device_type = "cpu"; > + enable-method = "psci"; > + i-cache-size = <0xC000>; > + i-cache-line-size = <64>; > + i-cache-sets = <256>; > + d-cache-size = <0x8000>; > + d-cache-line-size = <64>; > + d-cache-sets = <128>; > + next-level-cache = <&L2_0>; > + }; > + > + cpu1: cpu@1 { > + compatible = "arm,cortex-a72"; > + reg = <0x001>; > + device_type = "cpu"; > + enable-method = "psci"; > + i-cache-size = <0xC000>; > + i-cache-line-size = <64>; > + i-cache-sets = <256>; > + d-cache-size = <0x8000>; > + d-cache-line-size = <64>; > + d-cache-sets = <128>; > + next-level-cache = <&L2_0>; > + }; > + }; > + > + L2_0: l2-cache0 { > + compatible = "cache"; > + cache-level = <2>; > + cache-size = <0x100000>; > + cache-line-size = <64>; > + cache-sets = <2048>; > + next-level-cache = <&msmc_l3>; > + }; > + > + msmc_l3: l3-cache0 { > + compatible = "cache"; > + cache-level = <3>; > + }; > + > + firmware { > + optee { > + compatible = "linaro,optee-tz"; > + method = "smc"; > + }; > + > + psci: psci { > + compatible = "arm,psci-1.0"; > + method = "smc"; > + }; > + }; > + > + a72_timer0: timer-cl0-cpu0 { > + compatible = "arm,armv8-timer"; > + interrupts = <GIC_PPI 13 IRQ_TYPE_LEVEL_LOW>, /* cntpsirq */ > + <GIC_PPI 14 IRQ_TYPE_LEVEL_LOW>, /* cntpnsirq */ > + <GIC_PPI 11 IRQ_TYPE_LEVEL_LOW>, /* cntvirq */ > + <GIC_PPI 10 IRQ_TYPE_LEVEL_LOW>; /* cnthpirq */ > + }; > + > + pmu: pmu { > + compatible = "arm,armv8-pmuv3"; > + interrupts = <GIC_PPI 7 IRQ_TYPE_LEVEL_HIGH>; > + }; > + > + cbass_main: bus@100000 { > + compatible = "simple-bus"; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges = <0x00 0x00100000 0x00 0x00100000 0x00 0x00020000>, /* ctrl mmr */ > + <0x00 0x00600000 0x00 0x00600000 0x00 0x00031100>, /* GPIO */ > + <0x00 0x00a40000 0x00 0x00a40000 0x00 0x00000800>, /* timesync router */ > + <0x00 0x01000000 0x00 0x01000000 0x00 0x0d000000>, /* Most peripherals */ > + <0x00 0x30000000 0x00 0x30000000 0x00 0x0c400000>, /* MAIN NAVSS */ > + <0x00 0x70000000 0x00 0x70000000 0x00 0x00800000>, /* MSMC RAM */ > + <0x41 0x00000000 0x41 0x00000000 0x01 0x00000000>, /* PCIe1 DAT */ > + > + /* MCUSS_WKUP Range */ > + <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>, > + <0x00 0x40200000 0x00 0x40200000 0x00 0x00998400>, > + <0x00 0x40f00000 0x00 0x40f00000 0x00 0x00020000>, > + <0x00 0x41000000 0x00 0x41000000 0x00 0x00020000>, > + <0x00 0x41400000 0x00 0x41400000 0x00 0x00020000>, > + <0x00 0x41c00000 0x00 0x41c00000 0x00 0x00100000>, > + <0x00 0x42040000 0x00 0x42040000 0x00 0x03ac2400>, > + <0x00 0x45100000 0x00 0x45100000 0x00 0x00c24000>, > + <0x00 0x46000000 0x00 0x46000000 0x00 0x00200000>, > + <0x00 0x47000000 0x00 0x47000000 0x00 0x00068400>, > + <0x00 0x50000000 0x00 0x50000000 0x00 0x10000000>; > + > + cbass_mcu_wakeup: bus@28380000 { > + compatible = "simple-bus"; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges = <0x00 0x28380000 0x00 0x28380000 0x00 0x03880000>, /* MCU NAVSS*/ > + <0x00 0x40200000 0x00 0x40200000 0x00 0x00998400>, /* First peripheral window */ > + <0x00 0x40f00000 0x00 0x40f00000 0x00 0x00020000>, /* CTRL_MMR0 */ > + <0x00 0x41000000 0x00 0x41000000 0x00 0x00020000>, /* MCU R5F Core0 */ > + <0x00 0x41400000 0x00 0x41400000 0x00 0x00020000>, /* MCU R5F Core1 */ > + <0x00 0x41c00000 0x00 0x41c00000 0x00 0x00100000>, /* MCU SRAM */ > + <0x00 0x42040000 0x00 0x42040000 0x00 0x03ac2400>, /* WKUP peripheral window */ > + <0x00 0x45100000 0x00 0x45100000 0x00 0x00c24000>, /* MMRs, remaining NAVSS */ > + <0x00 0x46000000 0x00 0x46000000 0x00 0x00200000>, /* CPSW */ > + <0x00 0x47000000 0x00 0x47000000 0x00 0x00068400>, /* OSPI register space */ > + <0x00 0x50000000 0x00 0x50000000 0x00 0x10000000>; /* FSS OSPI0/1 data region 0 */ > + }; > + }; > +}; > + > +/* Now include the peripherals for each bus segments */ > +#include "k3-j7200-main.dtsi" > +#include "k3-j7200-mcu-wakeup.dtsi" > _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 3/4] arm64: dts: ti: Add support for J7200 SoC 2020-08-27 17:04 ` Suman Anna @ 2020-09-08 11:47 ` Nishanth Menon 0 siblings, 0 replies; 23+ messages in thread From: Nishanth Menon @ 2020-09-08 11:47 UTC (permalink / raw) To: Suman Anna Cc: Device Tree Mailing List, Grygorii Strashko, Lokesh Vutla, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List On 12:04-20200827, Suman Anna wrote: will just piggy on this thread.. > > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > > new file mode 100644 > > index 000000000000..70c8f7e941fb > > --- /dev/null > > +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > > @@ -0,0 +1,199 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Device Tree Source for J7200 SoC Family Main Domain peripherals > > + * > > + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ > > + */ > > + > > +&cbass_main { > > + msmc_ram: sram@70000000 { > > + compatible = "mmio-sram"; > > + reg = <0x0 0x70000000 0x0 0x100000>; > > nit, I prefer that we use a consistent style across all nodes. Most of the > places we are using 0x00 on the first cells of address and size. yes please. Will be great if you could address this. > [...] > > + > > + main_pmx0: pinmux@11c000 { > > + compatible = "pinctrl-single"; > > + /* Proxy 0 addressing */ > > + reg = <0x0 0x11c000 0x0 0x2b4>; > > This is the other node that uses a different style compared to all other nodes. > > Otherwise, > > Reviewed-by: Suman Anna <s-anna@ti.com> > > regards > Suman > [..] > > diff --git a/arch/arm64/boot/dts/ti/k3-j7200.dtsi b/arch/arm64/boot/dts/ti/k3-j7200.dtsi > > new file mode 100644 > > index 000000000000..aadf707f25f5 > > --- /dev/null > > +++ b/arch/arm64/boot/dts/ti/k3-j7200.dtsi > > @@ -0,0 +1,165 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Device Tree Source for J7200 SoC Family > > + * > > + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ > > + */ > > + > > +#include <dt-bindings/interrupt-controller/irq.h> > > +#include <dt-bindings/interrupt-controller/arm-gic.h> > > +#include <dt-bindings/pinctrl/k3.h> > > +#include <dt-bindings/soc/ti,sci_pm_domain.h> > > + > > +/ { > > + model = "Texas Instruments K3 J7200 SoC"; > > + compatible = "ti,j7200"; > > + interrupt-parent = <&gic500>; > > + #address-cells = <2>; > > + #size-cells = <2>; > > + > > + aliases { > > + serial0 = &wkup_uart0; > > + serial1 = &mcu_uart0; > > + serial2 = &main_uart0; > > + serial3 = &main_uart1; > > + serial4 = &main_uart2; > > + serial5 = &main_uart3; > > + serial6 = &main_uart4; > > + serial7 = &main_uart5; > > + serial8 = &main_uart6; > > + serial9 = &main_uart7; > > + serial10 = &main_uart8; > > + serial11 = &main_uart9; > > + }; > > + > > + cpus { > > + #address-cells = <1>; > > + #size-cells = <0>; > > + cpu-map { > > + cluster0: cluster0 { > > + core0 { > > + cpu = <&cpu0>; > > + }; > > + > > + core1 { > > + cpu = <&cpu1>; > > + }; > > + }; > > + > > + }; > > + > > + cpu0: cpu@0 { > > + compatible = "arm,cortex-a72"; > > + reg = <0x000>; > > + device_type = "cpu"; > > + enable-method = "psci"; > > + i-cache-size = <0xC000>; minor nitpick comment -> 0xc000 ? I just saw j721e has the same as well.. heck.. I thought I found them all, but looks like I missed. > > + i-cache-line-size = <64>; > > + i-cache-sets = <256>; > > + d-cache-size = <0x8000>; > > + d-cache-line-size = <64>; > > + d-cache-sets = <128>; > > + next-level-cache = <&L2_0>; > > + }; > > + > > + cpu1: cpu@1 { > > + compatible = "arm,cortex-a72"; > > + reg = <0x001>; > > + device_type = "cpu"; > > + enable-method = "psci"; > > + i-cache-size = <0xC000>; same.. > > + i-cache-line-size = <64>; > > + i-cache-sets = <256>; > > + d-cache-size = <0x8000>; > > + d-cache-line-size = <64>; > > + d-cache-sets = <128>; > > + next-level-cache = <&L2_0>; > > + }; > > + }; > > + > Other wise, looks fine to me. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 3/4] arm64: dts: ti: Add support for J7200 SoC 2020-08-27 6:51 ` [PATCH v2 3/4] arm64: dts: ti: Add support " Lokesh Vutla 2020-08-27 17:04 ` Suman Anna @ 2020-08-31 9:13 ` Peter Ujfalusi 2020-09-02 3:51 ` Lokesh Vutla 1 sibling, 1 reply; 23+ messages in thread From: Peter Ujfalusi @ 2020-08-31 9:13 UTC (permalink / raw) To: Lokesh Vutla, Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Linux ARM Mailing List Hi Lokesh, On 27/08/2020 9.51, Lokesh Vutla wrote: > The J7200 SoC is a part of the K3 Multicore SoC architecture platform. > It is targeted for automotive gateway, vehicle compute systems, > Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. > The SoC aims to meet the complex processing needs of modern embedded > products. > > Some highlights of this SoC are: > * Dual Cortex-A72s in a single cluster, two clusters of lockstep > capable dual Cortex-R5F MCUs and a Centralized Device Management and > Security Controller (DMSC). > * Configurable L3 Cache and IO-coherent architecture with high data > throughput capable distributed DMA architecture under NAVSS. > * Integrated Ethernet switch supporting up to a total of 4 external ports > in addition to legacy Ethernet switch of up to 2 ports. > * Upto 1 PCIe-GEN3 controller, 1 USB3.0 Dual-role device subsystems, > 20 MCANs, 3 McASP, eMMC and SD, OSPI/HyperBus memory controller, I3C > and I2C, eCAP/eQEP, eHRPWM among other peripherals. > * One hardware accelerator block containing AES/DES/SHA/MD5 called SA2UL > management. > > See J7200 Technical Reference Manual (SPRUIU1, June 2020) > for further details: https://www.ti.com/lit/pdf/spruiu1 > > Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> > --- > arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 199 ++++++++++++++++++ > .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 84 ++++++++ > arch/arm64/boot/dts/ti/k3-j7200.dtsi | 165 +++++++++++++++ > 3 files changed, 448 insertions(+) > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200.dtsi > > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > new file mode 100644 > index 000000000000..70c8f7e941fb > --- /dev/null > +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > @@ -0,0 +1,199 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Device Tree Source for J7200 SoC Family Main Domain peripherals > + * > + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ > + */ > + > +&cbass_main { > + msmc_ram: sram@70000000 { > + compatible = "mmio-sram"; > + reg = <0x0 0x70000000 0x0 0x100000>; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x0 0x0 0x70000000 0x100000>; > + > + atf-sram@0 { > + reg = <0x0 0x20000>; > + }; > + }; > + > + gic500: interrupt-controller@1800000 { > + compatible = "arm,gic-v3"; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + #interrupt-cells = <3>; > + interrupt-controller; > + reg = <0x00 0x01800000 0x00 0x10000>, /* GICD */ > + <0x00 0x01900000 0x00 0x100000>; /* GICR */ > + > + /* vcpumntirq: virtual CPU interface maintenance interrupt */ > + interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>; > + > + gic_its: msi-controller@1820000 { > + compatible = "arm,gic-v3-its"; > + reg = <0x00 0x01820000 0x00 0x10000>; > + socionext,synquacer-pre-its = <0x1000000 0x400000>; > + msi-controller; > + #msi-cells = <1>; > + }; > + }; > + > + main_navss: navss@30000000 { > + compatible = "simple-mfd"; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges = <0x00 0x30000000 0x00 0x30000000 0x00 0x0c400000>; > + > + secure_proxy_main: mailbox@32c00000 { > + compatible = "ti,am654-secure-proxy"; > + #mbox-cells = <1>; > + reg-names = "target_data", "rt", "scfg"; > + reg = <0x00 0x32c00000 0x00 0x100000>, > + <0x00 0x32400000 0x00 0x100000>, > + <0x00 0x32800000 0x00 0x100000>; > + interrupt-names = "rx_011"; > + interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>; > + }; Would it make sense to have the nodes needed for DMA also in the initial commit? mainline is prepared for it. - Péter Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 3/4] arm64: dts: ti: Add support for J7200 SoC 2020-08-31 9:13 ` Peter Ujfalusi @ 2020-09-02 3:51 ` Lokesh Vutla 0 siblings, 0 replies; 23+ messages in thread From: Lokesh Vutla @ 2020-09-02 3:51 UTC (permalink / raw) To: Peter Ujfalusi, Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Linux ARM Mailing List Hi Peter, On 31/08/20 2:43 pm, Peter Ujfalusi wrote: > Hi Lokesh, > > On 27/08/2020 9.51, Lokesh Vutla wrote: >> The J7200 SoC is a part of the K3 Multicore SoC architecture platform. >> It is targeted for automotive gateway, vehicle compute systems, >> Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. >> The SoC aims to meet the complex processing needs of modern embedded >> products. >> >> Some highlights of this SoC are: >> * Dual Cortex-A72s in a single cluster, two clusters of lockstep >> capable dual Cortex-R5F MCUs and a Centralized Device Management and >> Security Controller (DMSC). >> * Configurable L3 Cache and IO-coherent architecture with high data >> throughput capable distributed DMA architecture under NAVSS. >> * Integrated Ethernet switch supporting up to a total of 4 external ports >> in addition to legacy Ethernet switch of up to 2 ports. >> * Upto 1 PCIe-GEN3 controller, 1 USB3.0 Dual-role device subsystems, >> 20 MCANs, 3 McASP, eMMC and SD, OSPI/HyperBus memory controller, I3C >> and I2C, eCAP/eQEP, eHRPWM among other peripherals. >> * One hardware accelerator block containing AES/DES/SHA/MD5 called SA2UL >> management. >> >> See J7200 Technical Reference Manual (SPRUIU1, June 2020) >> for further details: https://www.ti.com/lit/pdf/spruiu1 >> >> Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> >> --- >> arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 199 ++++++++++++++++++ >> .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 84 ++++++++ >> arch/arm64/boot/dts/ti/k3-j7200.dtsi | 165 +++++++++++++++ >> 3 files changed, 448 insertions(+) >> create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-main.dtsi >> create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi >> create mode 100644 arch/arm64/boot/dts/ti/k3-j7200.dtsi >> >> diff --git a/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi >> new file mode 100644 >> index 000000000000..70c8f7e941fb >> --- /dev/null >> +++ b/arch/arm64/boot/dts/ti/k3-j7200-main.dtsi >> @@ -0,0 +1,199 @@ >> +// SPDX-License-Identifier: GPL-2.0 >> +/* >> + * Device Tree Source for J7200 SoC Family Main Domain peripherals >> + * >> + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ >> + */ >> + >> +&cbass_main { >> + msmc_ram: sram@70000000 { >> + compatible = "mmio-sram"; >> + reg = <0x0 0x70000000 0x0 0x100000>; >> + #address-cells = <1>; >> + #size-cells = <1>; >> + ranges = <0x0 0x0 0x70000000 0x100000>; >> + >> + atf-sram@0 { >> + reg = <0x0 0x20000>; >> + }; >> + }; >> + >> + gic500: interrupt-controller@1800000 { >> + compatible = "arm,gic-v3"; >> + #address-cells = <2>; >> + #size-cells = <2>; >> + ranges; >> + #interrupt-cells = <3>; >> + interrupt-controller; >> + reg = <0x00 0x01800000 0x00 0x10000>, /* GICD */ >> + <0x00 0x01900000 0x00 0x100000>; /* GICR */ >> + >> + /* vcpumntirq: virtual CPU interface maintenance interrupt */ >> + interrupts = <GIC_PPI 9 IRQ_TYPE_LEVEL_HIGH>; >> + >> + gic_its: msi-controller@1820000 { >> + compatible = "arm,gic-v3-its"; >> + reg = <0x00 0x01820000 0x00 0x10000>; >> + socionext,synquacer-pre-its = <0x1000000 0x400000>; >> + msi-controller; >> + #msi-cells = <1>; >> + }; >> + }; >> + >> + main_navss: navss@30000000 { >> + compatible = "simple-mfd"; >> + #address-cells = <2>; >> + #size-cells = <2>; >> + ranges = <0x00 0x30000000 0x00 0x30000000 0x00 0x0c400000>; >> + >> + secure_proxy_main: mailbox@32c00000 { >> + compatible = "ti,am654-secure-proxy"; >> + #mbox-cells = <1>; >> + reg-names = "target_data", "rt", "scfg"; >> + reg = <0x00 0x32c00000 0x00 0x100000>, >> + <0x00 0x32400000 0x00 0x100000>, >> + <0x00 0x32800000 0x00 0x100000>; >> + interrupt-names = "rx_011"; >> + interrupts = <GIC_SPI 37 IRQ_TYPE_LEVEL_HIGH>; >> + }; > > Would it make sense to have the nodes needed for DMA also in the initial > commit? > mainline is prepared for it. They are fairly independent patches. IMHO, they can come separately. This series is lying around for long time. I would prefer to get this base support in asap. Thanks and regards, Lokesh > > - Péter > > Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. > Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki > _______________________________________________ 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] 23+ messages in thread
* [PATCH v2 4/4] arm64: dts: ti: Add support for J7200 Common Processor Board 2020-08-27 6:51 [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Lokesh Vutla ` (2 preceding siblings ...) 2020-08-27 6:51 ` [PATCH v2 3/4] arm64: dts: ti: Add support " Lokesh Vutla @ 2020-08-27 6:51 ` Lokesh Vutla 2020-09-08 11:57 ` Nishanth Menon 2020-08-27 9:09 ` [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Grygorii Strashko 2020-09-07 12:02 ` Lokesh Vutla 5 siblings, 1 reply; 23+ messages in thread From: Lokesh Vutla @ 2020-08-27 6:51 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Lokesh Vutla, Sekhar Nori, Linux ARM Mailing List Add support for J7200 Common Processor Board. The EVM architecture is very similar to J721E as follows: +------------------------------------------------------+ | +-------------------------------------------+ | | | | | | | Add-on Card 1 Options | | | | | | | +-------------------------------------------+ | | | | | | +-------------------+ | | | | | | | SOM | | | +--------------+ | | | | | | | | | | | Add-on | +-------------------+ | | | Card 2 | | Power Supply | | Options | | | | | | | | | +--------------+ | <--- +------------------------------------------------------+ Common Processor Board Common Processor board is the baseboard that has most of the actual connectors, power supply etc. A SOM (System on Module) is plugged on to the common processor board and this contains the SoC, PMIC, DDR and basic high speed components necessary for functionality. Note: * The minimum configuration required to boot up the board is System On Module(SOM) + Common Processor Board. * Since there is just a single SOM and Common Processor Board, we are maintaining common processor board as the base dts and SOM as the dtsi that we include. In the future as more SOM's appear, we should move common processor board as a dtsi and include configurations as dts. * All daughter cards beyond the basic boards shall be maintained as overlays. * Since J7200 is derivative of J721E re-using the CONFIG_ARCH_K3_J721E_SOC config for building J7200 dts files. Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com> --- arch/arm64/boot/dts/ti/Makefile | 3 +- .../dts/ti/k3-j7200-common-proc-board.dts | 64 +++++++++++++++++++ arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi | 29 +++++++++ 3 files changed, 95 insertions(+), 1 deletion(-) create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile index 05c0bebf65d4..60ab9b72c130 100644 --- a/arch/arm64/boot/dts/ti/Makefile +++ b/arch/arm64/boot/dts/ti/Makefile @@ -8,4 +8,5 @@ dtb-$(CONFIG_ARCH_K3_AM6_SOC) += k3-am654-base-board.dtb -dtb-$(CONFIG_ARCH_K3_J721E_SOC) += k3-j721e-common-proc-board.dtb +dtb-$(CONFIG_ARCH_K3_J721E_SOC) += k3-j721e-common-proc-board.dtb \ + k3-j7200-common-proc-board.dtb diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts new file mode 100644 index 000000000000..e27069317c4e --- /dev/null +++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts @@ -0,0 +1,64 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ + */ + +/dts-v1/; + +#include "k3-j7200-som-p0.dtsi" + +/ { + chosen { + stdout-path = "serial2:115200n8"; + bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x02800000"; + }; +}; + +&wkup_uart0 { + /* Wakeup UART is used by System firmware */ + status = "disabled"; +}; + +&main_uart0 { + power-domains = <&k3_pds 146 TI_SCI_PD_SHARED>; +}; + +&main_uart2 { + /* MAIN UART 2 is used by R5F firmware */ + status = "disabled"; +}; + +&main_uart3 { + /* UART not brought out */ + status = "disabled"; +}; + +&main_uart4 { + /* UART not brought out */ + status = "disabled"; +}; + +&main_uart5 { + /* UART not brought out */ + status = "disabled"; +}; + +&main_uart6 { + /* UART not brought out */ + status = "disabled"; +}; + +&main_uart7 { + /* UART not brought out */ + status = "disabled"; +}; + +&main_uart8 { + /* UART not brought out */ + status = "disabled"; +}; + +&main_uart9 { + /* UART not brought out */ + status = "disabled"; +}; diff --git a/arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi new file mode 100644 index 000000000000..22fc50bd5c4c --- /dev/null +++ b/arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi @@ -0,0 +1,29 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2020 Texas Instruments Incorporated - https://www.ti.com/ + */ + +/dts-v1/; + +#include "k3-j7200.dtsi" + +/ { + memory@80000000 { + device_type = "memory"; + /* 4G RAM */ + reg = <0x00000000 0x80000000 0x00000000 0x80000000>, + <0x00000008 0x80000000 0x00000000 0x80000000>; + }; + + reserved_memory: reserved-memory { + #address-cells = <2>; + #size-cells = <2>; + ranges; + + secure_ddr: optee@9e800000 { + reg = <0x00 0x9e800000 0x00 0x01800000>; + alignment = <0x1000>; + no-map; + }; + }; +}; -- 2.28.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] 23+ messages in thread
* Re: [PATCH v2 4/4] arm64: dts: ti: Add support for J7200 Common Processor Board 2020-08-27 6:51 ` [PATCH v2 4/4] arm64: dts: ti: Add support for J7200 Common Processor Board Lokesh Vutla @ 2020-09-08 11:57 ` Nishanth Menon 0 siblings, 0 replies; 23+ messages in thread From: Nishanth Menon @ 2020-09-08 11:57 UTC (permalink / raw) To: Lokesh Vutla, ssantosh Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List On 12:21-20200827, Lokesh Vutla wrote: [..] > > Signed-off-by: Lokesh Vutla <lokeshvutla@ti.com> > Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com> > --- > arch/arm64/boot/dts/ti/Makefile | 3 +- > .../dts/ti/k3-j7200-common-proc-board.dts | 64 +++++++++++++++++++ > arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi | 29 +++++++++ > 3 files changed, 95 insertions(+), 1 deletion(-) > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi > > diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile > index 05c0bebf65d4..60ab9b72c130 100644 > --- a/arch/arm64/boot/dts/ti/Makefile > +++ b/arch/arm64/boot/dts/ti/Makefile > @@ -8,4 +8,5 @@ > > dtb-$(CONFIG_ARCH_K3_AM6_SOC) += k3-am654-base-board.dtb > > -dtb-$(CONFIG_ARCH_K3_J721E_SOC) += k3-j721e-common-proc-board.dtb > +dtb-$(CONFIG_ARCH_K3_J721E_SOC) += k3-j721e-common-proc-board.dtb \ > + k3-j7200-common-proc-board.dtb As we discussed in [1], to allow lesser dependencies and better maintainability, a) Lets first s/CONFIG_ARCH_K3_AM6_SOC/CONFIG_ARCH_K3/ and same with CONFIG_ARCH_K3_J721E_SOC in the dts Makefile. Santosh: headsup, we are cleaning up users of the SOC config options[2] - we will clean the Kconfig up in a follow-on rc/kernel rev. b) lets not use the \ and instead, use += - it will be lesser diff as we add new dtbs. +dtb-$(CONFIG_ARCH_K3) += k3-j7200-common-proc-board.dtb [1] https://lore.kernel.org/linux-arm-kernel/20200908112534.t5bgrjf7y3a6l2ss@akan/ [2] https://mailman.alsa-project.org/pipermail/alsa-devel/2020-September/173791.html > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > new file mode 100644 > index 000000000000..e27069317c4e > --- /dev/null > +++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts Otherwise looks fine to me. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform 2020-08-27 6:51 [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Lokesh Vutla ` (3 preceding siblings ...) 2020-08-27 6:51 ` [PATCH v2 4/4] arm64: dts: ti: Add support for J7200 Common Processor Board Lokesh Vutla @ 2020-08-27 9:09 ` Grygorii Strashko 2020-09-07 12:02 ` Lokesh Vutla 5 siblings, 0 replies; 23+ messages in thread From: Grygorii Strashko @ 2020-08-27 9:09 UTC (permalink / raw) To: Lokesh Vutla, Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Sekhar Nori, Linux ARM Mailing List On 27/08/2020 09:51, Lokesh Vutla wrote: > This series adds initial support for latest new SoC, J7200, from Texas Instruments. > > The J7200 SoC is a part of the K3 Multicore SoC architecture platform. > It is targeted for for automotive gateway, vehicle compute systems, > Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. > The SoC aims to meet the complex processing needs of modern embedded products. > > See J7200 Technical Reference Manual (SPRUIU1, June 2020) > for further details: https://www.ti.com/lit/pdf/spruiu1 > > Changes since v1: > - Swapped Patch 1 and 2 as suggested by Nishanth. > - Added description for each SoC in yaml bindings. > > Testing: > - ./scripts/checkpatch --strict > - Few warningns about Line length exceeding 100 columns. > But these are corresponding to comments > - v8make dtbs_check > - DT_SCHEMA_FLAGS="-u" > DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" > v8make dtbs_check > - DT_SCHEMA_FLAGS="-u" > DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" > v8make dt_binding_check > > > Lokesh Vutla (4): > dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema > dt-bindings: arm: ti: Add bindings for J7200 SoC > arm64: dts: ti: Add support for J7200 SoC > arm64: dts: ti: Add support for J7200 Common Processor Board > > .../devicetree/bindings/arm/ti/k3.txt | 26 --- > .../devicetree/bindings/arm/ti/k3.yaml | 35 +++ > MAINTAINERS | 2 +- > arch/arm64/boot/dts/ti/Makefile | 3 +- > .../dts/ti/k3-j7200-common-proc-board.dts | 64 ++++++ > arch/arm64/boot/dts/ti/k3-j7200-main.dtsi | 199 ++++++++++++++++++ > .../boot/dts/ti/k3-j7200-mcu-wakeup.dtsi | 84 ++++++++ > arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi | 29 +++ > arch/arm64/boot/dts/ti/k3-j7200.dtsi | 165 +++++++++++++++ > 9 files changed, 579 insertions(+), 28 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/arm/ti/k3.txt > create mode 100644 Documentation/devicetree/bindings/arm/ti/k3.yaml > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-main.dtsi > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi > create mode 100644 arch/arm64/boot/dts/ti/k3-j7200.dtsi > Thank you Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com> -- Best regards, grygorii _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform 2020-08-27 6:51 [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Lokesh Vutla ` (4 preceding siblings ...) 2020-08-27 9:09 ` [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Grygorii Strashko @ 2020-09-07 12:02 ` Lokesh Vutla 2020-09-07 14:14 ` Nishanth Menon 5 siblings, 1 reply; 23+ messages in thread From: Lokesh Vutla @ 2020-09-07 12:02 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Linux ARM Mailing List Hi, On 27/08/20 12:21 pm, Lokesh Vutla wrote: > This series adds initial support for latest new SoC, J7200, from Texas Instruments. > > The J7200 SoC is a part of the K3 Multicore SoC architecture platform. > It is targeted for for automotive gateway, vehicle compute systems, > Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. > The SoC aims to meet the complex processing needs of modern embedded products. > > See J7200 Technical Reference Manual (SPRUIU1, June 2020) > for further details: https://www.ti.com/lit/pdf/spruiu1 > > Changes since v1: > - Swapped Patch 1 and 2 as suggested by Nishanth. > - Added description for each SoC in yaml bindings. > > Testing: > - ./scripts/checkpatch --strict > - Few warningns about Line length exceeding 100 columns. > But these are corresponding to comments > - v8make dtbs_check > - DT_SCHEMA_FLAGS="-u" > DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" > v8make dtbs_check > - DT_SCHEMA_FLAGS="-u" > DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" > v8make dt_binding_check This series has been lying around for soo long with no major comments. It will be nice to get this merged. I understand we are waiting for Acks on yaml documentation but it would be bad to miss a merge window for a new platform because we are waiting for Acks on yaml conversion. Thanks and regards, Lokesh _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform 2020-09-07 12:02 ` Lokesh Vutla @ 2020-09-07 14:14 ` Nishanth Menon 2020-09-07 14:23 ` Lokesh Vutla 0 siblings, 1 reply; 23+ messages in thread From: Nishanth Menon @ 2020-09-07 14:14 UTC (permalink / raw) To: Lokesh Vutla Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List On 17:32-20200907, Lokesh Vutla wrote: > Hi, > > On 27/08/20 12:21 pm, Lokesh Vutla wrote: > > This series adds initial support for latest new SoC, J7200, from Texas Instruments. > > > > The J7200 SoC is a part of the K3 Multicore SoC architecture platform. > > It is targeted for for automotive gateway, vehicle compute systems, > > Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. > > The SoC aims to meet the complex processing needs of modern embedded products. > > > > See J7200 Technical Reference Manual (SPRUIU1, June 2020) > > for further details: https://www.ti.com/lit/pdf/spruiu1 > > > > Changes since v1: > > - Swapped Patch 1 and 2 as suggested by Nishanth. > > - Added description for each SoC in yaml bindings. > > > > Testing: > > - ./scripts/checkpatch --strict > > - Few warningns about Line length exceeding 100 columns. > > But these are corresponding to comments > > - v8make dtbs_check > > - DT_SCHEMA_FLAGS="-u" > > DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" > > v8make dtbs_check > > - DT_SCHEMA_FLAGS="-u" > > DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" > > v8make dt_binding_check > > This series has been lying around for soo long with no major comments. It will > be nice to get this merged. I understand we are waiting for Acks on yaml > documentation but it would be bad to miss a merge window for a new platform > because we are waiting for Acks on yaml conversion. I do require Rob / DT maintainer to ack the DT yaml conversion and the j7200 binding addition. Besides yaml and compatibility acks, there are a few ancillary comments to fix up.. Kconfig -> I think we should either stay with status quo and create a new config option per SoC OR rename the config to be generic (using j7200 with j721e SoC config is not very consistent). In addition, around the stuff that is going to next in parallel, the dts is generating additional warnings as well (DSS etc). I think it might be easier if we wait for DT maintainer ack on bindings prior to giving further cosmetic comments (To allow for any additional changes to come in to -next). -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform 2020-09-07 14:14 ` Nishanth Menon @ 2020-09-07 14:23 ` Lokesh Vutla 2020-09-07 23:48 ` Nishanth Menon 0 siblings, 1 reply; 23+ messages in thread From: Lokesh Vutla @ 2020-09-07 14:23 UTC (permalink / raw) To: Nishanth Menon Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List On 07/09/20 7:44 pm, Nishanth Menon wrote: > On 17:32-20200907, Lokesh Vutla wrote: >> Hi, >> >> On 27/08/20 12:21 pm, Lokesh Vutla wrote: >>> This series adds initial support for latest new SoC, J7200, from Texas Instruments. >>> >>> The J7200 SoC is a part of the K3 Multicore SoC architecture platform. >>> It is targeted for for automotive gateway, vehicle compute systems, >>> Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. >>> The SoC aims to meet the complex processing needs of modern embedded products. >>> >>> See J7200 Technical Reference Manual (SPRUIU1, June 2020) >>> for further details: https://www.ti.com/lit/pdf/spruiu1 >>> >>> Changes since v1: >>> - Swapped Patch 1 and 2 as suggested by Nishanth. >>> - Added description for each SoC in yaml bindings. >>> >>> Testing: >>> - ./scripts/checkpatch --strict >>> - Few warningns about Line length exceeding 100 columns. >>> But these are corresponding to comments >>> - v8make dtbs_check >>> - DT_SCHEMA_FLAGS="-u" >>> DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" >>> v8make dtbs_check >>> - DT_SCHEMA_FLAGS="-u" >>> DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" >>> v8make dt_binding_check >> >> This series has been lying around for soo long with no major comments. It will >> be nice to get this merged. I understand we are waiting for Acks on yaml >> documentation but it would be bad to miss a merge window for a new platform >> because we are waiting for Acks on yaml conversion. > > I do require Rob / DT maintainer to ack the DT yaml conversion and the > j7200 binding addition. > > Besides yaml and compatibility acks, there are a few ancillary > comments to fix up.. Kconfig -> I think we should either stay with > status quo and create a new config option per SoC OR rename the > config to be generic (using j7200 with j721e SoC config is not very Please suggest your preference here. I guess separate defconfig for J7200? > consistent). In addition, around the stuff that is going to next in > parallel, the dts is generating additional warnings as well (DSS hmm..there is no DSS being added in this series. There is one checkpatch warning for PATCH 1: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst This can be ignored. > etc). I think it might be easier if we wait for DT maintainer ack on > bindings prior to giving further cosmetic comments (To allow for any > additional changes to come in to -next). > okay. Thanks and regards, Lokesh _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform 2020-09-07 14:23 ` Lokesh Vutla @ 2020-09-07 23:48 ` Nishanth Menon 2020-09-08 9:55 ` Tero Kristo 0 siblings, 1 reply; 23+ messages in thread From: Nishanth Menon @ 2020-09-07 23:48 UTC (permalink / raw) To: Lokesh Vutla Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Tero Kristo, Rob Herring, Linux ARM Mailing List On 19:53-20200907, Lokesh Vutla wrote: [... I should have responded to the correct patch..] > > Besides yaml and compatibility acks, there are a few ancillary > > comments to fix up.. Kconfig -> I think we should either stay with > > status quo and create a new config option per SoC OR rename the > > config to be generic (using j7200 with j721e SoC config is not very > > Please suggest your preference here. I guess separate defconfig for J7200? I was just scanning through remaining arm64 additions to see what others have done. We seem to have two options here: a) Just use ARCH_K3 and no specific SoC configs b) Specific SoC configs In both cases, use += instead of \ to incrementally add dtbs We have been going with (b) so far, Tero: any specific preference here? (a) has the aspect of simplicity and reduced dependencies. (b) Allows downstream kernels to save just a little bit and focus purely on SoC of interest. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform 2020-09-07 23:48 ` Nishanth Menon @ 2020-09-08 9:55 ` Tero Kristo 2020-09-08 11:25 ` Nishanth Menon 0 siblings, 1 reply; 23+ messages in thread From: Tero Kristo @ 2020-09-08 9:55 UTC (permalink / raw) To: Nishanth Menon, Lokesh Vutla Cc: Device Tree Mailing List, Grygorii Strashko, Sekhar Nori, Rob Herring, Linux ARM Mailing List On 08/09/2020 02:48, Nishanth Menon wrote: > On 19:53-20200907, Lokesh Vutla wrote: > > [... I should have responded to the correct patch..] >>> Besides yaml and compatibility acks, there are a few ancillary >>> comments to fix up.. Kconfig -> I think we should either stay with >>> status quo and create a new config option per SoC OR rename the >>> config to be generic (using j7200 with j721e SoC config is not very >> >> Please suggest your preference here. I guess separate defconfig for J7200? > > > I was just scanning through remaining arm64 additions to see what others have > done. We seem to have two options here: > a) Just use ARCH_K3 and no specific SoC configs > b) Specific SoC configs > In both cases, use += instead of \ to incrementally add dtbs > > We have been going with (b) so far, Tero: any specific preference here? > > (a) has the aspect of simplicity and reduced dependencies. > (b) Allows downstream kernels to save just a little bit and focus purely > on SoC of interest. If possible, I think we should aim for a) at least for now. We have the soc type detection code in place anyways that can be used on driver level. Creating compile time flags should be avoided imo as much as possible and just go with runtime detection. I can't see why saving maybe a megabyte of memory with SoC specific kernels would be of any importance on K3 arch with the memory amounts we have in our disposal. -Tero -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki _______________________________________________ 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] 23+ messages in thread
* Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform 2020-09-08 9:55 ` Tero Kristo @ 2020-09-08 11:25 ` Nishanth Menon 0 siblings, 0 replies; 23+ messages in thread From: Nishanth Menon @ 2020-09-08 11:25 UTC (permalink / raw) To: Tero Kristo Cc: Device Tree Mailing List, Grygorii Strashko, Lokesh Vutla, Sekhar Nori, Rob Herring, Linux ARM Mailing List On 12:55-20200908, Tero Kristo wrote: > On 08/09/2020 02:48, Nishanth Menon wrote: > > On 19:53-20200907, Lokesh Vutla wrote: > > > > [... I should have responded to the correct patch..] > > > > Besides yaml and compatibility acks, there are a few ancillary > > > > comments to fix up.. Kconfig -> I think we should either stay with > > > > status quo and create a new config option per SoC OR rename the > > > > config to be generic (using j7200 with j721e SoC config is not very > > > > > > Please suggest your preference here. I guess separate defconfig for J7200? > > > > > > I was just scanning through remaining arm64 additions to see what others have > > done. We seem to have two options here: > > a) Just use ARCH_K3 and no specific SoC configs > > b) Specific SoC configs > > In both cases, use += instead of \ to incrementally add dtbs > > > > We have been going with (b) so far, Tero: any specific preference here? > > > > (a) has the aspect of simplicity and reduced dependencies. > > (b) Allows downstream kernels to save just a little bit and focus purely > > on SoC of interest. > > If possible, I think we should aim for a) at least for now. We have the soc > type detection code in place anyways that can be used on driver level. > Creating compile time flags should be avoided imo as much as possible and > just go with runtime detection. I can't see why saving maybe a megabyte of > memory with SoC specific kernels would be of any importance on K3 arch with > the memory amounts we have in our disposal. Agreed on (a). I see one other user (SND) beyond dtb Makefile, So, to order this right, lets first switch the users over from SOC config builds to ARCH_K3, before we drop the Kconfig definition/defconfig update in a follow on rc/version. -- Regards, Nishanth Menon Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D _______________________________________________ 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] 23+ messages in thread
end of thread, other threads:[~2020-09-08 11:58 UTC | newest] Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-08-27 6:51 [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema Lokesh Vutla 2020-08-28 0:41 ` Nishanth Menon 2020-08-28 3:14 ` Lokesh Vutla 2020-08-28 3:47 ` Suman Anna 2020-08-28 13:07 ` Nishanth Menon 2020-09-04 11:55 ` Nishanth Menon 2020-09-04 7:15 ` Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 2/4] dt-bindings: arm: ti: Add bindings for J7200 SoC Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 3/4] arm64: dts: ti: Add support " Lokesh Vutla 2020-08-27 17:04 ` Suman Anna 2020-09-08 11:47 ` Nishanth Menon 2020-08-31 9:13 ` Peter Ujfalusi 2020-09-02 3:51 ` Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 4/4] arm64: dts: ti: Add support for J7200 Common Processor Board Lokesh Vutla 2020-09-08 11:57 ` Nishanth Menon 2020-08-27 9:09 ` [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Grygorii Strashko 2020-09-07 12:02 ` Lokesh Vutla 2020-09-07 14:14 ` Nishanth Menon 2020-09-07 14:23 ` Lokesh Vutla 2020-09-07 23:48 ` Nishanth Menon 2020-09-08 9:55 ` Tero Kristo 2020-09-08 11:25 ` Nishanth Menon
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).