mbox series

[v2,0/7] clk: mediatek: use mtk_clk_simple_probe to simplify driver

Message ID 20220524083843.9994-1-miles.chen@mediatek.com (mailing list archive)
Headers show
Series clk: mediatek: use mtk_clk_simple_probe to simplify driver | expand

Message

Miles Chen May 24, 2022, 8:38 a.m. UTC
Hi everyone,

This is v2 patches to use mtk_clk_simple_probe to simplify driver.

Change since v1:
- Angelo's patch [1] already exported mtk_clk_simple_probe and mtk_clk_simple_remove,
so drop this part from my v1 and depends on [1].
- add a reviewed-by tag

mtk_clk_simple_probe() was introduced by Chun-Jie in commit
c58cd0e40ffa ("clk: mediatek: Add mtk_clk_simple_probe() to simplify clock providers")
and mtk_clk_simple_remove() was added in commit
commit 2204d96b1919 ("clk: mediatek: Add mtk_clk_simple_remove()")

With mtk_clk_simple_probe and mtk_clk_simple_remove,
we can get rid of defining the same probe function in clk drivers.
mtk_clk_simple_probe and mtk_clk_simple_remove are used in new platforms
now, let's apply them to old SoCs.

The process is quite straightforward: 
a. Look up probe functions which can be replaced by mtk_clk_simple_probe()
b. Add mtk_clk_desc and add .data
c. Replace the probe function with mtk_clk_simple_probe and mtk_clk_simple_remove

I tested the series on MT8192 and MT6779.
This patch is based on next-20220523 and [1].

Thanks,
Miles

[1] https://patchwork.kernel.org/project/linux-mediatek/patch/20220518111652.223727-7-angelogioacchino.delregno@collabora.com/



*** BLURB HERE ***

Miles Chen (7):
  clk: mediatek: mt2701: use mtk_clk_simple_probe to simplify driver
  clk: mediatek: mt2712: use mtk_clk_simple_probe to simplify driver
  clk: mediatek: mt6765: use mtk_clk_simple_probe to simplify driver
  clk: mediatek: mt6779: use mtk_clk_simple_probe to simplify driver
  clk: mediatek: mt6797: use mtk_clk_simple_probe to simplify driver
  clk: mediatek: mt8183: use mtk_clk_simple_probe to simplify driver
  clk: mediatek: mt8192: add mtk_clk_simple_remove

 drivers/clk/mediatek/clk-mt2701-bdp.c         | 36 +++++++------------
 drivers/clk/mediatek/clk-mt2701-img.c         | 36 +++++++------------
 drivers/clk/mediatek/clk-mt2701-vdec.c        | 36 +++++++------------
 drivers/clk/mediatek/clk-mt2712-bdp.c         | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt2712-img.c         | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt2712-jpgdec.c      | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt2712-mfg.c         | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt2712-vdec.c        | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt2712-venc.c        | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt6765-audio.c       | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt6765-cam.c         | 33 +++++++----------
 drivers/clk/mediatek/clk-mt6765-img.c         | 33 +++++++----------
 drivers/clk/mediatek/clk-mt6765-mipi0a.c      | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt6765-mm.c          | 33 +++++++----------
 drivers/clk/mediatek/clk-mt6765-vcodec.c      | 34 +++++++-----------
 drivers/clk/mediatek/clk-mt6779-aud.c         | 29 +++++++--------
 drivers/clk/mediatek/clk-mt6779-cam.c         | 29 +++++++--------
 drivers/clk/mediatek/clk-mt6779-img.c         | 29 +++++++--------
 drivers/clk/mediatek/clk-mt6779-ipe.c         | 29 +++++++--------
 drivers/clk/mediatek/clk-mt6779-mfg.c         | 27 +++++++-------
 drivers/clk/mediatek/clk-mt6779-vdec.c        | 29 +++++++--------
 drivers/clk/mediatek/clk-mt6779-venc.c        | 29 +++++++--------
 drivers/clk/mediatek/clk-mt6797-img.c         | 36 +++++++------------
 drivers/clk/mediatek/clk-mt6797-vdec.c        | 36 +++++++------------
 drivers/clk/mediatek/clk-mt6797-venc.c        | 36 +++++++------------
 drivers/clk/mediatek/clk-mt8183-cam.c         | 27 +++++++-------
 drivers/clk/mediatek/clk-mt8183-img.c         | 27 +++++++-------
 drivers/clk/mediatek/clk-mt8183-ipu0.c        | 27 +++++++-------
 drivers/clk/mediatek/clk-mt8183-ipu1.c        | 27 +++++++-------
 drivers/clk/mediatek/clk-mt8183-ipu_adl.c     | 27 +++++++-------
 drivers/clk/mediatek/clk-mt8183-ipu_conn.c    | 27 +++++++-------
 drivers/clk/mediatek/clk-mt8183-mfgcfg.c      | 29 +++++++--------
 drivers/clk/mediatek/clk-mt8183-vdec.c        | 27 +++++++-------
 drivers/clk/mediatek/clk-mt8183-venc.c        | 27 +++++++-------
 drivers/clk/mediatek/clk-mt8192-cam.c         |  1 +
 drivers/clk/mediatek/clk-mt8192-img.c         |  1 +
 .../clk/mediatek/clk-mt8192-imp_iic_wrap.c    |  1 +
 drivers/clk/mediatek/clk-mt8192-ipe.c         |  1 +
 drivers/clk/mediatek/clk-mt8192-mdp.c         |  1 +
 drivers/clk/mediatek/clk-mt8192-mfg.c         |  1 +
 drivers/clk/mediatek/clk-mt8192-msdc.c        |  1 +
 drivers/clk/mediatek/clk-mt8192-scp_adsp.c    |  1 +
 drivers/clk/mediatek/clk-mt8192-vdec.c        |  1 +
 drivers/clk/mediatek/clk-mt8192-venc.c        |  1 +
 44 files changed, 430 insertions(+), 647 deletions(-)

Comments

AngeloGioacchino Del Regno May 24, 2022, 9:37 a.m. UTC | #1
Il 24/05/22 10:38, Miles Chen ha scritto:
> Hi everyone,
> 
> This is v2 patches to use mtk_clk_simple_probe to simplify driver.
> 
> Change since v1:
> - Angelo's patch [1] already exported mtk_clk_simple_probe and mtk_clk_simple_remove,
> so drop this part from my v1 and depends on [1].
> - add a reviewed-by tag
> 
> mtk_clk_simple_probe() was introduced by Chun-Jie in commit
> c58cd0e40ffa ("clk: mediatek: Add mtk_clk_simple_probe() to simplify clock providers")
> and mtk_clk_simple_remove() was added in commit
> commit 2204d96b1919 ("clk: mediatek: Add mtk_clk_simple_remove()")
> 
> With mtk_clk_simple_probe and mtk_clk_simple_remove,
> we can get rid of defining the same probe function in clk drivers.
> mtk_clk_simple_probe and mtk_clk_simple_remove are used in new platforms
> now, let's apply them to old SoCs.
> 
> The process is quite straightforward:
> a. Look up probe functions which can be replaced by mtk_clk_simple_probe()
> b. Add mtk_clk_desc and add .data
> c. Replace the probe function with mtk_clk_simple_probe and mtk_clk_simple_remove
> 
> I tested the series on MT8192 and MT6779.
> This patch is based on next-20220523 and [1].
> 
> Thanks,
> Miles
> 
> [1] https://patchwork.kernel.org/project/linux-mediatek/patch/20220518111652.223727-7-angelogioacchino.delregno@collabora.com/
> 

For the entire series:

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

> 
> Miles Chen (7):
>    clk: mediatek: mt2701: use mtk_clk_simple_probe to simplify driver
>    clk: mediatek: mt2712: use mtk_clk_simple_probe to simplify driver
>    clk: mediatek: mt6765: use mtk_clk_simple_probe to simplify driver
>    clk: mediatek: mt6779: use mtk_clk_simple_probe to simplify driver
>    clk: mediatek: mt6797: use mtk_clk_simple_probe to simplify driver
>    clk: mediatek: mt8183: use mtk_clk_simple_probe to simplify driver
>    clk: mediatek: mt8192: add mtk_clk_simple_remove
> 
>   drivers/clk/mediatek/clk-mt2701-bdp.c         | 36 +++++++------------
>   drivers/clk/mediatek/clk-mt2701-img.c         | 36 +++++++------------
>   drivers/clk/mediatek/clk-mt2701-vdec.c        | 36 +++++++------------
>   drivers/clk/mediatek/clk-mt2712-bdp.c         | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt2712-img.c         | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt2712-jpgdec.c      | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt2712-mfg.c         | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt2712-vdec.c        | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt2712-venc.c        | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt6765-audio.c       | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt6765-cam.c         | 33 +++++++----------
>   drivers/clk/mediatek/clk-mt6765-img.c         | 33 +++++++----------
>   drivers/clk/mediatek/clk-mt6765-mipi0a.c      | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt6765-mm.c          | 33 +++++++----------
>   drivers/clk/mediatek/clk-mt6765-vcodec.c      | 34 +++++++-----------
>   drivers/clk/mediatek/clk-mt6779-aud.c         | 29 +++++++--------
>   drivers/clk/mediatek/clk-mt6779-cam.c         | 29 +++++++--------
>   drivers/clk/mediatek/clk-mt6779-img.c         | 29 +++++++--------
>   drivers/clk/mediatek/clk-mt6779-ipe.c         | 29 +++++++--------
>   drivers/clk/mediatek/clk-mt6779-mfg.c         | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt6779-vdec.c        | 29 +++++++--------
>   drivers/clk/mediatek/clk-mt6779-venc.c        | 29 +++++++--------
>   drivers/clk/mediatek/clk-mt6797-img.c         | 36 +++++++------------
>   drivers/clk/mediatek/clk-mt6797-vdec.c        | 36 +++++++------------
>   drivers/clk/mediatek/clk-mt6797-venc.c        | 36 +++++++------------
>   drivers/clk/mediatek/clk-mt8183-cam.c         | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt8183-img.c         | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt8183-ipu0.c        | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt8183-ipu1.c        | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt8183-ipu_adl.c     | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt8183-ipu_conn.c    | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt8183-mfgcfg.c      | 29 +++++++--------
>   drivers/clk/mediatek/clk-mt8183-vdec.c        | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt8183-venc.c        | 27 +++++++-------
>   drivers/clk/mediatek/clk-mt8192-cam.c         |  1 +
>   drivers/clk/mediatek/clk-mt8192-img.c         |  1 +
>   .../clk/mediatek/clk-mt8192-imp_iic_wrap.c    |  1 +
>   drivers/clk/mediatek/clk-mt8192-ipe.c         |  1 +
>   drivers/clk/mediatek/clk-mt8192-mdp.c         |  1 +
>   drivers/clk/mediatek/clk-mt8192-mfg.c         |  1 +
>   drivers/clk/mediatek/clk-mt8192-msdc.c        |  1 +
>   drivers/clk/mediatek/clk-mt8192-scp_adsp.c    |  1 +
>   drivers/clk/mediatek/clk-mt8192-vdec.c        |  1 +
>   drivers/clk/mediatek/clk-mt8192-venc.c        |  1 +
>   44 files changed, 430 insertions(+), 647 deletions(-)
>