coreboot October 2012

coreboot@coreboot.org

34 participants
312 discussions

Re: [coreboot] [SeaBIOS] VideoBios question
by Peter Stuge 03 Oct '12

03 Oct '12

Christian Gmeiner wrote: > What is the default state/video mode the VideoBios should enter > during/after init? Traditional VGA BIOSes always init mode 3. //Peter

1 0

VideoBios question
by Christian Gmeiner 03 Oct '12

03 Oct '12

Hi all. As you may noticed I am working on the SeaVideoBios (Geode part). And I am running into a simple question: What is the default state/video mode the VideoBios should enter during/after init? In general I need to set the wanted mode via int10 AH=00h Would it be okay to have no mode set during init and trust that the mode is set before using it? thanks --- Christian Gmeiner, MSc

1 0

Patch merged into coreboot/master: 83a039f libpayload: Set 8bits per char for serial port
by gerrit＠coreboot.org 03 Oct '12

03 Oct '12

the following patch was just integrated into master: commit 83a039fde2ec1516d0b38971b1e9db1605f38745 Author: Anton Kochkov <anton.kochkov(a)gmail.com> Date: Wed Sep 26 22:31:04 2012 +0400 libpayload: Set 8bits per char for serial port Previously we assume that hardware using 8 bits per char by default, but on Asrock A53 Pro this is not true (7 bit per char by default). Forcing use 8n1 now. Change-Id: Ib701725d2ec6dacd7862016b2045270956b27029 Signed-off-by: Anton Kochkov <anton.kochkov(a)gmail.com> Build-Tested: build bot (Jenkins) at Wed Sep 26 21:50:13 2012, giving +1 Reviewed-By: Marc Jones <marcj303(a)gmail.com> at Tue Oct 2 23:50:50 2012, giving +2 See http://review.coreboot.org/1541 for details. -gerrit

1 0

Patch merged into coreboot/master: 1c54244 superiotool: Fix for FreeBSD
by gerrit＠coreboot.org 03 Oct '12

03 Oct '12

the following patch was just integrated into master: commit 1c542442c5d2a6a382942c4db9d3fa3b0d0ac122 Author: Andriy Gapon <avg(a)FreeBSD.org> Date: Wed Oct 3 08:19:34 2012 +0200 superiotool: Fix for FreeBSD Makefile still used SVNDEF on FreeBSD. Change-Id: I45c7fbc66c33e82a2146ef7df87b63bc7edea4cd Signed-off-by: Patrick Georgi <patrick(a)georgi-clan.de> Build-Tested: build bot (Jenkins) at Wed Oct 3 08:59:07 2012, giving +1 Reviewed-By: Anton Kochkov <anton.kochkov(a)gmail.com> at Wed Oct 3 09:26:59 2012, giving +2 See http://review.coreboot.org/1554 for details. -gerrit

1 0

Patch merged into coreboot/master: 12ce567 libpayload: UHCI driver contained too much magic
by gerrit＠coreboot.org 03 Oct '12

03 Oct '12

the following patch was just integrated into master: commit 12ce567ab2937dc8358fd12cadac8ba9f18853e0 Author: Patrick Georgi <patrick(a)georgi-clan.de> Date: Wed Oct 3 08:23:56 2012 +0200 libpayload: UHCI driver contained too much magic The handling of finalize in uhci_bulk was confusing, and so its behaviour changed. If set, the driver is supposed to add a trailing empty packet iff the last packet is of maximum packet size. This helps the device to decide if the transfer is completed simply by waiting for a packet that isn't full length. Change-Id: I162e8c1e034924d0de6fdcb971c94cf3a5ea31eb Signed-off-by: Patrick Georgi <patrick(a)georgi-clan.de> Build-Tested: build bot (Jenkins) at Wed Oct 3 09:20:37 2012, giving +1 Reviewed-By: Anton Kochkov <anton.kochkov(a)gmail.com> at Wed Oct 3 09:25:24 2012, giving +2 See http://review.coreboot.org/1555 for details. -gerrit

1 0

New patch to review for coreboot: 12ce567 libpayload: UHCI driver contained too much magic
by Patrick Georgi 03 Oct '12

03 Oct '12

