Re: [Mesa-dev] [PATCH 1/2] mesa: add R10G10B10{A, X}2 to MESA <-> DRI format translations

2018-05-07 Thread Dylan Baker
Quoting Dylan Baker (2018-05-06 12:38:54) > > > On May 4, 2018 5:44:25 PM PDT, Chad Versace wrote: > > > >The BUG line should be converted to any of the following trailer lines: > > > >Bug: https://crbug.com/776903 > >(This is my favorite). > >Fixes:

Re: [Mesa-dev] [PATCH 1/2] mesa: add R10G10B10{A, X}2 to MESA <-> DRI format translations

2018-05-06 Thread Dylan Baker
On May 4, 2018 5:44:25 PM PDT, Chad Versace wrote: >Thanks for the patches. The code looks good. All my suggestions are >merely nitpicks to make the patches follow Mesa conventions. > >In general, if you have questions about commit message style, examine >the git log

Re: [Mesa-dev] [PATCH 1/2] mesa: add R10G10B10{A, X}2 to MESA <-> DRI format translations

2018-05-04 Thread Chad Versace
Thanks for the patches. The code looks good. All my suggestions are merely nitpicks to make the patches follow Mesa conventions. In general, if you have questions about commit message style, examine the git log for previous patches that touched the same files and directories as yours. Sometimes,

[Mesa-dev] [PATCH 1/2] mesa: add R10G10B10{A, X}2 to MESA <-> DRI format translations

2018-05-03 Thread Miguel Casas
This patch adds R10G10B10{A,X}2 MESA <-> DRI translation entries in the appropriate places for dri2 functions to accept them. BUG=https://crbug.com/776093 TEST=Compile and deploy mesa+this patch, then playback a VP9 Profile 2 video with sw decoder using crrev.com/c/897894. ---