From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Warren Subject: Re: [PATCH 1/2] ARM: tegra: add Acer Chromebook 13 device tree Date: Wed, 13 Aug 2014 11:12:01 -0600 Message-ID: <53EB9C61.1070406@wwwdotorg.org> References: <1407894967-18300-1-git-send-email-dgreid@chromium.org> <20140813085304.GE7735@ulmo> <53EB9786.6010809@wwwdotorg.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Andrew Bresticker Cc: Thierry Reding , Dylan Reid , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org" , Olof Johansson List-Id: linux-tegra@vger.kernel.org On 08/13/2014 11:04 AM, Andrew Bresticker wrote: > On Wed, Aug 13, 2014 at 9:51 AM, Stephen Warren wrote: >> On 08/13/2014 02:53 AM, Thierry Reding wrote: >>> >>> On Tue, Aug 12, 2014 at 06:56:07PM -0700, Dylan Reid wrote: >>>> >>>> The Acer Chromebook 13, codenamed "Big", contains an NVIDIA tegra124 >>>> processor and is similar to the Venice2 reference platform. >>>> >>>> The keyboard, USB 2, audio, HDMI, sdcard and emmc have been tested >>>> and work on the 1366x768 models. I haven't tried on the HD systems >>>> yet. >>> >>> >>> Presumably the HD systems will have a different compatible? Is it only >>> the panel that's different or are there other changes? >> >> >> Yes, we definitely need to plan ahead for the different SKUs. I assume we'll >> have different board names for each board, which then translates int DT >> filenames and U-Boot/cbootimage-configs/tegra-uboot-flasher board > > The firmware does not differentiate between the HD and non-HD boards, > so it will look for the same compatible string. As I mentioned in my > response to Thierry, we'd need to parse the EDID to figure out which > panel is being used. Well, that's if we do it at run-time. We could just have a separate DT per SKU, couldn't we?