From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH 2/3] ARM: sunxi: Add an ahci-platform compatible AHCI driver for the Allwinner SUNXi series of SoCs Date: Sun, 15 Dec 2013 14:04:54 -0500 Message-ID: <20131215190454.GA32509@htj.dyndns.org> References: <1386159055-10264-1-git-send-email-oliver@schinagl.nl> <1386159055-10264-3-git-send-email-oliver@schinagl.nl> <20131204123708.GD3158@htj.dyndns.org> <529F2677.3070208@schinagl.nl> <20131204131402.GG3158@htj.dyndns.org> <529F2B41.8090009@schinagl.nl> <20131204132312.GH3158@htj.dyndns.org> <20131206100117.59609a8a@skate> <52A19502.1090409@schinagl.nl> <52ADFC44.7040003@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <52ADFC44.7040003@redhat.com> Sender: linux-ide-owner@vger.kernel.org To: Hans de Goede Cc: Oliver Schinagl , Thomas Petazzoni , Olliver Schinagl , grant.likely@linaro.org, "rob.herring@calxeda.com" , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, dev@linux-sunxi.org, maxime.ripard@free-electrons.com, ijc@hellion.org.uk, Richard Zhu , Shawn Guo List-Id: devicetree@vger.kernel.org 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. > 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. Thanks. -- tejun