[PATCH V2] mtd: bcm47xxpart: find NVRAM partitions in middle blocks

[Brian Norris]

On Mon, Aug 18, 2014 at 08:20:27PM +0200, Rafał Miłecki wrote:
> Old devices used to have NVRAM at the very end of flash and they could
> be unaligned (starting at some offset in a block).
> In new devices NVRAM can be located quite randomly, however it seems to
> always start at the beginning of a block. For example Netgear R6250 has
> NVRAM located right after the bootloader, before the kernel partition.
> 
> Signed-off-by: Rafał Miłecki <zajec5 at gmail.com>
> ---
> V2: Ignore last block to avoid having two "nvram" partitions.

Tweaked the multi-line comment style and pushed to l2-mtd.git. Thanks!

> ---
>  drivers/mtd/bcm47xxpart.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/drivers/mtd/bcm47xxpart.c b/drivers/mtd/bcm47xxpart.c
> index 035690e..da641ac 100644
> --- a/drivers/mtd/bcm47xxpart.c
> +++ b/drivers/mtd/bcm47xxpart.c
> @@ -224,6 +224,16 @@ static int bcm47xxpart_parse(struct mtd_info *master,
>  			continue;
>  		}
>  
> +		/* New (ARM?) devices may have NVRAM in some middle block. Last
> +		 * block will be checked later, so skip it.
> +		 */
> +		if (offset != master->size - blocksize &&
> +		    buf[0x000 / 4] == NVRAM_HEADER) {
> +			bcm47xxpart_add_part(&parts[curr_part++], "nvram",
> +					     offset, 0);
> +			continue;
> +		}
> +
>  		/* Read middle of the block */
>  		if (mtd_read(master, offset + 0x8000, 0x4,
>  			     &bytes_read, (uint8_t *)buf) < 0) {

Brian