Patrick Georgi (patrick(a)georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1555 -gerrit commit 12ce567ab2937dc8358fd12cadac8ba9f18853e0 Author: Patrick Georgi <patrick(a)georgi-clan.de> Date: Wed Oct 3 08:23:56 2012 +0200 libpayload: UHCI driver contained too much magic The handling of finalize in uhci_bulk was confusing, and so its behaviour changed. If set, the driver is supposed to add a trailing empty packet iff the last packet is of maximum packet size. This helps the device to decide if the transfer is completed simply by waiting for a packet that isn't full length. Change-Id: I162e8c1e034924d0de6fdcb971c94cf3a5ea31eb Signed-off-by: Patrick Georgi <patrick(a)georgi-clan.de> --- payloads/libpayload/drivers/usb/uhci.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/payloads/libpayload/drivers/usb/uhci.c b/payloads/libpayload/drivers/usb/uhci.c index 6cc6b28..7647707 100644 --- a/payloads/libpayload/drivers/usb/uhci.c +++ b/payloads/libpayload/drivers/usb/uhci.c @@ -429,7 +429,10 @@ uhci_bulk (endpoint_t *ep, int size, u8 *data, int finalize) int maxpsize = ep->maxpacketsize; if (maxpsize == 0) fatal("MaxPacketSize == 0!!!"); - int numpackets = (size + maxpsize - 1) / maxpsize + finalize; + int numpackets = (size + maxpsize - 1) / maxpsize; + if (finalize && ((size % maxpsize) == 0)) { + numpackets++; + } if (numpackets == 0) return 0; td_t *tds = create_schedule (numpackets);

1 0

New patch to review for coreboot: 1c54244 superiotool: Fix for FreeBSD
by Patrick Georgi 03 Oct '12

03 Oct '12

Patrick Georgi (patrick(a)georgi-clan.de) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1554 -gerrit commit 1c542442c5d2a6a382942c4db9d3fa3b0d0ac122 Author: Andriy Gapon <avg(a)FreeBSD.org> Date: Wed Oct 3 08:19:34 2012 +0200 superiotool: Fix for FreeBSD Makefile still used SVNDEF on FreeBSD. Change-Id: I45c7fbc66c33e82a2146ef7df87b63bc7edea4cd Signed-off-by: Patrick Georgi <patrick(a)georgi-clan.de> --- util/superiotool/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/util/superiotool/Makefile b/util/superiotool/Makefile index 41625d3..2388f4e 100644 --- a/util/superiotool/Makefile +++ b/util/superiotool/Makefile @@ -41,7 +41,7 @@ LIBS = -framework IOKit -framework DirectHW -lpci -lz endif ifeq ($(OS_ARCH), FreeBSD) CFLAGS = -O2 -Wall -Werror -Wstrict-prototypes -Wundef -Wstrict-aliasing \ - -Werror-implicit-function-declaration -ansi $(SVNDEF) \ + -Werror-implicit-function-declaration -ansi $(VERSION) \ -I/usr/local/include LDFLAGS += -L/usr/local/lib LIBS = -lz

1 0

Patch merged into coreboot/master: afb58bf buildsystem: ensure directory exists before use
by gerrit＠coreboot.org 03 Oct '12

03 Oct '12

the following patch was just integrated into master: commit afb58bfd0df28c0d59126129322be953d1db6060 Author: Patrick Georgi <patrick.georgi(a)secunet.com> Date: Fri Sep 21 18:11:59 2012 +0200 buildsystem: ensure directory exists before use In some cases we request mktemp to create a temporary file in $(obj)/mainboard/... before it exists. Let's make sure the directory exists Change-Id: I51f0065c30b1f25eb501a6fd5edefb3f4c15d0ab Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> See http://review.coreboot.org/1532 for details. -gerrit

1 0

new Trinity APUs supported?
by Rudolf Marek 02 Oct '12

02 Oct '12

Hi all I'm considering to buy FM2 A10 CPU & Asus F2A85-V PRO (or some cheaper variant) http://www.asus.com/Motherboards/AMD_Socket_FM2/F2A85M_PRO/ (looks it has also serial port). The CPU is I think family 12h. I checked the coreboot sources and there seems to be only LLANO mentioned. Any clue? Thanks Rudolf

3 3

Patch merged into coreboot/master: dcec334 Fix compilation without CONFIG_WRITE_HIGH_TABLES.
by gerrit＠coreboot.org 02 Oct '12

02 Oct '12

the following patch was just integrated into master: commit dcec334895aa187e256543bc17382d567b3d1d94 Author: Denis 'GNUtoo' Carikli <GNUtoo(a)no-log.org> Date: Sat Sep 22 14:52:24 2012 +0200 Fix compilation without CONFIG_WRITE_HIGH_TABLES. Without that fix we have: CC boot/hardwaremain.ramstage.o src/boot/hardwaremain.c: In function 'hardwaremain': src/boot/hardwaremain.c:136:6: error: 'cbmem_post_handling' undeclared (first use in this function) src/boot/hardwaremain.c:136:6: note: each undeclared identifier is reported only once for each function it appears in src/boot/hardwaremain.c:137:3: error: implicit declaration of function 'cbmem_post_handling' [-Werror=implicit-function-declaration] cc1: all warnings being treated as errors make: *** [build/boot/hardwaremain.ramstage.o] Error 1 When compiling without CONFIG_WRITE_HIGH_TABLES Change-Id: Ie45f684a6db0ab55ef469bfcef57e539ae7e994c Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo(a)no-log.org> Build-Tested: build bot (Jenkins) at Sat Sep 22 15:24:31 2012, giving +1 Reviewed-By: Peter Stuge <peter(a)stuge.se> at Tue Oct 2 19:52:07 2012, giving +2 See http://review.coreboot.org/1533 for details. -gerrit

1 0

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 October 2012