From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Mon, 2 Apr 2018 20:33:08 -0400 Subject: [U-Boot] Please pull u-boot-dm In-Reply-To: References: <20180127005016.GB32085@bill-the-cat> <20180203181342.GT4297@bill-the-cat> Message-ID: <20180403003308.GH19725@bill-the-cat.ec.rr.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de On Mon, Apr 02, 2018 at 02:11:59PM +0800, Simon Glass wrote: > Hi Tom, > > On 4 February 2018 at 02:13, Tom Rini wrote: > > On Sat, Feb 03, 2018 at 10:12:41AM -0700, Simon Glass wrote: > >> Hi Tom, > >> > >> On 26 January 2018 at 17:50, Tom Rini wrote: > >> > On Fri, Jan 26, 2018 at 02:45:29PM -0700, Simon Glass wrote: > >> > > >> >> Hi Tom, > >> >> > >> >> Here are some additions to logging and 64-bit sandbox support. > >> >> > >> >> > >> >> The following changes since commit ab12aa24e619b5e81cbde7de88c6d9a19f04313b: > >> >> > >> >> ARM: socfpga: Convert callers of cm_write_with_phase for > >> >> wait_for_bit_le32 (2018-01-26 13:08:03 -0500) > >> >> > >> >> are available in the Git repository at: > >> >> > >> >> git://git.denx.de/u-boot-dm.git > >> >> > >> >> for you to fetch changes up to 4e94617bf911595efcecf52d6c9f323744a81cda: > >> >> > >> >> log: add category LOGC_EFI (2018-01-26 12:20:55 -0700) > >> >> > >> > > >> > NAK: > >> > Starting LLVM-3.8 test > >> > common/log.c:44:17: warning: implicit conversion from enumeration type 'enum log_category_t' to different enumeration type 'enum uclass_id' [-Wenum-conversion] > >> > if (log_uc_cat(cat) >= LOGC_NONE) > >> > ~~~~~~~~~~ ^~~ > >> > common/log.c:47:36: warning: implicit conversion from enumeration type 'enum log_category_t' to different enumeration type 'enum uclass_id' [-Wenum-conversion] > >> > return uclass_get_name(log_uc_cat(cat)); > >> > ~~~~~~~~~~ ^~~ > >> > 2 warnings generated. > >> > > >> > Also, checkpatch.pl notes that the sandbox64 defconfig isn't caught by > >> > MAINTAINERS and then I see the 32bit one already isn't. > >> > > >> > Finally, I see a bunch of warnings on sandbox64: > >> > lib/fdtdec.c: In function ‘fdtdec_get_addr_size_auto_noparent’: > >> > include/fdtdec.h:27:25: warning: large integer implicitly truncated to unsigned type [-Woverflow] > >> > #define FDT_ADDR_T_NONE (-1ULL) > >> > > >> > That's with my normal 64bit host gcc (5.4.0, Ubuntu 16.04). > >> > >> OK I'll drop sandbox64 for now. I'm not sure why buildman was happy, > >> but I may have missed it because there are already a lot of warnings > >> with sandbox. I'll see if I can fix those. I need to add clang to my > >> workflow... > > > > It's not too hard. Note that if you go newer than 3.8 there's a ton of > > warnings I haven't had time to look into. And I only test sandbox with > > clang atm. > > I don't see any warnings doing this: > > $ make -s -j10 HOSTCC=clang-3.8 O=b/sandbox/ sandbox_defconfig all > $ make -s -j10 HOSTCC=clang-3.8 O=b/sandbox64/ sandbox64_defconfig all > $ > > How do I get the warnings? They have been fixed since then, I guess it was something else that was also in that PR at the time? Thanks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: