[U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

[U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

[Holger Brunck]

This prevents u-boot to display during bootime:
Out:   serial
Err:   serial
Net:   UEC3Warning: failed to set MAC address

Now corrected and it will display:
Out:   serial
Err:   serial
Net:   UEC3
Warning: failed to set MAC address

Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
---
 net/eth.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/eth.c b/net/eth.c
index 02baa37..1c0c780 100644
--- a/net/eth.c
+++ b/net/eth.c
@@ -302,7 +302,7 @@ int eth_initialize(bd_t *bis)
 				puts("\nWarning: eth device name has a space!\n");
 
 			if (eth_write_hwaddr(dev, "eth", eth_number))
-				puts("Warning: failed to set MAC address\n");
+				puts("\nWarning: failed to set MAC address\n");
 
 			eth_number++;
 			dev = dev->next;
-- 
1.7.1

[U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

[Philip Balister]

On 09/23/2011 08:47 AM, Holger Brunck wrote:
> This prevents u-boot to display during bootime:
> Out:   serial
> Err:   serial
> Net:   UEC3Warning: failed to set MAC address
>
> Now corrected and it will display:
> Out:   serial
> Err:   serial
> Net:   UEC3
> Warning: failed to set MAC address
>
> Signed-off-by: Holger Brunck<holger.brunck@keymile.com>

Acked-by: Philip Balister <philip@opensdr.com>

But, when i submitted the same patch, I was told this is not the right 
way to solve the problem. I would still like to see this patch go in 
until someone has time to rewrite the problem section of code.

Philip

> ---
>   net/eth.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/net/eth.c b/net/eth.c
> index 02baa37..1c0c780 100644
> --- a/net/eth.c
> +++ b/net/eth.c
> @@ -302,7 +302,7 @@ int eth_initialize(bd_t *bis)
>   				puts("\nWarning: eth device name has a space!\n");
>
>   			if (eth_write_hwaddr(dev, "eth", eth_number))
> -				puts("Warning: failed to set MAC address\n");
> +				puts("\nWarning: failed to set MAC address\n");
>
>   			eth_number++;
>   			dev = dev->next;

[U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

[Mike Frysinger]

On Monday, September 26, 2011 10:24:54 Philip Balister wrote:
> On 09/23/2011 08:47 AM, Holger Brunck wrote:
> > This prevents u-boot to display during bootime:
> > Out:   serial
> > Err:   serial
> > Net:   UEC3Warning: failed to set MAC address
> > 
> > Now corrected and it will display:
> > Out:   serial
> > Err:   serial
> > Net:   UEC3
> > Warning: failed to set MAC address
> > 
> > Signed-off-by: Holger Brunck<holger.brunck@keymile.com>
> 
> Acked-by: Philip Balister <philip@opensdr.com>
> 
> But, when i submitted the same patch, I was told this is not the right
> way to solve the problem. I would still like to see this patch go in
> until someone has time to rewrite the problem section of code.

i don't think it is the right way long term, but short term, i think it's 
correct wrt the surrounding code.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110926/efaf5d93/attachment.pgp 

[U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

[Holger Brunck]

On 09/26/2011 06:07 PM, Mike Frysinger wrote:
> On Monday, September 26, 2011 10:24:54 Philip Balister wrote:
>> On 09/23/2011 08:47 AM, Holger Brunck wrote:
>>> This prevents u-boot to display during bootime:
>>> Out:   serial
>>> Err:   serial
>>> Net:   UEC3Warning: failed to set MAC address
>>>
>>> Now corrected and it will display:
>>> Out:   serial
>>> Err:   serial
>>> Net:   UEC3
>>> Warning: failed to set MAC address
>>>
>>> Signed-off-by: Holger Brunck<holger.brunck@keymile.com>
>>
>> Acked-by: Philip Balister <philip@opensdr.com>
>>
>> But, when i submitted the same patch, I was told this is not the right
>> way to solve the problem. I would still like to see this patch go in
>> until someone has time to rewrite the problem section of code.
> 
> i don't think it is the right way long term, but short term, i think it's 
> correct wrt the surrounding code.

Yes I agree. It never meant to be more than a bootlog improvement. The root
cause should be fixed in the long term.

Regards
Holger

[U-Boot] [PATCH] net/eth: add newline to "failed MAC address" warning

[Wolfgang Denk]

Dear Holger Brunck,

In message <1316782071-8988-1-git-send-email-holger.brunck@keymile.com> you wrote:
> This prevents u-boot to display during bootime:
> Out:   serial
> Err:   serial
> Net:   UEC3Warning: failed to set MAC address
> 
> Now corrected and it will display:
> Out:   serial
> Err:   serial
> Net:   UEC3
> Warning: failed to set MAC address
> 
> Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
> ---
>  net/eth.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Applied, 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
On the subject of C program indentation: "In My Egotistical  Opinion,
most  people's  C  programs  should be indented six feet downward and
covered with dirt."                               - Blair P. Houghton