Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933357AbbHKAkz (ORCPT ); Mon, 10 Aug 2015 20:40:55 -0400 Received: from mail-yk0-f169.google.com ([209.85.160.169]:36617 "EHLO mail-yk0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932678AbbHKAkw (ORCPT ); Mon, 10 Aug 2015 20:40:52 -0400 MIME-Version: 1.0 In-Reply-To: <20150810232552.GB13472@usrtlx11787.corpusers.net> References: <1438087956-17307-1-git-send-email-srinivas.kandagatla@linaro.org> <1438088076-17606-1-git-send-email-srinivas.kandagatla@linaro.org> <20150810232552.GB13472@usrtlx11787.corpusers.net> Date: Mon, 10 Aug 2015 20:40:52 -0400 Message-ID: Subject: Re: [PATCH v1 7/7] ARM: dts: ifc6410: add inforce LVDS panel support From: Rob Clark To: Bjorn Andersson Cc: Srinivas Kandagatla , Andy Gross , linux-arm-msm , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , Russell King , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1373 Lines: 37 On Mon, Aug 10, 2015 at 7:25 PM, Bjorn Andersson wrote: > On Tue 28 Jul 05:54 PDT 2015, Srinivas Kandagatla wrote: > >> This patch adds LVDS panel for IFC6410. >> > > My ifc6410 doesn't have a LVDS display, so I don't think these nodes > belong in this dts file. fwiw, my hope was that we could implement a panel-picker for lvds+ddc, in a similar way to handling devices that could have multiple different dsi panels, to make this a bit more automagically plug-and-play.. srini has done some experiments with probing panel's edid, so we just need to figure out if vendor+device-id in edid is reliable enough to have a global table mapping them back to panel compat strings or whether that table needs to go in dt.. BR, -R > I was thinking we could have a variant including this dts file adding > the panel, but I assume that you can mount other panels than the auo > panel defined below? > > > Either way, I think we should keep this file to define the stuff that's > on the IFC6410 itself and then figure out how to aid people in adding > the rest of their additions... > > Regards, > Bjorn -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/