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

[Simon Glass]

Hi Tom,

On Tue, 23 Jul 2019 at 20:23, Tom Rini <trini at konsulko.com> wrote:
>
> 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.

Well that is because I updated the travis config but not the gitlab. Doing that.

> The good news is the gitlab job runs in a
> defined docker container you can pull and try locally :)

OK good. Will give this all a crack at some point.

Regards,
Simon