On 01/04/14 22:47, Arnd Bergmann wrote:
On Saturday 04 January 2014 22:39:50 Arnd Bergmann wrote:
+Required properties:
+- compatible : compatible list, contains "allwinner,sun4i-a10-ahci"
+- reg : <registers mapping>
+- interrupts : <interrupt mapping for AHCI IRQ>
+- clocks : clocks for ACHI
+- clock-names : clock names for AHCI
The binding needs to specify the required names for the clocks.
fwiw, the imx driver uses "ahb" and "sata_ref" as the clock names.
I would strongly suggest using the same names here, and documenting
them in the ahci binding as optional.
This is my fault, and you just reminded me I should have fixed that from
the previous comments. It just slipped my mind and I'm sorry for that!
Hans, I'll go over the original commit thread and pick up all changes
and send them as a patch to you Monday.
Oliver
Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
