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

From	Grygorii Strashko <grygorii.strashko@ti.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY
Date	2016-06-03 18:50 +0200
Message-ID	<rG0si-5FK-11@gated-at.bofh.it> (permalink)
References	<rDuwx-6Pt-11@gated-at.bofh.it> <rDuwx-6Pt-13@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

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

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

Re: [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only  one PHY Grygorii Strashko <grygorii.strashko@ti.com> - 2016-06-03 18:50 +0200
  Re: [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having  only one PHY Tony Lindgren <tony@atomide.com> - 2016-06-10 13:40 +0200

csiph-web