Messages in this thread |  | | Date | Mon, 16 Dec 2013 07:21:11 +0100 | From | Oliver Schinagl <> | Subject | Re: [PATCH 2/3] ARM: sunxi: Add an ahci-platform compatible AHCI driver for the Allwinner SUNXi series of SoCs |
| |
On 15-12-13 20:04, Tejun Heo wrote: > Hello, Hans. > > On Sun, Dec 15, 2013 at 08:00:20PM +0100, Hans de Goede wrote: >> I think it would be a good idea to merge ahci upstream using the >> ahci_imx.c method for now. You already indicated that you were not against >> doing that for now. > Well, the thing is nothing actually happened since ahci_imx got > merged, so I'm wondering maybe there isn't enough pressure. I a slowly progressing on the improved platform driver, but still waiting on an answer about my clock question that I asked a few days ago. > >> Oliver is working on getting a cleaner solution for this, but doing this >> properly takes tinme, and we would like to move forward with getting >> sunxi ahci support upstream in the mean time. >> >> So if it is ok with you we would like to move forward with the version >> initially posted. Therefor I would like to request you to review it >> (glossing over the platform device instantiating a platform device >> approach for now), and provide us with feedback so that we can do a v2 >> and start moving this towards the mainline kernel. > We still have some time left before the next merge window and if it's > really necessary, I can submit drivers post -rc1 too, so I'd still > *much* prefer doing it properly rather than creating more drivers > which would need to be cleaned up later. Since I do this in my spare time and my technical skill isn't as advanced as some, I don't think I have something reviewed and merge-able before that time frame. I do enjoy the challenge and do like working on the new ahci_platform framework. and do wish to continue to work on it however.
Oliver > > Thanks. >
|  |