Re: [PATCH 0/3] mtd: nand: add Broadcom NAND controller support

From: Brian Norris
Date: Mon Mar 09 2015 - 13:59:56 EST


On Sun, Mar 08, 2015 at 12:18:39PM +0100, RafaÅ MiÅecki wrote:
> On 8 March 2015 at 01:57, Brian Norris <computersforpeace@xxxxxxxxx> wrote:
> > 3. I was told that there were only 2 or 3 chips that were released with
> > a v6.1 NAND controller, and BCM4708 wasn't one of them. Apparently I was
> > told wrong... I'll have to see if there are any other quirks we should
> > be accounting for.
>
> Please note I'm speaking about 6.01, not 6.10. Maybe it make a difference?
>
> Buffalo WZR-600DHP2 (BCM47081)
> [ 0.325732] [brcmnand_revision_init:371] ctrl->nand_version:0x0601
> [ 4.838105] bcma: bus0: Found chip with id 53010, rev 0x00 and package 0x02
>
> Buffalo WZR-1750DHP (BCM4708)
> [ 0.326823] [brcmnand_revision_init:371] ctrl->nand_version:0x0601
> [ 6.674288] bcma: bus0: Found chip with id 53010, rev 0x00 and package 0x02
>
> Netgear R6250 V1 (BCM4708)
> [ 0.326248] [brcmnand_revision_init:371] ctrl->nand_version:0x0601
> [ 8.721335] bcma: bus0: Found chip with id 53010, rev 0x00 and package 0x02
>
> Netgear R8000 (BCM4709)
> [ 0.241961] bcma: bus0: Found chip with id 53010, rev 0x00 and package 0x00
> [ 1.990330] bcm_nand bcma0:18: NAND Controller rev 6.01

We've never had a MINOR number larger than 4 or 5, I think, so I haven't
actually bothered with two digits past the decimal. It's 8 bits of MAJOR
and 8 bits of MINOR, but we just call it 6.1 in all our documentation.

0x0601 == v6.1
0x0701 == v7.1

I suppose if we ever have to care (e.g., we get past 7.9), we'd probably
call it 7.10 (0x070a). Kinda like kernel versioning (and not like Ubuntu
versioning).

Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/