[PATCH] spi-nor: Kconfig: Remove unneeded help message

[PATCH] spi-nor: Kconfig: Remove unneeded help message

[Fabio Estevam]

From: Fabio Estevam <fabio.estevam@freescale.com>

The "We only connect the NOR to this controller now" phrase
is not really useful for a Kconfig help text, so remove it.

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 drivers/mtd/spi-nor/Kconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/mtd/spi-nor/Kconfig b/drivers/mtd/spi-nor/Kconfig
index 64a4f0e..baa8442 100644
--- a/drivers/mtd/spi-nor/Kconfig
+++ b/drivers/mtd/spi-nor/Kconfig
@@ -26,6 +26,5 @@ config SPI_FSL_QUADSPI
 	depends on ARCH_MXC
 	help
 	  This enables support for the Quad SPI controller in master mode.
-	  We only connect the NOR to this controller now.
 
 endif # MTD_SPI_NOR
-- 
1.9.1

Re: [PATCH] spi-nor: Kconfig: Remove unneeded help message

[Brian Norris]

On Wed, Aug 12, 2015 at 08:28:19AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> The "We only connect the NOR to this controller now" phrase
> is not really useful for a Kconfig help text, so remove it.
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  drivers/mtd/spi-nor/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/mtd/spi-nor/Kconfig b/drivers/mtd/spi-nor/Kconfig
> index 64a4f0e..baa8442 100644
> --- a/drivers/mtd/spi-nor/Kconfig
> +++ b/drivers/mtd/spi-nor/Kconfig
> @@ -26,6 +26,5 @@ config SPI_FSL_QUADSPI
>  	depends on ARCH_MXC
>  	help
>  	  This enables support for the Quad SPI controller in master mode.
> -	  We only connect the NOR to this controller now.

I'm inclined to accept this patch, but I'm curious what the intention
for the original statement was. Does this mean "this controller could
support generic SPI, but this driver only bothers to implement the NOR
flash protocol"?

>  
>  endif # MTD_SPI_NOR
> -- 
> 1.9.1
> 

Brian

RE: [PATCH] spi-nor: Kconfig: Remove unneeded help message

[Li Frank]

> -----Original Message-----
> From: linux-mtd [mailto:linux-mtd-bounces@lists.infradead.org] On Behalf Of
> Brian Norris
> Sent: Wednesday, August 12, 2015 10:13 PM
> To: Fabio Estevam
> Cc: Estevam Fabio-R49496; Huang Shijie; linux-mtd@lists.infradead.org; Xu
> Han-B45815
> Subject: Re: [PATCH] spi-nor: Kconfig: Remove unneeded help message
> 
> On Wed, Aug 12, 2015 at 08:28:19AM -0300, Fabio Estevam wrote:
> > From: Fabio Estevam <fabio.estevam@freescale.com>
> >
> > The "We only connect the NOR to this controller now" phrase is not
> > really useful for a Kconfig help text, so remove it.
> >
> > Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> > ---
> >  drivers/mtd/spi-nor/Kconfig | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/drivers/mtd/spi-nor/Kconfig b/drivers/mtd/spi-nor/Kconfig
> > index 64a4f0e..baa8442 100644
> > --- a/drivers/mtd/spi-nor/Kconfig
> > +++ b/drivers/mtd/spi-nor/Kconfig
> > @@ -26,6 +26,5 @@ config SPI_FSL_QUADSPI
> >  	depends on ARCH_MXC
> >  	help
> >  	  This enables support for the Quad SPI controller in master mode.
> > -	  We only connect the NOR to this controller now.
> 
> I'm inclined to accept this patch, but I'm curious what the intention for the
> original statement was. Does this mean "this controller could support generic
> SPI, but this driver only bothers to implement the NOR flash protocol"?

This controller doesn't support generic SPI. 
Just support connect to SPI NOR flash.

Best regards
Frank Li

> 
> >
> >  endif # MTD_SPI_NOR
> > --
> > 1.9.1
> >
> 
> Brian
> 
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/

Re: [PATCH] spi-nor: Kconfig: Remove unneeded help message

[Fabio Estevam]

Hi Frank,

On Thu, Aug 13, 2015 at 1:49 PM, Li Frank <Frank.Li@freescale.com> wrote:

>> I'm inclined to accept this patch, but I'm curious what the intention for the
>> original statement was. Does this mean "this controller could support generic
>> SPI, but this driver only bothers to implement the NOR flash protocol"?
>
> This controller doesn't support generic SPI.
> Just support connect to SPI NOR flash.

Thanks for the clarification.

I changed the Kconfig in v2 to make this clear as per your suggestion.

Regards,

Fabio Estevam