diff mbox series

mmc: bcm283x: fix int to pointer cast

Message ID 20200515101426.5330-1-sw0312.kim@samsung.com
State Accepted
Commit 221c5e42a68e44916af5bacb8a8008ec7ef455d5
Headers show
Series mmc: bcm283x: fix int to pointer cast | expand

Commit Message

Seung-Woo Kim May 15, 2020, 10:14 a.m. UTC
On build with 32 bit, there is a warning for int-to-pointer-cast.
Fix the int to pointer cast by using uintptr_t.

Signed-off-by: Seung-Woo Kim <sw0312.kim at samsung.com>
---
 drivers/mmc/bcm2835_sdhci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jaehoon Chung May 15, 2020, 11:07 a.m. UTC | #1
On 5/15/20 7:14 PM, Seung-Woo Kim wrote:
> On build with 32 bit, there is a warning for int-to-pointer-cast.
> Fix the int to pointer cast by using uintptr_t.

Could you share in more detail? I didn't see any warning with gcc 9.2 / 8.3 / 7.4 /6.5.


Best Regards,
Jaehoon Chung


> 
> Signed-off-by: Seung-Woo Kim <sw0312.kim at samsung.com>
> ---
>  drivers/mmc/bcm2835_sdhci.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/mmc/bcm2835_sdhci.c b/drivers/mmc/bcm2835_sdhci.c
> index 39c93db275..c31f099f25 100644
> --- a/drivers/mmc/bcm2835_sdhci.c
> +++ b/drivers/mmc/bcm2835_sdhci.c
> @@ -209,7 +209,7 @@ static int bcm2835_sdhci_probe(struct udevice *dev)
>  	priv->last_write = 0;
>  
>  	host->name = dev->name;
> -	host->ioaddr = (void *)base;
> +	host->ioaddr = (void *)(uintptr_t)base;
>  	host->quirks = SDHCI_QUIRK_BROKEN_VOLTAGE | SDHCI_QUIRK_BROKEN_R1B |
>  		SDHCI_QUIRK_WAIT_SEND_CMD | SDHCI_QUIRK_NO_HISPD_BIT;
>  	host->max_clk = emmc_freq;
>
Seung-Woo Kim May 16, 2020, 2:57 a.m. UTC | #2
Hi Jaehoon,

On 2020? 05? 15? 20:07, Jaehoon Chung wrote:
> On 5/15/20 7:14 PM, Seung-Woo Kim wrote:
>> On build with 32 bit, there is a warning for int-to-pointer-cast.
>> Fix the int to pointer cast by using uintptr_t.
> 
> Could you share in more detail? I didn't see any warning with gcc 9.2 / 8.3 / 7.4 /6.5.

Sorry for not giving detail. If I applied PICe XHCI support [1], and
build rpi_4_32b, then I got related warning.

If [1] is not applied, then this can be ignored.

[1]
https://patchwork.ozlabs.org/project/uboot/patch/20200504124523.23484-7-s.nawrocki at samsung.com/

Best Regards,
- Seung-Woo Kim

> 
> 
> Best Regards,
> Jaehoon Chung
> 
> 
>>
>> Signed-off-by: Seung-Woo Kim <sw0312.kim at samsung.com>
>> ---
>>  drivers/mmc/bcm2835_sdhci.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/mmc/bcm2835_sdhci.c b/drivers/mmc/bcm2835_sdhci.c
>> index 39c93db275..c31f099f25 100644
>> --- a/drivers/mmc/bcm2835_sdhci.c
>> +++ b/drivers/mmc/bcm2835_sdhci.c
>> @@ -209,7 +209,7 @@ static int bcm2835_sdhci_probe(struct udevice *dev)
>>  	priv->last_write = 0;
>>  
>>  	host->name = dev->name;
>> -	host->ioaddr = (void *)base;
>> +	host->ioaddr = (void *)(uintptr_t)base;
>>  	host->quirks = SDHCI_QUIRK_BROKEN_VOLTAGE | SDHCI_QUIRK_BROKEN_R1B |
>>  		SDHCI_QUIRK_WAIT_SEND_CMD | SDHCI_QUIRK_NO_HISPD_BIT;
>>  	host->max_clk = emmc_freq;
>>
> 
> 
>
Matthias Brugger May 18, 2020, 7:24 a.m. UTC | #3
On 16/05/2020 04:57, Seung-Woo Kim wrote:
> Hi Jaehoon,
> 
> On 2020? 05? 15? 20:07, Jaehoon Chung wrote:
>> On 5/15/20 7:14 PM, Seung-Woo Kim wrote:
>>> On build with 32 bit, there is a warning for int-to-pointer-cast.
>>> Fix the int to pointer cast by using uintptr_t.
>>
>> Could you share in more detail? I didn't see any warning with gcc 9.2 / 8.3 / 7.4 /6.5.
> 
> Sorry for not giving detail. If I applied PICe XHCI support [1], and
> build rpi_4_32b, then I got related warning.
> 
> If [1] is not applied, then this can be ignored.

This is actually discussed in a new series [1]. If you have the same finding
there then we should include your patch to the series.

Please check and follow up there if necessary.

Regards,
Matthias

[1] https://www.mail-archive.com/u-boot at lists.denx.de/msg369283.html

> 
> [1]
> https://patchwork.ozlabs.org/project/uboot/patch/20200504124523.23484-7-s.nawrocki at samsung.com/
> 
> Best Regards,
> - Seung-Woo Kim
> 
>>
>>
>> Best Regards,
>> Jaehoon Chung
>>
>>
>>>
>>> Signed-off-by: Seung-Woo Kim <sw0312.kim at samsung.com>
>>> ---
>>>  drivers/mmc/bcm2835_sdhci.c | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/drivers/mmc/bcm2835_sdhci.c b/drivers/mmc/bcm2835_sdhci.c
>>> index 39c93db275..c31f099f25 100644
>>> --- a/drivers/mmc/bcm2835_sdhci.c
>>> +++ b/drivers/mmc/bcm2835_sdhci.c
>>> @@ -209,7 +209,7 @@ static int bcm2835_sdhci_probe(struct udevice *dev)
>>>  	priv->last_write = 0;
>>>  
>>>  	host->name = dev->name;
>>> -	host->ioaddr = (void *)base;
>>> +	host->ioaddr = (void *)(uintptr_t)base;
>>>  	host->quirks = SDHCI_QUIRK_BROKEN_VOLTAGE | SDHCI_QUIRK_BROKEN_R1B |
>>>  		SDHCI_QUIRK_WAIT_SEND_CMD | SDHCI_QUIRK_NO_HISPD_BIT;
>>>  	host->max_clk = emmc_freq;
>>>
>>
>>
>>
>
diff mbox series

Patch

diff --git a/drivers/mmc/bcm2835_sdhci.c b/drivers/mmc/bcm2835_sdhci.c
index 39c93db275..c31f099f25 100644
--- a/drivers/mmc/bcm2835_sdhci.c
+++ b/drivers/mmc/bcm2835_sdhci.c
@@ -209,7 +209,7 @@  static int bcm2835_sdhci_probe(struct udevice *dev)
 	priv->last_write = 0;
 
 	host->name = dev->name;
-	host->ioaddr = (void *)base;
+	host->ioaddr = (void *)(uintptr_t)base;
 	host->quirks = SDHCI_QUIRK_BROKEN_VOLTAGE | SDHCI_QUIRK_BROKEN_R1B |
 		SDHCI_QUIRK_WAIT_SEND_CMD | SDHCI_QUIRK_NO_HISPD_BIT;
 	host->max_clk = emmc_freq;