Re: hw/misc/aspeed_scu: 5d971f9e breaks Supermicro AST2400

[Erik Smit]

Hi Philippe,

On Tue, 30 Jun 2020 at 17:29, Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> Hi Erik,
>
> On 6/30/20 5:11 PM, Erik Smit wrote:
> > Hello,
> >
> > 5d971f9e memory: Revert "memory: accept mismatching sizes in
> > memory_region_access_valid" breaks Supermicro AST2400 u-boot.
> > Supermicro AST2500 u-boot is fine.
> >
> > The u-boot tries to make a 2-byte read from address 0x84, but
> > aspeed_ast2400_scu_ops has min_access = 4. When I change min_access to
> > 2 or revert above commit, u-boot boots again.
> >
> > Is changing min_access to 2 the right way to fix this?
>
> If you have access to the datasheet and can verify, then yes.
> Else I suppose Cédric, Andrew or Joel can check for you.

I do not have a datasheet. Aspeed seems quite picky about sharing this
and I'm just a random researcher.

Best regards,

Erik Smit