Re: [PATCH v2] pwm: kconfig: enable kona pwm to be built for cygnus arch

[Scott Branden]

On 2018-11-21 11:00 a.m., Florian Fainelli wrote:
> On 11/21/2018 4:35 AM, ClÃment PÃron wrote:
> > The Cygnus architecture use a Kona PWM. This is already present
> > in the device tree but can't be built actually. Hence, allow the
> > Kona PWM to be built for Cygnus arch.
> >
> > Signed-off-by: ClÃment PÃron <peron.clem@xxxxxxxxx>
> Reviewed-by: Florian Fainelli <f.fainelli@xxxxxxxxx>

Reviewed-by: Scott Branden <scott.branden@xxxxxxxxxxxx>

> The "default ARCH_BCM_CYGNUS" could arguably cover ARCH_BCM_MOBILE as
> well, but this is not probably worth a respin just to address that.
Yes, it's fine as is.
> > ---
> >   drivers/pwm/Kconfig | 4 +++-
> >   1 file changed, 3 insertions(+), 1 deletion(-)
> >
> > diff --git a/drivers/pwm/Kconfig b/drivers/pwm/Kconfig
> > index 27e5dd47a01f..982f4059f4e8 100644
> > --- a/drivers/pwm/Kconfig
> > +++ b/drivers/pwm/Kconfig
> > @@ -88,7 +88,9 @@ config PWM_BCM_IPROC
> >   
> >   config PWM_BCM_KONA
> >   	tristate "Kona PWM support"
> > -	depends on ARCH_BCM_MOBILE
> > +	depends on ARCH_BCM_MOBILE || ARCH_BCM_CYGNUS || COMPILE_TEST
> > +	depends on HAVE_CLK && HAS_IOMEM
> > +	default ARCH_BCM_CYGNUS
> >   	help
> >   	  Generic PWM framework driver for Broadcom Kona PWM block.
> >