[8/8] spi: Kconfig: Mark LPC32XX_SSP has BROKEN

Message ID 20190428204801.9578-9-jagan@amarulasolutions.com
State New
Headers show
Series
  • [1/8] Kconfig: Update CONFIG_BROKEN help text
Related show

Commit Message

Jagan Teki April 28, 2019, 8:48 p.m. UTC
Mark LPC32XX_SSP has BROKEN, this so the resulting build shows
warning for broken configuration enabled and associated code
will remove in v2019.07 release.

Cc: Vladimir Zapolskiy <vz@mleia.com>
Cc: Albert ARIBAUD <albert.aribaud@3adev.fr>
Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
---
 drivers/spi/Kconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Vladimir Zapolskiy April 30, 2019, 8:48 p.m. UTC | #1
Hi Jagan,

On 04/28/2019 11:48 PM, Jagan Teki wrote:
> Mark LPC32XX_SSP has BROKEN, this so the resulting build shows
> warning for broken configuration enabled and associated code
> will remove in v2019.07 release.
> 
> Cc: Vladimir Zapolskiy <vz@mleia.com>
> Cc: Albert ARIBAUD <albert.aribaud@3adev.fr>
> Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> ---
>  drivers/spi/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
> index 55f0d6cf2b..5fbe17bb20 100644
> --- a/drivers/spi/Kconfig
> +++ b/drivers/spi/Kconfig
> @@ -369,6 +369,7 @@ config KIRKWOOD_SPI
>  
>  config LPC32XX_SSP
>  	bool "LPC32XX SPI Driver"
> +	select BROKEN
>  	help
>  	  Enable support for SPI on LPC32xx
>  
> 

Acked-by: Vladimir Zapolskiy <vz@mleia.com>

Thank you for the change, as we've discussed earlier I won't have
objections against the driver removal when time is up.

Thus you can locally prepare a removal change in advance, the one
which you've sent earlier needs a minor update, please also remove
lpc32xx_ssp_init() function and its usage in the board files.

--
Best wishes,
Vladimir
Sylvain Lemieux May 9, 2019, 2:11 p.m. UTC | #2
Acked-by: Sylvain Lemieux <slemieux@tycoint.com>

On Tue, Apr 30, 2019 at 4:48 PM Vladimir Zapolskiy <vz@mleia.com> wrote:
>
> Hi Jagan,
>
> On 04/28/2019 11:48 PM, Jagan Teki wrote:
> > Mark LPC32XX_SSP has BROKEN, this so the resulting build shows
> > warning for broken configuration enabled and associated code
> > will remove in v2019.07 release.
> >
> > Cc: Vladimir Zapolskiy <vz@mleia.com>
> > Cc: Albert ARIBAUD <albert.aribaud@3adev.fr>
> > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com>
> > ---
> >  drivers/spi/Kconfig | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
> > index 55f0d6cf2b..5fbe17bb20 100644
> > --- a/drivers/spi/Kconfig
> > +++ b/drivers/spi/Kconfig
> > @@ -369,6 +369,7 @@ config KIRKWOOD_SPI
> >
> >  config LPC32XX_SSP
> >       bool "LPC32XX SPI Driver"
> > +     select BROKEN
> >       help
> >         Enable support for SPI on LPC32xx
> >
> >
>
> Acked-by: Vladimir Zapolskiy <vz@mleia.com>
>
> Thank you for the change, as we've discussed earlier I won't have
> objections against the driver removal when time is up.
>
> Thus you can locally prepare a removal change in advance, the one
> which you've sent earlier needs a minor update, please also remove
> lpc32xx_ssp_init() function and its usage in the board files.
>
> --
> Best wishes,
> Vladimir

Patch

diff --git a/drivers/spi/Kconfig b/drivers/spi/Kconfig
index 55f0d6cf2b..5fbe17bb20 100644
--- a/drivers/spi/Kconfig
+++ b/drivers/spi/Kconfig
@@ -369,6 +369,7 @@  config KIRKWOOD_SPI
 
 config LPC32XX_SSP
 	bool "LPC32XX SPI Driver"
+	select BROKEN
 	help
 	  Enable support for SPI on LPC32xx