]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
mmc: tmio: Check 'addr' width before checking for 64bit limitation
authorMarek Vasut <marek.vasut+renesas@mailbox.org>
Tue, 28 Feb 2023 21:18:12 +0000 (22:18 +0100)
committerMarek Vasut <marek.vasut+renesas@mailbox.org>
Fri, 7 Apr 2023 15:13:27 +0000 (17:13 +0200)
The 64bit limitation check is compiled and optimized out on 32bit
platforms, but generates a type width warning:

drivers/mmc/tmio-common.c: In function ‘tmio_sd_addr_is_dmaable’:
drivers/mmc/tmio-common.c:376:26: warning: right shift count >= width of type [-Wshift-count-overflow]
  376 |                 if (addr >> 32)
      |                          ^~

Fix the warning by checking the addr type width to see whether the
shift even makes sense in the first place. The width check is also
optimized out at compile time.

Signed-off-by: Marek Vasut <marek.vasut+renesas@mailbox.org>
Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>
drivers/mmc/tmio-common.c

index e9c7d3a2e004eb1c4daf280bcdbaca86dc6f5922..0b24a5a7bdbf8086da0a65bccff9bd1d38c48539 100644 (file)
@@ -373,7 +373,7 @@ static bool tmio_sd_addr_is_dmaable(struct mmc_data *data)
        if (!(data->flags & MMC_DATA_READ) && !IS_ALIGNED(addr, 128))
                return false;
        /* Gen3 DMA has 32bit limit */
-       if (addr >> 32)
+       if (sizeof(addr) > 4 && addr >> 32)
                return false;
 #endif