From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chad Versace Subject: Re: [PATCH] drm/i915: s/valleyview/baytrail Date: Fri, 10 May 2013 08:53:10 -0700 Message-ID: <518D17E6.20405@linux.intel.com> References: <1368075212-6631-1-git-send-email-ben@bwidawsk.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id 137F7E5BF7 for ; Fri, 10 May 2013 08:53:58 -0700 (PDT) In-Reply-To: <1368075212-6631-1-git-send-email-ben@bwidawsk.net> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Ben Widawsky Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On 05/08/2013 09:53 PM, Ben Widawsky wrote: > Bay Trail is the marketing name for the new Silvermont based SOCs we've > been calling Valleyview. All recent Intel disclosed documents use the > term "Bay Trail." AFAICT, only leaked docs (and our code) use the term > Valleyview. I've also verified with internal sources that references to, > "Valleyview" should be dropped. > > Why? > We've always strived to make the already confusing generation > information as clear as possible in our driver. With the introduction > of the below, we made two names referring to the exact same silicon. > > commit 93c34e70ebf464a9ee142d93b681c5df094ec654 > Author: Kenneth Graunke > Date: Mon Apr 22 00:53:50 2013 -0700 > > drm/i915: Fix page table entries for Bay Trail. > > While it's tempting to simply do the renames in Ken's patch, it's simply > not correct. As an added bonus, Mesa also uses the byt/baytrail names. > > Yes. I know this is painful. Doing it now is the least painful way to do > this right (short of going back in time.) > > disclaimer: only compile tested. > disclaimer2: I didn't fix pre-existing checkpatch issues. > > CC: Kenneth Graunke > CC: Jesse Barnes > Signed-off-by: Ben Widawsky Acked-by: Chad Versace