> -----Original Message-----
> From: Patrick DELAUNAY <patrick.delau...@foss.st.com>
> Sent: Wednesday, April 17, 2024 6:02 PM
> To: Patrice Chotard <patrice.chot...@foss.st.com>; u-boot@lists.denx.de
> Cc: U-Boot STM32 <uboot-st...@st-md-mailman.stormreply.com>; Jaehoon Chung 
> <jh80.ch...@samsung.com>;
> Peng Fan <peng....@nxp.com>; Sean Anderson <sean...@gmail.com>; Simon Glass 
> <s...@chromium.org>; Tom
> Rini <tr...@konsulko.com>
> Subject: Re: [PATCH 2/2] mmc: stm32_sdmmc2: Fix AARCH64 compilation warnings
> 
> Hi,
> 
> On 3/8/24 15:26, Patrice Chotard wrote:
> > When building with AARCH64 defconfig, we got warnings, fix them.
> >
> > Signed-off-by: Patrice Chotard <patrice.chot...@foss.st.com>
> > ---
> >
> >   drivers/mmc/stm32_sdmmc2.c | 8 ++++----
> >   1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/mmc/stm32_sdmmc2.c b/drivers/mmc/stm32_sdmmc2.c
> > index d4982a14281..39ae79ba129 100644
> > --- a/drivers/mmc/stm32_sdmmc2.c
> > +++ b/drivers/mmc/stm32_sdmmc2.c
> > @@ -220,9 +220,9 @@ static void stm32_sdmmc2_start_data(struct udevice *dev,
> >
> >     if (data->flags & MMC_DATA_READ) {
> >             data_ctrl |= SDMMC_DCTRL_DTDIR;
> > -           idmabase0 = (u32)data->dest;
> > +           idmabase0 = (u32)(long)data->dest;
> >     } else {
> > -           idmabase0 = (u32)data->src;
> > +           idmabase0 = (u32)(long)data->src;
> >     }
> >
> >     /* Set the SDMMC DataLength value */
> > @@ -463,8 +463,8 @@ retry_cmd:
> >
> >     stm32_sdmmc2_start_cmd(dev, cmd, cmdat, &ctx);
> >
> > -   dev_dbg(dev, "send cmd %d data: 0x%x @ 0x%x\n",
> > -           cmd->cmdidx, data ? ctx.data_length : 0, (unsigned int)data);
> > +   dev_dbg(dev, "send cmd %d data: 0x%x @ 0x%p\n",
> > +           cmd->cmdidx, data ? ctx.data_length : 0, data);
> >
> >     ret = stm32_sdmmc2_end_cmd(dev, cmd, &ctx);
> >
> 
> 
> 
> Reviewed-by: Patrick Delaunay <patrick.delau...@foss.st.com>

Reviewed-by: Jaehoon Chung <jh80.ch...@samsung.com>

Best Regards,
Jaehoon Chung

> 
> Thanks
> Patrick


Reply via email to