diff mbox series

[2/2] ARM: dts: am335x-icev2: Add CPSW ethernet0 and ethernet1

Message ID 1489412554-2054-3-git-send-email-rogerq@ti.com
State Accepted
Commit c0bde0bf2bfefe2bae04e09d3741f6f9899c7858
Headers show
Series None | expand

Commit Message

Roger Quadros March 13, 2017, 1:42 p.m. UTC 
Enable the 2 ethernet ports as CPSW ports in dual-mac mode

Signed-off-by: Roger Quadros <rogerq@ti.com>

[nsekhar@ti.com: use AM33XX_IOPAD()]
Signed-off-by: Sekhar Nori <nsekhar@ti.com>

---
 arch/arm/boot/dts/am335x-icev2.dts | 113 +++++++++++++++++++++++++++++++++++++
 1 file changed, 113 insertions(+)

-- 
2.7.4

Comments

Grygorii Strashko March 13, 2017, 7:24 p.m. UTC | #1 
On 03/13/2017 08:42 AM, Roger Quadros wrote:
> Enable the 2 ethernet ports as CPSW ports in dual-mac mode

>

> Signed-off-by: Roger Quadros <rogerq@ti.com>

> [nsekhar@ti.com: use AM33XX_IOPAD()]

> Signed-off-by: Sekhar Nori <nsekhar@ti.com>

> ---

>  arch/arm/boot/dts/am335x-icev2.dts | 113 +++++++++++++++++++++++++++++++++++++

>  1 file changed, 113 insertions(+)

>

> diff --git a/arch/arm/boot/dts/am335x-icev2.dts b/arch/arm/boot/dts/am335x-icev2.dts

> index a2ad076..cc343b0 100644

> --- a/arch/arm/boot/dts/am335x-icev2.dts

> +++ b/arch/arm/boot/dts/am335x-icev2.dts

> @@ -201,6 +201,69 @@

>  			AM33XX_IOPAD(0x938, PIN_OUTPUT_PULLUP | MUX_MODE1) /* (L16) gmii1_rxd2.uart3_txd */

>  		>;

>  	};

> +


>

>  &i2c0 {

> @@ -350,3 +413,53 @@

>  	pinctrl-0 = <&uart3_pins_default>;

>  	status = "okay";

>  };

> +

> +&gpio3 {

> +	p4 {

> +		gpio-hog;

> +		gpios = <4 GPIO_ACTIVE_HIGH>;

> +		output-high;

> +		line-name = "PR1_MII_CTRL";

> +	};

> +

> +	p10 {

> +		gpio-hog;

> +		gpios = <10 GPIO_ACTIVE_HIGH>;

> +		/* ETH1 mux: Low for MII-PRU, high for RMII-CPSW */

> +		output-high;

> +		line-name = "MUX_MII_CTL1";

> +	};

> +};

> +

> +&cpsw_emac0 {

> +	phy_id = <&davinci_mdio>, <1>;


this is deprecated definition.
pls, use phy-handle.

> +	phy-mode = "rmii";

> +	dual_emac_res_vlan = <1>;

> +};

> +

> +&cpsw_emac1 {

> +	phy_id = <&davinci_mdio>, <3>;


same

> +	phy-mode = "rmii";

> +	dual_emac_res_vlan = <2>;

> +};

> +

> +&mac {

> +	pinctrl-names = "default", "sleep";

> +	pinctrl-0 = <&cpsw_default>;

> +	pinctrl-1 = <&cpsw_sleep>;

> +	status = "okay";

> +	dual_emac;

> +};

> +

> +&phy_sel {

> +	rmii-clock-ext;

> +};

> +

> +&davinci_mdio {

> +	pinctrl-names = "default", "sleep";

> +	pinctrl-0 = <&davinci_mdio_default>;

> +	pinctrl-1 = <&davinci_mdio_sleep>;

> +	status = "okay";

> +	reset-gpios = <&gpio2 5 GPIO_ACTIVE_LOW>;

> +	reset-delay-us = <2>;   /* PHY datasheet states 1uS min */

> +};

>


-- 
regards,
-grygorii
Sekhar Nori March 14, 2017, 6:07 a.m. UTC | #2 
On Tuesday 14 March 2017 12:54 AM, Grygorii Strashko wrote:
> 

> 

