[PATCH V4] mtd: bcm47part driver for BCM47XX chipsets

[Artem Bityutskiy]

On Wed, 2012-08-29 at 14:20 +0200, Rafał Miłecki wrote:
> This driver provides parser detecting partitions on BCM47XX flash
> memories. It has many differences in comparison to BCM63XX, like:
> 1) Different CFE with no more trivial MAGICs
> 2) More partitions types (board_data, ML, POT)
> 3) Supporting more than 1 flash on a device
> which resulted in decision of writing new parser.

Aiaiai! [1] [2]

$:~/git/maintaining$ ./verify ../l2-mtd/ mips-bcm47xx_defconfig < ~/tmp/rafal.mbox 
Tested the patch(es) on top of the following commits:
1f7fb90 Quick fixes - applied by aiaiai
340b100 mtd: mtd_nandecctest: make module_init() return an error code if test fails
ef3b0f6 mtd: mtd_nandecctest: remove unnecessary srandom32() call
0f06219 mtd: m25p80: add support for the EON EN25Q64 chip

--------------------------------------------------------------------------------

Successfully built configuration "mips-bcm47xx_defconfig,mips,mips-linux-", results:

--- before_patching.log
+++ after_patching.log
@@ @@
+drivers/mtd/bcm47xxpart.c:178 bcm47xxpart_parse() warn: possible memory leak of 'buf' [smatch]

--------------------------------------------------------------------------------

checkpatch.pl has some complaints:

--------------------------------------------------------------------------------

checkpatch.pl results for patch "[PATCH] mtd: bcm47part driver for BCM47XX chipsets"

WARNING:CONFIG_DESCRIPTION: please write a paragraph that describes the config symbol fully
#35: FILE: drivers/mtd/Kconfig:151:
+config MTD_BCM47XX_PARTS

total: 0 errors, 1 warnings, 218 lines checked

--------------------------------------------------------------------------------

References:

1. http://git.infradead.org/users/dedekind/aiaiai.git
2. http://git.infradead.org/users/dedekind/maintaining.git

-- 
Best Regards,
Artem Bityutskiy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/linux-mtd/attachments/20120829/de60a71c/attachment.sig>