linux/drivers/mtd/maps, branch v4.20

mtd: sa1100: avoid VLA in sa1100_setup_mtd

2018-11-06T09:38:17Z

Enabling -Wvla found another variable-length array with randconfig testing: drivers/mtd/maps/sa1100-flash.c: In function 'sa1100_setup_mtd': drivers/mtd/maps/sa1100-flash.c:224:10: error: ISO C90 forbids variable length array 'cdev' [-Werror=vla] Dynamically allocate the cdev array passed to mtd_concat_create() instead of using a VLA. Reported-by: Arnd Bergmann Signed-off-by: Boris Brezillon Cc: Kees Cook Cc: Olof Johansson

mtd: maps: gpio-addr-flash: Convert to gpiod

2018-10-08T19:32:58Z

Convert from legacy gpio API to gpiod. Board files will have to use gpiod_lookup_tables. Signed-off-by: Ricardo Ribalda Delgado Suggested-by: Boris Brezillon Signed-off-by: Boris Brezillon

mtd: maps: gpio-addr-flash: Replace array with an integer

2018-10-08T19:32:57Z

By replacing the array with an integer we can avoid completely the bit comparison loop if the value has not changed (by far the most common case). Signed-off-by: Ricardo Ribalda Delgado Signed-off-by: Boris Brezillon

mtd: maps: gpio-addr-flash: Use order instead of size

2018-10-08T19:32:49Z

By using the order of the window instead of the size, we can replace a lot of expensive division and modulus on the code with simple bit operations. Signed-off-by: Ricardo Ribalda Delgado Signed-off-by: Boris Brezillon

mtd: maps: gpio-addr-flash: Use devm_* functions

2018-10-05T08:20:34Z

By using devm functions we can make the code cleaner. Signed-off-by: Ricardo Ribalda Delgado Signed-off-by: Boris Brezillon

mtd: maps: gpio-addr-flash: Fix ioremapped size

2018-10-05T08:20:32Z

We should only iomap the area of the chip that is memory mapped. Otherwise we could be mapping devices beyond the memory space or that belong to other devices. Signed-off-by: Ricardo Ribalda Delgado Fixes: ebd71e3a4861 ("mtd: maps: gpio-addr-flash: fix warnings and make more portable") Cc: Signed-off-by: Boris Brezillon

mtd: maps: gpio-addr-flash: Replace custom printk

2018-10-05T08:20:31Z

Use preferred print methods dev_* Signed-off-by: Ricardo Ribalda Delgado Signed-off-by: Boris Brezillon

mtd: physmap_of: Release resources on error

2018-10-05T08:19:33Z

During probe, if there was an error the memory region and the memory map were not properly released.This can lead a system unusable if deferred probe is in use. Replace mem_request and map with devm_ioremap_resource Signed-off-by: Ricardo Ribalda Delgado Signed-off-by: Boris Brezillon

mtd: physmap_of: Remove unused struct of_device_id

2018-10-04T11:44:26Z

This struct does not seem to be used anywhere on the code Signed-off-by: Ricardo Ribalda Delgado Signed-off-by: Boris Brezillon

mtd/maps: fix solutionengine.c printk format warnings

2018-07-24T20:43:10Z

Fix 2 printk format warnings (this driver is currently only used by arch/sh/) by using "%pap" instead of "%lx". Fixes these build warnings: ../drivers/mtd/maps/solutionengine.c: In function 'init_soleng_maps': ../include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 2 has type 'resource_size_t' {aka 'unsigned int'} [-Wformat=] ../drivers/mtd/maps/solutionengine.c:62:54: note: format string is defined here printk(KERN_NOTICE "Solution Engine: Flash at 0x%08lx, EPROM at 0x%08lx\n", ~~~~^ %08x ../include/linux/kern_levels.h:5:18: warning: format '%lx' expects argument of type 'long unsigned int', but argument 3 has type 'resource_size_t' {aka 'unsigned int'} [-Wformat=] ../drivers/mtd/maps/solutionengine.c:62:72: note: format string is defined here printk(KERN_NOTICE "Solution Engine: Flash at 0x%08lx, EPROM at 0x%08lx\n", ~~~~^ %08x Cc: David Woodhouse Cc: Brian Norris Cc: Boris Brezillon Cc: Marek Vasut Cc: Richard Weinberger Cc: linux-mtd@lists.infradead.org Cc: Yoshinori Sato Cc: Rich Felker Cc: linux-sh@vger.kernel.org Cc: Sergei Shtylyov Signed-off-by: Randy Dunlap Signed-off-by: Boris Brezillon