On Mon, Jul 01, 2013 at 02:44:45PM +0200, Sascha Hauer wrote: > On Mon, Jul 01, 2013 at 06:02:55PM +0800, Richard Zhu wrote: > > imx6q contains one Synopsys AHCI SATA controller, > > But it can't shares ahci_platform driver with other > > controllers. > > Because there are some misalignments of the bits > > definitions of the HBA registers and the Vendor > > Specific registers > > - CAP_SSS(bit20) of the HOST_CAP is writable, default > > value is '0', should be configured to be '1' > > - bit0 (only one AHCI SATA port on imx6q) of the > > HOST_PORTS_IMPL should be set to be '1'.(default 0) > > - One Vendor Specific register HOST_TIMER1MS(offset:0xe0) > > should be configured regarding to the frequency of AHB > > bus clock. > > > > Setup its own ahci sata driver, enable the imx6q ahci > > sata support, and update the ahci sata binding document. > > > > Signed-off-by: Richard Zhu <r65037@xxxxxxxxxxxxx> > > Wait a minute. We suggested to add a i.MX specific ahci driver to put > the i.MX speicific setup in there. Now you really post a separate i.MX > driver, but instead of putting the setup in there, it's a nearly > identical copy of the generic driver *without* the setup and the setup > is still in arch/arm/. That doesn't make sense to me. +1 Richard, You could have confused by me with pointing you to the sata_highbank driver. Sorry. But Sascha posted his work [1] later in the same thread. That's obviously the way we should go, and I thought you knew you should start your v2 based on that. You missed that? Shawn [1] http://thread.gmane.org/gmane.linux.ide/54410/focus=54434 -- 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
