From: Geert Uytterhoeven <geert+renesas@glider.be>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: linux-renesas-soc@vger.kernel.org, linux-gpio@vger.kernel.org,
Geert Uytterhoeven <geert+renesas@glider.be>
Subject: [PATCH 00/10] pinctrl: renesas: checker: Miscellaneous improvements
Date: Thu, 23 Dec 2021 15:56:16 +0100 [thread overview]
Message-ID: <cover.1640270559.git.geert+renesas@glider.be> (raw)
Hi Linus,
This patch series improves the Renesas pin control table validator
(enabled when DEBUG is defined, e.g. with CONFIG_DEBUG_PINCTRL=y), by:
1. Suggesting which pin groups can share pins,
2. Adding more checks to catch common errors.
If you enable the checker, you may be disappointed by the (lack of)
output. Indeed, it is much easier to fix detected issues, than to get
the checker code in a state that is suitable for upstream submission
;-)
Hence most issues detected have been fixed already for quite some time,
or never arrived upstream in the first place. Recent fixes are [1] and
[2].
If accepted, I intend to queue this in renesas-pinctrl for v5.18.
Thanks for your comments!
[1] "[PATCH 00/60] pinctrl: renesas: Share more pin group data"
https://lore.kernel.org/r/cover.1640269757.git.geert+renesas@glider.be/
[2] "[PATCH] pinctrl: renesas: r8a7794: Add range checking to
.pin_to_pocctrl()"
https://lore.kernel.org/r/d23767ad7152327654192d7191f4b8ae19493966.1640269510.git.geert+renesas@glider.be/
Geert Uytterhoeven (10):
pinctrl: renesas: Remove unused pfc parameter from .pin_to_pocctrl()
pinctrl: renesas: Factor out .pin_to_portcr() address handling
pinctrl: renesas: Pass sh_pfc_soc_info to rcar_pin_to_bias_reg()
pinctrl: renesas: checker: Simplify same_name()
pinctrl: renesas: checker: Add pin group sharing checks
pinctrl: renesas: checker: Validate bias configs consistency
pinctrl: renesas: checker: Validate drive strength configs consistency
pinctrl: renesas: checker: Validate I/O voltage configs consistency
pinctrl: renesas: checker: Check bias pin conflicts
pinctrl: renesas: checker: Check drive pin conflicts
drivers/pinctrl/renesas/core.c | 208 ++++++++++++++++++++++---
drivers/pinctrl/renesas/pfc-r8a73a4.c | 4 +-
drivers/pinctrl/renesas/pfc-r8a7740.c | 6 +-
drivers/pinctrl/renesas/pfc-r8a77470.c | 3 +-
drivers/pinctrl/renesas/pfc-r8a7790.c | 2 +-
drivers/pinctrl/renesas/pfc-r8a7791.c | 2 +-
drivers/pinctrl/renesas/pfc-r8a7794.c | 2 +-
drivers/pinctrl/renesas/pfc-r8a77950.c | 3 +-
drivers/pinctrl/renesas/pfc-r8a77951.c | 3 +-
drivers/pinctrl/renesas/pfc-r8a7796.c | 3 +-
drivers/pinctrl/renesas/pfc-r8a77965.c | 3 +-
drivers/pinctrl/renesas/pfc-r8a77970.c | 3 +-
drivers/pinctrl/renesas/pfc-r8a77980.c | 3 +-
drivers/pinctrl/renesas/pfc-r8a77990.c | 3 +-
drivers/pinctrl/renesas/pfc-r8a77995.c | 4 +-
drivers/pinctrl/renesas/pfc-r8a779a0.c | 3 +-
drivers/pinctrl/renesas/pfc-sh73a0.c | 4 +-
drivers/pinctrl/renesas/pinctrl.c | 24 +--
drivers/pinctrl/renesas/sh_pfc.h | 6 +-
19 files changed, 227 insertions(+), 62 deletions(-)
--
2.25.1
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
next reply other threads:[~2021-12-23 14:56 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-23 14:56 Geert Uytterhoeven [this message]
2021-12-23 14:56 ` [PATCH 01/10] pinctrl: renesas: Remove unused pfc parameter from .pin_to_pocctrl() Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 02/10] pinctrl: renesas: Factor out .pin_to_portcr() address handling Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 03/10] pinctrl: renesas: Pass sh_pfc_soc_info to rcar_pin_to_bias_reg() Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 04/10] pinctrl: renesas: checker: Simplify same_name() Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 05/10] pinctrl: renesas: checker: Add pin group sharing checks Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 06/10] pinctrl: renesas: checker: Validate bias configs consistency Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 07/10] pinctrl: renesas: checker: Validate drive strength " Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 08/10] pinctrl: renesas: checker: Validate I/O voltage " Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 09/10] pinctrl: renesas: checker: Check bias pin conflicts Geert Uytterhoeven
2021-12-23 14:56 ` [PATCH 10/10] pinctrl: renesas: checker: Check drive " Geert Uytterhoeven
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1640270559.git.geert+renesas@glider.be \
--to=geert+renesas@glider.be \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).