RE: [PATCH 2/2] pinctrl: imx: Support building i.MX pinctrl driver as module
> Subject: RE: [PATCH 2/2] pinctrl: imx: Support building i.MX pinctrl driver as > module > > > From: Anson Huang > > Sent: Thursday, July 16, 2020 11:07 PM > > Subject: [PATCH 2/2] pinctrl: imx: Support building i.MX pinctrl > > driver as module > > > > S/pinctrl driver/pinctrl core driver > > This also applies for Patch 1/2. OK > > > Change PINCTRL_IMX to tristate to support loadable module build. > > > > And i.MX common pinctrl driver should depend on CONFIG_OF to make sure > > no build error when i.MX common pinctrl driver is enabled for > > different architectures without CONFIG_OF. > > > > Also add module author, description and license. > > > > Signed-off-by: Anson Huang > > --- > > drivers/pinctrl/freescale/Kconfig | 3 ++- > > drivers/pinctrl/freescale/pinctrl-imx.c | 5 + > > 2 files changed, 7 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/pinctrl/freescale/Kconfig > > b/drivers/pinctrl/freescale/Kconfig > > index 570355c..922ae4b 100644 > > --- a/drivers/pinctrl/freescale/Kconfig > > +++ b/drivers/pinctrl/freescale/Kconfig > > @@ -1,6 +1,7 @@ > > # SPDX-License-Identifier: GPL-2.0-only config PINCTRL_IMX > > - bool > > + tristate "IMX pinctrl driver" > > IMX pinctrl core driver OK > > > + depends on OF > > select GENERIC_PINCTRL_GROUPS > > select GENERIC_PINMUX_FUNCTIONS > > select GENERIC_PINCONF > > diff --git a/drivers/pinctrl/freescale/pinctrl-imx.c > > b/drivers/pinctrl/freescale/pinctrl-imx.c > > index b80c450..3eaafb6 100644 > > --- a/drivers/pinctrl/freescale/pinctrl-imx.c > > +++ b/drivers/pinctrl/freescale/pinctrl-imx.c > > @@ -11,6 +11,7 @@ > > #include > > #include > > #include > > +#include > > #include > > #include > > #include > > @@ -898,3 +899,7 @@ const struct dev_pm_ops imx_pinctrl_pm_ops = { > > imx_pinctrl_resume) > > }; > > EXPORT_SYMBOL_GPL(imx_pinctrl_pm_ops); > > + > > +MODULE_AUTHOR("Linus Walleij "); > > MODULE_AUTHOR("Dong Aisheng "); > OK. Anson
RE: [PATCH 2/2] pinctrl: imx: Support building i.MX pinctrl driver as module
> From: Anson Huang > Sent: Thursday, July 16, 2020 11:07 PM > Subject: [PATCH 2/2] pinctrl: imx: Support building i.MX pinctrl driver as > module > S/pinctrl driver/pinctrl core driver This also applies for Patch 1/2. > Change PINCTRL_IMX to tristate to support loadable module build. > > And i.MX common pinctrl driver should depend on CONFIG_OF to make sure no > build error when i.MX common pinctrl driver is enabled for different > architectures without CONFIG_OF. > > Also add module author, description and license. > > Signed-off-by: Anson Huang > --- > drivers/pinctrl/freescale/Kconfig | 3 ++- > drivers/pinctrl/freescale/pinctrl-imx.c | 5 + > 2 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/pinctrl/freescale/Kconfig > b/drivers/pinctrl/freescale/Kconfig > index 570355c..922ae4b 100644 > --- a/drivers/pinctrl/freescale/Kconfig > +++ b/drivers/pinctrl/freescale/Kconfig > @@ -1,6 +1,7 @@ > # SPDX-License-Identifier: GPL-2.0-only config PINCTRL_IMX > - bool > + tristate "IMX pinctrl driver" IMX pinctrl core driver > + depends on OF > select GENERIC_PINCTRL_GROUPS > select GENERIC_PINMUX_FUNCTIONS > select GENERIC_PINCONF > diff --git a/drivers/pinctrl/freescale/pinctrl-imx.c > b/drivers/pinctrl/freescale/pinctrl-imx.c > index b80c450..3eaafb6 100644 > --- a/drivers/pinctrl/freescale/pinctrl-imx.c > +++ b/drivers/pinctrl/freescale/pinctrl-imx.c > @@ -11,6 +11,7 @@ > #include > #include > #include > +#include > #include > #include > #include > @@ -898,3 +899,7 @@ const struct dev_pm_ops imx_pinctrl_pm_ops = { > imx_pinctrl_resume) > }; > EXPORT_SYMBOL_GPL(imx_pinctrl_pm_ops); > + > +MODULE_AUTHOR("Linus Walleij "); MODULE_AUTHOR("Dong Aisheng "); Regards Aisheng > +MODULE_DESCRIPTION("NXP i.MX common pinctrl driver"); > +MODULE_LICENSE("GPL v2"); > -- > 2.7.4