On Mon, Nov 4, 2013 at 8:11 PM, Keith Packard <keithp@xxxxxxxxxx> wrote: > Jordan Justen <jljusten@xxxxxxxxx> writes: >> After patch 6, this will add SARGB8, right? So, maybe add this to the >> commit message, or separate out adding SARGB8 into a separate commit? > > I added the SARGB8 define in patch 4; is there some other separation you > think would be warranted? I was just noting that a side effect of patch five is adding support for SARGB8. When you remove the code in patch 6, and use these, you've now added support for this new format. Not important, but I thought it might be worth noting in the commit message. Actually probably noting it in the commit message for patch 6 is better since the change happens at that point. -Jordan _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
