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
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list [email protected] https://lists.denx.de/listinfo/u-boot
