Re: [coreboot] Tester devices [was: QA contribution]

2011-02-02 Thread Juhana Helovuo
[ Note: There seems to be something wrong with mailing list delivery. I did not receive this Peter's latest message at all, but found it in the mailing list archive instead. Other coreboot lists messages have arrived to my inbox after that. ] Peter Stuge wrote: Flash emulator would be nice,

Re: [coreboot] Moved 'pci_set_subsystem' to a weak function: Support for Siemens Mainboard

2011-02-02 Thread Joseph Kellermann
Patrick, could you commit this patch ? Regards, Joseph Von: Georgi, Patrick [patrick.geo...@secunet.com] Gesendet: Dienstag, 25. Januar 2011 13:54 Bis: Joseph Kellermann Cc: coreboot@coreboot.org Betreff: Re: [coreboot] Moved 'pci_set_subsystem' to a

[coreboot] errata#89 patch for Family 0Fh Prozessors

2011-02-02 Thread Josef Kellermann
This patch fixes a potential system hang. Don't know why this patch isn't included if 'CONFIG_K8_REV_F_SUPPORT == 1'. Signed-off-by: Josef Kellermann mailto://se...@arcor.dese...@arcor.de mailto://se...@arcor.de Index: src/cpu/amd/model_fxx/model_fxx_init.c

[coreboot] #174: Unable to boot from qemu-kvm -- seems to be a cbfs problem

2011-02-02 Thread coreboot
#174: Unable to boot from qemu-kvm -- seems to be a cbfs problem -+- Reporter: czw[+]660[+]@…| Owner: stepan@… Type: defect|Status: new Priority: major

Re: [coreboot] errata#89 patch for Family 0Fh Prozessors

2011-02-02 Thread Alex G.
Erratum 89 is already handled on line 390: if (!is_cpu_pre_b3()) { /* Erratum 89 ... */ msr = rdmsr(NB_CFG_MSR); msr.lo |= 1 3; If this is also needed when CONFIG_K8_REV_F_SUPPORT == 1, then we should move the existing one out of the #if CONFIG_K8_REV_F_SUPPORT == 0 so it gets

[coreboot] [PATCH] Add license headers for AMD family Fh

2011-02-02 Thread Alex G.
See patch. Alex Add GPL license headers to all files in src/cpu/amd/model_fxx (except microcode). Signed-off-by Alexandru Gagniuc mr.nuke...@gmail.com Acked-by Alexandru Gagniuc mr.nuke...@gmail.com Trivial Index: src/cpu/amd/model_fxx/Kconfig

Re: [coreboot] [PATCH] Add license headers for AMD family Fh

2011-02-02 Thread Stefan Reinauer
* Alex G. mr.nuke...@gmail.com [110202 20:23]: Add GPL license headers to all files in src/cpu/amd/model_fxx (except microcode). Signed-off-by Alexandru Gagniuc mr.nuke...@gmail.com Acked-by Alexandru Gagniuc mr.nuke...@gmail.com Trivial Not trivial at all, but legally kind of troublesome.

Re: [coreboot] [PATCH] Add license headers for AMD family Fh

2011-02-02 Thread Patrick Georgi
Am 02.02.2011 20:39, schrieb Alex G.: coreboot is GPLv2, not GPLv2+ http://www.coreboot.org/Development_Guidelines says GPLv2+. So which is the correct license? We have some GPLv2 and some GPLv2+ files, so the combined work is GPLv2. As for the files that you added licensing information to, the

Re: [coreboot] [PATCH] Add license headers for AMD family Fh

2011-02-02 Thread Alex G.
On 02/02/2011 09:50 PM, Patrick Georgi wrote: Am 02.02.2011 20:39, schrieb Alex G.: coreboot is GPLv2, not GPLv2+ http://www.coreboot.org/Development_Guidelines says GPLv2+. So which is the correct license? We have some GPLv2 and some GPLv2+ files, so the combined work is GPLv2. As for the

Re: [coreboot] errata#89 patch for Family 0Fh Prozessors

2011-02-02 Thread Josef Kellermann
Am 02.02.2011 20:11, schrieb Alex G.: Erratum 89 is already handled on line 390: if (!is_cpu_pre_b3()) { /* Erratum 89 ... */ msr = rdmsr(NB_CFG_MSR); msr.lo |= 1 3; If this is also needed when CONFIG_K8_REV_F_SUPPORT == 1, then we should move the existing one out of the #if

Re: [coreboot] errata#89 patch for Family 0Fh Prozessors

2011-02-02 Thread Rudolf Marek
While we are at it. I'm attaching some very old patch (r2978 ;) which contains some errata fixes too, if you have some spare time please try to check if it is correct. Signed-off-by: Rudolf Marek r.ma...@assembler.cz Thanks, Rudolf Index: src/cpu/amd/model_fxx/model_fxx_init.c

Re: [coreboot] errata#89 patch for Family 0Fh Prozessors

2011-02-02 Thread Alex G.
That looks way better. :) Acked-by: Alexandru Gagniuc mr.nuke...@gmail.com On 02/02/2011 10:17 PM, Josef Kellermann wrote: Am 02.02.2011 20:11, schrieb Alex G.: Erratum 89 is already handled on line 390: if (!is_cpu_pre_b3()) { /* Erratum 89 ... */ msr = rdmsr(NB_CFG_MSR);

Re: [coreboot] errata#89 patch for Family 0Fh Prozessors

2011-02-02 Thread Alex G.
src/northbridge/amd/amdk8/coherent_ht_car.c was renamed to someting else since then (I wasn't around then), and the patch can no longer be applied. Can you please check this? Alex On 02/02/2011 11:20 PM, Rudolf Marek wrote: While we are at it. I'm attaching some very old patch (r2978 ;) which

[coreboot] [commit] r6327 - trunk/util/superiotool

2011-02-02 Thread repository service
Author: uwe Date: Thu Feb 3 00:49:41 2011 New Revision: 6327 URL: https://tracker.coreboot.org/trac/coreboot/changeset/6327 Log: Add detection/dump support for the NSC PC87382. It is a rather small 'Super I/O' device, containing a serial port, IR, GPIO, and a Docking LPC switch. It is used in

Re: [coreboot] [PATCH] add PC87382 to superiotool

2011-02-02 Thread Uwe Hermann
Hi, On Tue, Feb 01, 2011 at 11:58:04AM +0100, Sven Schnelle wrote: this patch adds the NSC PC87382 to superiotool. It is a rather small 'superio' device, containing one Serial Port, one Infrared Port, GPIO and a Docking LPC switch. It is used in various Thinkpads. It adds 0x164e/0x16ef to

[coreboot] [commit] r6328 - trunk/src/ec/lenovo/pmh7

2011-02-02 Thread repository service
Author: uwe Date: Thu Feb 3 00:56:15 2011 New Revision: 6328 URL: https://tracker.coreboot.org/trac/coreboot/changeset/6328 Log: pmh7.[ch]: Add missing license headers. Signed-off-by: Uwe Hermann u...@hermann-uwe.de Acked-by: Uwe Hermann u...@hermann-uwe.de Modified:

Re: [coreboot] [PATCH v2] Add Lenovo PMH7

2011-02-02 Thread Uwe Hermann
On Tue, Feb 01, 2011 at 09:08:38AM +0100, Sven Schnelle wrote: this patch adds support for the Lenovo PMH7 (Power Management Hardware Hub) found in most recent (starting with X60/T60 AFAIK) Lenovo/IBM Laptops. Please ignore the previous version, as it missed the src/ec/Kconfig file.

[coreboot] build service results for r6328

2011-02-02 Thread repository service
Dear coreboot readers! This is the automatic build system of coreboot. The developer uwe checked in revision 6328 to the coreboot repository. This caused the following changes: Change Log: pmh7.[ch]: Add missing license headers. Signed-off-by: Uwe Hermann u...@hermann-uwe.de Acked-by: Uwe

Re: [coreboot] #174: Unable to boot from qemu-kvm -- seems to be a cbfs problem

2011-02-02 Thread coreboot
#174: Unable to boot from qemu-kvm -- seems to be a cbfs problem -+- Reporter: czw[+]660[+]@…| Owner: stepan@… Type: defect|Status: new Priority: major

Re: [coreboot] #174: Unable to boot from qemu-kvm -- seems to be a cbfs problem

2011-02-02 Thread coreboot
#174: Unable to boot from qemu-kvm -- seems to be a cbfs problem -+- Reporter: czw[+]660[+]@…| Owner: stepan@… Type: defect|Status: new Priority: major