[U-Boot] Please pull u-boot-dm

[Tom Rini]

On Tue, Jul 23, 2019 at 08:08:03PM -0700, Simon Glass wrote:
> Hi Tom,
> 
> On Tue, 23 Jul 2019 at 11:16, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Mon, Jul 22, 2019 at 08:48:45AM -0600, Simon Glass wrote:
> >
> > > Hi Tom,
> > >
> > > Build is here: https://travis-ci.org/sglass68/u-boot/builds/561552377
> > >
> > > The following changes since commit 0de815356474912ef5bef9a69f0327a5a93bb2c2:
> > >
> > >   Merge branch '2019-07-17-master-imports' (2019-07-18 11:31:37 -0400)
> > >
> > > are available in the Git repository at:
> > >
> > >   git://git.denx.de/u-boot-dm.git tags/dm-pull-22jul19
> > >
> > > for you to fetch changes up to 857ad7985ff63989c3c7feff56c2dc353d7d7c9a:
> > >
> > >   dm: device: make power domain calls optional (2019-07-20 19:50:44 -0600)
> > >
> >
> > First, in my LLVM-7 test:
> > /home/trini/u-boot/u-boot/tools/ifwitool.c:584:23: warning: unused function 'read_le8' [-Wunused-function]
> > static inline uint8_t read_le8(const void *src)
> >                       ^
> > /home/trini/u-boot/u-boot/tools/ifwitool.c:695:20: warning: unused function 'zero_n' [-Wunused-function]
> > static inline void zero_n(void *dest, size_t n)
> >
> > Which is simple to fix.  But I don't see why it's not being caught in
> > the travis test off-hand.
> 
> I'm not sure either. Possible it depends on the compiler used?

I'm concerned the travis job isn't running LLVM like we think it is.

> >  And I further see I didn't get LLVM-7
> > included in GitLab at all yet.
> >
> > Next however I see:
> > https://gitlab.denx.de/u-boot/u-boot/-/jobs/1222
> > which is the binman tests failing and needs to be fixed.
> 
> Oh dear, I don't seem to have any idea about gitlab yet. Have updated
> the gitlab config file. Should I stop using travis completely now?

Long term I think GitLab CI is going to be better for custodians as
we'll have more build resources available and thus the jobs will
complete quicker.  I don't know why the binman testsuite doesn't fail in
travis like it does there.  The good news is the gitlab job runs in a
defined docker container you can pull and try locally :)

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20190723/744f0e44/attachment.sig>