Re: [PATCH 3/5] wifi: rt2x00: silence sparse warnings

[Stanislaw Gruszka <stf_xl@wp.pl>]

On Fri, Feb 23, 2024 at 11:40:25AM +0100, Johannes Berg wrote:
> From: Johannes Berg <johannes.berg@intel.com>
> 
> Clearly writing __le32 was intended here, so just put the
> right casts into the code to silence sparse about it.
> 
> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Acked-by: Stanislaw Gruszka <stf_xl@wp.pl>

> ---
>  drivers/net/wireless/ralink/rt2x00/rt2800lib.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> index aaf31857ae1e..3bb81bcff0ac 100644
> --- a/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> +++ b/drivers/net/wireless/ralink/rt2x00/rt2800lib.c
> @@ -10946,13 +10946,13 @@ static void rt2800_efuse_read(struct rt2x00_dev *rt2x00dev, unsigned int i)
>  	/* Apparently the data is read from end to start */
>  	reg = rt2800_register_read_lock(rt2x00dev, efuse_data3_reg);
>  	/* The returned value is in CPU order, but eeprom is le */
> -	*(u32 *)&rt2x00dev->eeprom[i] = cpu_to_le32(reg);
> +	*(__le32 *)&rt2x00dev->eeprom[i] = cpu_to_le32(reg);
>  	reg = rt2800_register_read_lock(rt2x00dev, efuse_data2_reg);
> -	*(u32 *)&rt2x00dev->eeprom[i + 2] = cpu_to_le32(reg);
> +	*(__le32 *)&rt2x00dev->eeprom[i + 2] = cpu_to_le32(reg);
>  	reg = rt2800_register_read_lock(rt2x00dev, efuse_data1_reg);
> -	*(u32 *)&rt2x00dev->eeprom[i + 4] = cpu_to_le32(reg);
> +	*(__le32 *)&rt2x00dev->eeprom[i + 4] = cpu_to_le32(reg);
>  	reg = rt2800_register_read_lock(rt2x00dev, efuse_data0_reg);
> -	*(u32 *)&rt2x00dev->eeprom[i + 6] = cpu_to_le32(reg);
> +	*(__le32 *)&rt2x00dev->eeprom[i + 6] = cpu_to_le32(reg);
>  
>  	mutex_unlock(&rt2x00dev->csr_mutex);
>  }