In D45321#1034234, @kib wrote:

In D45321#1034161, @markj wrote:

Logically, it doesn't make sense that this result would differ per-pmap.

...

While there's currently no difference between pmap types in the handling of memattrs, it's not obvious to me that that'll always be true. In other words, I don't really agree with the premise of the patch. Maybe I'm missing some reason that my concern is invalid.

There is already a PAT mode (UC-) which cannot be represented in EPT. Although the mode is a hack to fix broken sofware (I believe), it is still a demonstration of the issue.

LGTM

Thank you for finding the proper fix! Working fine on my end.

Thanks for this. Seems good to me, with one question inline.

LGTM. We will still have to wait until @manu has finished with the DTS upgrade to 6.10.

I am happy with this. There are a couple whitespace/style issues remaining, but I will do a pass to fix them before committing. Thanks for your work, I will aim to merge by the end of the week!

Add check for generic compatible string exported by current u-boot.

Taking this over so I can publish some additions/changes...

Overall this is looking good to me. The core clock logic seems right, as well as the SYS group. I did not verify the clock hierarchy in detail compared to the TRM.