[coreboot] [commit] r6212 - in trunk/src: drivers/dec/21143 mainboard/hp/dl145_g1 mainboard/hp/dl145_g3 mainboard/hp/dl165_g6_fam10 mainboard/nokia/ip530 southbridge/ti/pci1x2x
Stefan Reinauer
stefan.reinauer at coresystems.de
Sun Dec 26 16:10:36 CET 2010
On 12/26/10 3:12 PM, repository service wrote:
> Modified: trunk/src/drivers/dec/21143/21143.c
> ==============================================================================
> --- trunk/src/drivers/dec/21143/21143.c Sun Dec 26 06:24:50 2010 (r6211)
> +++ trunk/src/drivers/dec/21143/21143.c Sun Dec 26 15:12:38 2010 (r6212)
> @@ -38,33 +38,40 @@
> +
> + /* Command and status configuration (offset 0x04) */
> + pci_write_config32(dev, 0x04,
> + CONFIG_DEC21143_COMMAND_AND_STATUS_CONFIGURATION);
> + printk(BIOS_DEBUG, "0x04 = %08x (07 01 80 02)\n",
> + pci_read_config32(dev, 0x04));
> +
> + /* Cache line size (offset 0x0C) */
> + pci_write_config8(dev, 0x0C, CONFIG_DEC21143_CACHE_LINE_SIZE);
> + printk(BIOS_DEBUG, "0x0c = %08x (00 80 00 00)\n",
> + pci_read_config32(dev, 0x0C));
> +
> + /* Expansion ROM base address (offset 0x30) */
> + pci_write_config32(dev, 0x30,
> + CONFIG_DEC21143_EXPANSION_ROM_BASE_ADDRESS);
This does not seem to make much sense for the 21143 ... Why is it needed?
Generally such stuff does not belong in Kconfig, but in this particular
case the resource allocator should do the right thing already.
> + printk(BIOS_DEBUG, "0x30 = %08x (0x00000000)\n",
> + pci_read_config32(dev, 0x30));
> }
Merry christmas,
Stefan
More information about the coreboot
mailing list