[U-Boot] Marvell/Kirkwood: 6192 vs. 6281 - does U-Boot correctly distinguish between the two?

[Prafulla Wadaskar]

________________________________
From: David Purdy [mailto:david.c.purdy at gmail.com]
Sent: 02 April 2012 03:47
To: u-boot at lists.denx.de; Prafulla Wadaskar; albert.u.boot at aribaud.net
Subject: Marvell/Kirkwood: 6192 vs. 6281 - does U-Boot correctly distinguish between the two?

I've found either a bug, a "feature" or a misunderstanding of my own.  I've researched it before posting as much as I could, but since there are __no__ 6192 Kirkwoods in mainline (AFAICS), there isn't much to compare to...

_Marvell's_ U-Boot running on a Pogoplug V4, with its 6192_A1 ...   shows the following:
-----------------------------------------------------------------------------------------------
U-Boot 1.1.4 (Jun 13 2011 - 17:36:40) Cloud Engines 1.1.2 (3.4.27) PHYADDR=0

U-Boot code: 00600000 -> 0067FFF0  BSS: -> 006918B4

Soc: 88F6192 A1 (DDR2)
CPU running @ 800Mhz L2 running @ 400Mhz
SysClock = 200Mhz , TClock = 166Mhz
-------------------------------------------------------------------------------------------------



U-Boot 2011.12  and mainline show:
-------------------------------------------------------------------------------------------------
U-Boot 2011.12-00536-ge276fbc-dirty (Mar 29 2012 - 12:19:08)



Pogo v4



SoC:   Kirkwood 88F6281_A1

DRAM:  128 MiB

WARNING: Caches not enabled

NAND:  128 MiB



-----------------------------------------------------------------
Is this a purely cosmetic problem, or is it something else?  The 6182 is not just a _slower_ 6291: some of its registers are different, right?

If this is a bug and it is misidentifying the SoC, then wouldn't some of the masks be incorrectly applied?

[Prafulla]
Please kindly send a mail in text format.

On mainlined u-boot, device id is read from the PCIE_Express_Vendor_Device_ID register (offset 0x4000)
So it should display whatever value this register has.
As per specification it should display 6192 (in your case)
May be on this device 6192 this register reflects wrong value, in Marvell's u-boot there may be some code tweak to display correct value.

BTW: mainlined u-boot supports 6192 just for basic boot operation (I don't have a board based on 6192 :-( ).
Patches are welcomed to extend it's support.

Regards..
Prafulla . ..