[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <36388bfb-75b5-07cd-4d39-9f6d683e582e@ti.com>
Date: Tue, 7 Dec 2021 18:50:53 +0530
From: Apurva Nandan <a-nandan@...com>
To: Aswath Govindraju <a-govindraju@...com>
CC: Marc Kleine-Budde <mkl@...gutronix.de>,
Kishon Vijay Abraham I <kishon@...com>,
Faiz Abbas <faiz_abbas@...com>, Nishanth Menon <nm@...com>,
Vignesh Raghavendra <vigneshr@...com>,
Tero Kristo <kristo@...nel.org>,
Rob Herring <robh+dt@...nel.org>,
<linux-arm-kernel@...ts.infradead.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v5 4/6] arm64: dts: ti: k3-j721e-common-proc-board: Add
support for mcu and main mcan nodes
On 22/11/21 7:11 pm, Aswath Govindraju wrote:
> From: Faiz Abbas <faiz_abbas@...com>
>
> Add four MCAN nodes present on the common processor board and set a
> maximum data rate of 5 Mbps. Disable all other nodes as they
> are not brought out on the common processor board.
>
> Signed-off-by: Faiz Abbas <faiz_abbas@...com>
> Signed-off-by: Aswath Govindraju <a-govindraju@...com>
Reviewed-by: Apurva Nandan <a-nandan@...com>
> ---
> .../dts/ti/k3-j721e-common-proc-board.dts | 155 ++++++++++++++++++
> 1 file changed, 155 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> index dc2bc67f9f48..2d7596911b27 100644
> --- a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
> @@ -112,6 +112,42 @@
> "cpb-codec-scki",
> "cpb-codec-scki-48000", "cpb-codec-scki-44100";
> };
> +
> + transceiver1: can-phy0 {
> + compatible = "ti,tcan1043";
> + #phy-cells = <0>;
> + max-bitrate = <5000000>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&mcu_mcan0_gpio_pins_default>;
> + standby-gpios = <&wkup_gpio0 54 GPIO_ACTIVE_LOW>;
> + enable-gpios = <&wkup_gpio0 0 GPIO_ACTIVE_HIGH>;
> + };
> +
> + transceiver2: can-phy1 {
> + compatible = "ti,tcan1042";
> + #phy-cells = <0>;
> + max-bitrate = <5000000>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&mcu_mcan1_gpio_pins_default>;
> + standby-gpios = <&wkup_gpio0 2 GPIO_ACTIVE_HIGH>;
> + };
> +
> + transceiver3: can-phy2 {
> + compatible = "ti,tcan1043";
> + #phy-cells = <0>;
> + max-bitrate = <5000000>;
> + standby-gpios = <&exp2 7 GPIO_ACTIVE_LOW>;
> + enable-gpios = <&exp2 6 GPIO_ACTIVE_HIGH>;
> + };
> +
> + transceiver4: can-phy3 {
> + compatible = "ti,tcan1042";
> + #phy-cells = <0>;
> + max-bitrate = <5000000>;
> + pinctrl-names = "default";
> + pinctrl-0 = <&main_mcan2_gpio_pins_default>;
> + standby-gpios = <&main_gpio0 127 GPIO_ACTIVE_HIGH>;
> + };
> };
>
> &main_pmx0 {
> @@ -207,6 +243,26 @@
> J721E_IOPAD(0x1a4, PIN_OUTPUT, 3) /* (W26) RGMII6_RXC.AUDIO_EXT_REFCLK2 */
> >;
> };
> +
> + main_mcan0_pins_default: main-mcan0-pins-default {
> + pinctrl-single,pins = <
> + J721E_IOPAD(0x208, PIN_INPUT, 0) /* (W5) MCAN0_RX */
> + J721E_IOPAD(0x20c, PIN_OUTPUT, 0) /* (W6) MCAN0_TX */
> + >;
> + };
> +
> + main_mcan2_pins_default: main-mcan2-pins-default {
> + pinctrl-single,pins = <
> + J721E_IOPAD(0x01f0, PIN_INPUT, 3) /* (AC2) MCAN2_RX.GPIO0_123 */
> + J721E_IOPAD(0x01f4, PIN_OUTPUT, 3) /* (AB1) MCAN2_TX.GPIO0_124 */
> + >;
> + };
> +
> + main_mcan2_gpio_pins_default: main-mcan2-gpio-pins-default {
> + pinctrl-single,pins = <
> + J721E_IOPAD(0x200, PIN_INPUT, 7) /* (AC4) UART1_CTSn.GPIO0_127 */
> + >;
> + };
> };
>
> &wkup_pmx0 {
> @@ -252,6 +308,33 @@
> J721E_WKUP_IOPAD(0x0088, PIN_INPUT, 0) /* MCU_MDIO0_MDIO */
> >;
> };
> +
> + mcu_mcan0_pins_default: mcu-mcan0-pins-default {
> + pinctrl-single,pins = <
> + J721E_WKUP_IOPAD(0xac, PIN_INPUT, 0) /* (C29) MCU_MCAN0_RX */
> + J721E_WKUP_IOPAD(0xa8, PIN_OUTPUT, 0) /* (D29) MCU_MCAN0_TX */
> + >;
> + };
> +
> + mcu_mcan0_gpio_pins_default: mcu-mcan0-gpio-pins-default {
> + pinctrl-single,pins = <
> + J721E_WKUP_IOPAD(0xb0, PIN_INPUT, 7) /* (F26) WKUP_GPIO0_0 */
> + J721E_WKUP_IOPAD(0x98, PIN_INPUT, 7) /* (E28) MCU_SPI0_D1.WKUP_GPIO0_54 */
> + >;
> + };
> +
> + mcu_mcan1_pins_default: mcu-mcan1-pins-default {
> + pinctrl-single,pins = <
> + J721E_WKUP_IOPAD(0xc4, PIN_INPUT, 0) /* (G24) WKUP_GPIO0_5.MCU_MCAN1_RX */
> + J721E_WKUP_IOPAD(0xc0, PIN_OUTPUT, 0) /* (G25) WKUP_GPIO0_4.MCU_MCAN1_TX */
> + >;
> + };
> +
> + mcu_mcan1_gpio_pins_default: mcu-mcan1-gpio-pins-default {
> + pinctrl-single,pins = <
> + J721E_WKUP_IOPAD(0xb8, PIN_INPUT, 7) /* (F28) WKUP_GPIO0_2 */
> + >;
> + };
> };
>
> &wkup_uart0 {
> @@ -773,3 +856,75 @@
> &icssg1_mdio {
> status = "disabled";
> };
> +
> +&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 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&main_mcan0_pins_default>;
> + phys = <&transceiver3>;
> +};
> +
> +&main_mcan1 {
> + status = "disabled";
> +};
> +
> +&main_mcan2 {
> + pinctrl-names = "default";
> + pinctrl-0 = <&main_mcan2_pins_default>;
> + phys = <&transceiver4>;
> +};
> +
> +&main_mcan3 {
> + status = "disabled";
> +};
> +
> +&main_mcan4 {
> + status = "disabled";
> +};
> +
> +&main_mcan5 {
> + status = "disabled";
> +};
> +
> +&main_mcan6 {
> + status = "disabled";
> +};
> +
> +&main_mcan7 {
> + status = "disabled";
> +};
> +
> +&main_mcan8 {
> + status = "disabled";
> +};
> +
> +&main_mcan9 {
> + status = "disabled";
> +};
> +
> +&main_mcan10 {
> + status = "disabled";
> +};
> +
> +&main_mcan11 {
> + status = "disabled";
> +};
> +
> +&main_mcan12 {
> + status = "disabled";
> +};
> +
> +&main_mcan13 {
> + status = "disabled";
> +};
Powered by blists - more mailing lists