coreboot March 2008

coreboot@coreboot.org

75 participants
382 discussions

Start a nNew thread

genacpi status
by joe＠smittys.pointclark.net 20 Mar '08

20 Mar '08

Hello, Does anyone know the status of genacpi? There is just an empty directory in the utils? Thanks - Joe

2 1

Denver meeting discussion topics
by Peter Stuge 20 Mar '08

20 Mar '08

I may or may not go, but here's a thread to gather discussion topics. -- LAR * Make walking efficient (special padding for all voids?) * Think about storing number of segments for converted ELFs * Maybe more generally: Think about storing number of files in dirs -- K8 ? -- dts * Interrupt routing syntax * Filename vs struct name; I want the geode/apic file to be apic.dts -- ACPI * We as a group don't know so much ACPI, maybe we should try to get linux-pm people involved? * What tables should go where? (board dts vs. src/) * How to hand over to Linux? Very relevant for linux-pm. What would their dream scenario be? Let's make it happen. Please append. Should we move this to the wiki? //Peter

2 1

Problem while compiling flashrom files
by malatesh kamatad 20 Mar '08

20 Mar '08

Hi all... I am getting this error while compiling the flashrom files ........ [root@localhost flashrom]# make Checking for pciutils and zlib... not found. Please install pciutils-devel and zlib-devel. See README for more information. make: *** [pciutils] Error 1 And i am working on the system given below [root@localhost flashrom]# uname -a Linux localhost.localdomain 2.6.24-2.fc9 #1 SMP Fri Jan 25 12:52:32 EST 2008 x86_64 x86_64 x86_64 GNU/Linux [root@localhost flashrom]# /sbin/lspci 00:00.0 Host bridge: VIA Technologies, Inc. K8M800 Host Bridge 00:00.1 Host bridge: VIA Technologies, Inc. K8M800 Host Bridge 00:00.2 Host bridge: VIA Technologies, Inc. K8M800 Host Bridge 00:00.3 Host bridge: VIA Technologies, Inc. K8M800 Host Bridge 00:00.4 Host bridge: VIA Technologies, Inc. K8M800 Host Bridge 00:00.7 Host bridge: VIA Technologies, Inc. K8M800 Host Bridge 00:01.0 PCI bridge: VIA Technologies, Inc. VT8237 PCI bridge [K8T800/K8T890 South] 00:0f.0 RAID bus controller: VIA Technologies, Inc. VIA VT6420 SATA RAID Controller (rev 80) 00:0f.1 IDE interface: VIA Technologies, Inc. VT82C586A/B/VT82C686/A/B/VT823x/A/C PIPC Bus Master IDE (rev 06) 00:10.0 USB Controller: VIA Technologies, Inc. VT82xxxxx UHCI USB 1.1Controller (rev 81) 00:10.1 USB Controller: VIA Technologies, Inc. VT82xxxxx UHCI USB 1.1Controller (rev 81) 00:10.2 USB Controller: VIA Technologies, Inc. VT82xxxxx UHCI USB 1.1Controller (rev 81) 00:10.3 USB Controller: VIA Technologies, Inc. VT82xxxxx UHCI USB 1.1Controller (rev 81) 00:10.4 USB Controller: VIA Technologies, Inc. USB 2.0 (rev 86) 00:11.0 ISA bridge: VIA Technologies, Inc. VT8237 ISA bridge [KT600/K8T800/K8T890 South] 00:11.5 Multimedia audio controller: VIA Technologies, Inc. VT8233/A/8235/8237 AC97 Audio Controller (rev 60) 00:12.0 Ethernet controller: VIA Technologies, Inc. VT6102 [Rhine-II] (rev 78) 00:18.0 Host bridge: Advanced Micro Devices [AMD] K8 [Athlon64/Opteron] HyperTransport Technology Configuration 00:18.1 Host bridge: Advanced Micro Devices [AMD] K8 [Athlon64/Opteron] Address Map 00:18.2 Host bridge: Advanced Micro Devices [AMD] K8 [Athlon64/Opteron] DRAM Controller 00:18.3 Host bridge: Advanced Micro Devices [AMD] K8 [Athlon64/Opteron] Miscellaneous Control 01:00.0 VGA compatible controller: VIA Technologies, Inc. S3 Unichrome Pro VGA Adapter (rev 01) please guide me...... with regards, Malatesh kamatad

3 4

v3 struct resource related strangeness
by Carl-Daniel Hailfinger 20 Mar '08

20 Mar '08

