Hi

Please use the real compatibles here. It's much easier to search
for. Plus, your driver doesn't support all the SoCs you're mentionning here.
[...]
Please provide the real property name to use. No need for an example
here, you have a full-fledged one in a few lines.

Fixed that.

Ditto. Plus, this is not a mod0 clock.

Yes it is! But maybe the formulation hasn't been clear enough...

You never talked about the clock-names property, and which clocks were
supposed to be provided.

Yes I did? But I expanded the text a little bit further...

quoted

+	interrupts = <0 32 4>;
+	bus-width = <4>;

And you never talked about bus-width either.

I can throw in a line for refering to the mmc slot gpio lib docs.

Isn't the cd-gpios property requested too?

I can refer to the docs there as well if you like... :-)

cheers
david