> On 03/13/2017 08:42 AM, Roger Quadros wrote:

>> Enable the 2 ethernet ports as CPSW ports in dual-mac mode

>>

>> Signed-off-by: Roger Quadros <rogerq@ti.com>

>> [nsekhar@ti.com: use AM33XX_IOPAD()]

>> Signed-off-by: Sekhar Nori <nsekhar@ti.com>

>> ---

>>  arch/arm/boot/dts/am335x-icev2.dts | 113

>> +++++++++++++++++++++++++++++++++++++

>>  1 file changed, 113 insertions(+)

>>

>> diff --git a/arch/arm/boot/dts/am335x-icev2.dts

>> b/arch/arm/boot/dts/am335x-icev2.dts

>> index a2ad076..cc343b0 100644

>> --- a/arch/arm/boot/dts/am335x-icev2.dts

>> +++ b/arch/arm/boot/dts/am335x-icev2.dts

>> @@ -201,6 +201,69 @@

>>              AM33XX_IOPAD(0x938, PIN_OUTPUT_PULLUP | MUX_MODE1) /*

>> (L16) gmii1_rxd2.uart3_txd */

>>          >;

>>      };

>> +

> 

>>

>>  &i2c0 {

>> @@ -350,3 +413,53 @@

>>      pinctrl-0 = <&uart3_pins_default>;

>>      status = "okay";

>>  };

>> +

>> +&gpio3 {

>> +    p4 {

>> +        gpio-hog;

>> +        gpios = <4 GPIO_ACTIVE_HIGH>;

>> +        output-high;

>> +        line-name = "PR1_MII_CTRL";

>> +    };

>> +

>> +    p10 {

>> +        gpio-hog;

>> +        gpios = <10 GPIO_ACTIVE_HIGH>;

>> +        /* ETH1 mux: Low for MII-PRU, high for RMII-CPSW */

>> +        output-high;

>> +        line-name = "MUX_MII_CTL1";

>> +    };

>> +};

>> +

>> +&cpsw_emac0 {

>> +    phy_id = <&davinci_mdio>, <1>;

> 

> this is deprecated definition.

> pls, use phy-handle.


Perhaps cpsw driver should warn about using deprecated DT properties.

Thanks,
Sekhar
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/am335x-icev2.dts b/arch/arm/boot/dts/am335x-icev2.dts
index a2ad076..cc343b0 100644
--- a/arch/arm/boot/dts/am335x-icev2.dts
+++ b/arch/arm/boot/dts/am335x-icev2.dts
@@ -201,6 +201,69 @@ 
 			AM33XX_IOPAD(0x938, PIN_OUTPUT_PULLUP | MUX_MODE1) /* (L16) gmii1_rxd2.uart3_txd */
 		>;
 	};
