Re: [Mesa-dev] [PATCH] mesa/st: fix yuv EGLImage's

2017-05-17 Thread Rob Clark
On Wed, May 17, 2017 at 6:30 AM, Philipp Zabel wrote: > On Tue, 2017-05-16 at 10:33 -0400, Rob Clark wrote: >> Don't reject YUV formats that the driver doesn't handle natively, since >> mesa/st already knows how to lower this in shader. >> >> Fixes: 83e9de2 ("st/mesa: EGLImageTarget* error handlin

Re: [Mesa-dev] [PATCH] mesa/st: fix yuv EGLImage's

2017-05-17 Thread Philipp Zabel
On Tue, 2017-05-16 at 10:33 -0400, Rob Clark wrote: > Don't reject YUV formats that the driver doesn't handle natively, since > mesa/st already knows how to lower this in shader. > > Fixes: 83e9de2 ("st/mesa: EGLImageTarget* error handling") > Cc: 17.1 Signed-off-by: Rob Clark > --- > src/mesa/s

Re: [Mesa-dev] [PATCH] mesa/st: fix yuv EGLImage's

2017-05-16 Thread Nicolai Hähnle
On 16.05.2017 16:33, Rob Clark wrote: Don't reject YUV formats that the driver doesn't handle natively, since mesa/st already knows how to lower this in shader. Fixes: 83e9de2 ("st/mesa: EGLImageTarget* error handling") Cc: 17.1 Seems reasonable. Reviewed-by: Nicolai Hähnle --- src/mesa

Re: [Mesa-dev] [PATCH] mesa/st: fix yuv EGLImage's

2017-05-16 Thread Rob Clark
On Tue, May 16, 2017 at 10:33 AM, Rob Clark wrote: > Don't reject YUV formats that the driver doesn't handle natively, since > mesa/st already knows how to lower this in shader. I probably should have mentioned that 'kmscube -M nv12-1img' is a good way to test this BR, -R > Fixes: 83e9de2 ("st/

[Mesa-dev] [PATCH] mesa/st: fix yuv EGLImage's

2017-05-16 Thread Rob Clark
Don't reject YUV formats that the driver doesn't handle natively, since mesa/st already knows how to lower this in shader. Fixes: 83e9de2 ("st/mesa: EGLImageTarget* error handling") Cc: 17.1 --- src/mesa/state_tracker/st_cb_eglimage.c | 32 ++-- 1 file changed, 30 ins