[PATCH RFC 2/3] pinctrl: Add driver support for Amlogic SoCs
Linus Walleij
linus.walleij at linaro.org
Fri Dec 27 09:19:07 PST 2024
Newcomers, latest patch set:
https://lore.kernel.org/linux-gpio/20241226-amlogic-pinctrl-v2-0-cdae42a67b76@amlogic.com/
I included some of the prior meson authors on the to line to see if
their mail addresses still work and if they have some feedback on this.
On Sun, Dec 22, 2024 at 10:08 AM Linus Walleij <linus.walleij at linaro.org> wrote:
> > > - Renaming drivers/pinctrl/sunxi to drivers/pinctrl/amlogic
> > > so we keep this sorted by actual vendor, sunxi is apparently
> > > yours (AMlogic:s) isn't it?
> >
> > It isn't. Sunxi is Allwinner SoCs.
>
> My apologies. I mixed it up completely. :(
But wait a minute. I see there is meson. And in the "meson" subdirectory
there is stuff named "amlogic" ...
$ ls -1 drivers/pinctrl/meson/
Kconfig
Makefile
pinctrl-amlogic-c3.c
pinctrl-amlogic-t7.c
pinctrl-meson8b.c
pinctrl-meson8.c
pinctrl-meson8-pmx.c
pinctrl-meson8-pmx.h
pinctrl-meson-a1.c
pinctrl-meson-axg.c
pinctrl-meson-axg-pmx.c
pinctrl-meson-axg-pmx.h
pinctrl-meson.c
pinctrl-meson-g12a.c
pinctrl-meson-gxbb.c
pinctrl-meson-gxl.c
pinctrl-meson.h
pinctrl-meson-s4.c
> What do you think of the idea of a separate drivers/pinctrl/amlogic directory
> though? I think there are already quite a few amlogic SoCs that need
> to be supported and more will come.
So what about renaming the existing subdir "meson" to "amlogic"
and put the driver there.
Also I want to know if this driver and hardware shares anything with
the existing drivers in that directory. It sometimes happen that
developers start something from scratch despite the existence of
prior art simply because of organizational issues, and we don't want
that kind of situation to leak over to the kernel.
Yours,
Linus Walleij
More information about the linux-arm-kernel
mailing list