<div dir="ltr">On Sun, Jan 26, 2014 at 11:45 AM, Mark Mueller <span dir="ltr"><<a href="mailto:markkmueller@gmail.com" target="_blank">markkmueller@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote"><div>On Fri, Jan 24, 2014 at 2:50 AM, Marek Olšák <span dir="ltr"><<a href="mailto:maraeo@gmail.com" target="_blank">maraeo@gmail.com</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>On Thu, Jan 23, 2014 at 8:36 PM, Kenneth Graunke <<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>> wrote:<br>



> On 01/23/2014 11:19 AM, Mark Mueller wrote:<br>
</div><div>>> That works for sRGB, but what about other color spaces that may be added in<br>
>> the future? I'm fine with leaving that to another day and with using the<br>
>> _SRGB decoration, as you say.<br>
><br>
> Oy.  Let's not worry about things that hypothetically could be someday.<br>
>  We need to do what makes sense today.<br>
<br>
</div>Yeah, let's commit this. We can discuss the other things later,<br>
including if SRGB should be a type or a modifier. The series is<br>
already pretty good.<br>
<span><font color="#888888"><br>
Marek<br></font></span></blockquote><div><br></div></div><div> I'm not sure if further acknowledgement is needed to push the series, but, either way, I'm not authorized to push these.<br></div></div></div></div></blockquote>

<div><br></div><div>The series looks good to me too, but I'd also like to see the _UNORM8 -> _SRGB suffix change in a follow-up patch.<br><br></div><div>I tried applying the patch series to my local tree but it doesn't apply cleanly.  My recent "mesa: add missing ETC2_SRGB cases in formats.c" change, and possibly others are at fault.  Sorry.<br>
<br></div><div>If you can rebase the series on top of master I can apply them, unless Marek beats me to it (I'm in the midst of traveling today).<br><br></div><div>-Brian<br><br>
</div></div></div></div>