u-boot.git/drivers/mtd, branch v2020.01

mtd: spi-nor-core: Fix static checker warnings

2019-12-27T12:17:26+00:00

Static checker warns 'ret' variable may be used uninitialized in
spi_nor_erase() and spi_nor_write() in case of zero length requests.
Fix these warnings by checking for zero length requests and returning
early.

Reported-by: Dan Murphy 
Signed-off-by: Vignesh Raghavendra

mtd: spi-nor: ids: Add GigaDevice gd25q128

2019-12-18T14:35:40+00:00

Add gd25q128 128Mbit chip to spi-nor id table.

Tested on Pinebook Pro

Signed-off-by: Peter Robinson 
Acked-by: Vignesh Raghavendra 
Tested-by: Jagan Teki  # roc-rk3399-pc
Reviewed-by: Jagan Teki

mtd: Makefile: deep cleanup

2019-12-04T22:10:51+00:00

Move MTD-related lines out of the root Makefile. Put them in their
respective directories. Enclose some of these new lines to skip them
when building the SPL. MTD core files and some MTD device drivers are
compiled in a mtd.o object and included in the final object only if
MTD support is required (there are two different symbols for that, one
for U-Boot and one for the SPL).

Now that all defconfigs have been fixed, we can stop the logic where
enabling a command selects the core files to compile. This logic is
broken since selecting a symbol with a 'depends on' will not enforce
this secondary dependency.

Signed-off-by: Miquel Raynal

cmd: nand/sf: isolate legacy code

2019-12-04T22:10:51+00:00

The 'sf' command is not supposed to rely on the MTD stack, but both
'sf' and 'nand' commands use helpers located in mtd_uboot.c. Despite
their location, these functions do not depend at all on the MTD
stack.

This file (drivers/mtd/mtd_uboot.c) is only compiled if CONFIG_MTD is
selected, which is inconsistent with the current situation. Solve this
by moving these three functions (which are only used by the above two
commands) out of mtd_uboot.c and put them in a C file only compiled
with cmd/sf.c and cmd/nand.c.

Signed-off-by: Miquel Raynal 
[trini: Don't export get_part function now]
Signed-off-by: Tom Rini

mtd: spi: SPI_FLASH_MTD depends on MTD

2019-12-04T22:10:51+00:00

It is already the case that all defconfigs with SPI_FLASH_MTD also
declare using MTD, but let's make this consistent and enforce it in
Kconfig. Most of the time SPI_FLASH_MTD is used in conjunction with
UBI, which already depends on MTD.

Suggested-by: Vignesh R 
Signed-off-by: Miquel Raynal

mtd: nand: add includes in NAND core to avoid warnings

2019-12-04T22:10:51+00:00

Because of the include's game, when some files are compiled for a SPI
NAND device, no warning appears. But when it is for a raw NAND device,
GCC complains. Fix these warning by including .

Signed-off-by: Miquel Raynal 
Reviewed-by: Boris Brezillon

mtd: rename CONFIG_MTD_DEVICE -> CONFIG_MTD

2019-12-04T04:04:10+00:00

Like in Linux, just use CONFIG_MTD to compile the MTD stack.

Signed-off-by: Miquel Raynal 
Reviewed-by: Boris Brezillon

mtd: rename CONFIG_MTD -> CONFIG_DM_MTD

2019-12-04T04:04:10+00:00

CONFIG_MTD must be reserved for the MTD core. Like any other
subsystem, prefix the symbol by DM when it comes to DM support.

Signed-off-by: Miquel Raynal 
Reviewed-by: Boris Brezillon 
[trini: Add Kconfig files]
Signed-off-by: Tom Rini

mtd: rename CONFIG_NAND -> CONFIG_MTD_RAW_NAND

2019-12-04T04:04:10+00:00

Add more clarity by changing the Kconfig entry name.

Signed-off-by: Miquel Raynal 
[trini: Re-run migration, update a few more cases]
Signed-off-by: Tom Rini 
Reviewed-by: Boris Brezillon

mtd: mtdcore: Drop unused mtd_table

2019-12-03T13:43:24+00:00

The array 'mtd_table' is unused in the code.  Remove this as gcc doesn't
always discard unused global variables.

Signed-off-by: Tom Rini