On Mon, Nov 2, 2020 at 2:25 AM Mansur Alisha Shaik <mansur@xxxxxxxxxxxxxx> wrote: > > Currently in calculate_inst_freq() video driver is calculating > macro blocks per frame in steam of macro blocks per second(mpbs). > Which results frequency is always setting to lower frequency (150MB) > as per frequency table for sc7180. Hence the playback is not smooth. > > Corrected this by correcting the mbps calculation. > > Signed-off-by: Mansur Alisha Shaik <mansur@xxxxxxxxxxxxxx> > --- > drivers/media/platform/qcom/venus/pm_helpers.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/platform/qcom/venus/pm_helpers.c b/drivers/media/platform/qcom/venus/pm_helpers.c > index 57877ea..001513f 100644 > --- a/drivers/media/platform/qcom/venus/pm_helpers.c > +++ b/drivers/media/platform/qcom/venus/pm_helpers.c > @@ -928,7 +928,7 @@ static unsigned long calculate_inst_freq(struct venus_inst *inst, > u32 fps = (u32)inst->fps; > u32 mbs_per_sec; > > - mbs_per_sec = load_per_instance(inst) / fps; > + mbs_per_sec = load_per_instance(inst); > > vpp_freq = mbs_per_sec * inst->clk_data.codec_freq_data->vpp_freq; > /* 21 / 20 is overhead factor */ > -- > QUALCOMM INDIA, on behalf of Qualcomm Innovation Center, Inc. is a member > of Code Aurora Forum, hosted by The Linux Foundation > Reviewed-by: Fritz Koenig <frkoenig@xxxxxxxxxxxx>
