On Wednesday 27 April 2016 03:06 AM, Joe Hershberger wrote:
> On Mon, Apr 25, 2016 at 4:53 PM, Joe Hershberger
> <[email protected]> wrote:
>> On Tue, Apr 12, 2016 at 3:46 AM, Mugunthan V N <[email protected]> wrote:
>>> Different TI platforms has to read with different combination to
>>> get the mac address from efuse. So add support to read mac address
>>> based on machine/device compatibles.
>>>
>>> The code is taken from Linux drivers/net/ethernet/ti/cpsw-common.c
>>> done by Tony Lindgren.
>>>
>>> Signed-off-by: Mugunthan V N <[email protected]>
>>
>> Acked-by: Joe Hershberger <[email protected]>
> 
> This is not checkpatch.pl clean. Please resubmit.
> 
> 609306.mbox:92: WARNING: line over 80 characters
> 609306.mbox:132: WARNING: line over 80 characters
> 

WARNING: line over 80 characters
#103: FILE: drivers/net/cpsw-common.c:39:
+               error("Not able to get syscon address to get mac efuse 
address\n");

WARNING: line over 80 characters
#143: FILE: drivers/net/cpsw-common.c:79:
+               error("Not able to get syscon address to get mac efuse 
address\n");

total: 0 errors, 2 warnings, 0 checks, 184 lines checked

These warnings are from error logs which can be ignored as it makes life
easier to grep the source when issue is found. The same is followed in
kernel as well.

Regards
Mugunthan V N
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to