[PATCH 5/5] DRM: Armada: add support for drm tda19988 driver
robdclark at gmail.com
Thu Oct 17 20:20:45 EDT 2013
On Tue, Oct 8, 2013 at 11:34 AM, Jean-Francois Moine <moinejf at free.fr> wrote:
> On Tue, 8 Oct 2013 10:49:39 +0100
> Russell King - ARM Linux <linux at arm.linux.org.uk> wrote:
>> On Tue, Oct 08, 2013 at 11:19:13AM +0200, Jean-Francois Moine wrote:
>> > The Cubox is an open platform, and I use it just like a desktop PC.
>> > When its required drivers will be in the mainline, I will do the same
>> > as I do with PCs: I will not recompile a specific kernel each time
>> > there are kernel bugs or security issues. Instead, I will just upgrade
>> > my system from my distributor (Debian), and, in the packages, there
>> > will be a generic mvebu kernel as there is already one for Marvell
>> > Armada 370/xp, Freescale iMX5x/iMX6 (linux-image-3.10-3-armmp).
>> > But, for that, all the Cubox specific stuff must be described in a DT.
>> Which scenario is better:
>> 1. To have something in mainline which is capable of driving the hardware,
>> but may need some additional work to make it useful for DT based setups.
>> 2. To have nothing.
>> Now, you may prefer to have nothing, but personally, I prefer there to be
>> forward progress. Forward progress means getting some kind of DRM driver
>> into mainline. Yes, it may not work with DT setups yet, but - as per
>> the discussions that have happened _endlessly_ on this topic, it's
>> something that can be resolved at a later date.
>> We _still_ haven't worked out how to properly deal with the TDA998x
>> driver (or indeed any DRM based outputs) in a DT based setup, and all
>> the time that problem exists, it won't be possible to write a proper
>> stable DT binding for this.
>> So please, get off your hobby horse about this and allow us to make some
>> modicum of progress.
> You forgot:
> 3. To have all patches ready for submission and have a working DT driven
> Cubox kernel.
No need to block this patch on DT support for all platforms that can
use this driver. I don't see an issue with removing non-DT support
later if it is no longer needed, or adding DT support in a future
patch for platforms which can use it. So this patch looks fine to me.
Signed-off-by: Rob Clark <robdclark at gmail.com
> I will submit a patch to add DT to the tda998x driver as soon as I have
> checked the new audio properties we talked about yesterday.
> Normally, this should have no impact on your Armada drm driver, and,
> yes, I will add DT to your driver as soon as it will be accepted
> (sorry to not ack it now: I had no time yet to have a look at it).
> Ken ar c'hentañ | ** Breizh ha Linux atav! **
> Jef | http://moinejf.free.fr/
More information about the linux-arm-kernel