[U-Boot] Pull request: u-boot-net.git master

Joe Hershberger
2018-01-17T18:46:31Z

Hi Tom,

On Mon, Jan 15, 2018 at 8:25 PM, Tom Rini <trini@konsulko.com> wrote:
> On Mon, Jan 15, 2018 at 12:06:17PM -0600, Joe Hershberger wrote:
>
>> Hi Tom,
>>
>> The following changes since commit 3dde8f20377c3a051dda64497bdf0cdb23e03a2d:
>>
>> Merge git://git.denx.de/u-boot-mmc (2018-01-14 22:26:38 -0500)
>>
>> are available in the git repository at:
>>
>>
>> git://git.denx.de/u-boot-net.git master
>>
>> for you to fetch changes up to 1e2d2597a667e16adfe0a8a9be22e904cee84727:
>>
>> phy: atheros: set auto-negotiation for AR8021 (2018-01-15 12:05:27 -0600)
>>
>
> Applied to u-boot/master, thanks!
>
> But please note from ceheckpatch.pl (which is actually pretty good):

Sorry, I should have noted this in the PR. These remaining checkpatch
spew are deliberate. I reviewed each of them to be acceptable.

Cheers,
-Joe

>
> WARNING: please write a paragraph that describes the config symbol fully
> #37: FILE: drivers/net/Kconfig:188:
> +config MACB_ZYNQ
>
> total: 0 errors, 1 warnings, 0 checks, 213 lines checked
> ERROR: Unrecognized email address: 'Coverity (CID: 144423)'
> #8:
> Reported-by: Coverity (CID: 144423)
>
> total: 1 errors, 0 warnings, 0 checks, 19 lines checked
> WARNING: please write a paragraph that describes the config symbol fully
> #34: FILE: drivers/net/phy/Kconfig:15:
> +config B53_SWITCH
>
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> #67:
> new file mode 100644
>
> CHECK: Prefer using the BIT macro
> #134: FILE: drivers/net/phy/b53.c:63:
> +#define PORT_OVERRIDE_SPEED_100M (1 << PORT_OVERRIDE_SPEED_S)
>
> total: 0 errors, 2 warnings, 1 checks, 675 lines checked
> CHECK: Unbalanced braces around else statement
> #38: FILE: net/bootp.c:396:
> + } else
>
> total: 0 errors, 0 warnings, 1 checks, 24 lines checked
> WARNING: A patch subject line should describe the change not the tool that found it
> #4:
> Subject: [PATCH] net: sh-eth: Fix coding style checked by checkpatch.pl
>
> WARNING: 'alligned' may be misspelled - perhaps 'aligned'?
> #10:
> Change from alligned to aligned.
>
> total: 0 errors, 2 warnings, 0 checks, 92 lines checked
> WARNING: please write a paragraph that describes the config symbol fully
> #277: FILE: drivers/net/Kconfig:279:
> +config SH_ETHER
>
> total: 0 errors, 1 warnings, 0 checks, 273 lines checked
>
> --
> Tom
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
>