[coreboot] [patch v2] ck804 PCIe lane configuration in kconfig

Uwe Hermann uwe at hermann-uwe.de
Sun Nov 21 22:38:06 CET 2010

On Sun, Nov 21, 2010 at 08:31:55PM +0000, Jonathan A. Kollasch wrote:
> -#define CK804_CHIP_REV 3

Hm, why remove CK804_CHIP_REV? I have no idea if or how that's supposed
to be used etc., though. Here's a quick grep:

southbridge/nvidia/ck804/ck804_early_setup_car.c:#define CK804_CHIP_REV 3
southbridge/nvidia/ck804/ck804_early_setup.c:#define CK804_CHIP_REV 3
southbridge/nvidia/ck804/ck804_lpc.c:#define CK804_CHIP_REV 2
southbridge/nvidia/ck804/ck804_lpc.c:#if CK804_CHIP_REV==1
southbridge/nvidia/ck804/ck804_lpc.c:#if CK804_CHIP_REV == 1

Maybe we should keep the #define until we know more.

The rest of the patch looks ok.

Acked-by: Uwe Hermann <uwe at hermann-uwe.de>

http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org

More information about the coreboot mailing list