* [PATCH 1/2] ARM: dts: am437x-idk-evm: Mark MAC as having only one PHY
@ 2016-05-27 18:17 ` Andrew F. Davis
0 siblings, 0 replies; 9+ messages in thread
From: Andrew F. Davis @ 2016-05-27 18:17 UTC (permalink / raw)
To: Benoît Cousson, Tony Lindgren
Cc: linux-omap, devicetree, linux-kernel, 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
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 1/2] ARM: dts: am437x-idk-evm: Mark MAC as having only one PHY
@ 2016-05-27 18:17 ` Andrew F. Davis
0 siblings, 0 replies; 9+ messages in thread
From: Andrew F. Davis @ 2016-05-27 18:17 UTC (permalink / raw)
To: Benoît Cousson, Tony Lindgren
Cc: linux-omap, devicetree, linux-kernel, 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
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY
@ 2016-05-27 18:17 ` Andrew F. Davis
0 siblings, 0 replies; 9+ messages in thread
From: Andrew F. Davis @ 2016-05-27 18:17 UTC (permalink / raw)
To: Benoît Cousson, Tony Lindgren
Cc: linux-omap, devicetree, linux-kernel, 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
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY
@ 2016-05-27 18:17 ` Andrew F. Davis
0 siblings, 0 replies; 9+ messages in thread
From: Andrew F. Davis @ 2016-05-27 18:17 UTC (permalink / raw)
To: Benoît Cousson, Tony Lindgren
Cc: linux-omap-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA, 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-l0cyMroinI0@public.gmane.org>
---
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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] ARM: dts: am437x-idk-evm: Mark MAC as having only one PHY
2016-05-27 18:17 ` Andrew F. Davis
@ 2016-06-03 16:47 ` Grygorii Strashko
-1 siblings, 0 replies; 9+ messages in thread
From: Grygorii Strashko @ 2016-06-03 16:47 UTC (permalink / raw)
To: Andrew F. Davis, Benoît Cousson, Tony Lindgren
Cc: linux-omap, devicetree, linux-kernel
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] ARM: dts: am437x-idk-evm: Mark MAC as having only one PHY
@ 2016-06-03 16:47 ` Grygorii Strashko
0 siblings, 0 replies; 9+ messages in thread
From: Grygorii Strashko @ 2016-06-03 16:47 UTC (permalink / raw)
To: Andrew F. Davis, Benoît Cousson, Tony Lindgren
Cc: linux-omap, devicetree, linux-kernel
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY
@ 2016-06-03 16:47 ` Grygorii Strashko
0 siblings, 0 replies; 9+ messages in thread
From: Grygorii Strashko @ 2016-06-03 16:47 UTC (permalink / raw)
To: Andrew F. Davis, Benoît Cousson, Tony Lindgren
Cc: linux-omap, devicetree, linux-kernel
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
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY
@ 2016-06-03 16:47 ` Grygorii Strashko
0 siblings, 0 replies; 9+ messages in thread
From: Grygorii Strashko @ 2016-06-03 16:47 UTC (permalink / raw)
To: Andrew F. Davis, Benoît Cousson, Tony Lindgren
Cc: linux-omap-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA,
linux-kernel-u79uwXL29TY76Z2rM5mHXA
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-l0cyMroinI0@public.gmane.org>
> ---
> 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-l0cyMroinI0@public.gmane.org>
--
regards,
-grygorii
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 2/2] ARM: dts: am335x-bone-common: Mark MAC as having only one PHY
2016-06-03 16:47 ` Grygorii Strashko
(?)
@ 2016-06-10 11:31 ` Tony Lindgren
-1 siblings, 0 replies; 9+ messages in thread
From: Tony Lindgren @ 2016-06-10 11:31 UTC (permalink / raw)
To: Grygorii Strashko
Cc: Andrew F. Davis, Benoît Cousson, linux-omap, devicetree,
linux-kernel
* 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
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2016-06-10 11:31 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-27 18:17 [PATCH 1/2] ARM: dts: am437x-idk-evm: Mark MAC as having only one PHY Andrew F. Davis
2016-05-27 18:17 ` Andrew F. Davis
2016-05-27 18:17 ` [PATCH 2/2] ARM: dts: am335x-bone-common: " Andrew F. Davis
2016-05-27 18:17 ` Andrew F. Davis
2016-06-03 16:47 ` Grygorii Strashko
2016-06-03 16:47 ` Grygorii Strashko
2016-06-10 11:31 ` Tony Lindgren
2016-06-03 16:47 ` [PATCH 1/2] ARM: dts: am437x-idk-evm: " Grygorii Strashko
2016-06-03 16:47 ` Grygorii Strashko
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.