Re: [PATCH] git commit --signoff -m "mtd: cadence: Fix cast to pointer from integer of different size warning

From: Miquel Raynal
Date: Mon Dec 16 2019 - 05:09:52 EST


Hi Vasyl,

Vasyl Gomonovych <gomonovych@xxxxxxxxx> wrote on Sat, 14 Dec 2019
22:09:45 +0100:

> Use a cast to uintptr_t and next to a pointer
> In the final assignment the same casting in place
> memory_pointer = (uintptr_t)mem_ptr;
> Fix warning: cast to pointer from integer of different size

It seems like you're 'git send' script is messy :) (see the title).

>
> Signed-off-by: Vasyl Gomonovych <gomonovych@xxxxxxxxx>
> ---
> This commit fixes a minor issue with a warning
> Not sure if we will have problem here in case of
> dma_addr_t which can be 64-bit wide on 32-bit arch

Why not manipulating dma_addr_t everywhere instead?

>
> ---
> drivers/mtd/nand/raw/cadence-nand-controller.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/mtd/nand/raw/cadence-nand-controller.c b/drivers/mtd/nand/raw/cadence-nand-controller.c
> index 3a36285a8d8a..960c3a0be69c 100644
> --- a/drivers/mtd/nand/raw/cadence-nand-controller.c
> +++ b/drivers/mtd/nand/raw/cadence-nand-controller.c
> @@ -1280,8 +1280,8 @@ cadence_nand_cdma_transfer(struct cdns_nand_ctrl *cdns_ctrl, u8 chip_nr,
> }
>
> cadence_nand_cdma_desc_prepare(cdns_ctrl, chip_nr, page,
> - (void *)dma_buf, (void *)dma_ctrl_dat,
> - ctype);
> + (void *)(uintptr_t)dma_buf,
> + (void *)(uintptr_t)dma_ctrl_dat, ctype);
>
> status = cadence_nand_cdma_send_and_wait(cdns_ctrl, thread_nr);
>

Thanks,
MiquÃl