Re: [PATCH] drm/i915: s/valleyview/baytrail

[Chad Versace <chad.versace@linux.intel.com>]

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 <kenneth@whitecape.org>
> 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 <kenneth@whitecape.org>
> CC: Jesse Barnes <jbarnes@virtuousgeek.org>
> Signed-off-by: Ben Widawsky <ben@bwidawsk.net>

Acked-by: Chad Versace <chad.versace@linux.intel.com>