From mboxrd@z Thu Jan 1 00:00:00 1970 From: Liviu.Dudau@arm.com (Liviu Dudau) Date: Mon, 19 Oct 2015 16:23:46 +0100 Subject: [RFC PATCH v3 4/4] drm/armada: Convert the probe function to the generic drm_of_component_probe() In-Reply-To: <20151019151714.GI32532@n2100.arm.linux.org.uk> References: <1445267270-23126-1-git-send-email-Liviu.Dudau@arm.com> <1445267270-23126-5-git-send-email-Liviu.Dudau@arm.com> <20151019151714.GI32532@n2100.arm.linux.org.uk> Message-ID: <20151019152345.GX3394@e106497-lin.cambridge.arm.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Mon, Oct 19, 2015 at 04:17:14PM +0100, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 04:07:50PM +0100, Liviu Dudau wrote: > > The armada DRM driver keeps some old platform data compatibility in the > > probe function that makes moving to the generic drm_of_component_probe() > > a bit more complicated that it should. Refactor the probe function to do > > the platform_data processing after the generic probe (and only if that > > fails). This way future cleanup can further remove support for it. > > I'm mainly using the old code, even with DT based platforms, because it > needs a block of reserved memory which I haven't got around to converting > to DT stuff (each time I've looked into doing that, bits for dealing with > reserved memory nodes were missing.) Thanks for explaining this! > > Acked-by: Russell King And thanks for all the ACKs! Best regards, Liviu > > -- > FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up > according to speedtest.net. > -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ?\_(?)_/?