It seems to work fine on pine64-lts.
So when my comments are resolved I think you could commit this (with the patch I sent you for arm64).

The Allwinner part seems ok.

This looks okay for me except one comment.
But due to the fact that I/We had a lot of problems with different revision of the hardware IP let me test on different SoC before commiting.
Also I won't set this review as accepted as I don't know anything about CAM.

In D15884#336074, @imp wrote:

These all look good. Maybe consider MAINTAINER?= uboot@freebsd.org in the master file?

Well in fact just commit it, it's better to have a few broken clocks that nothing at all.
I accept the review but do not add me to reviewed-by as I haven't read the clock definition.

In D15810#334278, @kevans wrote:

In D15810#334274, @manu wrote:

I prefer spliting the A10 and A20 right now even if for now they are the same.

As in, duplicate all of these common clocks? There's only like five extra clocks that the a20 has, so I was planning on just doing something slightly different in ccu_a20_register_clocks after registering all of the common/a10 clocks.

I prefer spliting the A10 and A20 right now even if for now they are the same.

Looks fine

Why simply not add the file to the repo and copy it ?
Something like in share/example

s/all SDHCI/all MMC/

Ok for me on arm64, dump is in zstd format and compress ok (kgdb coredump but it does it too for standard dumps so the issue is elsewhere)

It seems that dump on arm64 currently doesn't work so I can't say if any of this options will be a benefit or not on this platform.

Not allwinner specific, hoping this will remove my blocking.

Ok for me, also tested on pine64