[PATCH 09/13] pinctrl: exynos: Add pinctrl support for Exynos850

[Sam Protsenko]

On Tue, Dec 19, 2023 at 5:32 AM Chanho Park <chanho61.park at samsung.com> wrote:
>
> > -----Original Message-----
> > From: U-Boot <u-boot-bounces at lists.denx.de> On Behalf Of Sam Protsenko
> > Sent: Wednesday, December 13, 2023 12:17 PM
> > To: Minkyu Kang <mk7.kang at samsung.com>; Tom Rini <trini at konsulko.com>;
> > Lukasz Majewski <lukma at denx.de>; Sean Anderson <seanga2 at gmail.com>
> > Cc: Simon Glass <sjg at chromium.org>; Heinrich Schuchardt
> > <xypron.glpk at gmx.de>; u-boot at lists.denx.de
> > Subject: [PATCH 09/13] pinctrl: exynos: Add pinctrl support for Exynos850
> >
> > Add pinctrl support for Exynos850 SoC. It was mostly extracted from
> > corresponding Linux kernel code [1]. Power down modes and external
> > interrupt data were removed while converting the code for U-Boot, but
> > everything else was kept almost unchanged.
> >
> > [1] drivers/pinctrl/samsung/pinctrl-exynos-arm64.c
> >
> > Signed-off-by: Sam Protsenko <semen.protsenko at linaro.org>
> > ---
> >  drivers/pinctrl/exynos/Kconfig             |   8 ++
> >  drivers/pinctrl/exynos/Makefile            |   1 +
> >  drivers/pinctrl/exynos/pinctrl-exynos850.c | 125 +++++++++++++++++++++
> >  3 files changed, 134 insertions(+)
> >  create mode 100644 drivers/pinctrl/exynos/pinctrl-exynos850.c
> >
> > diff --git a/drivers/pinctrl/exynos/Kconfig
> > b/drivers/pinctrl/exynos/Kconfig
> > index a60f49869b45..1b7fb62bc4ba 100644
> > --- a/drivers/pinctrl/exynos/Kconfig
> > +++ b/drivers/pinctrl/exynos/Kconfig
> > @@ -16,3 +16,11 @@ config PINCTRL_EXYNOS78x0
> >       help
> >         Support pin multiplexing and pin configuration control on
> >         Samsung's Exynos78x0 SoC.
> > +
> > +config PINCTRL_EXYNOS850
> > +     bool "Samsung Exynos850 pinctrl driver"
> > +     depends on ARCH_EXYNOS && PINCTRL_FULL
> > +     select PINCTRL_EXYNOS
> > +     help
> > +       Support pin multiplexing and pin configuration control on
> > +       Samsung's Exynos850 SoC.
> > diff --git a/drivers/pinctrl/exynos/Makefile
> > b/drivers/pinctrl/exynos/Makefile
> > index 07db970ca942..3abe1226eb74 100644
> > --- a/drivers/pinctrl/exynos/Makefile
> > +++ b/drivers/pinctrl/exynos/Makefile
> > @@ -6,3 +6,4 @@
> >  obj-$(CONFIG_PINCTRL_EXYNOS)         += pinctrl-exynos.o
> >  obj-$(CONFIG_PINCTRL_EXYNOS7420)     += pinctrl-exynos7420.o
> >  obj-$(CONFIG_PINCTRL_EXYNOS78x0)     += pinctrl-exynos78x0.o
> > +obj-$(CONFIG_PINCTRL_EXYNOS850)              += pinctrl-exynos850.o
> > diff --git a/drivers/pinctrl/exynos/pinctrl-exynos850.c
> > b/drivers/pinctrl/exynos/pinctrl-exynos850.c
> > new file mode 100644
> > index 000000000000..2445dd752ea8
> > --- /dev/null
> > +++ b/drivers/pinctrl/exynos/pinctrl-exynos850.c
> > @@ -0,0 +1,125 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Copyright (c) 2023 Linaro Ltd.
> > + * Author: Sam Protsenko <semen.protsenko at linaro.org>
> > + *
> > + * Samsung Exynos USI driver (Universal Serial Interface).
>
> Typo. It should be a subject for the pinctrl driver.
>

Nice catch! Will be fixed in v2.

> Otherwise,
> Reviewed-by: Chanho Park <chanho61.park at samsung.com>

Thanks for the review!

>