diff mbox

[U-Boot,06/20] sf: Kconfig: Add SPI_FLASH_MTD entry

Message ID 1435428843-29750-7-git-send-email-jteki@openedev.com
State Accepted
Delegated to: Jagannadha Sutradharudu Teki
Headers show

Commit Message

Jagan Teki June 27, 2015, 6:13 p.m. UTC
Added SPI_FLASH_MTD entry on Kconfig with help description.

Signed-off-by: Jagan Teki <jteki@openedev.com>
Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
Cc: Heiko Schocher <hs@denx.de>
---
 drivers/mtd/spi/Kconfig | 12 ++++++++++++
 1 file changed, 12 insertions(+)

Comments

Daniel Schwierzeck June 29, 2015, 3:55 p.m. UTC | #1
2015-06-27 20:13 GMT+02:00 Jagan Teki <jteki@openedev.com>:
> Added SPI_FLASH_MTD entry on Kconfig with help description.
>
> Signed-off-by: Jagan Teki <jteki@openedev.com>
> Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
> Cc: Heiko Schocher <hs@denx.de>
> ---
>  drivers/mtd/spi/Kconfig | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/drivers/mtd/spi/Kconfig b/drivers/mtd/spi/Kconfig
> index 9536aac..217e4d7 100644
> --- a/drivers/mtd/spi/Kconfig
> +++ b/drivers/mtd/spi/Kconfig
> @@ -35,4 +35,16 @@ config SPI_FLASH
>
>           If unsure, say N
>
> +config SPI_FLASH_MTD
> +       bool "SPI Flash MTD support"
> +       depends on SPI_FLASH
> +       help
> +          Enable the MTD support for spi flash layer, this adapter is for
> +         translating mtd_read/mtd_write commands into spi_flash_read/write
> +         commands. It is not intended to use it within sf_cmd or the SPI
> +         flash subsystem. Such an adapter is needed for subsystems like
> +         UBI which can only operate on top of the MTD layer.
> +
> +         If unsure, say N
> +
>  endmenu # menu "SPI Flash Support"
> --
> 1.9.1
>

looks good. Maybe you also should remove that help text from the README file.

Acked-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
Jagan Teki June 29, 2015, 4 p.m. UTC | #2
On 29 June 2015 at 21:25, Daniel Schwierzeck
<daniel.schwierzeck@gmail.com> wrote:
> 2015-06-27 20:13 GMT+02:00 Jagan Teki <jteki@openedev.com>:
>> Added SPI_FLASH_MTD entry on Kconfig with help description.
>>
>> Signed-off-by: Jagan Teki <jteki@openedev.com>
>> Cc: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
>> Cc: Heiko Schocher <hs@denx.de>
>> ---
>>  drivers/mtd/spi/Kconfig | 12 ++++++++++++
>>  1 file changed, 12 insertions(+)
>>
>> diff --git a/drivers/mtd/spi/Kconfig b/drivers/mtd/spi/Kconfig
>> index 9536aac..217e4d7 100644
>> --- a/drivers/mtd/spi/Kconfig
>> +++ b/drivers/mtd/spi/Kconfig
>> @@ -35,4 +35,16 @@ config SPI_FLASH
>>
>>           If unsure, say N
>>
>> +config SPI_FLASH_MTD
>> +       bool "SPI Flash MTD support"
>> +       depends on SPI_FLASH
>> +       help
>> +          Enable the MTD support for spi flash layer, this adapter is for
>> +         translating mtd_read/mtd_write commands into spi_flash_read/write
>> +         commands. It is not intended to use it within sf_cmd or the SPI
>> +         flash subsystem. Such an adapter is needed for subsystems like
>> +         UBI which can only operate on top of the MTD layer.
>> +
>> +         If unsure, say N
>> +
>>  endmenu # menu "SPI Flash Support"
>> --
>> 1.9.1
>>
>
> looks good. Maybe you also should remove that help text from the README file.

Thought the same, will send the patch.

>
> Acked-by: Daniel Schwierzeck <daniel.schwierzeck@gmail.com>
>

thanks!
diff mbox

Patch

diff --git a/drivers/mtd/spi/Kconfig b/drivers/mtd/spi/Kconfig
index 9536aac..217e4d7 100644
--- a/drivers/mtd/spi/Kconfig
+++ b/drivers/mtd/spi/Kconfig
@@ -35,4 +35,16 @@  config SPI_FLASH
 
 	  If unsure, say N
 
+config SPI_FLASH_MTD
+	bool "SPI Flash MTD support"
+	depends on SPI_FLASH
+	help
+          Enable the MTD support for spi flash layer, this adapter is for
+	  translating mtd_read/mtd_write commands into spi_flash_read/write
+	  commands. It is not intended to use it within sf_cmd or the SPI
+	  flash subsystem. Such an adapter is needed for subsystems like
+	  UBI which can only operate on top of the MTD layer.
+
+	  If unsure, say N
+
 endmenu # menu "SPI Flash Support"