Hi Anson, Patch series mostly looks good to me. I have a comment about adding the MODULE_LICENSE. This is a pretty important change. Can you please add this change in a separate patch with a proper explanation of why it is needed. Most likely it is because it was forgotten in the previous patches. thanks, daniel. On 10.06.2020 10:57, Anson Huang wrote:
There are more and mroe requirements that SoC specific modules should be built as module in order to support generic kernel image, such as Android GKI concept. This patch series supports i.MX8 SoCs pinctrl drivers to be built as module, including i.MX8MQ/MM/MN/MP/QXP/QM/DXL SoCs, and it also supports building i.MX common pinctrl driver and i.MX SCU common pinctrl driver as module. Compared to V3, the changes are as below: - change the config dependency back to original; - use function callbacks for SCU related functions, and all drivers using SCU pinctrl driver need to initialize the function callbacks, pinctrl-imx.c will check the SCU function callback and call it when it is valid, then no build issue when PINCTRL_IMX is built in and PINCTRL_IMX_SCU is built as module. Anson Huang (9): pinctrl: imx: Support building SCU pinctrl driver as module pinctrl: imx: Support building i.MX pinctrl driver as module pinctrl: imx8mm: Support building as module pinctrl: imx8mn: Support building as module pinctrl: imx8mq: Support building as module pinctrl: imx8mp: Support building as module pinctrl: imx8qxp: Support building as module pinctrl: imx8qm: Support building as module pinctrl: imx8dxl: Support building as module drivers/pinctrl/freescale/Kconfig | 19 +++++----- drivers/pinctrl/freescale/pinctrl-imx.c | 22 ++++++----- drivers/pinctrl/freescale/pinctrl-imx.h | 57 ++++++++++++----------------- drivers/pinctrl/freescale/pinctrl-imx8dxl.c | 12 +++--- drivers/pinctrl/freescale/pinctrl-imx8mm.c | 10 ++--- drivers/pinctrl/freescale/pinctrl-imx8mn.c | 10 ++--- drivers/pinctrl/freescale/pinctrl-imx8mp.c | 10 ++--- drivers/pinctrl/freescale/pinctrl-imx8mq.c | 9 ++--- drivers/pinctrl/freescale/pinctrl-imx8qm.c | 12 +++--- drivers/pinctrl/freescale/pinctrl-imx8qxp.c | 12 +++--- drivers/pinctrl/freescale/pinctrl-scu.c | 6 +++ 11 files changed, 86 insertions(+), 93 deletions(-)
