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

Message ID 20160527181743.8401-2-afd@ti.com
State New
Headers show

Commit Message

Andrew F. Davis May 27, 2016, 6:17 p.m. UTC
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(-)

Comments

Grygorii Strashko June 3, 2016, 4:47 p.m. UTC | #1
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>
Tony Lindgren June 10, 2016, 11:31 a.m. UTC | #2
* 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
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch
diff mbox

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>;