Message ID | 1481817905-6597-1-git-send-email-jagan@openedev.com |
---|---|
State | Accepted |
Delegated to: | Tom Rini |
Headers | show |
On Thu, Dec 15, 2016 at 05:05:05PM +0100, Jagan Teki wrote: > Hi Tom, > > Please pull this PR. > > Changes for v2: > - Skip "spi: Zap mpc52xx_spi.c and config" patch due to dependencies. > > thanks! > Jagan. > > The following changes since commit 53e8ca22538c2cec691fe74098684a359302688c: > > MAINTAINERS: DFU: Change e-mail address for DFU maintainer (2016-12-12 13:03:15 -0500) > > are available in the git repository at: > > git://git.denx.de/u-boot-spi.git master Sandbox fails to build, and I'm not sure what else yet. Please put your PR through a round with travis-ci before submitting v3, thanks! And for the record, it will build any branch so you can put things into WIP-... or TEST-... or whatever you wish. I personally use WIP-... for my tests.
On Thu, Dec 15, 2016 at 6:50 PM, Tom Rini <trini@konsulko.com> wrote: > On Thu, Dec 15, 2016 at 05:05:05PM +0100, Jagan Teki wrote: > >> Hi Tom, >> >> Please pull this PR. >> >> Changes for v2: >> - Skip "spi: Zap mpc52xx_spi.c and config" patch due to dependencies. >> >> thanks! >> Jagan. >> >> The following changes since commit 53e8ca22538c2cec691fe74098684a359302688c: >> >> MAINTAINERS: DFU: Change e-mail address for DFU maintainer (2016-12-12 13:03:15 -0500) >> >> are available in the git repository at: >> >> git://git.denx.de/u-boot-spi.git master > > Sandbox fails to build, and I'm not sure what else yet. Please put your > PR through a round with travis-ci before submitting v3, thanks! And for > the record, it will build any branch so you can put things into WIP-... > or TEST-... or whatever you wish. I personally use WIP-... for my > tests. I don't think Sandbox will fail, since none of these related to this anyway I will re-ran the build and send. thanks!
On Thu, Dec 15, 2016 at 7:08 PM, Jagan Teki <jagan@openedev.com> wrote: > On Thu, Dec 15, 2016 at 6:50 PM, Tom Rini <trini@konsulko.com> wrote: >> On Thu, Dec 15, 2016 at 05:05:05PM +0100, Jagan Teki wrote: >> >>> Hi Tom, >>> >>> Please pull this PR. >>> >>> Changes for v2: >>> - Skip "spi: Zap mpc52xx_spi.c and config" patch due to dependencies. >>> >>> thanks! >>> Jagan. >>> >>> The following changes since commit 53e8ca22538c2cec691fe74098684a359302688c: >>> >>> MAINTAINERS: DFU: Change e-mail address for DFU maintainer (2016-12-12 13:03:15 -0500) >>> >>> are available in the git repository at: >>> >>> git://git.denx.de/u-boot-spi.git master >> >> Sandbox fails to build, and I'm not sure what else yet. Please put your >> PR through a round with travis-ci before submitting v3, thanks! And for >> the record, it will build any branch so you can put things into WIP-... >> or TEST-... or whatever you wish. I personally use WIP-... for my >> tests. > > I don't think Sandbox will fail, since none of these related to this > anyway I will re-ran the build and send. Never find any issues [1]. [1] https://paste.ubuntu.com/23639445/ thanks!
On Fri, Dec 16, 2016 at 08:21:36PM +0100, Jagan Teki wrote: > On Thu, Dec 15, 2016 at 7:08 PM, Jagan Teki <jagan@openedev.com> wrote: > > On Thu, Dec 15, 2016 at 6:50 PM, Tom Rini <trini@konsulko.com> wrote: > >> On Thu, Dec 15, 2016 at 05:05:05PM +0100, Jagan Teki wrote: > >> > >>> Hi Tom, > >>> > >>> Please pull this PR. > >>> > >>> Changes for v2: > >>> - Skip "spi: Zap mpc52xx_spi.c and config" patch due to dependencies. > >>> > >>> thanks! > >>> Jagan. > >>> > >>> The following changes since commit 53e8ca22538c2cec691fe74098684a359302688c: > >>> > >>> MAINTAINERS: DFU: Change e-mail address for DFU maintainer (2016-12-12 13:03:15 -0500) > >>> > >>> are available in the git repository at: > >>> > >>> git://git.denx.de/u-boot-spi.git master > >> > >> Sandbox fails to build, and I'm not sure what else yet. Please put your > >> PR through a round with travis-ci before submitting v3, thanks! And for > >> the record, it will build any branch so you can put things into WIP-... > >> or TEST-... or whatever you wish. I personally use WIP-... for my > >> tests. > > > > I don't think Sandbox will fail, since none of these related to this > > anyway I will re-ran the build and send. > > Never find any issues [1]. > > [1] https://paste.ubuntu.com/23639445/ Note that sandbox isn't even in your list there. Please see https://travis-ci.org/trini/u-boot/jobs/184309388 for some examples of the build failure.
On Fri, Dec 16, 2016 at 8:40 PM, Tom Rini <trini@konsulko.com> wrote: > On Fri, Dec 16, 2016 at 08:21:36PM +0100, Jagan Teki wrote: >> On Thu, Dec 15, 2016 at 7:08 PM, Jagan Teki <jagan@openedev.com> wrote: >> > On Thu, Dec 15, 2016 at 6:50 PM, Tom Rini <trini@konsulko.com> wrote: >> >> On Thu, Dec 15, 2016 at 05:05:05PM +0100, Jagan Teki wrote: >> >> >> >>> Hi Tom, >> >>> >> >>> Please pull this PR. >> >>> >> >>> Changes for v2: >> >>> - Skip "spi: Zap mpc52xx_spi.c and config" patch due to dependencies. >> >>> >> >>> thanks! >> >>> Jagan. >> >>> >> >>> The following changes since commit 53e8ca22538c2cec691fe74098684a359302688c: >> >>> >> >>> MAINTAINERS: DFU: Change e-mail address for DFU maintainer (2016-12-12 13:03:15 -0500) >> >>> >> >>> are available in the git repository at: >> >>> >> >>> git://git.denx.de/u-boot-spi.git master >> >> >> >> Sandbox fails to build, and I'm not sure what else yet. Please put your >> >> PR through a round with travis-ci before submitting v3, thanks! And for >> >> the record, it will build any branch so you can put things into WIP-... >> >> or TEST-... or whatever you wish. I personally use WIP-... for my >> >> tests. >> > >> > I don't think Sandbox will fail, since none of these related to this >> > anyway I will re-ran the build and send. >> >> Never find any issues [1]. >> >> [1] https://paste.ubuntu.com/23639445/ > > Note that sandbox isn't even in your list there. Please see > https://travis-ci.org/trini/u-boot/jobs/184309388 for some examples of > the build failure. Fixed yesterday itself may be you are not re-based, and don't know why include that patch on run since this patch is not part of my PR? thanks!
On Fri, Dec 16, 2016 at 11:14:29PM +0100, Jagan Teki wrote: > On Fri, Dec 16, 2016 at 8:40 PM, Tom Rini <trini@konsulko.com> wrote: > > On Fri, Dec 16, 2016 at 08:21:36PM +0100, Jagan Teki wrote: > >> On Thu, Dec 15, 2016 at 7:08 PM, Jagan Teki <jagan@openedev.com> wrote: > >> > On Thu, Dec 15, 2016 at 6:50 PM, Tom Rini <trini@konsulko.com> wrote: > >> >> On Thu, Dec 15, 2016 at 05:05:05PM +0100, Jagan Teki wrote: > >> >> > >> >>> Hi Tom, > >> >>> > >> >>> Please pull this PR. > >> >>> > >> >>> Changes for v2: > >> >>> - Skip "spi: Zap mpc52xx_spi.c and config" patch due to dependencies. > >> >>> > >> >>> thanks! > >> >>> Jagan. > >> >>> > >> >>> The following changes since commit 53e8ca22538c2cec691fe74098684a359302688c: > >> >>> > >> >>> MAINTAINERS: DFU: Change e-mail address for DFU maintainer (2016-12-12 13:03:15 -0500) > >> >>> > >> >>> are available in the git repository at: > >> >>> > >> >>> git://git.denx.de/u-boot-spi.git master > >> >> > >> >> Sandbox fails to build, and I'm not sure what else yet. Please put your > >> >> PR through a round with travis-ci before submitting v3, thanks! And for > >> >> the record, it will build any branch so you can put things into WIP-... > >> >> or TEST-... or whatever you wish. I personally use WIP-... for my > >> >> tests. > >> > > >> > I don't think Sandbox will fail, since none of these related to this > >> > anyway I will re-ran the build and send. > >> > >> Never find any issues [1]. > >> > >> [1] https://paste.ubuntu.com/23639445/ > > > > Note that sandbox isn't even in your list there. Please see > > https://travis-ci.org/trini/u-boot/jobs/184309388 for some examples of > > the build failure. > > Fixed yesterday itself may be you are not re-based, and don't know why > include that patch on run since this patch is not part of my PR? OK, I guess I didn't hit reset hard enough, sorry!
On Thu, Dec 15, 2016 at 05:05:05PM +0100, Jagan Teki wrote: > Hi Tom, > > Please pull this PR. > > Changes for v2: > - Skip "spi: Zap mpc52xx_spi.c and config" patch due to dependencies. > > thanks! > Jagan. > > The following changes since commit 53e8ca22538c2cec691fe74098684a359302688c: > > MAINTAINERS: DFU: Change e-mail address for DFU maintainer (2016-12-12 13:03:15 -0500) > > are available in the git repository at: > > git://git.denx.de/u-boot-spi.git master > > for you to fetch changes up to db9225ba2686d6b7e249d00e1803bd07f71d6070: > > sf: Do not force the DT memory map size to exactly match the device (2016-12-15 16:57:28 +0100) > Applied to u-boot/master, thanks!