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

[Philippe Mathieu-Daudé]

On 6/30/20 5:32 PM, Erik Smit 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?
>>
>> 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.

Neither I have them. The I suggest to open a bug on
https://bugs.launchpad.net/qemu/+filebug and we'll ask the Aspeed folks
to write a patch.

> 
> Best regards,
> 
> Erik Smit
>