regmap bug fix
Simon Glass
sjg at chromium.org
Thu Mar 25 01:38:13 CET 2021
Hi Marek,
On Wed, 17 Mar 2021 at 04:19, Marek Behun <marek.behun at nic.cz> wrote:
>
> Simon, Heiko, Bin,
>
> Pratyush discovered that the solution implemented by the patch
> regmap: fix a serious pointer casting bug
> is wrong. The cpu_to_le32() / le32_to_cpu() shifts data to the correct
> position, but on big endian machines it also reverses byte order.
> Somehow this went right through my head when I thought this up.
>
> I have sent a new version, with subject
> [PATCH u-boot v3.1 01/39] regmap: fix a serious pointer casting bug
>
> The new solution utilizes an union { u8; u16; u32; u64; }, since all
> members of an union start at the same address.
>
> Could you please review this? Thanks.
You already have my review tag on that.
I dropped it from u-boot-dm/next. The problem was that it v1 was in my
queue, but not later versions.
Regards,
Simon
More information about the U-Boot
mailing list