On Mon, 23 Sep 2024, Bernhard Beschow wrote:
Am 23. September 2024 10:43:19 UTC schrieb BALATON Zoltan <balaton@eik.bme.hu>:
On Mon, 23 Sep 2024, Bernhard Beschow wrote:
Populate this read-only register with some arbitrary values which avoids
U-Boot's get_clocks() to hang().
Maybe this should be a property settable by the machine as each board may have
different values and it may need to use the correct value for the machine.
I actually considered this but went with the pragmatic solution to avoid
over-engineering. In particular, I wanted to avoid further machine-specitic
attributes in the machine class struct. Or do you expect a new e500 machine to
be added? In that case I'd set above arbitrary values as default and expect a
new machine to override these properties.
Can't override if there's no property for it. There's one machine I may be
interested in that uses a Freescale e500 SoC. That one seems to use 0x0606180c
for this value which I think corresponds to 0/1 Ratio both 3:1, DDR Ratio 12:1
and Plat Ratio 6:1. I think one property to set the 32 bit value without
individual fields would be enough and we can put comments next to the value if
needed to note what components it comes from. Or if you just need any value
here maybe you could take this one then that would be good for me as well.