+
+	cpsw_default: cpsw_default {
+		pinctrl-single,pins = <
+			/* Slave 1, RMII mode */
+			AM33XX_IOPAD(0x90c, (PIN_INPUT_PULLUP | MUX_MODE1))	/* mii1_crs.rmii1_crs_dv */
+			AM33XX_IOPAD(0x944, (PIN_INPUT_PULLUP | MUX_MODE0))	/* rmii1_refclk.rmii1_refclk */
+			AM33XX_IOPAD(0x940, (PIN_INPUT_PULLUP | MUX_MODE1))	/* mii1_rxd0.rmii1_rxd0 */
+			AM33XX_IOPAD(0x93c, (PIN_INPUT_PULLUP | MUX_MODE1))	/* mii1_rxd1.rmii1_rxd1 */
+			AM33XX_IOPAD(0x910, (PIN_INPUT_PULLUP | MUX_MODE1))	/* mii1_rxerr.rmii1_rxerr */
+			AM33XX_IOPAD(0x928, (PIN_OUTPUT_PULLDOWN | MUX_MODE1))	/* mii1_txd0.rmii1_txd0 */
+			AM33XX_IOPAD(0x924, (PIN_OUTPUT_PULLDOWN | MUX_MODE1))	/* mii1_txd1.rmii1_txd1 */
+			AM33XX_IOPAD(0x914, (PIN_OUTPUT_PULLDOWN | MUX_MODE1))	/* mii1_txen.rmii1_txen */
+			/* Slave 2, RMII mode */
+			AM33XX_IOPAD(0x870, (PIN_INPUT_PULLUP | MUX_MODE3))	/* gpmc_wait0.rmii2_crs_dv */
+			AM33XX_IOPAD(0x908, (PIN_INPUT_PULLUP | MUX_MODE1))	/* mii1_col.rmii2_refclk */
+			AM33XX_IOPAD(0x86c, (PIN_INPUT_PULLUP | MUX_MODE3))	/* gpmc_a11.rmii2_rxd0 */
+			AM33XX_IOPAD(0x868, (PIN_INPUT_PULLUP | MUX_MODE3))	/* gpmc_a10.rmii2_rxd1 */
+			AM33XX_IOPAD(0x874, (PIN_INPUT_PULLUP | MUX_MODE3))	/* gpmc_wpn.rmii2_rxerr */
+			AM33XX_IOPAD(0x854, (PIN_OUTPUT_PULLDOWN | MUX_MODE3))	/* gpmc_a5.rmii2_txd0 */
+			AM33XX_IOPAD(0x850, (PIN_OUTPUT_PULLDOWN | MUX_MODE3))	/* gpmc_a4.rmii2_txd1 */
+			AM33XX_IOPAD(0x840, (PIN_OUTPUT_PULLDOWN | MUX_MODE3))	/* gpmc_a0.rmii2_txen */
+		>;
+	};
+
+	cpsw_sleep: cpsw_sleep {
+		pinctrl-single,pins = <
+			/* Slave 1 reset value */
+			AM33XX_IOPAD(0x90c, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x944, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x940, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x93c, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x910, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x928, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x924, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x914, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+
+			/* Slave 2 reset value */
+			AM33XX_IOPAD(0x870, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x908, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x86c, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x868, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x874, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x854, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x850, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x840, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+		>;
+	};
+
+	davinci_mdio_default: davinci_mdio_default {
+		pinctrl-single,pins = <
+			/* MDIO */
+			AM33XX_IOPAD(0x948, (PIN_INPUT_PULLUP | SLEWCTRL_FAST | MUX_MODE0))	/* mdio_data.mdio_data */
+			AM33XX_IOPAD(0x94c, (PIN_OUTPUT_PULLUP | MUX_MODE0))			/* mdio_clk.mdio_clk */
+		>;
+	};
+
+	davinci_mdio_sleep: davinci_mdio_sleep {
+		pinctrl-single,pins = <
+			/* MDIO reset value */
+			AM33XX_IOPAD(0x948, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+			AM33XX_IOPAD(0x94c, (PIN_INPUT_PULLDOWN | MUX_MODE7))
+		>;
+	};
 };
 
 &i2c0 {
@@ -350,3 +413,53 @@ 
 	pinctrl-0 = <&uart3_pins_default>;
 	status = "okay";
 };
+
+&gpio3 {
+	p4 {
+		gpio-hog;
+		gpios = <4 GPIO_ACTIVE_HIGH>;
+		output-high;
+		line-name = "PR1_MII_CTRL";
+	};
+
+	p10 {
+		gpio-hog;
+		gpios = <10 GPIO_ACTIVE_HIGH>;
+		/* ETH1 mux: Low for MII-PRU, high for RMII-CPSW */
+		output-high;
+		line-name = "MUX_MII_CTL1";
+	};
+};
+
+&cpsw_emac0 {
+	phy_id = <&davinci_mdio>, <1>;
+	phy-mode = "rmii";
+	dual_emac_res_vlan = <1>;
+};
+
+&cpsw_emac1 {
+	phy_id = <&davinci_mdio>, <3>;
+	phy-mode = "rmii";
+	dual_emac_res_vlan = <2>;
+};
+
+&mac {
+	pinctrl-names = "default", "sleep";
+	pinctrl-0 = <&cpsw_default>;
+	pinctrl-1 = <&cpsw_sleep>;
+	status = "okay";
+	dual_emac;
+};
+
+&phy_sel {
+	rmii-clock-ext;
+};
+
+&davinci_mdio {
+	pinctrl-names = "default", "sleep";
+	pinctrl-0 = <&davinci_mdio_default>;
+	pinctrl-1 = <&davinci_mdio_sleep>;
+	status = "okay";
+	reset-gpios = <&gpio2 5 GPIO_ACTIVE_LOW>;
+	reset-delay-us = <2>;   /* PHY datasheet states 1uS min */
+};