Hi Loic, On 9/11/19 5:45 PM, Loic Poulain wrote: > In downstream driver, there are two frequency tables defined, > one for the encoder and one for the decoder: > > /* Encoders / > <972000 490000000 0x55555555>, / 4k UHD @ 30 / > <489600 320000000 0x55555555>, / 1080p @ 60 / > <244800 150000000 0x55555555>, / 1080p @ 30 / > <108000 75000000 0x55555555>, / 720p @ 30 */ > > /* Decoders / > <1944000 490000000 0xffffffff>, / 4k UHD @ 60 / > < 972000 320000000 0xffffffff>, / 4k UHD @ 30 / > < 489600 150000000 0xffffffff>, / 1080p @ 60 / > < 244800 75000000 0xffffffff>; / 1080p @ 30 */ > > It shows that encoder always needs a higher clock than decoder. > > In current venus driver, the unified frequency table is aligned > with the downstream decoder table which causes performance issues > in encoding scenarios. Fix that by aligning frequency table on > worst case (encoding). > > Signed-off-by: Loic Poulain <loic.poulain@xxxxxxxxxx> > --- > drivers/media/platform/qcom/venus/core.c | 9 +++++---- > 1 file changed, 5 insertions(+), 4 deletions(-) Acked-by: Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx> -- -- regards, Stan
