Hi Jagan, On Wed, Nov 4, 2015 at 8:47 AM, Fabio Estevam <feste...@gmail.com> wrote: > On Wed, Nov 4, 2015 at 8:46 AM, Fabio Estevam <feste...@gmail.com> wrote: >> Hi Tom, >> >> On Wed, Nov 4, 2015 at 1:45 AM, Fabio Estevam <feste...@gmail.com> wrote: >>> On Tue, Nov 3, 2015 at 11:47 PM, Tom Rini <tr...@konsulko.com> wrote: >>> >>>> NAK. >>>> >>>> So, this blows up badly for a lot of things when I do: >>>> ./tools/buildman/buildman -b master --force-build --step 0 -Cdvel >>>> 'blackfin|microblaze|m68k|nds32|x86|aarch64|sandbox|mips|avr32|arm|powerpc|sh4|sparc' >>> >>> It seems I missed to add bitops for avr32 and mips. Will work on it >>> tomorrow. >> >> I don't see issues after applying the three attached patches. > > Ops, I do see a build issue in blackfin that didn't happen in previous > versions. Will work on fixing it.
When you reworked my series you missed to updated these functions: diff --git a/drivers/mtd/spi/sf_ops.c b/drivers/mtd/spi/sf_ops.c index 31c79c6..d832464 100644 --- a/drivers/mtd/spi/sf_ops.c +++ b/drivers/mtd/spi/sf_ops.c @@ -652,7 +652,7 @@ int stm_is_locked(struct spi_flash *flash, u32 ofs, size_t len) * * Returns negative on errors, 0 on success. */ -int stm_lock(struct spi_flash *flash, u32 ofs, u32 len) +int stm_lock(struct spi_flash *flash, u32 ofs, size_t len) { u8 status_old, status_new; u8 mask = SR_BP2 | SR_BP1 | SR_BP0; @@ -703,7 +703,7 @@ int stm_lock(struct spi_flash *flash, u32 ofs, u32 len) * * Returns negative on errors, 0 on success. */ -int stm_unlock(struct spi_flash *flash, u32 ofs, u32 len) +int stm_unlock(struct spi_flash *flash, u32 ofs, size_t len) { uint8_t status_old, status_new; u8 mask = SR_BP2 | SR_BP1 | SR_BP0; -- With this change and the other 3 bitops patches I sent I don't see any more build issues here. Thanks _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot