Message ID | 20220129220221.2823127-3-colin.foster@in-advantage.com |
---|---|
State | Superseded |
Headers | show |
Series | None | expand |
On Sat, Jan 29, 2022 at 11:02 PM Colin Foster <colin.foster@in-advantage.com> wrote: > As the commit message suggests, this simply adds the ability to select > SGPIO pinctrl as a module. This becomes more practical when the SGPIO > hardware exists on an external chip, controlled indirectly by I2C or SPI. > This commit enables that level of control. > > Signed-off-by: Colin Foster <colin.foster@in-advantage.com> Reviewed-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij
On 1/29/2022 2:02 PM, Colin Foster wrote: > As the commit message suggests, this simply adds the ability to select > SGPIO pinctrl as a module. This becomes more practical when the SGPIO > hardware exists on an external chip, controlled indirectly by I2C or SPI. > This commit enables that level of control. > > Signed-off-by: Colin Foster <colin.foster@in-advantage.com> Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
On Mon, Jan 31, 2022 at 03:11:55PM -0800, Florian Fainelli wrote: > > > On 1/29/2022 2:02 PM, Colin Foster wrote: > > As the commit message suggests, this simply adds the ability to select > > SGPIO pinctrl as a module. This becomes more practical when the SGPIO > > hardware exists on an external chip, controlled indirectly by I2C or SPI. > > This commit enables that level of control. > > > > Signed-off-by: Colin Foster <colin.foster@in-advantage.com> > > Reviewed-by: Florian Fainelli <f.fainelli@gmail.com> Thanks Florian > -- > Florian
diff --git a/drivers/pinctrl/Kconfig b/drivers/pinctrl/Kconfig index 1b367f423ceb..7ff00c560775 100644 --- a/drivers/pinctrl/Kconfig +++ b/drivers/pinctrl/Kconfig @@ -292,7 +292,7 @@ config PINCTRL_MCP23S08 corresponding interrupt-controller. config PINCTRL_MICROCHIP_SGPIO - bool "Pinctrl driver for Microsemi/Microchip Serial GPIO" + tristate "Pinctrl driver for Microsemi/Microchip Serial GPIO" depends on OF depends on HAS_IOMEM select GPIOLIB diff --git a/drivers/pinctrl/pinctrl-microchip-sgpio.c b/drivers/pinctrl/pinctrl-microchip-sgpio.c index 8e081c90bdb2..8db3caf15cf2 100644 --- a/drivers/pinctrl/pinctrl-microchip-sgpio.c +++ b/drivers/pinctrl/pinctrl-microchip-sgpio.c @@ -912,6 +912,7 @@ static const struct of_device_id microchip_sgpio_gpio_of_match[] = { /* sentinel */ } }; +MODULE_DEVICE_TABLE(of, microchip_sgpio_gpio_of_match); static struct platform_driver microchip_sgpio_pinctrl_driver = { .driver = { @@ -922,3 +923,6 @@ static struct platform_driver microchip_sgpio_pinctrl_driver = { .probe = microchip_sgpio_probe, }; builtin_platform_driver(microchip_sgpio_pinctrl_driver); + +MODULE_DESCRIPTION("Microchip SGPIO Pinctrl Driver"); +MODULE_LICENSE("GPL v2");
As the commit message suggests, this simply adds the ability to select SGPIO pinctrl as a module. This becomes more practical when the SGPIO hardware exists on an external chip, controlled indirectly by I2C or SPI. This commit enables that level of control. Signed-off-by: Colin Foster <colin.foster@in-advantage.com> --- drivers/pinctrl/Kconfig | 2 +- drivers/pinctrl/pinctrl-microchip-sgpio.c | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-)