From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH 0/2] Documentation/EDID fixes Date: Wed, 08 Mar 2017 10:11:29 +0200 Message-ID: <87r328chn2.fsf@intel.com> References: <1488810456-4583-1-git-send-email-javi.merino@kernel.org> <87o9xdf4ek.fsf@intel.com> <20170307163346.GA2255@ct-lt-587> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by gabe.freedesktop.org (Postfix) with ESMTPS id CCC0C6E83B for ; Wed, 8 Mar 2017 08:11:32 +0000 (UTC) In-Reply-To: <20170307163346.GA2255@ct-lt-587> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Javi Merino Cc: linux-kernel@lists.codethink.co.uk, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gVHVlLCAwNyBNYXIgMjAxNywgSmF2aSBNZXJpbm8gPGphdmkubWVyaW5vQGtlcm5lbC5vcmc+ IHdyb3RlOgo+IE9uIFR1ZSwgTWFyIDA3LCAyMDE3IGF0IDA2OjE2OjUxUE0gKzAyMDAsIEphbmkg TmlrdWxhIHdyb3RlOgo+PiBPbiBNb24sIDA2IE1hciAyMDE3LCBKYXZpIE1lcmlubyA8amF2aS5t ZXJpbm9Aa2VybmVsLm9yZz4gd3JvdGU6Cj4+ID4gSSBmb3VuZCB0aGVzZSB0d28gbWlub3IgaXNz dWVzIHdoaWxlIGJ1aWxkaW5nIGFuIEVESUQuICBJJ20gbm90IHN1cmUKPj4gPiB3aGV0aGVyIHRo ZSBzZWNvbmQgcGF0Y2ggKEFkZCBPPSB0byBzdXBwb3J0KSBpcyB1cHN0cmVhbSBtYXRlcmlhbCwg YnV0Cj4+ID4gSSdtIHNlbmRpbmcgaXQganVzdCBpbiBjYXNlLgo+PiAKPj4gSSdtIG5vdCBvcHBv c2VkIHRvIGZpeGluZyBleGlzdGluZyBpc3N1ZXMgbGlrZSB0aGlzLCBidXQgcmVhbGx5IEkgdGhp bmsKPj4gdGhlcmUgc2hvdWxkIGJlIGFuIHVzZXJzcGFjZSB0b29sIGZvciB0aGlzLiBEZWZpbml0 ZWx5IG91dHNpZGUgb2YgdGhlCj4+IERvY3VtZW50YXRpb24gZGlyZWN0b3J5LCBwZXJoYXBzIGV2 ZW4gb3V0c2lkZSB0aGUga2VybmVsIHRyZWUKPj4gYWx0b2dldGhlci4KPgo+IEkgYW0gbm90IHN1 cmUgd2hldGhlciB0aGlzIHNob3VsZCBiZSBpbiB0aGUga2VybmVsIG9yIG5vdCwgYnV0IEkKPiBh Z3JlZSB0aGF0IERvY3VtZW50YXRpb24vIGRvZXNuJ3QgbG9vayBsaWtlIHRoZSByaWdodCBwbGFj ZSB0byBidWlsZAo+IHRoZXNlIGZpbGVzLgoKSnVzdCBzbyB0aGVyZSdzIG5vIGNvbmZ1c2lvbjog SSdtIGp1c3Qgc2F5aW5nIHdoYXQgSSB0aGluayBzaG91bGQgYmUKZG9uZSBpbiB0aGUgbG9uZyBy dW4uIEknbSBub3Qgb3Bwb3NlZCB0byB0aGUgZml4ZXMgYXQgaGFuZCwgYnV0IHNvbWVvbmUKZWxz ZSBuZWVkcyB0byByZXZpZXcgdGhlbS4KCkJSLApKYW5pLgoKLS0gCkphbmkgTmlrdWxhLCBJbnRl bCBPcGVuIFNvdXJjZSBUZWNobm9sb2d5IENlbnRlcgpfX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFpbGluZyBsaXN0CmRyaS1kZXZlbEBs aXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1h bi9saXN0aW5mby9kcmktZGV2ZWwK From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750989AbdCHIMZ (ORCPT ); Wed, 8 Mar 2017 03:12:25 -0500 Received: from mga11.intel.com ([192.55.52.93]:6436 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750707AbdCHIMX (ORCPT ); Wed, 8 Mar 2017 03:12:23 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,262,1486454400"; d="scan'208";a="832227479" From: Jani Nikula To: Javi Merino Cc: airlied@linux.ie, C.Emde@osadl.org, linux-kernel@lists.codethink.co.uk, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH 0/2] Documentation/EDID fixes In-Reply-To: <20170307163346.GA2255@ct-lt-587> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <1488810456-4583-1-git-send-email-javi.merino@kernel.org> <87o9xdf4ek.fsf@intel.com> <20170307163346.GA2255@ct-lt-587> Date: Wed, 08 Mar 2017 10:11:29 +0200 Message-ID: <87r328chn2.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 07 Mar 2017, Javi Merino wrote: > On Tue, Mar 07, 2017 at 06:16:51PM +0200, Jani Nikula wrote: >> On Mon, 06 Mar 2017, Javi Merino wrote: >> > I found these two minor issues while building an EDID. I'm not sure >> > whether the second patch (Add O= to support) is upstream material, but >> > I'm sending it just in case. >> >> I'm not opposed to fixing existing issues like this, but really I think >> there should be an userspace tool for this. Definitely outside of the >> Documentation directory, perhaps even outside the kernel tree >> altogether. > > I am not sure whether this should be in the kernel or not, but I > agree that Documentation/ doesn't look like the right place to build > these files. Just so there's no confusion: I'm just saying what I think should be done in the long run. I'm not opposed to the fixes at hand, but someone else needs to review them. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center