mbox series

[v2,0/5] MediaTek pinctrl drivers cleanups

Message ID 20220222111144.20796-1-angelogioacchino.delregno@collabora.com (mailing list archive)
Headers show
Series MediaTek pinctrl drivers cleanups | expand

Message

AngeloGioacchino Del Regno Feb. 22, 2022, 11:11 a.m. UTC
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.

Changes in v2:
- Patch 2/5: Changed -ENOENT to -ENODEV as per Chen-Yu's suggestion
- Patch 4/5: Removed unnoticed wrong comment change (Chen-Yu)

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 | 61 ++++++++++++++-----
 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, 164 insertions(+), 285 deletions(-)

Comments

AngeloGioacchino Del Regno March 16, 2022, 10:57 a.m. UTC | #1
Il 22/02/22 12:11, AngeloGioacchino Del Regno ha scritto:
> 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.
> 
> Changes in v2:
> - Patch 2/5: Changed -ENOENT to -ENODEV as per Chen-Yu's suggestion
> - Patch 4/5: Removed unnoticed wrong comment change (Chen-Yu)
> 
> 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 | 61 ++++++++++++++-----
>   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, 164 insertions(+), 285 deletions(-)
> 


Hello maintainers,

friendly ping for this series that already got a Reviewed-by tag on all patches :)

Cheers,
Angelo
Linus Walleij March 17, 2022, 1:07 a.m. UTC | #2
On Wed, Mar 16, 2022 at 11:57 AM AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com> wrote:

> Hello maintainers,
>
> friendly ping for this series that already got a Reviewed-by tag on all patches :)

Patches applied!

Sorry for missing them in yesterdays batch.

Yours,
Linus Walleij