[PATCH 1/2] ARM: dts: am437x-idk-evm: Mark MAC as having only one PHY

[PATCH 1/2] ARM: dts: am437x-idk-evm: Mark MAC as having only one PHY

[Andrew F. Davis]

Currently am4372.dtsi declares the MAC controller to have two
slave ports, on this board we only use one, so set the slave
count to one. This eliminates a console error message when
the non-existent PHY is not detected.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 arch/arm/boot/dts/am437x-idk-evm.dts | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/boot/dts/am437x-idk-evm.dts b/arch/arm/boot/dts/am437x-idk-evm.dts
index 76dcfc6..12a6951 100644
--- a/arch/arm/boot/dts/am437x-idk-evm.dts
+++ b/arch/arm/boot/dts/am437x-idk-evm.dts
@@ -382,6 +382,7 @@
 };
 
 &mac {
+	slaves = <1>;
 	pinctrl-names = "default", "sleep";
 	pinctrl-0 = <&cpsw_default>;
 	pinctrl-1 = <&cpsw_sleep>;
-- 
2.8.3

[PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY

[Andrew F. Davis]

Currently am33xx.dtsi declares the MAC controller to have two
slave ports, on these boards we only use one, so set the slave
count to one. This eliminates a console error message when
the non-existent PHY is not detected.

Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 arch/arm/boot/dts/am335x-bone-common.dtsi | 6 +-----
 1 file changed, 1 insertion(+), 5 deletions(-)

diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
index 0cc150b..8767bbe 100644
--- a/arch/arm/boot/dts/am335x-bone-common.dtsi
+++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
@@ -359,12 +359,8 @@
 	phy-mode = "mii";
 };
 
-&cpsw_emac1 {
-	phy_id = <&davinci_mdio>, <1>;
-	phy-mode = "mii";
-};
-
 &mac {
+	slaves = <1>;
 	pinctrl-names = "default", "sleep";
 	pinctrl-0 = <&cpsw_default>;
 	pinctrl-1 = <&cpsw_sleep>;
-- 
2.8.3

Re: [PATCH 1/2] ARM: dts: am437x-idk-evm: Mark MAC as having only one PHY

[Grygorii Strashko]

On 05/27/2016 09:17 PM, Andrew F. Davis wrote:
> Currently am4372.dtsi declares the MAC controller to have two
> slave ports, on this board we only use one, so set the slave
> count to one. This eliminates a console error message when
> the non-existent PHY is not detected.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---
>   arch/arm/boot/dts/am437x-idk-evm.dts | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/boot/dts/am437x-idk-evm.dts b/arch/arm/boot/dts/am437x-idk-evm.dts
> index 76dcfc6..12a6951 100644
> --- a/arch/arm/boot/dts/am437x-idk-evm.dts
> +++ b/arch/arm/boot/dts/am437x-idk-evm.dts
> @@ -382,6 +382,7 @@
>   };
>
>   &mac {
> +	slaves = <1>;
>   	pinctrl-names = "default", "sleep";
>   	pinctrl-0 = <&cpsw_default>;
>   	pinctrl-1 = <&cpsw_sleep>;
>

Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>

-- 
regards,
-grygorii

Re: [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY

[Grygorii Strashko]

On 05/27/2016 09:17 PM, Andrew F. Davis wrote:
> Currently am33xx.dtsi declares the MAC controller to have two
> slave ports, on these boards we only use one, so set the slave
> count to one. This eliminates a console error message when
> the non-existent PHY is not detected.
>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---
>   arch/arm/boot/dts/am335x-bone-common.dtsi | 6 +-----
>   1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/arch/arm/boot/dts/am335x-bone-common.dtsi b/arch/arm/boot/dts/am335x-bone-common.dtsi
> index 0cc150b..8767bbe 100644
> --- a/arch/arm/boot/dts/am335x-bone-common.dtsi
> +++ b/arch/arm/boot/dts/am335x-bone-common.dtsi
> @@ -359,12 +359,8 @@
>   	phy-mode = "mii";
>   };
>
> -&cpsw_emac1 {
> -	phy_id = <&davinci_mdio>, <1>;
> -	phy-mode = "mii";
> -};
> -
>   &mac {
> +	slaves = <1>;
>   	pinctrl-names = "default", "sleep";
>   	pinctrl-0 = <&cpsw_default>;
>   	pinctrl-1 = <&cpsw_sleep>;
>

Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>

-- 
regards,
-grygorii

Re: [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY

[Tony Lindgren]

* Grygorii Strashko <grygorii.strashko@ti.com> [160603 09:49]:
> On 05/27/2016 09:17 PM, Andrew F. Davis wrote:
> > Currently am33xx.dtsi declares the MAC controller to have two
> > slave ports, on these boards we only use one, so set the slave
> > count to one. This eliminates a console error message when
> > the non-existent PHY is not detected.
...

> Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>

Applying both into omap-for-v4.8/dt thanks.

Tony