• Seung-Woo Kim's avatar
    mmc: dw_mmc: remove UBSAN warning in dw_mci_setup_bus() · 65257a0d
    Seung-Woo Kim authored
    This patch removes following UBSAN warnings in dw_mci_setup_bus().
    
      UBSAN: Undefined behaviour in drivers/mmc/host/dw_mmc.c:1102:14
      shift exponent 250 is too large for 32-bit type 'unsigned int'
      Call trace:
      [<ffffff90080908a8>] dump_backtrace+0x0/0x380
      [<ffffff9008090c3c>] show_stack+0x14/0x20
      [<ffffff90087457b8>] dump_stack+0xe0/0x120
      [<ffffff90087b1360>] ubsan_epilogue+0x18/0x68
      [<ffffff90087b1a94>] __ubsan_handle_shift_out_of_bounds+0x18c/0x1bc
      [<ffffff9008d89cb8>] dw_mci_setup_bus+0x3a0/0x438
      [...]
    
      UBSAN: Undefined behaviour in drivers/mmc/host/dw_mmc.c:1132:27
      shift exponent 250 is too large for 32-bit type 'unsigned int'
      Call trace:
      [<ffffff90080908a8>] dump_backtrace+0x0/0x380
      [<ffffff9008090c3c>] show_stack+0x14/0x20
      [<ffffff90087457b8>] dump_stack+0xe0/0x120
      [<ffffff90087b1360>] ubsan_epilogue+0x18/0x68
      [<ffffff90087b1a94>] __ubsan_handle_shift_out_of_bounds+0x18c/0x1bc
      [<ffffff9008d89c9c>] dw_mci_setup_bus+0x384/0x438
      [...]
    
    The warnings are caused because of bit shift which is used to
    filter spamming message for CONFIG_MMC_CLKGATE, but the config is
    already removed. So this patch just removes the shift.
    Signed-off-by: default avatarSeung-Woo Kim <sw0312.kim@samsung.com>
    Signed-off-by: default avatarJaehoon Chung <jh80.chung@samsung.com>
    Signed-off-by: default avatarUlf Hansson <ulf.hansson@linaro.org>
    65257a0d
dw_mmc.c 82.3 KB