Re: [PATCH v9 12/15] media: mtk-vcodec: enc: Remove mtk_vcodec_release_enc_pm

From: Yong Wu
Date: Mon Jan 17 2022 - 04:41:48 EST


On Thu, 2022-01-13 at 17:10 +0100, Matthias Brugger wrote:
> Hi Hans,
>
> On 13/01/2022 11:15, Hans Verkuil wrote:
> > On 13/01/2022 11:11, AngeloGioacchino Del Regno wrote:
> > > Il 11/01/22 11:57, AngeloGioacchino Del Regno ha scritto:
> > > > Il 12/11/21 11:55, Yong Wu ha scritto:
> > > > > After this patchset, mtk_vcodec_release_enc_pm has only one
> > > > > line.
> > > > > then remove that function, use pm_runtime_disable instead.
> > > > >
> > > > > meanwhile, mtk_vcodec_init_enc_pm only operate for the
> > > > > clocks,
> > > > > rename it from the _pm to _clk.
> > > > >
> > > > > No functional change.
> > > > >
> > > > > CC: Tiffany Lin <tiffany.lin@xxxxxxxxxxxx>
> > > > > CC: Irui Wang <irui.wang@xxxxxxxxxxxx>
> > > > > Signed-off-by: Yong Wu <yong.wu@xxxxxxxxxxxx>
> > > >
> > > > Reviewed-by: AngeloGioacchino Del Regno <
> > > > angelogioacchino.delregno@xxxxxxxxxxxxx>
> > > >
> > >
> > > Hello Yong,
> > > the mtk-vcodec patches were merged in Yunfei's vcodec patch
> > > series and Hans has
> > > scheduled that for v5.18.
> > >
> > > Can you please send a v10 and drop patches 10/15, 11/15, 12/15
> > > (all of the
> > > media: mtk-vcodec: *) from this series?

Hi Angelo,

I have dropped 11/15 and 12/15 in v10 and rebase 10/15 on media tree
manually, thus I didn't keep your A-y for that one.

Please take a look at that one.

Thanks.

> > >
> > > For the records, I think that after sending v10 this series is
> > > ready to be merged,
> > > as it was well reviewed and also tested on many MTK platforms.
> >
> > Good to know. When I have the v10 I'll try to prioritize reviewing
> > it and running
> > my usual tests.

Hi Hans/Matthias,

v10:
https://patchwork.linuxtv.org/project/linux-media/list/?series=7130

Please help take a look that one.

> >
> > Yong, please make sure you run 'checkpatch.pl --strict' over the
> > v10 patches and fix
> > any issues (using common sense).

Have run this on v10.

Thanks.
> >
>
> Can you please take me in the look when you take the patches. I'll
> take the DTS
> related as soon as you queue up the others.
>
> Thanks!
> Matthias
>
> > Regards,
> >
> > Hans
> >
> > >
> > > Thank you,
> > > - Angelo