radeonsi: Expose support for 10-bit VP9 decode

Submitted by Vishwakarma, Pratik on June 28, 2019, 5:09 a.m.

Details

Message ID 20190628050856.208189-1-Pratik.Vishwakarma@amd.com
State New
Headers show
Series "radeonsi: Expose support for 10-bit VP9 decode" ( rev: 1 ) in Mesa

Not browsing as part of any series.

Commit Message

Vishwakarma, Pratik June 28, 2019, 5:09 a.m.
Fix si_vid_is_format_supported to expose support
for 10-bit VP9 decode using P016 format. Without
this change, 10-bit decode will be exposed only
for HEVC even though newer hardware support
10-bit decode for VP9.

Signed-off-by: Pratik Vishwakarma <Pratik.Vishwakarma@amd.com>

---
 src/gallium/drivers/radeonsi/si_get.c | 4 ++++
 1 file changed, 4 insertions(+)

-- 
2.17.1

Patch hide | download patch | download mbox

diff --git a/src/gallium/drivers/radeonsi/si_get.c b/src/gallium/drivers/radeonsi/si_get.c
index 4e23d283ab7..8cc5933f9bc 100644
--- a/src/gallium/drivers/radeonsi/si_get.c
+++ b/src/gallium/drivers/radeonsi/si_get.c
@@ -709,6 +709,10 @@  static boolean si_vid_is_format_supported(struct pipe_screen *screen,
 		return (format == PIPE_FORMAT_NV12) ||
 			(format == PIPE_FORMAT_P016);
 
+	/* Vp9 profile 2 supports 10 bit decoding using P016 */
+	if (profile == PIPE_VIDEO_PROFILE_VP9_PROFILE2)
+		return format == PIPE_FORMAT_P016;
+
 	/* we can only handle this one with UVD */
 	if (profile != PIPE_VIDEO_PROFILE_UNKNOWN)
 		return format == PIPE_FORMAT_NV12;

Comments

Am 28.06.19 um 07:09 schrieb Vishwakarma, Pratik:
> Fix si_vid_is_format_supported to expose support
> for 10-bit VP9 decode using P016 format. Without
> this change, 10-bit decode will be exposed only
> for HEVC even though newer hardware support
> 10-bit decode for VP9.
>
> Signed-off-by: Pratik Vishwakarma <Pratik.Vishwakarma@amd.com>

Reviewed-by: Christian König <christian.koenig@amd.com>

> ---
>   src/gallium/drivers/radeonsi/si_get.c | 4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/src/gallium/drivers/radeonsi/si_get.c b/src/gallium/drivers/radeonsi/si_get.c
> index 4e23d283ab7..8cc5933f9bc 100644
> --- a/src/gallium/drivers/radeonsi/si_get.c
> +++ b/src/gallium/drivers/radeonsi/si_get.c
> @@ -709,6 +709,10 @@ static boolean si_vid_is_format_supported(struct pipe_screen *screen,
>   		return (format == PIPE_FORMAT_NV12) ||
>   			(format == PIPE_FORMAT_P016);
>   
> +	/* Vp9 profile 2 supports 10 bit decoding using P016 */
> +	if (profile == PIPE_VIDEO_PROFILE_VP9_PROFILE2)
> +		return format == PIPE_FORMAT_P016;
> +
>   	/* we can only handle this one with UVD */
>   	if (profile != PIPE_VIDEO_PROFILE_UNKNOWN)
>   		return format == PIPE_FORMAT_NV12;
On 6/28/2019 12:32 PM, Christian König wrote:
> Am 28.06.19 um 07:09 schrieb Vishwakarma, Pratik:

>> Fix si_vid_is_format_supported to expose support

>> for 10-bit VP9 decode using P016 format. Without

>> this change, 10-bit decode will be exposed only

>> for HEVC even though newer hardware support

>> 10-bit decode for VP9.

>>

>> Signed-off-by: Pratik Vishwakarma <Pratik.Vishwakarma@amd.com>

>

> Reviewed-by: Christian König <christian.koenig@amd.com>


Thanks Christian. How to get this patch merged?

>

>> ---

>>   src/gallium/drivers/radeonsi/si_get.c | 4 ++++

>>   1 file changed, 4 insertions(+)

>>

>> diff --git a/src/gallium/drivers/radeonsi/si_get.c 

>> b/src/gallium/drivers/radeonsi/si_get.c

>> index 4e23d283ab7..8cc5933f9bc 100644

>> --- a/src/gallium/drivers/radeonsi/si_get.c

>> +++ b/src/gallium/drivers/radeonsi/si_get.c

>> @@ -709,6 +709,10 @@ static boolean si_vid_is_format_supported(struct 

>> pipe_screen *screen,

>>           return (format == PIPE_FORMAT_NV12) ||

>>               (format == PIPE_FORMAT_P016);

>>   +    /* Vp9 profile 2 supports 10 bit decoding using P016 */

>> +    if (profile == PIPE_VIDEO_PROFILE_VP9_PROFILE2)

>> +        return format == PIPE_FORMAT_P016;

>> +

>>       /* we can only handle this one with UVD */

>>       if (profile != PIPE_VIDEO_PROFILE_UNKNOWN)

>>           return format == PIPE_FORMAT_NV12;

>