[U-Boot] [PATCH] net, fec: Shorten device name as done for other drivers

[U-Boot] [PATCH] net, fec: Shorten device name as done for other drivers

[Stefano Babic]

After discussion on the ML it is suggested to drop unrequired
and not useful characters from the device name.
This patch changes the name for the fec_mxc driver from
"FEC_MXC" to "FEC".

Signed-off-by: Stefano Babic <sbabic@denx.de>
---
 drivers/net/fec_mxc.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c
index 57f89a3..6b801ce 100644
--- a/drivers/net/fec_mxc.c
+++ b/drivers/net/fec_mxc.c
@@ -743,7 +743,7 @@ static int fec_probe(bd_t *bd)
 	writel(0x05ee0024, &fec->eth->r_cntrl);	/* FIXME 0x05ee0004 */
 	fec_mii_setspeed(fec);
 
-	sprintf(edev->name, "FEC_MXC");
+	sprintf(edev->name, "FEC");
 
 	miiphy_register(edev->name, fec_miiphy_read, fec_miiphy_write);
 
-- 
1.6.3.3

[U-Boot] [PATCH] net, fec: Shorten device name as done for other drivers

[Ben Warren]

  Hi Stefano,

n 7/19/2010 10:57 PM, Stefano Babic wrote:
> After discussion on the ML it is suggested to drop unrequired
> and not useful characters from the device name.
> This patch changes the name for the fec_mxc driver from
> "FEC_MXC" to "FEC".
>
> Signed-off-by: Stefano Babic<sbabic@denx.de>
> ---
>   drivers/net/fec_mxc.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c
> index 57f89a3..6b801ce 100644
> --- a/drivers/net/fec_mxc.c
> +++ b/drivers/net/fec_mxc.c
> @@ -743,7 +743,7 @@ static int fec_probe(bd_t *bd)
>   	writel(0x05ee0024,&fec->eth->r_cntrl);	/* FIXME 0x05ee0004 */
>   	fec_mii_setspeed(fec);
>
> -	sprintf(edev->name, "FEC_MXC");
> +	sprintf(edev->name, "FEC");
>
>   	miiphy_register(edev->name, fec_miiphy_read, fec_miiphy_write);
>
Applied to net repo.

thanks,
Ben

[U-Boot] [PATCH] net, fec: Shorten device name as done for other drivers

[Wolfgang Denk]

Dear Ben Warren,

In message <4C46196C.8050007@gmail.com> you wrote:
>   Hi Stefano,
> 
> n 7/19/2010 10:57 PM, Stefano Babic wrote:
> > After discussion on the ML it is suggested to drop unrequired
> > and not useful characters from the device name.
> > This patch changes the name for the fec_mxc driver from
> > "FEC_MXC" to "FEC".
> >
> > Signed-off-by: Stefano Babic<sbabic@denx.de>
> > ---
> >   drivers/net/fec_mxc.c |    2 +-
> >   1 files changed, 1 insertions(+), 1 deletions(-)
...
> Applied to net repo.

I consider this a bug fix. Could you please send a pull request any
time soon?  Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Accident: A condition in which presence of mind is good, but  absence
of body is better.