Re: [Mesa-dev] [PATCH] radeon/vcn: use enc profile instead of pic profile

2018-03-02 Thread Boyuan Zhang
On 2018-02-28 08:44 AM, Leo Liu wrote: Boyuan, please also make sure whether this patch along with other one are needed to Cc stable or not. Regards, Leo Thanks. I double checked the 18.0 and 17.3 branch, the previous hevc encode related patch sets are not in there yet. So seems we don't

Re: [Mesa-dev] [PATCH] radeon/vcn: use enc profile instead of pic profile

2018-03-02 Thread Leo Liu
On 03/01/2018 02:30 PM, Boyuan Zhang wrote: Agree, I added the missing profile and entry_point to st/omx. Please see the attached patch below. On radeon driver side, do you think we should still check the profile in encoder instead since profile shouldn't been changed during encoding. Or we

Re: [Mesa-dev] [PATCH] radeon/vcn: use enc profile instead of pic profile

2018-03-01 Thread Boyuan Zhang
Agree, I added the missing profile and entry_point to st/omx. Please see the attached patch below. On radeon driver side, do you think we should still check the profile in encoder instead since profile shouldn't been changed during encoding. Or we can just leave it with picture profile with this

Re: [Mesa-dev] [PATCH] radeon/vcn: use enc profile instead of pic profile

2018-02-28 Thread Leo Liu
Boyuan, please also make sure whether this patch along with other one are needed to Cc stable or not. Regards, Leo On 02/28/2018 03:03 AM, Christian König wrote: Am 28.02.2018 um 00:56 schrieb boyuan.zh...@amd.com: From: Boyuan Zhang Picture profile might not be set

Re: [Mesa-dev] [PATCH] radeon/vcn: use enc profile instead of pic profile

2018-02-28 Thread Christian König
Am 28.02.2018 um 00:56 schrieb boyuan.zh...@amd.com: From: Boyuan Zhang Picture profile might not be set in some cases. Therefore, better to use the profile stored in encoder base. Signed-off-by: Boyuan Zhang NAK, when the picture profile isn't

[Mesa-dev] [PATCH] radeon/vcn: use enc profile instead of pic profile

2018-02-27 Thread boyuan.zhang
From: Boyuan Zhang Picture profile might not be set in some cases. Therefore, better to use the profile stored in encoder base. Signed-off-by: Boyuan Zhang --- src/gallium/drivers/radeon/radeon_vcn_enc.c | 4 ++--