diff mbox

pinctrl: bcm: Default PINCTRL_BCM281XX to y for ARCH_BCM_MOBILE

Message ID 1449020490-27084-1-git-send-email-f.fainelli@gmail.com
State New
Headers show

Commit Message

Florian Fainelli Dec. 2, 2015, 1:41 a.m. UTC
This driver is mandatory for proper operation on the platforms covered
by the ARCH_BCM_MOBILE Kconfig symbol, make sure we do that driver on
for these platforms.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
---
 drivers/pinctrl/bcm/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Ray Jui Dec. 2, 2015, 5:42 p.m. UTC | #1
On 12/1/2015 5:41 PM, Florian Fainelli wrote:
> This driver is mandatory for proper operation on the platforms covered
> by the ARCH_BCM_MOBILE Kconfig symbol, make sure we do that driver on
> for these platforms.
>
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
> ---
>   drivers/pinctrl/bcm/Kconfig | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/pinctrl/bcm/Kconfig b/drivers/pinctrl/bcm/Kconfig
> index cd11d4d9ad58..b63d7068f26c 100644
> --- a/drivers/pinctrl/bcm/Kconfig
> +++ b/drivers/pinctrl/bcm/Kconfig
> @@ -9,6 +9,7 @@ config PINCTRL_BCM281XX
>   	select PINCONF
>   	select GENERIC_PINCONF
>   	select REGMAP_MMIO
> +	default ARCH_BCM_MOBILE
>   	help
>   	  Say Y here to support Broadcom BCM281xx pinctrl driver, which is used
>   	  for the BCM281xx SoC family, including BCM11130, BCM11140, BCM11351,
>

pinmux and pinctrl related configurations from the bcm281xx pinctrl 
driver are indeed mandatory for ARCH_BCM_MOBILE related platforms for 
some peripherals to function properly. It looks like we are missing some 
required configurations in the DT files though, but I guess that needs 
to be addressed by separate patches.

This change looks good to me!

Reviewed-by: Ray Jui <rjui@broadcom.com>

Thanks!

Ray
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Linus Walleij Dec. 10, 2015, 10:54 p.m. UTC | #2
On Wed, Dec 2, 2015 at 2:41 AM, Florian Fainelli <f.fainelli@gmail.com> wrote:

> This driver is mandatory for proper operation on the platforms covered
> by the ARCH_BCM_MOBILE Kconfig symbol, make sure we do that driver on
> for these platforms.
>
> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>

Patch applied with Ray's Review tag.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/pinctrl/bcm/Kconfig b/drivers/pinctrl/bcm/Kconfig
index cd11d4d9ad58..b63d7068f26c 100644
--- a/drivers/pinctrl/bcm/Kconfig
+++ b/drivers/pinctrl/bcm/Kconfig
@@ -9,6 +9,7 @@  config PINCTRL_BCM281XX
 	select PINCONF
 	select GENERIC_PINCONF
 	select REGMAP_MMIO
+	default ARCH_BCM_MOBILE
 	help
 	  Say Y here to support Broadcom BCM281xx pinctrl driver, which is used
 	  for the BCM281xx SoC family, including BCM11130, BCM11140, BCM11351,