[U-Boot] [PATCH 07/26] pinctrl: add support for MediaTek MT7628
sr at denx.de
Wed Aug 28 12:37:37 UTC 2019
On 28.08.19 14:26, Stefan Roese wrote:
> On 28.08.19 08:37, Weijie Gao wrote:
>> This patch adds pinctrl support for mt7628, with a file for common pinmux
>> functions and a file for mt7628 which has additional support for pinconf.
>> Signed-off-by: Weijie Gao <weijie.gao at mediatek.com>
>> drivers/pinctrl/Kconfig | 1 +
>> drivers/pinctrl/Makefile | 1 +
>> drivers/pinctrl/mtmips/Kconfig | 13 +
>> drivers/pinctrl/mtmips/Makefile | 7 +
>> drivers/pinctrl/mtmips/pinctrl-mt7628.c | 585 ++++++++++++++++++
>> .../pinctrl/mtmips/pinctrl-mtmips-common.c | 87 +++
>> .../pinctrl/mtmips/pinctrl-mtmips-common.h | 53 ++
>> 7 files changed, 747 insertions(+)
>> create mode 100644 drivers/pinctrl/mtmips/Kconfig
>> create mode 100644 drivers/pinctrl/mtmips/Makefile
>> create mode 100644 drivers/pinctrl/mtmips/pinctrl-mt7628.c
>> create mode 100644 drivers/pinctrl/mtmips/pinctrl-mtmips-common.c
>> create mode 100644 drivers/pinctrl/mtmips/pinctrl-mtmips-common.h
> Nice patch. I do have 2 questions though:
> a) Why are you introducing a new "mtmips" directory and don't re-use
> the already available "mediatek" directory? Is there nothing in
> common with these "mediatek" drivers?
> b) Somewhat related: You introduce a mtmips-common file. For which
> platforms is this targeted (non-mt7628)? Is there nothing in common
> with the "mediatek" files already available?
> Other than that I've tested this on my MT7688 board and it works
> just fine. Thanks a lot!
I do have another comment though:
I've used the common "pinctrl-single" driver in Linux a few weeks ago as
there is no need for a separate MT7628 specific pin-mux driver  etc.
Frankly, I don't know that status of the "pinctrl-single" U-Boot driver
in depth. If its compatible with the Linux one (which I really hope), then
we don't need a MT7628 specific pinctrl driver but can use the "pinctrl-single"
driver as I've done in the Linux .
It would be great if you could check this and change this pinctrl support
to the common "single" driver is possible.
More information about the U-Boot