Hi, reading through v3 resource code caused a few questions: Why does ram_resource() live in device/pci_device.c? What does the index member of struct resource really mean? Sometimes it is a hardcoded number with no obvious meaning, sometimes it is a memory address, sometimes it is a resource type. Also this snippet of northbridge/amd/geodelx/geodelx.c looks strange: > /** > * Set resources in the PCI domain. > * > * Also, as a side effect, create a RAM resource in the child which, > * interestingly enough, is the northbridge PCI device, for later > * allocation of address space. > * > * @param dev The device. > */ > static void geodelx_pci_domain_set_resources(struct device *dev) > { > int idx; > struct device *mc_dev; > struct northbridge_amd_geodelx_domain_config *nb_dm = > (struct northbridge_amd_geodelx_domain_config *)dev->device_configuration; > > printk(BIOS_SPEW, ">> Entering northbridge.c: %s\n", __FUNCTION__); > > mc_dev = dev->link[0].children; > if (mc_dev) { > /* Report the memory regions. */ > idx = 10; Magical value? > /* 0 .. 640 KB */ > ram_resource(dev, idx++, 0, 640); > /* 1 MB .. (Systop - 1 MB) (in KB) */ > ram_resource(dev, idx++, 1024, > (get_systop(nb_dm) / 1024) - 1024); > } > > phase4_assign_resources(&dev->link[0]); > } Regards, Carl-Daniel -- http://www.hailfinger.org/

2 1

[RFC] v3: device_path_type and device_id_type
by Carl-Daniel Hailfinger 20 Mar '08

20 Mar '08

Hi all, the following structs are quite similar in purpose, but only one of them is debug-friendly: enum device_path_type { DEVICE_PATH_NONE = 0, DEVICE_PATH_ROOT, DEVICE_PATH_PCI_DOMAIN, DEVICE_PATH_PCI_BUS, DEVICE_PATH_PCI, DEVICE_PATH_PNP, DEVICE_PATH_I2C, DEVICE_PATH_APIC, DEVICE_PATH_APIC_CLUSTER, DEVICE_PATH_CPU, DEVICE_PATH_CPU_BUS, DEVICE_PATH_IOPORT, }; #define TYPENAME(a,b,c,d) ((a<<24)|(b<<16)|(c<<8)|(d)) enum device_id_type { DEVICE_ID_NONE = 0, DEVICE_ID_ROOT = TYPENAME('R','O','O','T'), DEVICE_ID_PCI_DOMAIN = TYPENAME('P','C','I','D'), DEVICE_ID_PCI = TYPENAME(' ','P','C','I'), DEVICE_ID_PNP = TYPENAME(' ','P','N','P'), DEVICE_ID_I2C = TYPENAME(' ','I','2','C'), DEVICE_ID_APIC = TYPENAME('A','P','I','C'), DEVICE_ID_APIC_CLUSTER = TYPENAME('A','P','C','C'), DEVICE_ID_CPU = TYPENAME(' ','C','P','U'), DEVICE_ID_CPU_BUS = TYPENAME(' ','B','U','S'), }; The only difference in the listing is that device_id_type does not have DEVICE_ID_PCI_BUS and DEVICE_ID_IOPORT. Comments? Regards, Carl-Daniel -- http://www.hailfinger.org/

1 0

[PATCH] v3: Add build time warning for NULL dereference
by Carl-Daniel Hailfinger 20 Mar '08

20 Mar '08

Since I didn't get an answer about whether it is legal to have bus->dev == NULL, simply add a build time warning because we will dereference a NULL pointer at runtime on some targets. If bus->dev == NULL is illegal, we'll have to add some checks and a hefty warning to the code. I admit I was tempted to make this an #error just to have someone look at the problem more closely. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Index: LinuxBIOSv3-stuff/device/device.c =================================================================== --- LinuxBIOSv3-stuff/device/device.c (Revision 644) +++ LinuxBIOSv3-stuff/device/device.c (Arbeitskopie) @@ -290,6 +290,7 @@ for (curdev = bus->children; curdev; curdev = curdev->sibling) { unsigned int links; int i; +#warning Missing NULL check, will explode at runtime printk(BIOS_SPEW, "%s: %s(%s) dtsname %s have_resources %d enabled %d\n", __func__, bus->dev->dtsname, dev_path(bus->dev), -- http://www.hailfinger.org/

1 0

[PATCH] v3: improve printing of leftover static devices
by Carl-Daniel Hailfinger 20 Mar '08

20 Mar '08

