<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Dec 20, 2013 at 7:38 AM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On 12/19/2013 06:47 PM, Michel Dänzer wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Don, 2013-12-19 at 13:56 -0800, Mark Mueller wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Adjust MESA_FORMAT color component ordering to match API docs, driver<br>
specific formats (e.g. PIPE_FORMATs),<br>
</blockquote>
<br>
Actually, there are a couple of examples of other format definitions<br>
which match the Mesa formats before your change but no longer after it,<br>
e.g. in the DRI and i915, nouveau and radeon driver code.<br>
<br>
Changing the Mesa format definitions will be confusing for people<br>
switching between branches with and without your change.<br>
</blockquote>
<br></div>
Unfortunately, I don't think there's a solution to that issue.  There'll always be stable branches that we're cherry-picking to.  If that's a major concern, we'll never change any of the MESA_FORMATs.<div class="im">
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also, because these Mesa formats are defined as packed values, you're<br>
essentially changing the notation from big endian (aka human readable)<br>
to little endian. It's unfortunate that the packed PIPE_FORMATs are<br>
named in little endian order, that's a concession we had to make when<br>
adding them.<br>
<br>
Overall, I'm afraid this change doesn't look very good at all to me. At<br>
the very least though, you'd also have to change the order of component<br>
sizes for formats such as MESA_FORMAT_BGRA2101010_UNORM or<br>
MESA_FORMAT_BGRA1555_UNORM, otherwise they're just plain wrong.<br>
</blockquote>
<br></div>
Yeah, I missed that, and I agree that if we're going to rename things to follow the Gallium style, we should fix those too.<br><br></blockquote><div> </div><div>That wasn't an oversight, that part is in my next phase. If it's preferred, I can move this patch into the subsequent series that I've been working on. I'd prefer not to because the first 2 patches are huge but straight forward changes. The subsequent series is more challenging because further name changing exposes format names that are redundant as well as cases where formats are being used incorrectly, probably because the name was misunderstood. The series will be smaller but will require more thorough review and testing by reviewers.</div>
<div><br></div><div>Mark</div></div></div></div>