* [PATCH 0/3] AM68 SK: Add initial support
@ 2022-10-18 12:38 Sinthu Raja
2022-10-18 12:38 ` [PATCH 1/3] dt-bindings: arm: ti: Add binding for AM68 SK Sinthu Raja
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: Sinthu Raja @ 2022-10-18 12:38 UTC (permalink / raw)
To: Nishanth Menon, Tero Kristo, Rob Herring
Cc: Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel,
devicetree, linux-kernel, Sinthu Raja
From: Sinthu Raja <sinthu.raja@ti.com>
AM68 Starter Kit (SK) is a low cost, small form factor board designed
for TI’s J721S2/AM68 SoC. TI’s AM68 SoC comprises of dual core A72, high
performance vision accelerators, hardware accelerators, latest C71x
DSP, high bandwidth real-time IPs for capture and display. The SoC is
power optimized to provide best in class performance for industrial
and automotive applications.
Refer below link to J721S2/AM68 Technical Reference Manual for further details:
http://www.ti.com/lit/pdf/spruj28
AM68 SK supports the following interfaces:
* 16 GB LPDDR4 RAM
* x1 Gigabit Ethernet interface
* x1 USB 3.1 Type-C port
* x2 USB 3.1 Type-A ports
* x1 PCIe M.2 M Key
* 512 Mbit OSPI flash
* x2 CSI2 Camera interface (RPi and TI Camera connector)
* 40-pin Raspberry Pi GPIO header
This series of patch add initial support for AM68 starter kit.
Design files can be referrred from https://www.ti.com/lit/zip/SPRR463
Sinthu Raja (3):
dt-bindings: arm: ti: Add binding for AM68 SK
arm64: dts: ti: Add initial support for AM68 SK System on Module
arch: arm64: ti: Add support for AM68 SK base board
.../devicetree/bindings/arm/ti/k3.yaml | 1 +
arch/arm64/boot/dts/ti/Makefile | 2 +
.../boot/dts/ti/k3-am68-sk-base-board.dts | 459 ++++++++++++++++++
arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi | 129 +++++
4 files changed, 591 insertions(+)
create mode 100644 arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts
create mode 100644 arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi
--
2.36.1
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH 1/3] dt-bindings: arm: ti: Add binding for AM68 SK 2022-10-18 12:38 [PATCH 0/3] AM68 SK: Add initial support Sinthu Raja @ 2022-10-18 12:38 ` Sinthu Raja 2022-10-18 12:58 ` Krzysztof Kozlowski 2022-10-18 12:38 ` [PATCH 2/3] arm64: dts: ti: Add initial support for AM68 SK System on Module Sinthu Raja 2022-10-18 12:38 ` [PATCH 3/3] arch: arm64: ti: Add support for AM68 SK base board Sinthu Raja 2 siblings, 1 reply; 9+ messages in thread From: Sinthu Raja @ 2022-10-18 12:38 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel, devicetree, linux-kernel, Sinthu Raja From: Sinthu Raja <sinthu.raja@ti.com> AM68 Starter Kit is a low cost, small form factor board designed for TI's AM68 SoC which is optimized to provide best in class performance for industrial applications and add binding for the same. Signed-off-by: Sinthu Raja <sinthu.raja@ti.com> --- Documentation/devicetree/bindings/arm/ti/k3.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml index 28b8232e1c5b..54f983fb23cf 100644 --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml @@ -69,6 +69,7 @@ properties: items: - enum: - ti,j721s2-evm + - ti,am68-sk - const: ti,j721s2 additionalProperties: true -- 2.36.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/3] dt-bindings: arm: ti: Add binding for AM68 SK 2022-10-18 12:38 ` [PATCH 1/3] dt-bindings: arm: ti: Add binding for AM68 SK Sinthu Raja @ 2022-10-18 12:58 ` Krzysztof Kozlowski 0 siblings, 0 replies; 9+ messages in thread From: Krzysztof Kozlowski @ 2022-10-18 12:58 UTC (permalink / raw) To: Sinthu Raja, Nishanth Menon, Tero Kristo, Rob Herring Cc: Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel, devicetree, linux-kernel, Sinthu Raja On 18/10/2022 08:38, Sinthu Raja wrote: > From: Sinthu Raja <sinthu.raja@ti.com> > > AM68 Starter Kit is a low cost, small form factor board designed for > TI's AM68 SoC which is optimized to provide best in class performance > for industrial applications and add binding for the same. > > Signed-off-by: Sinthu Raja <sinthu.raja@ti.com> > --- > Documentation/devicetree/bindings/arm/ti/k3.yaml | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml > index 28b8232e1c5b..54f983fb23cf 100644 > --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml > +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml > @@ -69,6 +69,7 @@ properties: > items: > - enum: > - ti,j721s2-evm > + - ti,am68-sk Alphabetical order please. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 2/3] arm64: dts: ti: Add initial support for AM68 SK System on Module 2022-10-18 12:38 [PATCH 0/3] AM68 SK: Add initial support Sinthu Raja 2022-10-18 12:38 ` [PATCH 1/3] dt-bindings: arm: ti: Add binding for AM68 SK Sinthu Raja @ 2022-10-18 12:38 ` Sinthu Raja 2022-10-18 12:58 ` Krzysztof Kozlowski 2022-10-18 12:38 ` [PATCH 3/3] arch: arm64: ti: Add support for AM68 SK base board Sinthu Raja 2 siblings, 1 reply; 9+ messages in thread From: Sinthu Raja @ 2022-10-18 12:38 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel, devicetree, linux-kernel, Sinthu Raja From: Sinthu Raja <sinthu.raja@ti.com> AM68 Starter Kit (SK)[1] is a low cost, small form factor board designed for TI’s AM68 SoC. TI’s AM68 SoC comprises of dual core A72, high performance vision accelerators, hardware accelerators, latest C71x DSP, high bandwidth real-time IPs for capture and display. The SoC is power optimized to provide best in class performance for industrial applications. AM68 SK supports the following interfaces: * 16 GB LPDDR4 RAM * x1 Gigabit Ethernet interface * x1 USB 3.1 Type-C port * x2 USB 3.1 Type-A ports * x1 PCIe M.2 M Key * 512 Mbit OSPI flash * x2 CSI2 Camera interface (RPi and TI Camera connector) * 40-pin Raspberry Pi GPIO header SK's System on Module (SoM) contains the SoC and DDR. Therefore, add DT node for the SOC and DDR on the SoM. Schematics: https://www.ti.com/lit/zip/SPRR463 TRM: http://www.ti.com/lit/pdf/spruj28 Signed-off-by: Sinthu Raja <sinthu.raja@ti.com> --- arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi | 129 +++++++++++++++++++++ 1 file changed, 129 insertions(+) create mode 100644 arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi b/arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi new file mode 100644 index 000000000000..734c13eb0490 --- /dev/null +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi @@ -0,0 +1,129 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2022 Texas Instruments Incorporated - https://www.ti.com/ + */ + +/dts-v1/; + +#include "k3-j721s2.dtsi" +#include <dt-bindings/gpio/gpio.h> + +/ { + memory@80000000 { + device_type = "memory"; + /* 16 GB RAM */ + reg = <0x00 0x80000000 0x00 0x80000000>, + <0x08 0x80000000 0x03 0x80000000>; + }; + + /* Reserving memory regions still pending */ + reserved_memory: reserved-memory { + #address-cells = <2>; + #size-cells = <2>; + ranges; + + secure_ddr: optee@9e800000 { + reg = <0x00 0x9e800000 0x00 0x01800000>; + alignment = <0x1000>; + no-map; + }; + + }; + +}; + +&mailbox0_cluster0 { + status = "disabled"; +}; + +&mailbox0_cluster1 { + status = "disabled"; +}; + +&mailbox0_cluster2 { + status = "disabled"; +}; + +&mailbox0_cluster3 { + status = "disabled"; +}; + +&mailbox0_cluster4 { + status = "disabled"; +}; + +&mailbox0_cluster5 { + status = "disabled"; +}; + +&mailbox0_cluster6 { + status = "disabled"; +}; + +&mailbox0_cluster7 { + status = "disabled"; +}; + +&mailbox0_cluster8 { + status = "disabled"; +}; + +&mailbox0_cluster9 { + status = "disabled"; +}; + +&mailbox0_cluster10 { + status = "disabled"; +}; + +&mailbox0_cluster11 { + status = "disabled"; +}; + +&mailbox1_cluster0 { + status = "disabled"; +}; + +&mailbox1_cluster1 { + status = "disabled"; +}; + +&mailbox1_cluster2 { + status = "disabled"; +}; + +&mailbox1_cluster3 { + status = "disabled"; +}; + +&mailbox1_cluster4 { + status = "disabled"; +}; + +&mailbox1_cluster5 { + status = "disabled"; +}; + +&mailbox1_cluster6 { + status = "disabled"; +}; + +&mailbox1_cluster7 { + status = "disabled"; +}; + +&mailbox1_cluster8 { + status = "disabled"; +}; + +&mailbox1_cluster9 { + status = "disabled"; +}; + +&mailbox1_cluster10 { + status = "disabled"; +}; + +&mailbox1_cluster11 { + status = "disabled"; +}; -- 2.36.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 2/3] arm64: dts: ti: Add initial support for AM68 SK System on Module 2022-10-18 12:38 ` [PATCH 2/3] arm64: dts: ti: Add initial support for AM68 SK System on Module Sinthu Raja @ 2022-10-18 12:58 ` Krzysztof Kozlowski 0 siblings, 0 replies; 9+ messages in thread From: Krzysztof Kozlowski @ 2022-10-18 12:58 UTC (permalink / raw) To: Sinthu Raja, Nishanth Menon, Tero Kristo, Rob Herring Cc: Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel, devicetree, linux-kernel, Sinthu Raja On 18/10/2022 08:38, Sinthu Raja wrote: > From: Sinthu Raja <sinthu.raja@ti.com> > > AM68 Starter Kit (SK)[1] is a low cost, small form factor board designed > for TI’s AM68 SoC. TI’s AM68 SoC comprises of dual core A72, high > performance vision accelerators, hardware accelerators, latest C71x > DSP, high bandwidth real-time IPs for capture and display. The SoC is > power optimized to provide best in class performance for industrial > applications. > > AM68 SK supports the following interfaces: > * 16 GB LPDDR4 RAM > * x1 Gigabit Ethernet interface > * x1 USB 3.1 Type-C port > * x2 USB 3.1 Type-A ports > * x1 PCIe M.2 M Key > * 512 Mbit OSPI flash > * x2 CSI2 Camera interface (RPi and TI Camera connector) > * 40-pin Raspberry Pi GPIO header > > SK's System on Module (SoM) contains the SoC and DDR. > Therefore, add DT node for the SOC and DDR on the SoM. > > Schematics: https://www.ti.com/lit/zip/SPRR463 > TRM: http://www.ti.com/lit/pdf/spruj28 > > Signed-off-by: Sinthu Raja <sinthu.raja@ti.com> > --- > arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi | 129 +++++++++++++++++++++ > 1 file changed, 129 insertions(+) > create mode 100644 arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi > > diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi b/arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi > new file mode 100644 > index 000000000000..734c13eb0490 > --- /dev/null > +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-som.dtsi > @@ -0,0 +1,129 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (C) 2022 Texas Instruments Incorporated - https://www.ti.com/ > + */ > + > +/dts-v1/; > + > +#include "k3-j721s2.dtsi" > +#include <dt-bindings/gpio/gpio.h> > + > +/ { > + memory@80000000 { > + device_type = "memory"; > + /* 16 GB RAM */ > + reg = <0x00 0x80000000 0x00 0x80000000>, > + <0x08 0x80000000 0x03 0x80000000>; > + }; > + > + /* Reserving memory regions still pending */ > + reserved_memory: reserved-memory { > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + > + secure_ddr: optee@9e800000 { > + reg = <0x00 0x9e800000 0x00 0x01800000>; > + alignment = <0x1000>; > + no-map; > + }; > + Drop empty line. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/3] arch: arm64: ti: Add support for AM68 SK base board 2022-10-18 12:38 [PATCH 0/3] AM68 SK: Add initial support Sinthu Raja 2022-10-18 12:38 ` [PATCH 1/3] dt-bindings: arm: ti: Add binding for AM68 SK Sinthu Raja 2022-10-18 12:38 ` [PATCH 2/3] arm64: dts: ti: Add initial support for AM68 SK System on Module Sinthu Raja @ 2022-10-18 12:38 ` Sinthu Raja 2022-10-18 13:02 ` Krzysztof Kozlowski 2 siblings, 1 reply; 9+ messages in thread From: Sinthu Raja @ 2022-10-18 12:38 UTC (permalink / raw) To: Nishanth Menon, Tero Kristo, Rob Herring Cc: Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel, devicetree, linux-kernel, Sinthu Raja From: Sinthu Raja <sinthu.raja@ti.com> The SK architecture comprises of baseboard and a SOM board. It is as follows, +--------------------------------------+---+ | | |<--Connectors | +---| | +---------------------------+ | | | | SOM | +---| | | | | | | | | +---| Power Supply | | | | | | +---------------------------+ | | | |<--+ | BASE BOARD | +------------------------------------------+ AM68 Starter Kit's baseboard contains most of the actual connectors, power supply etc. The System on Module (SoM) is plugged on to the base board. Therefore, add support for peripherals brought out in the base board. Schematics: https://www.ti.com/lit/zip/SPRR463 Signed-off-by: Sinthu Raja <sinthu.raja@ti.com> --- arch/arm64/boot/dts/ti/Makefile | 2 + .../boot/dts/ti/k3-am68-sk-base-board.dts | 459 ++++++++++++++++++ 2 files changed, 461 insertions(+) create mode 100644 arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile index 4555a5be2257..498b089d96f8 100644 --- a/arch/arm64/boot/dts/ti/Makefile +++ b/arch/arm64/boot/dts/ti/Makefile @@ -19,6 +19,8 @@ dtb-$(CONFIG_ARCH_K3) += k3-j7200-common-proc-board.dtb dtb-$(CONFIG_ARCH_K3) += k3-j721s2-common-proc-board.dtb +dtb-$(CONFIG_ARCH_K3) += k3-am68-sk-base-board.dtb + dtb-$(CONFIG_ARCH_K3) += k3-am642-evm.dtb dtb-$(CONFIG_ARCH_K3) += k3-am642-sk.dtb diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts new file mode 100644 index 000000000000..f51cbd2e3b72 --- /dev/null +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts @@ -0,0 +1,459 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (C) 2022 Texas Instruments Incorporated - https://www.ti.com/ + * + * Base Board: https://www.ti.com/lit/zip/SPRR463 + */ + +/dts-v1/; + +#include "k3-am68-sk-som.dtsi" +#include <dt-bindings/net/ti-dp83867.h> +#include <dt-bindings/phy/phy-cadence.h> +#include <dt-bindings/phy/phy.h> +#include <dt-bindings/mux/ti-serdes.h> + +/ { + compatible = "ti,am68-sk", "ti,j721s2"; + model = "Texas Instruments AM68 SK"; + + chosen { + stdout-path = "serial2:115200n8"; + bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x2880000"; + }; + + aliases { + serial2 = &main_uart8; + mmc1 = &main_sdhci1; + can0 = &mcu_mcan0; + can1 = &mcu_mcan1; + can2 = &main_mcan6; + can3 = &main_mcan7; + }; + + vusb_main: fixedregulator-vusb-main5v0 { + /* USB MAIN INPUT 5V DC */ + compatible = "regulator-fixed"; + regulator-name = "vusb-main5v0"; + regulator-min-microvolt = <5000000>; + regulator-max-microvolt = <5000000>; + regulator-always-on; + regulator-boot-on; + }; + + vsys_3v3: fixedregulator-vsys3v3 { + /* Output of LM5141 */ + compatible = "regulator-fixed"; + regulator-name = "vsys_3v3"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + vin-supply = <&vusb_main>; + regulator-always-on; + regulator-boot-on; + }; + + vdd_mmc1: fixedregulator-sd { + /* Output of TPS22918 */ + compatible = "regulator-fixed"; + regulator-name = "vdd_mmc1"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-boot-on; + enable-active-high; + vin-supply = <&vsys_3v3>; + gpio = <&exp1 10 GPIO_ACTIVE_HIGH>; + }; + + vdd_sd_dv: gpio-regulator-TLV71033 { + /* Output of TLV71033 */ + compatible = "regulator-gpio"; + regulator-name = "tlv71033"; + pinctrl-names = "default"; + pinctrl-0 = <&vdd_sd_dv_pins_default>; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <3300000>; + regulator-boot-on; + vin-supply = <&vsys_3v3>; + gpios = <&main_gpio0 49 GPIO_ACTIVE_HIGH>; + states = <1800000 0x0>, + <3300000 0x1>; + }; + + vsys_io_1v8: regulator-vsys-io-1v8 { + compatible = "regulator-fixed"; + regulator-name = "vsys_io_1v8"; + regulator-min-microvolt = <1800000>; + regulator-max-microvolt = <1800000>; + regulator-always-on; + regulator-boot-on; + }; + + vsys_io_1v2: regulator-vsys-io-1v2 { + compatible = "regulator-fixed"; + regulator-name = "vsys_io_1v2"; + regulator-min-microvolt = <1200000>; + regulator-max-microvolt = <1200000>; + regulator-always-on; + regulator-boot-on; + }; + + transceiver1: can-phy0 { + compatible = "ti,tcan1042"; + #phy-cells = <0>; + max-bitrate = <5000000>; + pinctrl-names = "default"; + pinctrl-0 = <&mcu_mcan0_pins_default>; + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; + }; + + transceiver2: can-phy1 { + compatible = "ti,tcan1042"; + #phy-cells = <0>; + max-bitrate = <5000000>; + pinctrl-names = "default"; + pinctrl-0 = <&mcu_mcan1_pins_default>; + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; + }; + + transceiver3: can-phy2 { + compatible = "ti,tcan1042"; + #phy-cells = <0>; + max-bitrate = <5000000>; + pinctrl-names = "default"; + pinctrl-0 = <&main_mcan6_pins_default>; + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; + }; + + transceiver4: can-phy3 { + compatible = "ti,tcan1042"; + #phy-cells = <0>; + max-bitrate = <5000000>; + pinctrl-names = "default"; + pinctrl-0 = <&main_mcan7_pins_default>; + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; + }; +}; + +&main_pmx0 { + + main_uart8_pins_default: main-uart8-pins-default { + pinctrl-single,pins = < + J721S2_IOPAD(0x0d0, PIN_INPUT, 11) /* (AF26) SPI0_CS1.UART8_RXD */ + J721S2_IOPAD(0x0d4, PIN_OUTPUT, 11) /* (AH27) SPI0_CLK.UART8_TXD */ + >; + }; + + main_i2c0_pins_default: i2c0_pins_default { + pinctrl-single,pins = < + J721S2_IOPAD(0x0e0, PIN_INPUT, 0) /* (AH25) I2C0_SCL */ + J721S2_IOPAD(0x0e4, PIN_INPUT, 0) /* (AE24) I2C0_SDA */ + >; + }; + + main_mmc1_pins_default: main-mmc1-pins-default { + pinctrl-single,pins = < + J721S2_IOPAD(0x104, PIN_INPUT, 0) /* (P23) MMC1_CLK */ + J721S2_IOPAD(0x108, PIN_INPUT, 0) /* (N24) MMC1_CMD */ + J721S2_IOPAD(0x0fc, PIN_INPUT, 0) /* (M23) MMC1_DAT0 */ + J721S2_IOPAD(0x0f8, PIN_INPUT, 0) /* (P24) MMC1_DAT1 */ + J721S2_IOPAD(0x0f4, PIN_INPUT, 0) /* (R24) MMC1_DAT2 */ + J721S2_IOPAD(0x0f0, PIN_INPUT, 0) /* (R22) MMC1_DAT3 */ + J721S2_IOPAD(0x0e8, PIN_INPUT, 8) /* (AE25) TIMER_IO0.MMC1_SDCD */ + >; + }; + + vdd_sd_dv_pins_default: vdd-sd-dv-pins-default { + pinctrl-single,pins = < + J721S2_IOPAD(0x0c4, PIN_INPUT, 7) /* (AB26) ECAP0_IN_APWM_OUT.GPIO0_49 */ + >; + }; + + main_usbss0_pins_default: main-usbss0-pins-default { + pinctrl-single,pins = < + J721S2_IOPAD(0x0ec, PIN_OUTPUT, 6) /* (AG25) TIMER_IO1.USB0_DRVVBUS */ + >; + }; + + main_mcan6_pins_default: mcan6_pins_default { + pinctrl-single,pins = < + J721S2_IOPAD(0x098, PIN_INPUT, 0) /* (V25) MCASP0_AXR10.MCAN6_RX */ + J721S2_IOPAD(0x094, PIN_INPUT, 0) /* (AA25) MCASP0_AXR9.MCAN6_TX */ + >; + }; + + main_mcan7_pins_default: mcan7_pins_default { + pinctrl-single,pins = < + J721S2_IOPAD(0x0a0, PIN_INPUT, 0) /* (AB25) MCASP0_AXR12.MCAN7_RX */ + J721S2_IOPAD(0x09c, PIN_INPUT, 0) /* (T24) MCASP0_AXR11.MCAN7_TX */ + >; + }; + +}; + +&wkup_pmx0 { + mcu_cpsw_pins_default: mcu-cpsw-pins-default { + pinctrl-single,pins = < + J721S2_WKUP_IOPAD(0x094, PIN_INPUT, 0) /* (B22) MCU_RGMII1_RD0 */ + J721S2_WKUP_IOPAD(0x090, PIN_INPUT, 0) /* (B21) MCU_RGMII1_RD1 */ + J721S2_WKUP_IOPAD(0x08c, PIN_INPUT, 0) /* (C22) MCU_RGMII1_RD2 */ + J721S2_WKUP_IOPAD(0x088, PIN_INPUT, 0) /* (D23) MCU_RGMII1_RD3 */ + J721S2_WKUP_IOPAD(0x084, PIN_INPUT, 0) /* (D22) MCU_RGMII1_RXC */ + J721S2_WKUP_IOPAD(0x06c, PIN_INPUT, 0) /* (E23) MCU_RGMII1_RX_CTL */ + J721S2_WKUP_IOPAD(0x07c, PIN_OUTPUT, 0) /* (F23) MCU_RGMII1_TD0 */ + J721S2_WKUP_IOPAD(0x078, PIN_OUTPUT, 0) /* (G22) MCU_RGMII1_TD1 */ + J721S2_WKUP_IOPAD(0x074, PIN_OUTPUT, 0) /* (E21) MCU_RGMII1_TD2 */ + J721S2_WKUP_IOPAD(0x070, PIN_OUTPUT, 0) /* (E22) MCU_RGMII1_TD3 */ + J721S2_WKUP_IOPAD(0x080, PIN_OUTPUT, 0) /* (F21) MCU_RGMII1_TXC */ + J721S2_WKUP_IOPAD(0x068, PIN_OUTPUT, 0) /* (F22) MCU_RGMII1_TX_CTL */ + >; + }; + + mcu_mdio_pins_default: mcu-mdio-pins-default { + pinctrl-single,pins = < + J721S2_WKUP_IOPAD(0x09c, PIN_OUTPUT, 0) /* (A21) MCU_MDIO0_MDC */ + J721S2_WKUP_IOPAD(0x098, PIN_INPUT, 0) /* (A22) MCU_MDIO0_MDIO */ + >; + }; + + mcu_mcan0_pins_default: mcu-mcan0-pins-default { + pinctrl-single,pins = < + J721S2_WKUP_IOPAD(0x0bc, PIN_INPUT, 0) /* (E28) MCU_MCAN0_RX */ + J721S2_WKUP_IOPAD(0x0b8, PIN_OUTPUT, 0) /* (E27) MCU_MCAN0_TX */ + >; + }; + + mcu_mcan1_pins_default: mcu-mcan1-pins-default { + pinctrl-single,pins = < + J721S2_WKUP_IOPAD(0x0d4, PIN_INPUT, 0) /* (F26) WKUP_GPIO0_5.MCU_MCAN1_RX */ + J721S2_WKUP_IOPAD(0x0d0, PIN_OUTPUT, 0) /* (C23) WKUP_GPIO0_4.MCU_MCAN1_TX*/ + >; + }; + + mcu_i2c1_pins_default: mcu_i2c1_pins_default { + pinctrl-single,pins = < + J721S2_WKUP_IOPAD(0x0e0, PIN_INPUT, 0) /* (F24) WKUP_GPIO0_8.MCU_I2C1_SCL */ + J721S2_WKUP_IOPAD(0x0e4, PIN_INPUT, 0) /* (H26) WKUP_GPIO0_9.MCU_I2C1_SDA */ + >; + }; + +}; + +&main_gpio2 { + status = "disabled"; +}; + +&main_gpio4 { + status = "disabled"; +}; + +&main_gpio6 { + status = "disabled"; +}; + +&wkup_gpio1 { + status = "disabled"; +}; + +&wkup_uart0 { + status = "reserved"; +}; + +&main_uart0 { + status = "disabled"; +}; + +&main_uart1 { + status = "disabled"; +}; + +&main_uart2 { + status = "disabled"; +}; + +&main_uart3 { + status = "disabled"; +}; + +&main_uart4 { + status = "disabled"; +}; + +&main_uart5 { + status = "disabled"; +}; + +&main_uart6 { + status = "disabled"; +}; + +&main_uart7 { + status = "disabled"; +}; + +&main_uart8 { + pinctrl-names = "default"; + pinctrl-0 = <&main_uart8_pins_default>; + /* Shared with TFA on this platform */ + power-domains = <&k3_pds 357 TI_SCI_PD_SHARED>; +}; + +&main_uart9 { + status = "disabled"; +}; + +&main_i2c0 { + clock-frequency = <400000>; + + exp1: gpio@21 { + compatible = "ti,tca6424"; + reg = <0x21>; + gpio-controller; + #gpio-cells = <2>; + gpio-line-names = "CSI_VIO_SEL", "CSI_SEL_FPC_EXPn", "HDMI_PDn", + "HDMI_LS_OE", "DP0_3V3 _EN", "BOARDID_EEPROM_WP", + "CAN_STB", " ", "GPIO_uSD_PWR_EN", "eDP_ENABLE", + "IO_EXP_PCIe1_M.2_RTSz", "IO_EXP_MCU_RGMII_RST#", + "IO_EXP_CSI2_EXP_RSTz", " ", "CSI0_B_GPIO1", + "CSI1_B_GPIO1"; + }; +}; + +&main_i2c1 { + status = "disabled"; +}; + +&main_i2c2 { + status = "disabled"; +}; + +&main_i2c3 { + status = "disabled"; +}; + +&main_i2c4 { + status = "disabled"; +}; + +&main_i2c5 { + status = "disabled"; +}; + +&main_i2c6 { + status = "disabled"; +}; + +&main_sdhci0 { + status = "disabled"; +}; + +&main_sdhci1 { + /* SD card */ + pinctrl-0 = <&main_mmc1_pins_default>; + pinctrl-names = "default"; + disable-wp; + vmmc-supply = <&vdd_mmc1>; + vqmmc-supply = <&vdd_sd_dv>; +}; + +&mcu_cpsw { + pinctrl-names = "default"; + pinctrl-0 = <&mcu_cpsw_pins_default &mcu_mdio_pins_default>; +}; + +&davinci_mdio { + phy0: ethernet-phy@0 { + reg = <0>; + ti,rx-internal-delay = <DP83867_RGMIIDCTL_2_00_NS>; + ti,fifo-depth = <DP83867_PHYCR_FIFO_DEPTH_4_B_NIB>; + ti,min-output-impedance; + }; +}; + +&cpsw_port1 { + phy-mode = "rgmii-rxid"; + phy-handle = <&phy0>; +}; + +&mcu_mcan0 { + pinctrl-names = "default"; + pinctrl-0 = <&mcu_mcan0_pins_default>; + phys = <&transceiver1>; +}; + +&mcu_mcan1 { + pinctrl-names = "default"; + pinctrl-0 = <&mcu_mcan1_pins_default>; + phys = <&transceiver2>; +}; + +&main_mcan0 { + status = "disabled"; +}; + +&main_mcan1 { + status = "disabled"; +}; + +&main_mcan2 { + status = "disabled"; +}; + +&main_mcan3 { + status = "disabled"; +}; + +&main_mcan4 { + status = "disabled"; +}; + +&main_mcan5 { + status = "disabled"; +}; + +&main_mcan6 { + pinctrl-names = "default"; + pinctrl-0 = <&main_mcan6_pins_default>; + phys = <&transceiver3>; +}; + +&main_mcan7 { + pinctrl-names = "default"; + pinctrl-0 = <&main_mcan7_pins_default>; + phys = <&transceiver4>; +}; + +&main_mcan8 { + status = "disabled"; +}; + +&main_mcan9 { + status = "disabled"; +}; + +&main_mcan10 { + status = "disabled"; +}; + +&main_mcan11 { + status = "disabled"; +}; + +&main_mcan12 { + status = "disabled"; +}; + +&main_mcan13 { + status = "disabled"; +}; + +&main_mcan14 { + status = "disabled"; +}; + +&main_mcan15 { + status = "disabled"; +}; + +&main_mcan17 { + status = "disabled"; +}; -- 2.36.1 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3] arch: arm64: ti: Add support for AM68 SK base board 2022-10-18 12:38 ` [PATCH 3/3] arch: arm64: ti: Add support for AM68 SK base board Sinthu Raja @ 2022-10-18 13:02 ` Krzysztof Kozlowski 2022-11-03 8:38 ` Sinthu Raja M 0 siblings, 1 reply; 9+ messages in thread From: Krzysztof Kozlowski @ 2022-10-18 13:02 UTC (permalink / raw) To: Sinthu Raja, Nishanth Menon, Tero Kristo, Rob Herring Cc: Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel, devicetree, linux-kernel, Sinthu Raja On 18/10/2022 08:38, Sinthu Raja wrote: > From: Sinthu Raja <sinthu.raja@ti.com> > > The SK architecture comprises of baseboard and a SOM board. It is > as follows, > > +--------------------------------------+---+ > | | |<--Connectors > | +---| > | +---------------------------+ | | > | | SOM | +---| > | | | | | > | | | +---| Power Supply > | | | | | > | +---------------------------+ | | > | |<--+ > | BASE BOARD | > +------------------------------------------+ This is pretty straightforward for SoM, I am not sure what's the benefit to add it. > > AM68 Starter Kit's baseboard contains most of the actual connectors, > power supply etc. The System on Module (SoM) is plugged on to the base > board. Therefore, add support for peripherals brought out in the base > board. > > Schematics: https://www.ti.com/lit/zip/SPRR463 > > Signed-off-by: Sinthu Raja <sinthu.raja@ti.com> > --- > arch/arm64/boot/dts/ti/Makefile | 2 + > .../boot/dts/ti/k3-am68-sk-base-board.dts | 459 ++++++++++++++++++ > 2 files changed, 461 insertions(+) > create mode 100644 arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > > diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile > index 4555a5be2257..498b089d96f8 100644 > --- a/arch/arm64/boot/dts/ti/Makefile > +++ b/arch/arm64/boot/dts/ti/Makefile > @@ -19,6 +19,8 @@ dtb-$(CONFIG_ARCH_K3) += k3-j7200-common-proc-board.dtb > > dtb-$(CONFIG_ARCH_K3) += k3-j721s2-common-proc-board.dtb > > +dtb-$(CONFIG_ARCH_K3) += k3-am68-sk-base-board.dtb > + Alphabetical order, although does not look like other entries are ordered... > dtb-$(CONFIG_ARCH_K3) += k3-am642-evm.dtb > dtb-$(CONFIG_ARCH_K3) += k3-am642-sk.dtb > > diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > new file mode 100644 > index 000000000000..f51cbd2e3b72 > --- /dev/null > +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > @@ -0,0 +1,459 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Copyright (C) 2022 Texas Instruments Incorporated - https://www.ti.com/ > + * > + * Base Board: https://www.ti.com/lit/zip/SPRR463 > + */ > + > +/dts-v1/; > + > +#include "k3-am68-sk-som.dtsi" > +#include <dt-bindings/net/ti-dp83867.h> > +#include <dt-bindings/phy/phy-cadence.h> > +#include <dt-bindings/phy/phy.h> > +#include <dt-bindings/mux/ti-serdes.h> > + > +/ { > + compatible = "ti,am68-sk", "ti,j721s2"; > + model = "Texas Instruments AM68 SK"; > + > + chosen { > + stdout-path = "serial2:115200n8"; > + bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x2880000"; None of these bootargs are suitable for mainline. earlycon for sure - it is pure debug option. > + }; > + > + aliases { > + serial2 = &main_uart8; > + mmc1 = &main_sdhci1; > + can0 = &mcu_mcan0; > + can1 = &mcu_mcan1; > + can2 = &main_mcan6; > + can3 = &main_mcan7; > + }; > + > + vusb_main: fixedregulator-vusb-main5v0 { Not "fixedregulator-" but just "regulator-" Same in other places. > + /* USB MAIN INPUT 5V DC */ > + compatible = "regulator-fixed"; > + regulator-name = "vusb-main5v0"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + regulator-always-on; > + regulator-boot-on; > + }; > + > + vsys_3v3: fixedregulator-vsys3v3 { > + /* Output of LM5141 */ > + compatible = "regulator-fixed"; > + regulator-name = "vsys_3v3"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + vin-supply = <&vusb_main>; > + regulator-always-on; > + regulator-boot-on; > + }; > + > + vdd_mmc1: fixedregulator-sd { > + /* Output of TPS22918 */ > + compatible = "regulator-fixed"; > + regulator-name = "vdd_mmc1"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-boot-on; > + enable-active-high; > + vin-supply = <&vsys_3v3>; > + gpio = <&exp1 10 GPIO_ACTIVE_HIGH>; > + }; > + > + vdd_sd_dv: gpio-regulator-TLV71033 { 1. regulator-xxxxxx 2. lowercase node name > + /* Output of TLV71033 */ > + compatible = "regulator-gpio"; > + regulator-name = "tlv71033"; > + pinctrl-names = "default"; > + pinctrl-0 = <&vdd_sd_dv_pins_default>; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <3300000>; > + regulator-boot-on; > + vin-supply = <&vsys_3v3>; > + gpios = <&main_gpio0 49 GPIO_ACTIVE_HIGH>;> + states = <1800000 0x0>, > + <3300000 0x1>; > + }; > + > + vsys_io_1v8: regulator-vsys-io-1v8 { > + compatible = "regulator-fixed"; > + regulator-name = "vsys_io_1v8"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + regulator-always-on; > + regulator-boot-on; > + }; > + > + vsys_io_1v2: regulator-vsys-io-1v2 { > + compatible = "regulator-fixed"; > + regulator-name = "vsys_io_1v2"; > + regulator-min-microvolt = <1200000>; > + regulator-max-microvolt = <1200000>; > + regulator-always-on; > + regulator-boot-on; > + }; > + > + transceiver1: can-phy0 { > + compatible = "ti,tcan1042"; > + #phy-cells = <0>; > + max-bitrate = <5000000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&mcu_mcan0_pins_default>; > + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; > + }; > + > + transceiver2: can-phy1 { > + compatible = "ti,tcan1042"; > + #phy-cells = <0>; > + max-bitrate = <5000000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&mcu_mcan1_pins_default>; > + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; > + }; > + > + transceiver3: can-phy2 { > + compatible = "ti,tcan1042"; > + #phy-cells = <0>; > + max-bitrate = <5000000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&main_mcan6_pins_default>; > + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; > + }; > + > + transceiver4: can-phy3 { > + compatible = "ti,tcan1042"; > + #phy-cells = <0>; > + max-bitrate = <5000000>; > + pinctrl-names = "default"; > + pinctrl-0 = <&main_mcan7_pins_default>; > + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; > + }; > +}; > + > +&main_pmx0 { > + No need for blank line. > + main_uart8_pins_default: main-uart8-pins-default { > + pinctrl-single,pins = < > + J721S2_IOPAD(0x0d0, PIN_INPUT, 11) /* (AF26) SPI0_CS1.UART8_RXD */ > + J721S2_IOPAD(0x0d4, PIN_OUTPUT, 11) /* (AH27) SPI0_CLK.UART8_TXD */ > + >; > + }; > + > + main_i2c0_pins_default: i2c0_pins_default { No underscores in node names. Why do you use here different style than in other places? > + pinctrl-single,pins = < > + J721S2_IOPAD(0x0e0, PIN_INPUT, 0) /* (AH25) I2C0_SCL */ > + J721S2_IOPAD(0x0e4, PIN_INPUT, 0) /* (AE24) I2C0_SDA */ > + >; > + }; > + > + main_mmc1_pins_default: main-mmc1-pins-default { > + pinctrl-single,pins = < > + J721S2_IOPAD(0x104, PIN_INPUT, 0) /* (P23) MMC1_CLK */ > + J721S2_IOPAD(0x108, PIN_INPUT, 0) /* (N24) MMC1_CMD */ > + J721S2_IOPAD(0x0fc, PIN_INPUT, 0) /* (M23) MMC1_DAT0 */ > + J721S2_IOPAD(0x0f8, PIN_INPUT, 0) /* (P24) MMC1_DAT1 */ > + J721S2_IOPAD(0x0f4, PIN_INPUT, 0) /* (R24) MMC1_DAT2 */ > + J721S2_IOPAD(0x0f0, PIN_INPUT, 0) /* (R22) MMC1_DAT3 */ > + J721S2_IOPAD(0x0e8, PIN_INPUT, 8) /* (AE25) TIMER_IO0.MMC1_SDCD */ > + >; > + }; > + > + vdd_sd_dv_pins_default: vdd-sd-dv-pins-default { > + pinctrl-single,pins = < > + J721S2_IOPAD(0x0c4, PIN_INPUT, 7) /* (AB26) ECAP0_IN_APWM_OUT.GPIO0_49 */ > + >; > + }; > + > + main_usbss0_pins_default: main-usbss0-pins-default { > + pinctrl-single,pins = < > + J721S2_IOPAD(0x0ec, PIN_OUTPUT, 6) /* (AG25) TIMER_IO1.USB0_DRVVBUS */ > + >; > + }; > + > + main_mcan6_pins_default: mcan6_pins_default { Be sure you tested the DTS against bindings. Please run `make dtbs_check` (see Documentation/devicetree/bindings/writing-schema.rst for instructions). > + pinctrl-single,pins = < > + J721S2_IOPAD(0x098, PIN_INPUT, 0) /* (V25) MCASP0_AXR10.MCAN6_RX */ > + J721S2_IOPAD(0x094, PIN_INPUT, 0) /* (AA25) MCASP0_AXR9.MCAN6_TX */ > + >; > + }; > + > + main_mcan7_pins_default: mcan7_pins_default { > + pinctrl-single,pins = < > + J721S2_IOPAD(0x0a0, PIN_INPUT, 0) /* (AB25) MCASP0_AXR12.MCAN7_RX */ > + J721S2_IOPAD(0x09c, PIN_INPUT, 0) /* (T24) MCASP0_AXR11.MCAN7_TX */ > + >; > + }; > + > +}; > + > +&wkup_pmx0 { > + mcu_cpsw_pins_default: mcu-cpsw-pins-default { > + pinctrl-single,pins = < > + J721S2_WKUP_IOPAD(0x094, PIN_INPUT, 0) /* (B22) MCU_RGMII1_RD0 */ > + J721S2_WKUP_IOPAD(0x090, PIN_INPUT, 0) /* (B21) MCU_RGMII1_RD1 */ > + J721S2_WKUP_IOPAD(0x08c, PIN_INPUT, 0) /* (C22) MCU_RGMII1_RD2 */ > + J721S2_WKUP_IOPAD(0x088, PIN_INPUT, 0) /* (D23) MCU_RGMII1_RD3 */ > + J721S2_WKUP_IOPAD(0x084, PIN_INPUT, 0) /* (D22) MCU_RGMII1_RXC */ > + J721S2_WKUP_IOPAD(0x06c, PIN_INPUT, 0) /* (E23) MCU_RGMII1_RX_CTL */ > + J721S2_WKUP_IOPAD(0x07c, PIN_OUTPUT, 0) /* (F23) MCU_RGMII1_TD0 */ > + J721S2_WKUP_IOPAD(0x078, PIN_OUTPUT, 0) /* (G22) MCU_RGMII1_TD1 */ > + J721S2_WKUP_IOPAD(0x074, PIN_OUTPUT, 0) /* (E21) MCU_RGMII1_TD2 */ > + J721S2_WKUP_IOPAD(0x070, PIN_OUTPUT, 0) /* (E22) MCU_RGMII1_TD3 */ > + J721S2_WKUP_IOPAD(0x080, PIN_OUTPUT, 0) /* (F21) MCU_RGMII1_TXC */ > + J721S2_WKUP_IOPAD(0x068, PIN_OUTPUT, 0) /* (F22) MCU_RGMII1_TX_CTL */ > + >; > + }; > + > + mcu_mdio_pins_default: mcu-mdio-pins-default { > + pinctrl-single,pins = < > + J721S2_WKUP_IOPAD(0x09c, PIN_OUTPUT, 0) /* (A21) MCU_MDIO0_MDC */ > + J721S2_WKUP_IOPAD(0x098, PIN_INPUT, 0) /* (A22) MCU_MDIO0_MDIO */ > + >; > + }; > + > + mcu_mcan0_pins_default: mcu-mcan0-pins-default { > + pinctrl-single,pins = < > + J721S2_WKUP_IOPAD(0x0bc, PIN_INPUT, 0) /* (E28) MCU_MCAN0_RX */ > + J721S2_WKUP_IOPAD(0x0b8, PIN_OUTPUT, 0) /* (E27) MCU_MCAN0_TX */ > + >; > + }; > + > + mcu_mcan1_pins_default: mcu-mcan1-pins-default { > + pinctrl-single,pins = < > + J721S2_WKUP_IOPAD(0x0d4, PIN_INPUT, 0) /* (F26) WKUP_GPIO0_5.MCU_MCAN1_RX */ > + J721S2_WKUP_IOPAD(0x0d0, PIN_OUTPUT, 0) /* (C23) WKUP_GPIO0_4.MCU_MCAN1_TX*/ > + >; > + }; > + > + mcu_i2c1_pins_default: mcu_i2c1_pins_default { > + pinctrl-single,pins = < > + J721S2_WKUP_IOPAD(0x0e0, PIN_INPUT, 0) /* (F24) WKUP_GPIO0_8.MCU_I2C1_SCL */ > + J721S2_WKUP_IOPAD(0x0e4, PIN_INPUT, 0) /* (H26) WKUP_GPIO0_9.MCU_I2C1_SDA */ > + >; > + }; > + Drop line > +}; > + > +&main_gpio2 { > + status = "disabled"; > +}; > + Best regards, Krzysztof ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3] arch: arm64: ti: Add support for AM68 SK base board 2022-10-18 13:02 ` Krzysztof Kozlowski @ 2022-11-03 8:38 ` Sinthu Raja M 2022-11-03 12:42 ` Krzysztof Kozlowski 0 siblings, 1 reply; 9+ messages in thread From: Sinthu Raja M @ 2022-11-03 8:38 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Nishanth Menon, Tero Kristo, Rob Herring, Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel, devicetree, linux-kernel, Sinthu Raja On Tue, Oct 18, 2022 at 6:33 PM Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > > On 18/10/2022 08:38, Sinthu Raja wrote: > > From: Sinthu Raja <sinthu.raja@ti.com> > > > > The SK architecture comprises of baseboard and a SOM board. It is > > as follows, > > > > +--------------------------------------+---+ > > | | |<--Connectors > > | +---| > > | +---------------------------+ | | > > | | SOM | +---| > > | | | | | > > | | | +---| Power Supply > > | | | | | > > | +---------------------------+ | | > > | |<--+ > > | BASE BOARD | > > +------------------------------------------+ > > This is pretty straightforward for SoM, I am not sure what's the benefit > to add it. > > > > > AM68 Starter Kit's baseboard contains most of the actual connectors, > > power supply etc. The System on Module (SoM) is plugged on to the base > > board. Therefore, add support for peripherals brought out in the base > > board. > > > > Schematics: https://www.ti.com/lit/zip/SPRR463 > > > > Signed-off-by: Sinthu Raja <sinthu.raja@ti.com> > > --- > > arch/arm64/boot/dts/ti/Makefile | 2 + > > .../boot/dts/ti/k3-am68-sk-base-board.dts | 459 ++++++++++++++++++ > > 2 files changed, 461 insertions(+) > > create mode 100644 arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > > > > diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile > > index 4555a5be2257..498b089d96f8 100644 > > --- a/arch/arm64/boot/dts/ti/Makefile > > +++ b/arch/arm64/boot/dts/ti/Makefile > > @@ -19,6 +19,8 @@ dtb-$(CONFIG_ARCH_K3) += k3-j7200-common-proc-board.dtb > > > > dtb-$(CONFIG_ARCH_K3) += k3-j721s2-common-proc-board.dtb > > > > +dtb-$(CONFIG_ARCH_K3) += k3-am68-sk-base-board.dtb > > + > > Alphabetical order, although does not look like other entries are ordered... > > > dtb-$(CONFIG_ARCH_K3) += k3-am642-evm.dtb > > dtb-$(CONFIG_ARCH_K3) += k3-am642-sk.dtb > > > > diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > > new file mode 100644 > > index 000000000000..f51cbd2e3b72 > > --- /dev/null > > +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts > > @@ -0,0 +1,459 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +/* > > + * Copyright (C) 2022 Texas Instruments Incorporated - https://www.ti.com/ > > + * > > + * Base Board: https://www.ti.com/lit/zip/SPRR463 > > + */ > > + > > +/dts-v1/; > > + > > +#include "k3-am68-sk-som.dtsi" > > +#include <dt-bindings/net/ti-dp83867.h> > > +#include <dt-bindings/phy/phy-cadence.h> > > +#include <dt-bindings/phy/phy.h> > > +#include <dt-bindings/mux/ti-serdes.h> > > + > > +/ { > > + compatible = "ti,am68-sk", "ti,j721s2"; > > + model = "Texas Instruments AM68 SK"; > > + > > + chosen { > > + stdout-path = "serial2:115200n8"; > > + bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x2880000"; > > None of these bootargs are suitable for mainline. earlycon for sure - it > is pure debug option. Krzysztof, In AM68 SK the Linux console is on main_uart8, so to be consistent with other J7 families of devices aliasing the serial port to ttyS2. I am quite confused why these bootargs are not suitable for mainline. I check the below documentation section 2.3, as mentioned it can be used as kernel parameters. Please correct me if I am wrong. https://www.kernel.org/doc/html/latest/devicetree/usage-model.html?highlight=bootargs > > > + }; > > + > > + aliases { > > + serial2 = &main_uart8; > > + mmc1 = &main_sdhci1; > > + can0 = &mcu_mcan0; > > + can1 = &mcu_mcan1; > > + can2 = &main_mcan6; > > + can3 = &main_mcan7; > > + }; > > + > > + vusb_main: fixedregulator-vusb-main5v0 { > > Not "fixedregulator-" but just "regulator-" > > Same in other places. > > > + /* USB MAIN INPUT 5V DC */ > > + compatible = "regulator-fixed"; > > + regulator-name = "vusb-main5v0"; > > + regulator-min-microvolt = <5000000>; > > + regulator-max-microvolt = <5000000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + vsys_3v3: fixedregulator-vsys3v3 { > > + /* Output of LM5141 */ > > + compatible = "regulator-fixed"; > > + regulator-name = "vsys_3v3"; > > + regulator-min-microvolt = <3300000>; > > + regulator-max-microvolt = <3300000>; > > + vin-supply = <&vusb_main>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + vdd_mmc1: fixedregulator-sd { > > + /* Output of TPS22918 */ > > + compatible = "regulator-fixed"; > > + regulator-name = "vdd_mmc1"; > > + regulator-min-microvolt = <3300000>; > > + regulator-max-microvolt = <3300000>; > > + regulator-boot-on; > > + enable-active-high; > > + vin-supply = <&vsys_3v3>; > > + gpio = <&exp1 10 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + vdd_sd_dv: gpio-regulator-TLV71033 { > > 1. regulator-xxxxxx > 2. lowercase node name > > > + /* Output of TLV71033 */ > > + compatible = "regulator-gpio"; > > + regulator-name = "tlv71033"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&vdd_sd_dv_pins_default>; > > + regulator-min-microvolt = <1800000>; > > + regulator-max-microvolt = <3300000>; > > + regulator-boot-on; > > + vin-supply = <&vsys_3v3>; > > + gpios = <&main_gpio0 49 GPIO_ACTIVE_HIGH>;> + states = <1800000 0x0>, > > + <3300000 0x1>; > > + }; > > + > > + vsys_io_1v8: regulator-vsys-io-1v8 { > > + compatible = "regulator-fixed"; > > + regulator-name = "vsys_io_1v8"; > > + regulator-min-microvolt = <1800000>; > > + regulator-max-microvolt = <1800000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + vsys_io_1v2: regulator-vsys-io-1v2 { > > + compatible = "regulator-fixed"; > > + regulator-name = "vsys_io_1v2"; > > + regulator-min-microvolt = <1200000>; > > + regulator-max-microvolt = <1200000>; > > + regulator-always-on; > > + regulator-boot-on; > > + }; > > + > > + transceiver1: can-phy0 { > > + compatible = "ti,tcan1042"; > > + #phy-cells = <0>; > > + max-bitrate = <5000000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&mcu_mcan0_pins_default>; > > + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + transceiver2: can-phy1 { > > + compatible = "ti,tcan1042"; > > + #phy-cells = <0>; > > + max-bitrate = <5000000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&mcu_mcan1_pins_default>; > > + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + transceiver3: can-phy2 { > > + compatible = "ti,tcan1042"; > > + #phy-cells = <0>; > > + max-bitrate = <5000000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&main_mcan6_pins_default>; > > + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; > > + }; > > + > > + transceiver4: can-phy3 { > > + compatible = "ti,tcan1042"; > > + #phy-cells = <0>; > > + max-bitrate = <5000000>; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&main_mcan7_pins_default>; > > + standby-gpios = <&exp1 6 GPIO_ACTIVE_HIGH>; > > + }; > > +}; > > + > > +&main_pmx0 { > > + > > No need for blank line. > > > + main_uart8_pins_default: main-uart8-pins-default { > > + pinctrl-single,pins = < > > + J721S2_IOPAD(0x0d0, PIN_INPUT, 11) /* (AF26) SPI0_CS1.UART8_RXD */ > > + J721S2_IOPAD(0x0d4, PIN_OUTPUT, 11) /* (AH27) SPI0_CLK.UART8_TXD */ > > + >; > > + }; > > + > > + main_i2c0_pins_default: i2c0_pins_default { > > No underscores in node names. Why do you use here different style than > in other places? > > > + pinctrl-single,pins = < > > + J721S2_IOPAD(0x0e0, PIN_INPUT, 0) /* (AH25) I2C0_SCL */ > > + J721S2_IOPAD(0x0e4, PIN_INPUT, 0) /* (AE24) I2C0_SDA */ > > + >; > > + }; > > + > > + main_mmc1_pins_default: main-mmc1-pins-default { > > + pinctrl-single,pins = < > > + J721S2_IOPAD(0x104, PIN_INPUT, 0) /* (P23) MMC1_CLK */ > > + J721S2_IOPAD(0x108, PIN_INPUT, 0) /* (N24) MMC1_CMD */ > > + J721S2_IOPAD(0x0fc, PIN_INPUT, 0) /* (M23) MMC1_DAT0 */ > > + J721S2_IOPAD(0x0f8, PIN_INPUT, 0) /* (P24) MMC1_DAT1 */ > > + J721S2_IOPAD(0x0f4, PIN_INPUT, 0) /* (R24) MMC1_DAT2 */ > > + J721S2_IOPAD(0x0f0, PIN_INPUT, 0) /* (R22) MMC1_DAT3 */ > > + J721S2_IOPAD(0x0e8, PIN_INPUT, 8) /* (AE25) TIMER_IO0.MMC1_SDCD */ > > + >; > > + }; > > + > > + vdd_sd_dv_pins_default: vdd-sd-dv-pins-default { > > + pinctrl-single,pins = < > > + J721S2_IOPAD(0x0c4, PIN_INPUT, 7) /* (AB26) ECAP0_IN_APWM_OUT.GPIO0_49 */ > > + >; > > + }; > > + > > + main_usbss0_pins_default: main-usbss0-pins-default { > > + pinctrl-single,pins = < > > + J721S2_IOPAD(0x0ec, PIN_OUTPUT, 6) /* (AG25) TIMER_IO1.USB0_DRVVBUS */ > > + >; > > + }; > > + > > + main_mcan6_pins_default: mcan6_pins_default { > > Be sure you tested the DTS against bindings. Please run `make > dtbs_check` (see Documentation/devicetree/bindings/writing-schema.rst > for instructions). > > > + pinctrl-single,pins = < > > + J721S2_IOPAD(0x098, PIN_INPUT, 0) /* (V25) MCASP0_AXR10.MCAN6_RX */ > > + J721S2_IOPAD(0x094, PIN_INPUT, 0) /* (AA25) MCASP0_AXR9.MCAN6_TX */ > > + >; > > + }; > > + > > + main_mcan7_pins_default: mcan7_pins_default { > > + pinctrl-single,pins = < > > + J721S2_IOPAD(0x0a0, PIN_INPUT, 0) /* (AB25) MCASP0_AXR12.MCAN7_RX */ > > + J721S2_IOPAD(0x09c, PIN_INPUT, 0) /* (T24) MCASP0_AXR11.MCAN7_TX */ > > + >; > > + }; > > + > > +}; > > + > > +&wkup_pmx0 { > > + mcu_cpsw_pins_default: mcu-cpsw-pins-default { > > + pinctrl-single,pins = < > > + J721S2_WKUP_IOPAD(0x094, PIN_INPUT, 0) /* (B22) MCU_RGMII1_RD0 */ > > + J721S2_WKUP_IOPAD(0x090, PIN_INPUT, 0) /* (B21) MCU_RGMII1_RD1 */ > > + J721S2_WKUP_IOPAD(0x08c, PIN_INPUT, 0) /* (C22) MCU_RGMII1_RD2 */ > > + J721S2_WKUP_IOPAD(0x088, PIN_INPUT, 0) /* (D23) MCU_RGMII1_RD3 */ > > + J721S2_WKUP_IOPAD(0x084, PIN_INPUT, 0) /* (D22) MCU_RGMII1_RXC */ > > + J721S2_WKUP_IOPAD(0x06c, PIN_INPUT, 0) /* (E23) MCU_RGMII1_RX_CTL */ > > + J721S2_WKUP_IOPAD(0x07c, PIN_OUTPUT, 0) /* (F23) MCU_RGMII1_TD0 */ > > + J721S2_WKUP_IOPAD(0x078, PIN_OUTPUT, 0) /* (G22) MCU_RGMII1_TD1 */ > > + J721S2_WKUP_IOPAD(0x074, PIN_OUTPUT, 0) /* (E21) MCU_RGMII1_TD2 */ > > + J721S2_WKUP_IOPAD(0x070, PIN_OUTPUT, 0) /* (E22) MCU_RGMII1_TD3 */ > > + J721S2_WKUP_IOPAD(0x080, PIN_OUTPUT, 0) /* (F21) MCU_RGMII1_TXC */ > > + J721S2_WKUP_IOPAD(0x068, PIN_OUTPUT, 0) /* (F22) MCU_RGMII1_TX_CTL */ > > + >; > > + }; > > + > > + mcu_mdio_pins_default: mcu-mdio-pins-default { > > + pinctrl-single,pins = < > > + J721S2_WKUP_IOPAD(0x09c, PIN_OUTPUT, 0) /* (A21) MCU_MDIO0_MDC */ > > + J721S2_WKUP_IOPAD(0x098, PIN_INPUT, 0) /* (A22) MCU_MDIO0_MDIO */ > > + >; > > + }; > > + > > + mcu_mcan0_pins_default: mcu-mcan0-pins-default { > > + pinctrl-single,pins = < > > + J721S2_WKUP_IOPAD(0x0bc, PIN_INPUT, 0) /* (E28) MCU_MCAN0_RX */ > > + J721S2_WKUP_IOPAD(0x0b8, PIN_OUTPUT, 0) /* (E27) MCU_MCAN0_TX */ > > + >; > > + }; > > + > > + mcu_mcan1_pins_default: mcu-mcan1-pins-default { > > + pinctrl-single,pins = < > > + J721S2_WKUP_IOPAD(0x0d4, PIN_INPUT, 0) /* (F26) WKUP_GPIO0_5.MCU_MCAN1_RX */ > > + J721S2_WKUP_IOPAD(0x0d0, PIN_OUTPUT, 0) /* (C23) WKUP_GPIO0_4.MCU_MCAN1_TX*/ > > + >; > > + }; > > + > > + mcu_i2c1_pins_default: mcu_i2c1_pins_default { > > + pinctrl-single,pins = < > > + J721S2_WKUP_IOPAD(0x0e0, PIN_INPUT, 0) /* (F24) WKUP_GPIO0_8.MCU_I2C1_SCL */ > > + J721S2_WKUP_IOPAD(0x0e4, PIN_INPUT, 0) /* (H26) WKUP_GPIO0_9.MCU_I2C1_SDA */ > > + >; > > + }; > > + > > Drop line > > > +}; > > + > > +&main_gpio2 { > > + status = "disabled"; > > +}; > > + > Best regards, > Krzysztof > -- With Regards Sinthu Raja ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 3/3] arch: arm64: ti: Add support for AM68 SK base board 2022-11-03 8:38 ` Sinthu Raja M @ 2022-11-03 12:42 ` Krzysztof Kozlowski 0 siblings, 0 replies; 9+ messages in thread From: Krzysztof Kozlowski @ 2022-11-03 12:42 UTC (permalink / raw) To: Sinthu Raja M Cc: Nishanth Menon, Tero Kristo, Rob Herring, Vignesh Raghavendra, Krzysztof Kozlowski, linux-arm-kernel, devicetree, linux-kernel, Sinthu Raja On 03/11/2022 04:38, Sinthu Raja M wrote: > On Tue, Oct 18, 2022 at 6:33 PM Krzysztof Kozlowski > <krzysztof.kozlowski@linaro.org> wrote: >> >> On 18/10/2022 08:38, Sinthu Raja wrote: >>> From: Sinthu Raja <sinthu.raja@ti.com> >>> >>> The SK architecture comprises of baseboard and a SOM board. It is >>> as follows, >>> >>> +--------------------------------------+---+ >>> | | |<--Connectors >>> | +---| >>> | +---------------------------+ | | >>> | | SOM | +---| >>> | | | | | >>> | | | +---| Power Supply >>> | | | | | >>> | +---------------------------+ | | >>> | |<--+ >>> | BASE BOARD | >>> +------------------------------------------+ >> >> This is pretty straightforward for SoM, I am not sure what's the benefit >> to add it. >> >>> >>> AM68 Starter Kit's baseboard contains most of the actual connectors, >>> power supply etc. The System on Module (SoM) is plugged on to the base >>> board. Therefore, add support for peripherals brought out in the base >>> board. >>> >>> Schematics: https://www.ti.com/lit/zip/SPRR463 >>> >>> Signed-off-by: Sinthu Raja <sinthu.raja@ti.com> >>> --- >>> arch/arm64/boot/dts/ti/Makefile | 2 + >>> .../boot/dts/ti/k3-am68-sk-base-board.dts | 459 ++++++++++++++++++ >>> 2 files changed, 461 insertions(+) >>> create mode 100644 arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts >>> >>> diff --git a/arch/arm64/boot/dts/ti/Makefile b/arch/arm64/boot/dts/ti/Makefile >>> index 4555a5be2257..498b089d96f8 100644 >>> --- a/arch/arm64/boot/dts/ti/Makefile >>> +++ b/arch/arm64/boot/dts/ti/Makefile >>> @@ -19,6 +19,8 @@ dtb-$(CONFIG_ARCH_K3) += k3-j7200-common-proc-board.dtb >>> >>> dtb-$(CONFIG_ARCH_K3) += k3-j721s2-common-proc-board.dtb >>> >>> +dtb-$(CONFIG_ARCH_K3) += k3-am68-sk-base-board.dtb >>> + >> >> Alphabetical order, although does not look like other entries are ordered... >> >>> dtb-$(CONFIG_ARCH_K3) += k3-am642-evm.dtb >>> dtb-$(CONFIG_ARCH_K3) += k3-am642-sk.dtb >>> >>> diff --git a/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts >>> new file mode 100644 >>> index 000000000000..f51cbd2e3b72 >>> --- /dev/null >>> +++ b/arch/arm64/boot/dts/ti/k3-am68-sk-base-board.dts >>> @@ -0,0 +1,459 @@ >>> +// SPDX-License-Identifier: GPL-2.0 >>> +/* >>> + * Copyright (C) 2022 Texas Instruments Incorporated - https://www.ti.com/ >>> + * >>> + * Base Board: https://www.ti.com/lit/zip/SPRR463 >>> + */ >>> + >>> +/dts-v1/; >>> + >>> +#include "k3-am68-sk-som.dtsi" >>> +#include <dt-bindings/net/ti-dp83867.h> >>> +#include <dt-bindings/phy/phy-cadence.h> >>> +#include <dt-bindings/phy/phy.h> >>> +#include <dt-bindings/mux/ti-serdes.h> >>> + >>> +/ { >>> + compatible = "ti,am68-sk", "ti,j721s2"; >>> + model = "Texas Instruments AM68 SK"; >>> + >>> + chosen { >>> + stdout-path = "serial2:115200n8"; >>> + bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x2880000"; >> >> None of these bootargs are suitable for mainline. earlycon for sure - it >> is pure debug option. > Krzysztof, > In AM68 SK the Linux console is on main_uart8, so to be consistent > with other J7 families of devices aliasing the serial port to ttyS2. I > am quite confused why these bootargs are not suitable for mainline. I > check the below documentation section 2.3, as mentioned it can be used > as kernel parameters. Please correct me if I am wrong. > https://www.kernel.org/doc/html/latest/devicetree/usage-model.html?highlight=bootargs Why? Console is defined by stdout path, so why do you need to redefine it? Bootargs are for debugging so why all users in entire world by default must debug? Best regards, Krzysztof ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-11-03 12:42 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-10-18 12:38 [PATCH 0/3] AM68 SK: Add initial support Sinthu Raja 2022-10-18 12:38 ` [PATCH 1/3] dt-bindings: arm: ti: Add binding for AM68 SK Sinthu Raja 2022-10-18 12:58 ` Krzysztof Kozlowski 2022-10-18 12:38 ` [PATCH 2/3] arm64: dts: ti: Add initial support for AM68 SK System on Module Sinthu Raja 2022-10-18 12:58 ` Krzysztof Kozlowski 2022-10-18 12:38 ` [PATCH 3/3] arch: arm64: ti: Add support for AM68 SK base board Sinthu Raja 2022-10-18 13:02 ` Krzysztof Kozlowski 2022-11-03 8:38 ` Sinthu Raja M 2022-11-03 12:42 ` Krzysztof Kozlowski
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).