[U-Boot] [GIT PULL v2] u-boot-sunxi/master

[Maxime Ripard]

Hi,

On Thu, Sep 28, 2017 at 01:02:58PM +0000, Tom Rini wrote:
> On Thu, Sep 28, 2017 at 11:12:55AM +0200, Maxime Ripard wrote:
> 
> > Hi Tom,
> > 
> > Here is the second version of the pull request sent yesterday. It
> > shouldn't show the config warnings anymore.
> > 
> > Thanks!
> > Maxime
> > 
> > The following changes since commit 6e9b109aa9bcdd094e77da9fb03de8b44da8eb9d:
> > 
> >   Merge git://git.denx.de/u-boot-usb (2017-09-27 09:39:48 -0400)
> > 
> > are available in the git repository at:
> > 
> >   git://git.denx.de/u-boot-sunxi master
> > 
> > for you to fetch changes up to 6a191c5ef1294de8c09a3c4539b76a3d9fa60036:
> > 
> >   sunxi: usb_phy: invert the USB phy_ctl condition (2017-09-28 10:44:12 +0200)
> > 
> 
> OK, there's still some problems.  As a rule, you should at least run
> 'make tests' before posting and that shows:
> +make O=/home/trini/work/u-boot/u-boot/build-sandbox -s -j8
> ../drivers/core/read.c: In function ‘dev_read_addr_ptr’:
> ../drivers/core/read.c:64:44: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
>   return (addr == FDT_ADDR_T_NONE) ? NULL : (void *)addr;

This one is in the current master.

> ../lib/uuid.c: In function ‘gen_rand_uuid’:
> ../lib/uuid.c:246:2: warning: implicit declaration of function ‘clrsetbits_be16’ [-Wimplicit-function-declaration]
>   clrsetbits_be16(&uuid.time_hi_and_version,
>   ^
> ../lib/uuid.c:250:2: warning: implicit declaration of function ‘clrsetbits_8’ [-Wimplicit-function-declaration]
>   clrsetbits_8(&uuid.clock_seq_hi_and_reserved,
>   ^
> lib/built-in.o: In function `gen_rand_uuid':
> /home/trini/work/u-boot/u-boot/build-sandbox/../lib/uuid.c:246: undefined reference to `clrsetbits_be16'
> /home/trini/work/u-boot/u-boot/build-sandbox/../lib/uuid.c:250: undefined reference to `clrsetbits_8'
> collect2: error: ld returned 1 exit status
> make[2]: *** [u-boot] Error 1
> make[1]: *** [sub-make] Error 2

And these two are only seen on the sandbox, since arch/sandbox doesn't
define the clrsetbits. How am I supposed to fix that?

> Next, it's a good idea to hook your own github up to travis-ci and make
> sure that the build there completes for everything (note that it won't
> fail on warnings).  Thanks!

Ok, will do.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170929/4d8525bf/attachment.sig>