From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sean Paul Subject: Re: [PATCH v8 2/3] drm/panel: Add support for S6E3HA2 panel driver on TM2 board Date: Wed, 1 Feb 2017 14:03:32 -0500 Message-ID: <20170201190331.GB14264@art_vandelay> References: <08c5d94b-c76f-af14-c08f-478e26a34a7c@samsung.com> <588FD3C3.7080508@samsung.com> <20170131085449.GA19348@ulmo.ba.sec> <20170131143853.GU20076@art_vandelay> <20170131150226.GB4519@ulmo.ba.sec> <87r33j85ap.fsf@eliezer.anholt.net> <20170131213132.GC872@mithrandir.ba.sec> <87d1f2kfgi.fsf@eliezer.anholt.net> <20170201145249.GB17698@ulmo.ba.sec> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Emil Velikov Cc: devicetree , "moderated list:ARM/S5P EXYNOS AR..." , ML dri-devel , Donghwa Lee , "Linux-Kernel@Vger. Kernel. Org" , andi.shyti@samsung.com, jh80.chung@samsung.com, cw00.choi@samsung.com, Kukjin Kim , Hyungwon Hwang , Krzysztof Kozlowski , Hoegeun Kwon List-Id: linux-samsung-soc@vger.kernel.org T24gV2VkLCBGZWIgMDEsIDIwMTcgYXQgMDM6Mjk6NDBQTSArMDAwMCwgRW1pbCBWZWxpa292IHdy b3RlOgo+IE9uIDEgRmVicnVhcnkgMjAxNyBhdCAxNDo1MiwgVGhpZXJyeSBSZWRpbmcgPHRoaWVy cnkucmVkaW5nQGdtYWlsLmNvbT4gd3JvdGU6Cj4gPiBPbiBUdWUsIEphbiAzMSwgMjAxNyBhdCAw Mjo1NDo1M1BNIC0wODAwLCBFcmljIEFuaG9sdCB3cm90ZToKPiA+PiBUaGllcnJ5IFJlZGluZyA8 dGhpZXJyeS5yZWRpbmdAZ21haWwuY29tPiB3cml0ZXM6Cj4gPj4KPiA+PiA+IFsgVW5rbm93biBz aWduYXR1cmUgc3RhdHVzIF0KPiA+PiA+IE9uIFR1ZSwgSmFuIDMxLCAyMDE3IGF0IDEwOjE1OjEw QU0gLTA4MDAsIEVyaWMgQW5ob2x0IHdyb3RlOgo+ID4+ID4+IFRoaWVycnkgUmVkaW5nIDx0aGll cnJ5LnJlZGluZ0BnbWFpbC5jb20+IHdyaXRlczoKPiA+PiA+Pgo+ID4+ID4+ID4gWyBVbmtub3du IHNpZ25hdHVyZSBzdGF0dXMgXQo+ID4+ID4+ID4gT24gVHVlLCBKYW4gMzEsIDIwMTcgYXQgMDk6 Mzg6NTNBTSAtMDUwMCwgU2VhbiBQYXVsIHdyb3RlOgo+ID4+ID4+ID4+IE9uIFR1ZSwgSmFuIDMx LCAyMDE3IGF0IDA5OjU0OjQ5QU0gKzAxMDAsIFRoaWVycnkgUmVkaW5nIHdyb3RlOgo+ID4+ID4+ ID4+ID4gT24gVHVlLCBKYW4gMzEsIDIwMTcgYXQgMDk6MDE6MDdBTSArMDkwMCwgSW5raSBEYWUg d3JvdGU6Cj4gPj4gPj4gPj4gPiA+Cj4gPj4gPj4gPj4gPiA+Cj4gPj4gPj4gPj4gPiA+IDIwMTfr hYQgMDHsm5QgMjTsnbwgMTA6NTDsl5AgSG9lZ2V1biBLd29uIOydtCjqsIApIOyTtCDquIA6Cj4g Pj4gPj4gPj4gPiA+ID4gRGVhciBUaGllcnJ5LAo+ID4+ID4+ID4+ID4gPiA+Cj4gPj4gPj4gPj4g PiA+ID4gQ291bGQgeW91IHBsZWFzZSByZXZpZXcgdGhpcyBwYXRjaD8KPiA+PiA+PiA+PiA+ID4K PiA+PiA+PiA+PiA+ID4gVGhpZXJyeSwgSSB0aGluayB0aGlzIHBhdGNoIGhhcyBiZWVuIHJldmll d2VkIGVub3VnaCBidXQgbm8gY29tbWVudAo+ID4+ID4+ID4+ID4gPiBmcm9tIHlvdS4gU2VlbXMg eW91IGFyZSBidXN5LiBJIHdpbGwgcGljayB1cCB0aGlzLgo+ID4+ID4+ID4+ID4KPiA+PiA+PiA+ PiA+IFNvcnJ5LCBidXQgdGhhdCdzIG5vdCBob3cgaXQgd29ya3MuIFRoaXMgcGF0Y2ggaGFzIGdv bmUgdGhyb3VnaCA4Cj4gPj4gPj4gPj4gPiByZXZpc2lvbnMgd2l0aGluIDQgd2Vla3MsIGFuZCBJ IHRlbmQgdG8gaWdub3JlIHBhdGNoZXMgbGlrZSB0aGF0IHVudGlsCj4gPj4gPj4gPj4gPiB0aGUg ZHVzdCBzZXR0bGVzLgo+ID4+ID4+ID4+ID4KPiA+PiA+PiA+Pgo+ID4+ID4+ID4+IFNlZW1zIGxp a2UgdGhlIGR1c3Qgd2FzIHByZXR0eSBzZXR0bGVkLiBJdCB3YXMgcG9zdGVkIG9uIDEvMTEsIHBp bmdlZCBvbiAxLzI0LAo+ID4+ID4+ID4+IGFuZCBwaWNrZWQgdXAgb24gMS8zMS4gSSBkb24ndCB0 aGluayBpdCdzIHVucmVhc29uYWJsZSB0byB0YWtlIGl0IHRocm91Z2gKPiA+PiA+PiA+PiBhbm90 aGVyIHRyZWUgYWZ0ZXIgdGhhdC4KPiA+PiA+PiA+Pgo+ID4+ID4+ID4+IEkgd29uZGVyIGlmIGRy bV9wYW5lbCB3b3VsZCBiZW5lZml0IGZyb20gdGhlIC1taXNjIGdyb3VwIG1haW50YWluZXJzaGlw IG1vZGVsCj4gPj4gPj4gPj4gYXMgZHJtX2JyaWRnZSBkb2VzLiBCeSBzcHJlYWRpbmcgb3V0IHRo ZSB3b3JrbG9hZCwgdGhlIGhpZ2gtbWFpbnRlbmFuY2UKPiA+PiA+PiA+PiBwYXRjaGVzIHdvdWxk IGhvcGVmdWxseSBmaW5kIHNvbWVvbmUgdG8gc2hlcGhlcmQgdGhlbSB0aHJvdWdoLgo+ID4+ID4+ ID4KPiA+PiA+PiA+IEV4Y2VwdCB0aGF0IG5vYm9keSBleGNlcHQgbWUgcmVhbGx5IGNhcmVzLiBJ ZiB3ZSBsZXQgcGVvcGxlIHRha2UgcGF0Y2hlcwo+ID4+ID4+ID4gdGhyb3VnaCBzZXBhcmF0ZSB0 cmVlcyBvciBncm91cC1tYWludGFpbmVkIHRyZWVzIHRoZXknbGwgbGlrZWx5IGdvIGluCj4gPj4g Pj4gPiB3aXRob3V0IHRvbyBtdWNoIHRob3VnaHQuIERSTSBwYW5lbCBpcyBzb21ld2hhdCBkaWZm ZXJlbnQgZnJvbSBjb3JlIERSTQo+ID4+ID4+ID4gaW4gdGhpcyByZWdhcmQgYmVjYXVzZSBpdHMg aW5mcmFzdHJ1Y3R1cmUgaXMgbWluaW1hbCBhbmQgdGhlcmUncyBsaXR0bGUKPiA+PiA+PiA+IG91 dHNpZGUgdGhlIHBhbmVsLXNpbXBsZSBkcml2ZXIuIFNvIHdlJ3JlIHN0aWxsIGF0IGEgc3RhZ2Ug d2hlcmUgd2UgbmVlZAo+ID4+ID4+ID4gdG8gZmluZS10dW5lIHdoYXQgZHJpdmVycyBzaG91bGQg bG9vayBsaWtlIGFuZCBob3cgd2UgY2FuIGltcHJvdmUuCj4gPj4gPj4KPiA+PiA+PiBJIHdvdWxk IGxvdmUgdG8gY2FyZSBhbmQgcGFydGljaXBhdGUgaW4gcmV2aWV3LCBidXQgd2l0aCB0aGUgc3Ry dWN0dXJlCj4gPj4gPj4gb2YgeW91ciB0cmVlIHlvdSdyZSB0aGUgb25seSBvbmUgd2hvc2UgcmV2 aWV3IGNvdW50cywgc28gSSBkb24ndAo+ID4+ID4+IHBhcnRpY2lwYXRlLgo+ID4+ID4KPiA+PiA+ IFJlYWxseT8gV2hhdCBleGFjdGx5IGRvIHlvdSB0aGluayBpcyBzcGVjaWFsIGFib3V0IHRoZSBz dHJ1Y3R1cmUgb2YgbXkKPiA+PiA+IHRyZWU/IEkgcmVxdWlyZSBwYXRjaGVzIHRvIGJlIG9uIGRy aS1kZXZlbCAoSSBwaWNrIHRoZW0gdXAgZnJvbSB0aGUKPiA+PiA+IHBhdGNod29yayBpbnN0YW5j ZSBhdCBmcmVlZGVza3RvcC5vcmcpLCB0aGUgdHJlZSBpcyBwdWJsaWNseSBhdmFpbGFibGUKPiA+ PiA+IGFuZCByZXZpZXdlZC1ieSB0YWdzIGdldCBwaWNrZWQgdXAgYXV0b21hdGljYWxseSBieSBw YXRjaHdvcmsuCj4gPj4gPgo+ID4+ID4gVGhlIHBhbmVsIHRyZWUgd29ya3MgZXhhY3RseSBsaWtl IGFueSBvdGhlciBtYWludGFpbmVyIHRyZWUuIEFuZCBteQo+ID4+ID4gcmV2aWV3IGlzICpub3Qq IHRoZSBvbmx5IG9uZSB0aGF0IGNvdW50cy4gSSBhcHByZWNpYXRlIGV2ZXJ5IFJldmlld2VkLWJ5 Cj4gPj4gPiB0YWcgSSBzZWUgb24gcGFuZWwgcGF0Y2hlcyBiZWNhdXNlIGl0IG1lYW5zIHRoYXQg SSBkb24ndCBoYXZlIHRvIGxvb2sgYXMKPiA+PiA+IGNsb3NlbHkgYXMgSSBoYXZlIHRvIG90aGVy d2lzZS4KPiA+PiA+Cj4gPj4gPiBJdCBpcyB0cnVlIHRoYXQgSSBhbSByZXNwb25zaWJsZSBmb3Ig dGhvc2UgcGF0Y2hlcywgdGhhdCdzIHdoeSBJIGdldCB0bwo+ID4+ID4gaGF2ZSB0aGUgZmluYWwg d29yZCBvbiB3aGV0aGVyIG9yIG5vdCBhIHBhdGNoIGdldHMgYXBwbGllZC4gQW5kIHRoYXQncwo+ ID4+ID4gbm8gZGlmZmVyZW50IGZyb20gYW55IG90aGVyIG1haW50YWluZXIgdHJlZSBlaXRoZXIu Cj4gPj4KPiA+PiBJZiBtZSByZXZpZXdpbmcgYSBwYXRjaCBpc24ndCBwYXJ0IG9mIHVuYmxvY2tp bmcgdGhhdCBwYXRjaCBnZXR0aW5nIGluLAo+ID4+IHRoZW4gSSB3b24ndCBib3RoZXIgYmVjYXVz ZSBhbGwgSSBjb3VsZCBlbmQgdXAgZG9pbmcgaXMgcHVuaXNoaW5nIHRoZQo+ID4+IGRldmVsb3Bl ciBvZiB0aGUgcGF0Y2guICBDb250cmlidXRvcnMgaGF2ZSBhIGhhcmQgZW5vdWdoIHRpbWUgYWxy ZWFkeS4KPiA+Cj4gPiBNYXliZSB5b3Ugc2hvdWxkIGdvIGFuZCByZWFkIG15IHByZXZpb3VzIHJl cGx5IGFnYWluIG1vcmUgY2FyZWZ1bGx5Lgo+ID4gUGVyaGFwcyB0aGVuIHlvdSdsbCByZWFsaXpl IHRoYXQgcmV2aWV3cyBhcmUgaW4gZmFjdCBoZWxwaW5nIGluIGdldHRpbmcKPiA+IHBhdGNoZXMg bWVyZ2VkLgo+ID4KPiA+IEludGVyZXN0aW5nbHkgbXkgaW5ib3ggZG9lc24ndCBzaG93IHlvdSBl dmVyIGJvdGhlcmluZyB0byByZXZpZXcgcGFuZWwKPiA+IHBhdGNoZXMsIHNvIG1heWJlIHlvdSBz aG91bGQgYmUgbW9yZSBjYXJlZnVsIGFib3V0IHlvdXIgYXNzdW1wdGlvbnMuCj4gPgo+IEdlbnRz LCBpdCdzIHVuZGVyc3RhbmRhYmxlIHRoYXQgZW1vdGlvbnMgbWlnaHQgYmUgcnVubmluZyBoaWdo Lgo+IAo+IFdoYXQncyB0aGUgcG9pbnQgaW4gcG9pbnRpbmcgZmluZ2VycyBhdCBlYWNoIG90aGVy IC0gdGhlcmUgaXMgZW5vdWdoCj4gdG8gZ28gaW4gZWFjaCBkaXJlY3Rpb24uCj4gTGV0IHVzIGFs bCBzdGVwIGJhY2sgZm9yIGEgc2Vjb25kIGFuZCBjb25zaWRlciBob3cgd2UgY2FuIG1ha2UgdGhp bmdzIGJldHRlci4KPiAKClNlZW1zIGxpa2UgSSBraWNrZWQgdXAgc29tZSBkdXN0IGhlcmUsIGZv ciB0aGF0IEkgYXBvbG9naXplLiBJIGNlcnRhaW5seSBkaWQgbm90CmludGVuZCB0byBkaW1pbmlz aCBUaGllcnJ5J3MgKG9yIGFueW9uZSBlbHNlJ3MpIHJvbGUgYXMgbWFpbnRhaW5lci4KClRvIHB1 dCB0aGlzIGFzIGNvbmNpc2VseSBhcyBJIGNhbiwgSSB0aG91Z2h0IGRybV9wYW5lbCB3b3VsZCBi ZSBhIGdvb2QgY2FuZGlkYXRlCmZvciAtbWlzYyBnaXZlbjoKICAgICAgICAtIGRybV9icmlkZ2Ug aXMgYWxyZWFkeSBtYWludGFpbmVkIHRoZXJlCiAgICAgICAgLSB0aGUgZHJpdmVycyBhcmUgc21h bGwsIGFuZCB3ZSBqdXN0IHJlc29sdmVkIHRvIG1haW50YWluIHNtYWxsIGRyaXZlcnMKICAgICAg ICAgIGluIC1taXNjCiAgICAgICAgLSBuZXcgcGF0Y2hlcyBhcmUgYmxvY2tlZCBvbiBhIHNpbmds ZSByZXZpZXdlci9jb21taXR0ZXIgYXMgb3Bwb3NlZCB0byBhCiAgICAgICAgICBxdWFsaWZpZWQg Y29tbWl0dGVlICh3aGljaCBJIGhhdmUgY29tZSB0byB1bmRlcnN0YW5kIGlzIGEgZmVhdHVyZSBp bgogICAgICAgICAgdGhpcyBpbnN0YW5jZSkKClNvIGlmIHdlIGNhbid0IG1pZ3JhdGUgaXQgdG8g LW1pc2Mgbm93LCBmb3IgZmVhciBvZiBxdWFsaXR5IGlzc3Vlcywgd2hhdCBhcmUgdGhlCnN0ZXBz IG5lY2Vzc2FyeSB0byAiZGUtc3RhZ2UiIGl0PwoKU2VhbgoKCj4gSSB0aGluayBpdCdsbCBiZSBu aWNlIHRvIGhhdmUgc29tZS9tb3N0IG9mIHRoZSBjb21tb24gY29uY2VybnMgdGhhdAo+IFRoaWVy cnkvb3RoZXJzIGNvbWVzIGFjcm9zcyBkb2N1bWVudGVkIC0gaW4ta2VybmVsLCBibG9nIHBvc3Qs IG90aGVyLgo+IFN1Y2ggdGhhdCBvbmUgY2FuIHJlZmVyZW5jZSB0byBzcGVjaWZpYyBwb2ludHMg YXMgcGF0Y2ggZmFsbHMgc3ViLXBhci4KPiBXZSBhbGwgd2FudCB0byBoYXZlIGEgYmFsYW5jZSBv ZiBuaWNlbHkgd3JpdHRlbiBkcml2ZXIgYW5kIHF1aWNrCj4gbWVyZ2UuCj4gCj4gSW5raSwgSSBi ZWxpZXZlIG15c2VsZiBhbmQgb3RoZXJzIGhhdmUgaW52aXRlZCB5b3UgYmVmb3JlIG9uCj4gI2Ry aS1kZXZlbC4gVGhpcyBpcyBhbm90aGVyIG1lZGl1bSB3aGVyZSB5b3UgY2FuIHBva2UgZGV2cyBh bmQgZnJvbSBteQo+IGV4cGVyaWVuY2UgLSBpdCB0ZW5kcyB0byBiZSBtb3JlIGVmZmljaWVudCwg bW9zdCBvZiB0aGUgdGltZS4KPiAKPiBUaGFua3MKPiBFbWlsCj4gX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX18KPiBkcmktZGV2ZWwgbWFpbGluZyBsaXN0Cj4g ZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwo+IGh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0 b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCgotLSAKU2VhbiBQYXVsLCBTb2Z0d2Fy ZSBFbmdpbmVlciwgR29vZ2xlIC8gQ2hyb21pdW0gT1MKX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2ZWxA bGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21haWxt YW4vbGlzdGluZm8vZHJpLWRldmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751349AbdBATDi (ORCPT ); Wed, 1 Feb 2017 14:03:38 -0500 Received: from mail-yw0-f171.google.com ([209.85.161.171]:34876 "EHLO mail-yw0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750885AbdBATDf (ORCPT ); Wed, 1 Feb 2017 14:03:35 -0500 Date: Wed, 1 Feb 2017 14:03:32 -0500 From: Sean Paul To: Emil Velikov Cc: Thierry Reding , devicetree , "moderated list:ARM/S5P EXYNOS AR..." , cw00.choi@samsung.com, Donghwa Lee , "Linux-Kernel@Vger. Kernel. Org" , andi.shyti@samsung.com, jh80.chung@samsung.com, Kukjin Kim , ML dri-devel , Hyungwon Hwang , Krzysztof Kozlowski , Hoegeun Kwon Subject: Re: [PATCH v8 2/3] drm/panel: Add support for S6E3HA2 panel driver on TM2 board Message-ID: <20170201190331.GB14264@art_vandelay> References: <08c5d94b-c76f-af14-c08f-478e26a34a7c@samsung.com> <588FD3C3.7080508@samsung.com> <20170131085449.GA19348@ulmo.ba.sec> <20170131143853.GU20076@art_vandelay> <20170131150226.GB4519@ulmo.ba.sec> <87r33j85ap.fsf@eliezer.anholt.net> <20170131213132.GC872@mithrandir.ba.sec> <87d1f2kfgi.fsf@eliezer.anholt.net> <20170201145249.GB17698@ulmo.ba.sec> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Feb 01, 2017 at 03:29:40PM +0000, Emil Velikov wrote: > On 1 February 2017 at 14:52, Thierry Reding wrote: > > On Tue, Jan 31, 2017 at 02:54:53PM -0800, Eric Anholt wrote: > >> Thierry Reding writes: > >> > >> > [ Unknown signature status ] > >> > On Tue, Jan 31, 2017 at 10:15:10AM -0800, Eric Anholt wrote: > >> >> Thierry Reding writes: > >> >> > >> >> > [ Unknown signature status ] > >> >> > On Tue, Jan 31, 2017 at 09:38:53AM -0500, Sean Paul wrote: > >> >> >> On Tue, Jan 31, 2017 at 09:54:49AM +0100, Thierry Reding wrote: > >> >> >> > On Tue, Jan 31, 2017 at 09:01:07AM +0900, Inki Dae wrote: > >> >> >> > > > >> >> >> > > > >> >> >> > > 2017년 01월 24일 10:50에 Hoegeun Kwon 이(가) 쓴 글: > >> >> >> > > > Dear Thierry, > >> >> >> > > > > >> >> >> > > > Could you please review this patch? > >> >> >> > > > >> >> >> > > Thierry, I think this patch has been reviewed enough but no comment > >> >> >> > > from you. Seems you are busy. I will pick up this. > >> >> >> > > >> >> >> > Sorry, but that's not how it works. This patch has gone through 8 > >> >> >> > revisions within 4 weeks, and I tend to ignore patches like that until > >> >> >> > the dust settles. > >> >> >> > > >> >> >> > >> >> >> Seems like the dust was pretty settled. It was posted on 1/11, pinged on 1/24, > >> >> >> and picked up on 1/31. I don't think it's unreasonable to take it through > >> >> >> another tree after that. > >> >> >> > >> >> >> I wonder if drm_panel would benefit from the -misc group maintainership model > >> >> >> as drm_bridge does. By spreading out the workload, the high-maintenance > >> >> >> patches would hopefully find someone to shepherd them through. > >> >> > > >> >> > Except that nobody except me really cares. If we let people take patches > >> >> > through separate trees or group-maintained trees they'll likely go in > >> >> > without too much thought. DRM panel is somewhat different from core DRM > >> >> > in this regard because its infrastructure is minimal and there's little > >> >> > outside the panel-simple driver. So we're still at a stage where we need > >> >> > to fine-tune what drivers should look like and how we can improve. > >> >> > >> >> I would love to care and participate in review, but with the structure > >> >> of your tree you're the only one whose review counts, so I don't > >> >> participate. > >> > > >> > Really? What exactly do you think is special about the structure of my > >> > tree? I require patches to be on dri-devel (I pick them up from the > >> > patchwork instance at freedesktop.org), the tree is publicly available > >> > and reviewed-by tags get picked up automatically by patchwork. > >> > > >> > The panel tree works exactly like any other maintainer tree. And my > >> > review is *not* the only one that counts. I appreciate every Reviewed-by > >> > tag I see on panel patches because it means that I don't have to look as > >> > closely as I have to otherwise. > >> > > >> > It is true that I am responsible for those patches, that's why I get to > >> > have the final word on whether or not a patch gets applied. And that's > >> > no different from any other maintainer tree either. > >> > >> If me reviewing a patch isn't part of unblocking that patch getting in, > >> then I won't bother because all I could end up doing is punishing the > >> developer of the patch. Contributors have a hard enough time already. > > > > Maybe you should go and read my previous reply again more carefully. > > Perhaps then you'll realize that reviews are in fact helping in getting > > patches merged. > > > > Interestingly my inbox doesn't show you ever bothering to review panel > > patches, so maybe you should be more careful about your assumptions. > > > Gents, it's understandable that emotions might be running high. > > What's the point in pointing fingers at each other - there is enough > to go in each direction. > Let us all step back for a second and consider how we can make things better. > Seems like I kicked up some dust here, for that I apologize. I certainly did not intend to diminish Thierry's (or anyone else's) role as maintainer. To put this as concisely as I can, I thought drm_panel would be a good candidate for -misc given: - drm_bridge is already maintained there - the drivers are small, and we just resolved to maintain small drivers in -misc - new patches are blocked on a single reviewer/committer as opposed to a qualified committee (which I have come to understand is a feature in this instance) So if we can't migrate it to -misc now, for fear of quality issues, what are the steps necessary to "de-stage" it? Sean > I think it'll be nice to have some/most of the common concerns that > Thierry/others comes across documented - in-kernel, blog post, other. > Such that one can reference to specific points as patch falls sub-par. > We all want to have a balance of nicely written driver and quick > merge. > > Inki, I believe myself and others have invited you before on > #dri-devel. This is another medium where you can poke devs and from my > experience - it tends to be more efficient, most of the time. > > Thanks > Emil > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Sean Paul, Software Engineer, Google / Chromium OS