Skip to content

Commit 5904c9d

Browse files
arndbbroonie
authored andcommitted
spi: imx: fix little-endian build
The newly added dynamic burst code produces a harmless warning on big-endian configurations: drivers/spi/spi-imx.c: In function 'spi_imx_buf_rx_swap_u32': drivers/spi/spi-imx.c:284:15: error: unused variable 'bytes_per_word' [-Werror=unused-variable] unsigned int bytes_per_word; ^~~~~~~~~~~~~~ drivers/spi/spi-imx.c: In function 'spi_imx_buf_tx_swap_u32': drivers/spi/spi-imx.c:319:15: error: unused variable 'bytes_per_word' [-Werror=unused-variable] unsigned int bytes_per_word; This adds another #ifdef around the variable declaration matching the one on the use. Fixes: 1673c81 ("spi: imx: dynamic burst length adjust for PIO mode") Signed-off-by: Arnd Bergmann <[email protected]> Signed-off-by: Mark Brown <[email protected]>
1 parent 1673c81 commit 5904c9d

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

drivers/spi/spi-imx.c

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -281,7 +281,9 @@ static bool spi_imx_can_dma(struct spi_master *master, struct spi_device *spi,
281281
static void spi_imx_buf_rx_swap_u32(struct spi_imx_data *spi_imx)
282282
{
283283
unsigned int val = readl(spi_imx->base + MXC_CSPIRXDATA);
284+
#ifdef __LITTLE_ENDIAN
284285
unsigned int bytes_per_word;
286+
#endif
285287

286288
if (spi_imx->rx_buf) {
287289
#ifdef __LITTLE_ENDIAN
@@ -316,7 +318,9 @@ static void spi_imx_buf_rx_swap(struct spi_imx_data *spi_imx)
316318
static void spi_imx_buf_tx_swap_u32(struct spi_imx_data *spi_imx)
317319
{
318320
u32 val = 0;
321+
#ifdef __LITTLE_ENDIAN
319322
unsigned int bytes_per_word;
323+
#endif
320324

321325
if (spi_imx->tx_buf) {
322326
val = *(u32 *)spi_imx->tx_buf;

0 commit comments

Comments
 (0)