* [PATCH] dt: fsl-fec.txt: Fix GPIO comment and remove hardcoded MAC address
@ 2012-04-24 3:58 Fabio Estevam
[not found] ` <1335239915-2563-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
0 siblings, 1 reply; 2+ messages in thread
From: Fabio Estevam @ 2012-04-24 3:58 UTC (permalink / raw)
To: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ
Cc: Fabio Estevam, davem-fT/PcQaiUtIeIZ0/mPfg9Q,
rob.herring-bsGFqQB8/DxBDgjK7y7TUQ
Fix the comment for the GPIO number used for PHY reset and also avoid passing a
hardcoded MAC address.
Signed-off-by: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
---
Documentation/devicetree/bindings/net/fsl-fec.txt | 6 +-----
1 files changed, 1 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/net/fsl-fec.txt b/Documentation/devicetree/bindings/net/fsl-fec.txt
index de43951..3261203 100644
--- a/Documentation/devicetree/bindings/net/fsl-fec.txt
+++ b/Documentation/devicetree/bindings/net/fsl-fec.txt
@@ -9,9 +9,6 @@ Required properties:
"rgmii", "rgmii-id", "rgmii-rxid", "rgmii-txid", "rtbi", "smii".
- phy-reset-gpios : Should specify the gpio for phy reset
-Optional properties:
-- local-mac-address : 6 bytes, mac address
-
Example:
fec@83fec000 {
@@ -19,6 +16,5 @@ fec@83fec000 {
reg = <0x83fec000 0x4000>;
interrupts = <87>;
phy-mode = "mii";
- phy-reset-gpios = <&gpio1 14 0>; /* GPIO2_14 */
- local-mac-address = [00 04 9F 01 1B B9];
+ phy-reset-gpios = <&gpio1 14 0>; /* GPIO1_14 */
};
--
1.7.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] dt: fsl-fec.txt: Fix GPIO comment and remove hardcoded MAC address
[not found] ` <1335239915-2563-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
@ 2012-04-24 6:31 ` Shawn Guo
0 siblings, 0 replies; 2+ messages in thread
From: Shawn Guo @ 2012-04-24 6:31 UTC (permalink / raw)
To: Fabio Estevam
Cc: Fabio Estevam, devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ,
davem-fT/PcQaiUtIeIZ0/mPfg9Q, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ
On 24 April 2012 11:58, Fabio Estevam <festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> Fix the comment for the GPIO number used for PHY reset and also avoid passing a
> hardcoded MAC address.
>
You can choose to not pass MAC address from device tree, but someone
may want to use it for some case, e.g. local network just for
debugging. In any case, it's a binding supported by driver, you can
choose not use it, but you can not remove it from the binding doc.
Regards,
Shawn
> Signed-off-by: Fabio Estevam <fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> ---
> Documentation/devicetree/bindings/net/fsl-fec.txt | 6 +-----
> 1 files changed, 1 insertions(+), 5 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/net/fsl-fec.txt b/Documentation/devicetree/bindings/net/fsl-fec.txt
> index de43951..3261203 100644
> --- a/Documentation/devicetree/bindings/net/fsl-fec.txt
> +++ b/Documentation/devicetree/bindings/net/fsl-fec.txt
> @@ -9,9 +9,6 @@ Required properties:
> "rgmii", "rgmii-id", "rgmii-rxid", "rgmii-txid", "rtbi", "smii".
> - phy-reset-gpios : Should specify the gpio for phy reset
>
> -Optional properties:
> -- local-mac-address : 6 bytes, mac address
> -
> Example:
>
> fec@83fec000 {
> @@ -19,6 +16,5 @@ fec@83fec000 {
> reg = <0x83fec000 0x4000>;
> interrupts = <87>;
> phy-mode = "mii";
> - phy-reset-gpios = <&gpio1 14 0>; /* GPIO2_14 */
> - local-mac-address = [00 04 9F 01 1B B9];
> + phy-reset-gpios = <&gpio1 14 0>; /* GPIO1_14 */
> };
> --
> 1.7.1
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-04-24 6:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-24 3:58 [PATCH] dt: fsl-fec.txt: Fix GPIO comment and remove hardcoded MAC address Fabio Estevam
[not found] ` <1335239915-2563-1-git-send-email-festevam-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2012-04-24 6:31 ` Shawn Guo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox