From mboxrd@z Thu Jan 1 00:00:00 1970 From: daniel@ffwll.ch (Daniel Vetter) Date: Tue, 14 Aug 2018 12:42:42 +0200 Subject: [PATCH v2 4/7] drm/i2c: tda998x: convert to bridge driver In-Reply-To: <20180810171630.GR30658@n2100.armlinux.org.uk> References: <20180730164137.GD17271@n2100.armlinux.org.uk> <20180808190930.GQ20303@art_vandelay> <20180808221547.GN30658@n2100.armlinux.org.uk> <20180810161105.GT20303@art_vandelay> <20180810165037.GQ30658@n2100.armlinux.org.uk> <20180810170229.GU20303@art_vandelay> <20180810171630.GR30658@n2100.armlinux.org.uk> Message-ID: <20180814104242.GO21634@phenom.ffwll.local> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Aug 10, 2018 at 06:16:30PM +0100, Russell King - ARM Linux wrote: > On Fri, Aug 10, 2018 at 01:02:29PM -0400, Sean Paul wrote: > > On Fri, Aug 10, 2018 at 05:50:37PM +0100, Russell King - ARM Linux wrote: > > > Almost none of my DRM specific patches on dri-devel this time around > > > received any feedback what so ever, even after myself and David chasing > > > them up. Over the Armada changes, David Airlie eventually said to me > > > "if it works for you, send it to me I suppose". > > > > > > David also tried poking the tda998x/component discussion thread as well > > > from the beginning of July (or so David told me), with seemingly the > > > same result. > > > > > > So, in summary, it's been almost impossible to get any feedback on any > > > of the patches and discussions I've sent - I think extracting blood > > > from a rock might be easier! ;) > > > > Yeah, I suspect a lot of people have blinders wrt tda given that it's off in > > i2c/. Feel free to ping me on irc in future if you're looking for a review. Once > > tda moves to bridge/ and drm-misc, it'll likely be much easier to get people > > engaged. > > I suspect that those who actually use it don't have such blinders, yet > they also have ignored the patches despite being copied with them. I > can't see that the location of the driver would make any difference > for them. Expecting review to magically happen doesn't work. You need to actively solicit that, and often that involves a bit of reviewing trading. Especially anywhere where a part of the kernel is officially group-maintained like drm_bridge. Given your past track record of handling other contributors I think it's entirely understandably that people do not choose to collaborate with you voluntarily. Fixing that is entirely up to you though. I good first step here would imo be to move the tda driver under bridge, formally put it under the drm-misc group maintainership, actively solict and ack for that from other bridge maintainers and then also suggest other authors and contributors to the tda driver to join as committers and co-maintainers. Thanks, Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH v2 4/7] drm/i2c: tda998x: convert to bridge driver Date: Tue, 14 Aug 2018 12:42:42 +0200 Message-ID: <20180814104242.GO21634@phenom.ffwll.local> References: <20180730164137.GD17271@n2100.armlinux.org.uk> <20180808190930.GQ20303@art_vandelay> <20180808221547.GN30658@n2100.armlinux.org.uk> <20180810161105.GT20303@art_vandelay> <20180810165037.GQ30658@n2100.armlinux.org.uk> <20180810170229.GU20303@art_vandelay> <20180810171630.GR30658@n2100.armlinux.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail-ed1-x541.google.com (mail-ed1-x541.google.com [IPv6:2a00:1450:4864:20::541]) by gabe.freedesktop.org (Postfix) with ESMTPS id 51EF36E12D for ; Tue, 14 Aug 2018 10:42:47 +0000 (UTC) Received: by mail-ed1-x541.google.com with SMTP id s16-v6so9832442edq.12 for ; Tue, 14 Aug 2018 03:42:47 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20180810171630.GR30658@n2100.armlinux.org.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Russell King - ARM Linux Cc: David Airlie , Liviu Dudau , Jyri Sarha , Tomi Valkeinen , dri-devel@lists.freedesktop.org, Peter Rosin , linux-arm-kernel@lists.infradead.org, Laurent Pinchart List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBBdWcgMTAsIDIwMTggYXQgMDY6MTY6MzBQTSArMDEwMCwgUnVzc2VsbCBLaW5nIC0g QVJNIExpbnV4IHdyb3RlOgo+IE9uIEZyaSwgQXVnIDEwLCAyMDE4IGF0IDAxOjAyOjI5UE0gLTA0 MDAsIFNlYW4gUGF1bCB3cm90ZToKPiA+IE9uIEZyaSwgQXVnIDEwLCAyMDE4IGF0IDA1OjUwOjM3 UE0gKzAxMDAsIFJ1c3NlbGwgS2luZyAtIEFSTSBMaW51eCB3cm90ZToKPiA+ID4gQWxtb3N0IG5v bmUgb2YgbXkgRFJNIHNwZWNpZmljIHBhdGNoZXMgb24gZHJpLWRldmVsIHRoaXMgdGltZSBhcm91 bmQKPiA+ID4gcmVjZWl2ZWQgYW55IGZlZWRiYWNrIHdoYXQgc28gZXZlciwgZXZlbiBhZnRlciBt eXNlbGYgYW5kIERhdmlkIGNoYXNpbmcKPiA+ID4gdGhlbSB1cC4gIE92ZXIgdGhlIEFybWFkYSBj aGFuZ2VzLCBEYXZpZCBBaXJsaWUgZXZlbnR1YWxseSBzYWlkIHRvIG1lCj4gPiA+ICJpZiBpdCB3 b3JrcyBmb3IgeW91LCBzZW5kIGl0IHRvIG1lIEkgc3VwcG9zZSIuCj4gPiA+IAo+ID4gPiBEYXZp ZCBhbHNvIHRyaWVkIHBva2luZyB0aGUgdGRhOTk4eC9jb21wb25lbnQgZGlzY3Vzc2lvbiB0aHJl YWQgYXMgd2VsbAo+ID4gPiBmcm9tIHRoZSBiZWdpbm5pbmcgb2YgSnVseSAob3Igc28gRGF2aWQg dG9sZCBtZSksIHdpdGggc2VlbWluZ2x5IHRoZQo+ID4gPiBzYW1lIHJlc3VsdC4KPiA+ID4gCj4g PiA+IFNvLCBpbiBzdW1tYXJ5LCBpdCdzIGJlZW4gYWxtb3N0IGltcG9zc2libGUgdG8gZ2V0IGFu eSBmZWVkYmFjayBvbiBhbnkKPiA+ID4gb2YgdGhlIHBhdGNoZXMgYW5kIGRpc2N1c3Npb25zIEkn dmUgc2VudCAtIEkgdGhpbmsgZXh0cmFjdGluZyBibG9vZAo+ID4gPiBmcm9tIGEgcm9jayBtaWdo dCBiZSBlYXNpZXIhIDspCj4gPiAKPiA+IFllYWgsIEkgc3VzcGVjdCBhIGxvdCBvZiBwZW9wbGUg aGF2ZSBibGluZGVycyB3cnQgdGRhIGdpdmVuIHRoYXQgaXQncyBvZmYgaW4KPiA+IGkyYy8uIEZl ZWwgZnJlZSB0byBwaW5nIG1lIG9uIGlyYyBpbiBmdXR1cmUgaWYgeW91J3JlIGxvb2tpbmcgZm9y IGEgcmV2aWV3LiBPbmNlCj4gPiB0ZGEgbW92ZXMgdG8gYnJpZGdlLyBhbmQgZHJtLW1pc2MsIGl0 J2xsIGxpa2VseSBiZSBtdWNoIGVhc2llciB0byBnZXQgcGVvcGxlCj4gPiBlbmdhZ2VkLgo+IAo+ IEkgc3VzcGVjdCB0aGF0IHRob3NlIHdobyBhY3R1YWxseSB1c2UgaXQgZG9uJ3QgaGF2ZSBzdWNo IGJsaW5kZXJzLCB5ZXQKPiB0aGV5IGFsc28gaGF2ZSBpZ25vcmVkIHRoZSBwYXRjaGVzIGRlc3Bp dGUgYmVpbmcgY29waWVkIHdpdGggdGhlbS4gIEkKPiBjYW4ndCBzZWUgdGhhdCB0aGUgbG9jYXRp b24gb2YgdGhlIGRyaXZlciB3b3VsZCBtYWtlIGFueSBkaWZmZXJlbmNlCj4gZm9yIHRoZW0uCgpF eHBlY3RpbmcgcmV2aWV3IHRvIG1hZ2ljYWxseSBoYXBwZW4gZG9lc24ndCB3b3JrLiBZb3UgbmVl ZCB0byBhY3RpdmVseQpzb2xpY2l0IHRoYXQsIGFuZCBvZnRlbiB0aGF0IGludm9sdmVzIGEgYml0 IG9mIHJldmlld2luZyB0cmFkaW5nLgpFc3BlY2lhbGx5IGFueXdoZXJlIHdoZXJlIGEgcGFydCBv ZiB0aGUga2VybmVsIGlzIG9mZmljaWFsbHkKZ3JvdXAtbWFpbnRhaW5lZCBsaWtlIGRybV9icmlk Z2UuCgpHaXZlbiB5b3VyIHBhc3QgdHJhY2sgcmVjb3JkIG9mIGhhbmRsaW5nIG90aGVyIGNvbnRy aWJ1dG9ycyBJIHRoaW5rIGl0J3MKZW50aXJlbHkgdW5kZXJzdGFuZGFibHkgdGhhdCBwZW9wbGUg ZG8gbm90IGNob29zZSB0byBjb2xsYWJvcmF0ZSB3aXRoIHlvdQp2b2x1bnRhcmlseS4gRml4aW5n IHRoYXQgaXMgZW50aXJlbHkgdXAgdG8geW91IHRob3VnaC4KCkkgZ29vZCBmaXJzdCBzdGVwIGhl cmUgd291bGQgaW1vIGJlIHRvIG1vdmUgdGhlIHRkYSBkcml2ZXIgdW5kZXIgYnJpZGdlLApmb3Jt YWxseSBwdXQgaXQgdW5kZXIgdGhlIGRybS1taXNjIGdyb3VwIG1haW50YWluZXJzaGlwLCBhY3Rp dmVseSBzb2xpY3QKYW5kIGFjayBmb3IgdGhhdCBmcm9tIG90aGVyIGJyaWRnZSBtYWludGFpbmVy cyBhbmQgdGhlbiBhbHNvIHN1Z2dlc3Qgb3RoZXIKYXV0aG9ycyBhbmQgY29udHJpYnV0b3JzIHRv IHRoZSB0ZGEgZHJpdmVyIHRvIGpvaW4gYXMgY29tbWl0dGVycyBhbmQKY28tbWFpbnRhaW5lcnMu CgpUaGFua3MsIERhbmllbAotLSAKRGFuaWVsIFZldHRlcgpTb2Z0d2FyZSBFbmdpbmVlciwgSW50 ZWwgQ29ycG9yYXRpb24KaHR0cDovL2Jsb2cuZmZ3bGwuY2gKX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVsIG1haWxpbmcgbGlzdApkcmktZGV2 ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlzdHMuZnJlZWRlc2t0b3Aub3JnL21h aWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg==