On 6/30/20 5:32 PM, Erik Smit wrote: > Hi Philippe, > > On Tue, 30 Jun 2020 at 17:29, Philippe Mathieu-Daudé <[email protected]> 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 >