Improve printing of leftover static devices. The log level is debatable, though. Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006(a)gmx.net> Index: LinuxBIOSv3-stuff/device/hypertransport.c =================================================================== --- LinuxBIOSv3-stuff/device/hypertransport.c (Revision 644) +++ LinuxBIOSv3-stuff/device/hypertransport.c (Arbeitskopie) @@ -580,10 +580,11 @@ */ if (old_devices) { struct device *left; + printk(BIOS_ERR, "HT: Left over static devices:\n"); for (left = old_devices; left; left = left->sibling) { - printk(BIOS_DEBUG, "%s\n", dev_path(left)); + printk(BIOS_ERR, "%s\n", dev_path(left)); } - printk(BIOS_ERR, "HT: Left over static devices.\n"); + printk(BIOS_ERR, "HT: End of list.\n"); /* Put back the left over static device, and let * pci_scan_bus() disable it. */ Index: LinuxBIOSv3-stuff/device/pci_device.c =================================================================== --- LinuxBIOSv3-stuff/device/pci_device.c (Revision 644) +++ LinuxBIOSv3-stuff/device/pci_device.c (Arbeitskopie) @@ -1122,10 +1122,11 @@ */ if (old_devices) { struct device *left; + printk(BIOS_ERR, "PCI: Left over static devices:\n"); for (left = old_devices; left; left = left->sibling) { - printk(BIOS_SPEW, "%s\n", left->dtsname); + printk(BIOS_ERR, "%s\n", left->dtsname); } - banner(BIOS_SPEW, "PCI: Left over static devices.\n"); + printk(BIOS_ERR, "PCI: End of list.\n"); } /* For all children that implement scan_bus() (i.e. bridges) -- http://www.hailfinger.org/

1 0

[libpayload][patch] change kconfig references from buildrom to libpayload
by Chris Kilgour 20 Mar '08

20 Mar '08

- change kconfig references from 'buildrom' to 'libpayload' Signed-off-by: Christopher Kilgour <techie(a)whiterocker.com>

2 1

r3179 - trunk/payloads/libpayload/util/kconfig
by svn＠coreboot.org 20 Mar '08

20 Mar '08

Author: uwe Date: 2008-03-20 02:58:08 +0100 (Thu, 20 Mar 2008) New Revision: 3179 Modified: trunk/payloads/libpayload/util/kconfig/confdata.c trunk/payloads/libpayload/util/kconfig/zconf.tab.c_shipped trunk/payloads/libpayload/util/kconfig/zconf.y Log: Change kconfig references from 'buildrom' to 'libpayload'. Signed-off-by: Christopher Kilgour <techie(a)whiterocker.com> Acked-by: Uwe Hermann <uwe(a)hermann-uwe.de> Modified: trunk/payloads/libpayload/util/kconfig/confdata.c =================================================================== --- trunk/payloads/libpayload/util/kconfig/confdata.c 2008-03-20 01:56:05 UTC (rev 3178) +++ trunk/payloads/libpayload/util/kconfig/confdata.c 2008-03-20 01:58:08 UTC (rev 3179) @@ -381,7 +381,7 @@ fprintf(out, _("#\n" "# Automatically generated make config: don't edit\n" - "# Buildrom version: %s\n" + "# Libpayload version: %s\n" "%s%s" "#\n"), sym_get_string_value(sym), @@ -405,7 +405,7 @@ } fprintf(out_h, "/*\n" " * Automatically generated C config: don't edit\n" - " * Buildrom version: %s\n" + " * Libpayload version: %s\n" " */\n" "%s" "\n", Modified: trunk/payloads/libpayload/util/kconfig/zconf.tab.c_shipped =================================================================== --- trunk/payloads/libpayload/util/kconfig/zconf.tab.c_shipped 2008-03-20 01:56:05 UTC (rev 3178) +++ trunk/payloads/libpayload/util/kconfig/zconf.tab.c_shipped 2008-03-20 01:58:08 UTC (rev 3179) @@ -1949,7 +1949,7 @@ sym_init(); menu_init(); modules_sym = sym_lookup("MODULES", 0); - rootmenu.prompt = menu_add_prompt(P_MENU, "Buildrom Configuration", NULL); + rootmenu.prompt = menu_add_prompt(P_MENU, "Libpayload Configuration", NULL); #if YYDEBUG if (getenv("ZCONF_DEBUG")) Modified: trunk/payloads/libpayload/util/kconfig/zconf.y =================================================================== --- trunk/payloads/libpayload/util/kconfig/zconf.y 2008-03-20 01:56:05 UTC (rev 3178) +++ trunk/payloads/libpayload/util/kconfig/zconf.y 2008-03-20 01:58:08 UTC (rev 3179) @@ -459,7 +459,7 @@ sym_init(); menu_init(); modules_sym = sym_lookup("MODULES", 0); - rootmenu.prompt = menu_add_prompt(P_MENU, "Buildrom Configuration", NULL); + rootmenu.prompt = menu_add_prompt(P_MENU, "Libpayload Configuration", NULL); #if YYDEBUG if (getenv("ZCONF_DEBUG"))

1 0

[libpayload] [patch] fix "make menuconfig"
by Chris Kilgour 20 Mar '08

20 Mar '08

-allows 'make menuconfig' to function within libpayload -removes reference to BusyBox Signed-off-by: Christopher Kilgour <techie(a)whiterocker.com>

2 1

← Newer
1
...
10
11
12
13
14
15
16
...
39
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

coreboot March 2008