<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Jan 19, 2014 at 7:56 PM, Michel Dänzer <span dir="ltr"><<a href="mailto:michel@daenzer.net" target="_blank">michel@daenzer.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fre, 2014-01-17 at 03:47 -0800, Mark Mueller wrote:<br>
><br>
> diff --git a/src/mesa/main/formats.h b/src/mesa/main/formats.h<br>
> index 348d2f4..fb43c83 100644<br>
> --- a/src/mesa/main/formats.h<br>
> +++ b/src/mesa/main/formats.h<br>
> @@ -182,14 +182,14 @@ typedef enum<br>
>     MESA_FORMAT_RGB_UNORM8,<br>
><br>
>     /* Type P formats */<br>
> -   MESA_FORMAT_RGB565,               /*                     RRRR RGGG GGGB BBBB */<br>
> -   MESA_FORMAT_RGB565_REV,   /*                     GGGB BBBB RRRR RGGG */<br>
> -   MESA_FORMAT_ARGB4444,     /*                     AAAA RRRR GGGG BBBB */<br>
> -   MESA_FORMAT_ARGB4444_REV, /*                     GGGG BBBB AAAA RRRR */<br>
> -   MESA_FORMAT_RGBA5551,        /*                     RRRR RGGG GGBB BBBA */<br>
> -   MESA_FORMAT_ARGB1555,     /*                     ARRR RRGG GGGB BBBB */<br>
> -   MESA_FORMAT_ARGB1555_REV, /*                     GGGB BBBB ARRR RRGG */<br>
> -   MESA_FORMAT_AL44,         /*                               AAAA LLLL */<br>
> +   MESA_FORMAT_B5G6R5_UNORM,         /* BBBB BGGG GGGR RRRR */<br>
> +   MESA_FORMAT_R5G6B5_UNORM,     /* RRRR RGGG GGGB BBBB */<br>
> +   MESA_FORMAT_B4G4R4A4_UNORM,       /* BBBB GGGG RRRR AAAA */<br>
> +   MESA_FORMAT_A4R4G4B4_UNORM,       /* AAAA RRRR GGGG BBBB */<br>
> +   MESA_FORMAT_A1B5G5R5_UNORM,   /* ARRR RRGG GGGB BBBB */<br>
> +   MESA_FORMAT_B5G5R5A1_UNORM,       /* BBBB BGGG GGRR RRRA */<br>
> +   MESA_FORMAT_A1R5G5B5_UNORM,       /* ARRR RRGG GGGB BBBB */<br>
> +   MESA_FORMAT_L4A4_UNORM,       /* LLLL AAAA */<br>
<br>
Please keep these comments aligned with the other comments describing<br>
packed format layouts. (Please also don't remove the header comments<br>
explaining the format of these comments)<br></blockquote><div><br></div><div>Sorry, there were some fossilized tabs in there that I _had_ removed in an earlier patch but they came back like the night of the living dead.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Also, why are you changing the component order in the comments for<br>
these, but not for some other packed formats in the series?<br>
<br>
Last but not least, there are a few cases in the series where you're<br>
defining a format as 'type A', when these comments clearly show that<br>
they're packed formats. Please be careful.<br>
<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Unlike the 100's of global substitutions, all of the comments have to be done by hand and there were</div><div>some stragglers that got through. With the major shift to a lot more P type formats, much of that has</div>
<div>changed again. I should have a new patchset revision ready by tomorrow.</div><div><br></div><div>Mark</div><div><br></div></div></div></div>