From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Schwierzeck Date: Mon, 22 Jan 2018 21:26:45 +0100 Subject: [U-Boot] Pull request: u-boot-spi/master In-Reply-To: <20180122171400.GN32220@bill-the-cat> References: <1516600256-8152-1-git-send-email-jagan@amarulasolutions.com> <20180122125844.GA32220@bill-the-cat> <7fd360b9-0494-9927-d456-91c4d3c0cb1a@gmail.com> <20180122171400.GN32220@bill-the-cat> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de On 22.01.2018 18:14, Tom Rini wrote: > On Mon, Jan 22, 2018 at 05:49:39PM +0100, Daniel Schwierzeck wrote: >> >> >> On 22.01.2018 13:58, Tom Rini wrote: >>> On Mon, Jan 22, 2018 at 11:20:56AM +0530, Jagan Teki wrote: >>> >>>> Hi Tom, >>>> >>>> Please pull this PR. >>>> >>>> thanks! >>>> Jagan. >>>> >>>> The following changes since commit 98691a60abffb44303d7dae6e9e699d0daded930: >>>> >>>> Merge git://git.denx.de/u-boot-rockchip (2018-01-09 13:28:51 -0500) >>>> >>>> are available in the git repository at: >>>> >>>> git://git.denx.de/u-boot-spi.git master >>>> >>>> for you to fetch changes up to b23c685c6f295da3c01dd487f0e003b70299af91: >>>> >>>> mips: bmips: enable the SPI flash on the Comtrend AR-5387un (2018-01-22 10:39:13 +0530) >>>> >>> >>> NAK: >>> >>> commit 19e3a4856c1cba751a9ecb3931ff0d96a7f169be >>> Author: Álvaro Fernández Rojas >>> Date: Sat Jan 20 02:11:34 2018 +0100 >>> >>> wait_bit: add 8/16/32 BE/LE versions of wait_for_bit >>> >>> Add 8/16/32 bits and BE/LE versions of wait_for_bit. >>> This is needed for reading registers that are not aligned to 32 bits, and for >>> Big Endian platforms. >>> >>> Signed-off-by: Álvaro Fernández Rojas >>> Reviewed-by: Daniel Schwierzeck >>> Reviewed-by: Jagan Teki >>> >>> Adds warnings on almost all platforms: >>> w+(ls1088ardb_qspi_SECURE_BOOT) ../include/wait_bit.h: In function ?wait_for_bit_be16?: >>> w+(ls1088ardb_qspi_SECURE_BOOT) ../include/wait_bit.h:76:31: warning: implicit declaration of function ?readw_be? [-Wimplicit-function-declaration] >>> w+(ls1088ardb_qspi_SECURE_BOOT) ../include/wait_bit.h: In function ?wait_for_bit_be32?: w+(ls1088ardb_qspi_SECURE_BOOT) ../include/wait_bit.h:78:31: warning: implicit declaration of function ?readl_be? [-Wimplicit-function-declaration] >>> >>> >> >> Tom, would this change to the patch be acceptable? >> >> --- a/include/wait_bit.h >> +++ b/include/wait_bit.h >> @@ -73,8 +73,12 @@ static inline int wait_for_bit_##sfx(const void *reg, \ >> >> BUILD_WAIT_FOR_BIT(8, u8, readb) >> BUILD_WAIT_FOR_BIT(le16, u16, readw) >> +#ifdef readw_be >> BUILD_WAIT_FOR_BIT(be16, u16, readw_be) >> +#endif >> BUILD_WAIT_FOR_BIT(le32, u32, readl) >> +#ifdef readl_be >> BUILD_WAIT_FOR_BIT(be32, u32, readl_be) >> +#endif >> >> #endif >> >> This wouldn't define wait_bit_be*() on archs which doesn't implement >> readw_be or readl_be. >> >> A build with the updated patch is scheduled at >> https://travis-ci.org/danielschwierzeck/u-boot/builds/331899381 > > That seems reasonable, thanks! > Álvaro, could you send a v10 where the patch "wait_bit: add 8/16/32 BE/LE versions of wait_for_bit" is fixed like above? Thanks. -- - Daniel -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: OpenPGP digital signature URL: