[PATCH 0/5] MediaTek pinctrl drivers cleanups

AngeloGioacchino Del Regno posted 5 patches 2 years, 7 months ago
There is a newer version of this series
drivers/pinctrl/mediatek/pinctrl-mt2701.c     | 40 ++++--------
drivers/pinctrl/mediatek/pinctrl-mt2712.c     | 44 +++----------
drivers/pinctrl/mediatek/pinctrl-mt6765.c     |  9 +--
drivers/pinctrl/mediatek/pinctrl-mt6779.c     |  9 +--
drivers/pinctrl/mediatek/pinctrl-mt6797.c     |  9 +--
drivers/pinctrl/mediatek/pinctrl-mt8127.c     | 39 +++---------
drivers/pinctrl/mediatek/pinctrl-mt8135.c     | 17 ++---
drivers/pinctrl/mediatek/pinctrl-mt8167.c     | 40 +++---------
drivers/pinctrl/mediatek/pinctrl-mt8173.c     | 29 +++------
drivers/pinctrl/mediatek/pinctrl-mt8183.c     |  9 +--
drivers/pinctrl/mediatek/pinctrl-mt8192.c     |  9 +--
drivers/pinctrl/mediatek/pinctrl-mt8195.c     |  9 +--
drivers/pinctrl/mediatek/pinctrl-mt8365.c     | 40 +++---------
drivers/pinctrl/mediatek/pinctrl-mt8516.c     | 40 +++---------
drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 63 ++++++++++++++-----
drivers/pinctrl/mediatek/pinctrl-mtk-common.h | 33 +++++++---
drivers/pinctrl/mediatek/pinctrl-paris.c      |  9 ++-
drivers/pinctrl/mediatek/pinctrl-paris.h      |  3 +-
18 files changed, 165 insertions(+), 286 deletions(-)
[PATCH 0/5] MediaTek pinctrl drivers cleanups
Posted by AngeloGioacchino Del Regno 2 years, 7 months ago
Hello!
This series performs some cleanups to the MediaTek pinctrl drivers
by commonizing some functions (reducing code duplication) hence also
simplifying the probe mechanism by removing unnecessary per-driver
probe functions.

This code was tested against multiple platforms.

AngeloGioacchino Del Regno (5):
  pinctrl: mediatek: paris: Unify probe function by using OF match data
  pinctrl: mediatek: common-v1: Add common probe function
  pinctrl: mediatek: common-v1: Use common probe function
  pinctrl: mediatek: common-v1: Commonize spec_pupd callback
  pinctrl: mediatek: common-v1: Commonize spec_ies_smt_set callback

 drivers/pinctrl/mediatek/pinctrl-mt2701.c     | 40 ++++--------
 drivers/pinctrl/mediatek/pinctrl-mt2712.c     | 44 +++----------
 drivers/pinctrl/mediatek/pinctrl-mt6765.c     |  9 +--
 drivers/pinctrl/mediatek/pinctrl-mt6779.c     |  9 +--
 drivers/pinctrl/mediatek/pinctrl-mt6797.c     |  9 +--
 drivers/pinctrl/mediatek/pinctrl-mt8127.c     | 39 +++---------
 drivers/pinctrl/mediatek/pinctrl-mt8135.c     | 17 ++---
 drivers/pinctrl/mediatek/pinctrl-mt8167.c     | 40 +++---------
 drivers/pinctrl/mediatek/pinctrl-mt8173.c     | 29 +++------
 drivers/pinctrl/mediatek/pinctrl-mt8183.c     |  9 +--
 drivers/pinctrl/mediatek/pinctrl-mt8192.c     |  9 +--
 drivers/pinctrl/mediatek/pinctrl-mt8195.c     |  9 +--
 drivers/pinctrl/mediatek/pinctrl-mt8365.c     | 40 +++---------
 drivers/pinctrl/mediatek/pinctrl-mt8516.c     | 40 +++---------
 drivers/pinctrl/mediatek/pinctrl-mtk-common.c | 63 ++++++++++++++-----
 drivers/pinctrl/mediatek/pinctrl-mtk-common.h | 33 +++++++---
 drivers/pinctrl/mediatek/pinctrl-paris.c      |  9 ++-
 drivers/pinctrl/mediatek/pinctrl-paris.h      |  3 +-
 18 files changed, 165 insertions(+), 286 deletions(-)

-- 
2.33.1

Re: [PATCH 0/5] MediaTek pinctrl drivers cleanups
Posted by Linus Walleij 2 years, 7 months ago
On Thu, Feb 10, 2022 at 3:19 PM AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com> wrote:

> Hello!
> This series performs some cleanups to the MediaTek pinctrl drivers
> by commonizing some functions (reducing code duplication) hence also
> simplifying the probe mechanism by removing unnecessary per-driver
> probe functions.

This looks fine from my Mediatek-untrained eye but
I need some help to review this, Matthias? Guodong?

Yours,
Linus Walleij