> > > + compatible = "marvell,mvebu-sata-phy"; > > > + reg = <0x84000 0x0334>; > > > + clocks = <&gate_clk 15>; > > > + clock-names = "sata"; > > > + #phy-cells = <1>; > > > > Is it on purpose that your are having phy-cells value to 1? > > Yes. Each instance only controls one phy. Ah, sorry, got that wrong. As you say in a later comment, this should be 0. Will be fixed in v2. Thanks Andrew -- 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