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

[Erik Smit]

Hi Joel,

On Wed, 1 Jul 2020 at 02:54, Joel Stanley <joel@jms.id.au> wrote:
>
> On Tue, 30 Jun 2020 at 15:32, Erik Smit <erik.lucas.smit@gmail.com> wrote:
> >
> > 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?
>
> Ryan, do all addresses on the AST2400's APB respond to single byte accesses?
>
> > >
> > > 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.
>
> Erik, can you point us to the image you're using?

SMT_X11_158.bin

You can get it on
https://www.supermicro.com/en/products/motherboard/X11SSL-F by
clicking on "BMC/IPMI Firmware" on the right.

> What is the command line are you testing with?

qemu-system-arm -machine palmetto-bmc -m 512 -drive
file=SMT_X11_158.bin,format=raw,if=mtd -nodefaults -nographic -serial
stdio

Best regards,

Erik Smit