From mboxrd@z Thu Jan 1 00:00:00 1970 From: moinejf@free.fr (Jean-Francois Moine) Date: Tue, 8 Oct 2013 17:34:18 +0200 Subject: [PATCH 5/5] DRM: Armada: add support for drm tda19988 driver In-Reply-To: <20131008094939.GD25034@n2100.arm.linux.org.uk> References: <20131006220728.GG12758@n2100.arm.linux.org.uk> <20131007111807.5e86ea6e@armhf> <20131007094404.GI12758@n2100.arm.linux.org.uk> <20131007124820.2189a4c3@armhf> <20131007110902.GL12758@n2100.arm.linux.org.uk> <20131008111913.09bfe93e@armhf> <20131008094939.GD25034@n2100.arm.linux.org.uk> Message-ID: <20131008173418.0fc21d60@armhf> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, 8 Oct 2013 10:49:39 +0100 Russell King - ARM Linux 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. > > or > > 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. 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/ From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean-Francois Moine Subject: Re: [PATCH 5/5] DRM: Armada: add support for drm tda19988 driver Date: Tue, 8 Oct 2013 17:34:18 +0200 Message-ID: <20131008173418.0fc21d60@armhf> References: <20131006220728.GG12758@n2100.arm.linux.org.uk> <20131007111807.5e86ea6e@armhf> <20131007094404.GI12758@n2100.arm.linux.org.uk> <20131007124820.2189a4c3@armhf> <20131007110902.GL12758@n2100.arm.linux.org.uk> <20131008111913.09bfe93e@armhf> <20131008094939.GD25034@n2100.arm.linux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from smtp3-g21.free.fr (smtp3-g21.free.fr [212.27.42.3]) by gabe.freedesktop.org (Postfix) with ESMTP id 5E5E8E5CC6 for ; Tue, 8 Oct 2013 08:30:52 -0700 (PDT) In-Reply-To: <20131008094939.GD25034@n2100.arm.linux.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Russell King - ARM Linux Cc: Jason Cooper , "dri-devel@lists.freedesktop.org" , "linux-arm-kernel@lists.infradead.org" , Sebastian Hesselbarth List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCA4IE9jdCAyMDEzIDEwOjQ5OjM5ICswMTAwClJ1c3NlbGwgS2luZyAtIEFSTSBMaW51 eCA8bGludXhAYXJtLmxpbnV4Lm9yZy51az4gd3JvdGU6Cgo+IE9uIFR1ZSwgT2N0IDA4LCAyMDEz IGF0IDExOjE5OjEzQU0gKzAyMDAsIEplYW4tRnJhbmNvaXMgTW9pbmUgd3JvdGU6Cj4gPiBUaGUg Q3Vib3ggaXMgYW4gb3BlbiBwbGF0Zm9ybSwgYW5kIEkgdXNlIGl0IGp1c3QgbGlrZSBhIGRlc2t0 b3AgUEMuCj4gPiBXaGVuIGl0cyByZXF1aXJlZCBkcml2ZXJzIHdpbGwgYmUgaW4gdGhlIG1haW5s aW5lLCBJIHdpbGwgZG8gdGhlIHNhbWUKPiA+IGFzIEkgZG8gd2l0aCBQQ3M6IEkgd2lsbCBub3Qg cmVjb21waWxlIGEgc3BlY2lmaWMga2VybmVsIGVhY2ggdGltZQo+ID4gdGhlcmUgYXJlIGtlcm5l bCBidWdzIG9yIHNlY3VyaXR5IGlzc3Vlcy4gSW5zdGVhZCwgSSB3aWxsIGp1c3QgdXBncmFkZQo+ ID4gbXkgc3lzdGVtIGZyb20gbXkgZGlzdHJpYnV0b3IgKERlYmlhbiksIGFuZCwgaW4gdGhlIHBh Y2thZ2VzLCB0aGVyZQo+ID4gd2lsbCBiZSBhIGdlbmVyaWMgbXZlYnUga2VybmVsIGFzIHRoZXJl IGlzIGFscmVhZHkgb25lIGZvciBNYXJ2ZWxsCj4gPiBBcm1hZGEgMzcwL3hwLCBGcmVlc2NhbGUg aU1YNXgvaU1YNiAobGludXgtaW1hZ2UtMy4xMC0zLWFybW1wKS4KPiA+IEJ1dCwgZm9yIHRoYXQs IGFsbCB0aGUgQ3Vib3ggc3BlY2lmaWMgc3R1ZmYgbXVzdCBiZSBkZXNjcmliZWQgaW4gYSBEVC4K PiAKPiBXaGljaCBzY2VuYXJpbyBpcyBiZXR0ZXI6Cj4gCj4gMS4gVG8gaGF2ZSBzb21ldGhpbmcg aW4gbWFpbmxpbmUgd2hpY2ggaXMgY2FwYWJsZSBvZiBkcml2aW5nIHRoZSBoYXJkd2FyZSwKPiAg ICBidXQgbWF5IG5lZWQgc29tZSBhZGRpdGlvbmFsIHdvcmsgdG8gbWFrZSBpdCB1c2VmdWwgZm9y IERUIGJhc2VkIHNldHVwcy4KPiAKPiBvcgo+IAo+IDIuIFRvIGhhdmUgbm90aGluZy4KPiAKPiBO b3csIHlvdSBtYXkgcHJlZmVyIHRvIGhhdmUgbm90aGluZywgYnV0IHBlcnNvbmFsbHksIEkgcHJl ZmVyIHRoZXJlIHRvIGJlCj4gZm9yd2FyZCBwcm9ncmVzcy4gIEZvcndhcmQgcHJvZ3Jlc3MgbWVh bnMgZ2V0dGluZyBzb21lIGtpbmQgb2YgRFJNIGRyaXZlcgo+IGludG8gbWFpbmxpbmUuICBZZXMs IGl0IG1heSBub3Qgd29yayB3aXRoIERUIHNldHVwcyB5ZXQsIGJ1dCAtIGFzIHBlcgo+IHRoZSBk aXNjdXNzaW9ucyB0aGF0IGhhdmUgaGFwcGVuZWQgX2VuZGxlc3NseV8gb24gdGhpcyB0b3BpYywg aXQncwo+IHNvbWV0aGluZyB0aGF0IGNhbiBiZSByZXNvbHZlZCBhdCBhIGxhdGVyIGRhdGUuCj4g Cj4gV2UgX3N0aWxsXyBoYXZlbid0IHdvcmtlZCBvdXQgaG93IHRvIHByb3Blcmx5IGRlYWwgd2l0 aCB0aGUgVERBOTk4eAo+IGRyaXZlciAob3IgaW5kZWVkIGFueSBEUk0gYmFzZWQgb3V0cHV0cykg aW4gYSBEVCBiYXNlZCBzZXR1cCwgYW5kIGFsbAo+IHRoZSB0aW1lIHRoYXQgcHJvYmxlbSBleGlz dHMsIGl0IHdvbid0IGJlIHBvc3NpYmxlIHRvIHdyaXRlIGEgcHJvcGVyCj4gc3RhYmxlIERUIGJp bmRpbmcgZm9yIHRoaXMuCj4gCj4gU28gcGxlYXNlLCBnZXQgb2ZmIHlvdXIgaG9iYnkgaG9yc2Ug YWJvdXQgdGhpcyBhbmQgYWxsb3cgdXMgdG8gbWFrZSBzb21lCj4gbW9kaWN1bSBvZiBwcm9ncmVz cy4KCllvdSBmb3Jnb3Q6CgozLiBUbyBoYXZlIGFsbCBwYXRjaGVzIHJlYWR5IGZvciBzdWJtaXNz aW9uIGFuZCBoYXZlIGEgd29ya2luZyBEVCBkcml2ZW4KICAgQ3Vib3gga2VybmVsLgoKSSB3aWxs IHN1Ym1pdCBhIHBhdGNoIHRvIGFkZCBEVCB0byB0aGUgdGRhOTk4eCBkcml2ZXIgYXMgc29vbiBh cyBJIGhhdmUKY2hlY2tlZCB0aGUgbmV3IGF1ZGlvIHByb3BlcnRpZXMgd2UgdGFsa2VkIGFib3V0 IHllc3RlcmRheS4KCk5vcm1hbGx5LCB0aGlzIHNob3VsZCBoYXZlIG5vIGltcGFjdCBvbiB5b3Vy IEFybWFkYSBkcm0gZHJpdmVyLCBhbmQsCnllcywgSSB3aWxsIGFkZCBEVCB0byB5b3VyIGRyaXZl ciBhcyBzb29uIGFzIGl0IHdpbGwgYmUgYWNjZXB0ZWQKKHNvcnJ5IHRvIG5vdCBhY2sgaXQgbm93 OiBJIGhhZCBubyB0aW1lIHlldCB0byBoYXZlIGEgbG9vayBhdCBpdCkuCgotLSAKS2VuIGFyIGMn aGVudGHDsQl8CSAgICAgICoqIEJyZWl6aCBoYSBMaW51eCBhdGF2ISAqKgpKZWYJCXwJCWh0dHA6 Ly9tb2luZWpmLmZyZWUuZnIvCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fCmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNr dG9wLm9yZwpodHRwOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJp LWRldmVsCg==