From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kenneth Graunke Subject: Re: [PATCH] drm/i915: s/valleyview/baytrail Date: Wed, 08 May 2013 22:17:45 -0700 Message-ID: <518B3179.7000800@whitecape.org> 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 homiemail-a2.g.dreamhost.com (caiajhbdcbhh.dreamhost.com [208.97.132.177]) by gabe.freedesktop.org (Postfix) with ESMTP id 66ECBE5C02 for ; Wed, 8 May 2013 22:16:25 -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 Reviewed-by: Kenneth Graunke