[Buildroot] [PATCH 1/1] wf111: ARM bumped to 5.2.2-r4

[Buildroot] [PATCH 1/1] wf111: ARM bumped to 5.2.2-r4

[Geert Visser]

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180814/932cb53d/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-wf111-ARM-bumped-to-5.2.2-r4.patch
Type: text/x-patch
Size: 2001 bytes
Desc: not available
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20180814/932cb53d/attachment.bin>

[Buildroot] [PATCH 1/1] wf111: ARM bumped to 5.2.2-r4

[Thomas Petazzoni]

Hello Geert,

Thanks for your patch. Unfortunately, it is sent as attached file, and
not inline, so we can't review it properly. Could you resend it with
"git send-email", so that it is sent inline ?

(Please don't send it inline manually with your e-mail client, because
it will most likely rewrap long lines and damage the patch).

That being said, the patch looks mostly good. A couple of comments
though:

 - The fields in the .hash file are not properly aligned. I.e, on the
   first hash there is only one space between "sha256" and the hash
   itself, and on the following lines, there are two spaces.

 - We really want the WF111_VERSION variable to be defined, so could
   you change the code to:

ifeq ($(BR2_ARM_CPU_ARMV7A),y)
WF111_VERSION = $(WF111_VERSION_ARM)
...
else ifeq ($(BR2_ARM_CPU_ARMV5),y)
WF111_VERSION = $(WF111_VERSION_ARM)
...
else
WF111_VERSION = $(WF111_VERSION_X86)
...
endif

or alternatively:

ifeq ($(BR2_ARM_CPU_ARMV7A)$(BR2_ARM_CPU_ARMV5),y)
WF111_VERSION = 5.2.2-r4
else
WF111_VERSION = 5.2.2-r3
endif

Thanks!

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com