Re: [PATCH] rt2x00: Fix efuse EEPROM reading on PPC32.

[Ivo Van Doorn <ivdoorn@xxxxxxxxx>]

On Wed, Nov 16, 2011 at 11:16 PM, Gertjan van Wingerde
<gwingerde@xxxxxxxxx> wrote:
> Fix __le32 to __le16 conversion of the first word of an 8-word block
> of EEPROM read via the efuse method.
>
> Reported-and-tested-by: Ingvar Hagelund <ingvar@xxxxxxxxxxxxxxxxxx>
> Signed-off-by: Gertjan van Wingerde <gwingerde@xxxxxxxxx>
> CC: <stable@xxxxxxxxxxxxxxx>

Acked-by: Ivo van Doorn <IvDoorn@xxxxxxxxx>

> ---
>  drivers/net/wireless/rt2x00/rt2800lib.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/drivers/net/wireless/rt2x00/rt2800lib.c b/drivers/net/wireless/rt2x00/rt2800lib.c
> index 3f183a1..1ba079d 100644
> --- a/drivers/net/wireless/rt2x00/rt2800lib.c
> +++ b/drivers/net/wireless/rt2x00/rt2800lib.c
> @@ -3771,7 +3771,7 @@ static void rt2800_efuse_read(struct rt2x00_dev *rt2x00dev, unsigned int i)
>        /* Apparently the data is read from end to start */
>        rt2800_register_read_lock(rt2x00dev, EFUSE_DATA3, &reg);
>        /* The returned value is in CPU order, but eeprom is le */
> -       rt2x00dev->eeprom[i] = cpu_to_le32(reg);
> +       *(u32 *)&rt2x00dev->eeprom[i] = cpu_to_le32(reg);
>        rt2800_register_read_lock(rt2x00dev, EFUSE_DATA2, &reg);
>        *(u32 *)&rt2x00dev->eeprom[i + 2] = cpu_to_le32(reg);
>        rt2800_register_read_lock(rt2x00dev, EFUSE_DATA1, &reg);
> --
> 1.7.7
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html