Applied, thanks for the patch. > On 02/03/2015 02:15 PM, Gwenole Beauchesne wrote: > > Hi, > > > > 2015-02-03 22:11 GMT+01:00 Sean V Kelley <[email protected]>: > >> From: Heng-ruey Hsu <[email protected]> > >> > >> With this, we can get hardware supported resolution for video encode and > >> decode. This CL is used to prepare GetSupportedResolution in chrome. > >> > >> Signed-off-by: Heng-ruey Hsu <[email protected]> > >> Reviewed-by: Wu-cheng Li <[email protected]> > >> Reviewed-by: Sean V Kelley <[email protected]> > >> Cc: Pawel Osciak <[email protected]> > >> Cc: Stéphane Marchesin <[email protected]> > >> --- > >> src/i965_drv_video.c | 12 ++++++++++++ > >> 1 file changed, 12 insertions(+) > >> > >> diff --git a/src/i965_drv_video.c b/src/i965_drv_video.c > >> index ac2aeac..64f4876 100644 > >> --- a/src/i965_drv_video.c > >> +++ b/src/i965_drv_video.c > >> @@ -5149,6 +5149,18 @@ i965_QuerySurfaceAttributes(VADriverContextP ctx, > >> attribs[i].value.value.p = NULL; /* ignore */ > >> i++; > >> > >> + attribs[i].type = VASurfaceAttribMaxWidth; > >> + attribs[i].value.type = VAGenericValueTypeInteger; > >> + attribs[i].flags = VA_SURFACE_ATTRIB_GETTABLE; > >> + attribs[i].value.value.i = i965->codec_info->max_width; > >> + i++; > >> + > >> + attribs[i].type = VASurfaceAttribMaxHeight; > >> + attribs[i].value.type = VAGenericValueTypeInteger; > >> + attribs[i].flags = VA_SURFACE_ATTRIB_GETTABLE; > >> + attribs[i].value.value.i = i965->codec_info->max_height; > >> + i++; > >> + > >> if (i > *num_attribs) { > >> *num_attribs = i; > >> free(attribs); > > > > This would work as a temporary solution. > > > > However, a really correct solution requires a second patch to make > > codec_info really per-codec. i.e. max resolution constraints can vary > > on a per-codec basis, depending on the generation. > > yes, I agree and that is planned. Right now the max/min resolutions > are purely hard coded on per platform basis. > > Thanks, > > Sean > > > > > Regards, > > > > _______________________________________________ > Libva mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/libva
_______________________________________________ Libva mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libva
