coreboot June 2012

coreboot@coreboot.org

41 participants
241 discussions

Regarding contribution for coreboot
by manasa gv 04 Jun '12

04 Jun '12

Hi, I am new and interested to work on coreboot.I have gone through the coreboot website. I am trying to do patch review.. But I am taking more time to come up with the solution compared to others.. So,Please let me know any other ways to contribute for coreboot apart from patch review?? Thanks, Manasa

1 0

Re: [coreboot] New Motherboards?
by ron minnich 03 Jun '12

03 Jun '12

Get a chromebox. It's quite well supported and it's a very fine machine. ron

1 0

New Motherboards?
by gary sheppard 03 Jun '12

03 Jun '12

Hello everyone, Are there any new motherboards that are fully supported? About mid-year 2013 I will be looking to build a new desktop / workstation. I would like to avoid UEFI like the plague. I am serious. If a donation would help I would like to propose a "What board" type discussion. I am guessing Intel? I am open minded on what board folks. I simply will not buy into UEFI. I have read quite a bit of the specifications material, and I simply do not believe that it cures any of the problems it seems to be saying it will. I can foresee a security / user nightmare coming. Corporate Root kits were bad enough before UEFI, I can only imagine how bad it will get after we are all Secured using UEFI. Thank you for your time and consideration, Gary Sheppard

1 0

New patch to review for filo: 4193fd9 Make booting from ATAPI drives more resilient.
by Patrick Georgi 01 Jun '12

01 Jun '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/1087 -gerrit commit 4193fd9b4921a78831f9346b02cbb004b69b16e3 Author: Nico Huber <nico.huber(a)secunet.com> Date: Fri May 25 11:34:08 2012 +0200 Make booting from ATAPI drives more resilient. This fixes a semantic error, where the ATAPI driver requested sense from the drive instead of retrying a failed command (i.e. the actual command had been overwritten). The count of retries and delays are also adapted to behave more like the Linux kernel driver. Change-Id: Iabf20120c841bbc277af4efa48c6594804f60205 Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com> --- drivers/ide_new.c | 90 +++++++++++++++++++++++------------------------------ drivers/ide_new.h | 6 ++-- 2 files changed, 42 insertions(+), 54 deletions(-) diff --git a/drivers/ide_new.c b/drivers/ide_new.c index 658f177..cc1fb83 100644 --- a/drivers/ide_new.c +++ b/drivers/ide_new.c @@ -192,8 +192,10 @@ ob_ide_error(struct ide_drive *drive, unsigned char stat, char *msg) for (i = 0; i < sizeof(cmd->cdb); i++) debug("%02x ", cmd->cdb[i]); } - if (cmd->sense_valid) - debug(", sense: %02x/%02x/%02x", cmd->sense.sense_key, cmd->sense.asc, cmd->sense.ascq); + if (cmd->sense) + debug(", sense: %02x/%02x/%02x", + cmd->sense->sense_key, + cmd->sense->asc, cmd->sense->ascq); else debug(", no sense"); debug("\n"); @@ -535,7 +537,7 @@ ob_ide_pio_packet(struct ide_drive *drive, struct atapi_command *cmd) static int ob_ide_atapi_packet(struct ide_drive *drive, struct atapi_command *cmd) { - int retries = 5, ret; + int retries = 10, ret; if (drive->type != ide_type_atapi) return 1; @@ -550,33 +552,35 @@ ob_ide_atapi_packet(struct ide_drive *drive, struct atapi_command *cmd) if (!ret) break; - /* - * request sense failed, bummer - */ - if (cmd->cdb[0] == ATAPI_REQ_SENSE) - break; - if (ob_ide_atapi_request_sense(drive)) break; /* - * we know sense is valid. retry if the drive isn't ready, - * otherwise don't bother. - */ - if (cmd->sense.sense_key != ATAPI_SENSE_NOT_READY) - break; - /* - * ... except 'medium not present' + * We know sense is valid. Retry if the unit attention + * condition has been set (media change / reset) or the + * drive isn't ready, otherwise don't bother. */ - if (cmd->sense.asc == 0x3a) { - /* 'medium not present' is not an error */ - ret = 0; - /* force reevaluation */ - drive->channel->present = 0; + if (cmd->sense->sense_key == ATAPI_SENSE_UNIT_ATTENTION) { + /* Just retry. */ + } else if (cmd->sense->sense_key == ATAPI_SENSE_NOT_READY) { + if (cmd->sense->asc == 0x04) { + /* The drive is becoming ready, give it some + * more time. */ + mdelay(3000); + } else if (cmd->sense->asc == 0x3a) { + /* + * 'medium not present' is not an error, + * but we have to report nevertheless + */ + ret = RETURN_NO_MEDIUM; + /* force reevaluation */ + drive->channel->present = 0; + /* Don't retry in this case. */ + break; + } + } else { break; } - - mdelay(1000); } while (retries--); if (ret) @@ -588,26 +592,25 @@ ob_ide_atapi_packet(struct ide_drive *drive, struct atapi_command *cmd) static int ob_ide_atapi_request_sense(struct ide_drive *drive) { - struct atapi_command *cmd = &drive->channel->atapi_cmd; - unsigned char old_cdb; + struct atapi_command *orig_cmd = &drive->channel->atapi_cmd; + struct atapi_command cmd; + memset(&cmd, 0, sizeof(cmd)); + + cmd.cdb[0] = ATAPI_REQ_SENSE; + cmd.cdb[4] = 18; + cmd.buffer = (unsigned char *)&drive->channel->atapi_sense; + cmd.buflen = 18; + cmd.data_direction = atapi_ddir_read; /* * save old cdb for debug error */ - old_cdb = cmd->cdb[0]; - - memset(cmd, 0, sizeof(*cmd)); - cmd->cdb[0] = ATAPI_REQ_SENSE; - cmd->cdb[4] = 18; - cmd->buffer = (unsigned char *) &cmd->sense; - cmd->buflen = 18; - cmd->data_direction = atapi_ddir_read; - cmd->old_cdb = old_cdb; + cmd.old_cdb = orig_cmd->cdb[0]; - if (ob_ide_atapi_packet(drive, cmd)) + if (ob_ide_pio_packet(drive, &cmd)) return 1; - cmd->sense_valid = 1; + orig_cmd->sense = &drive->channel->atapi_sense; return 0; } @@ -621,21 +624,6 @@ ob_ide_atapi_drive_ready(struct ide_drive *drive) struct atapi_capacity cap; int i; - /* - * Test Unit Ready is like a ping - * But wait a bit, as the drive might take a while - */ - i = 30; - while (i-- != 0) { - memset(cmd, 0, sizeof(*cmd)); - cmd->cdb[0] = ATAPI_TUR; - - if (!ob_ide_atapi_packet(drive,cmd)) break; - - /* Give the drive some time to breathe */ - mdelay(500); - } - memset(cmd, 0, sizeof(*cmd)); cmd->cdb[0] = ATAPI_TUR; diff --git a/drivers/ide_new.h b/drivers/ide_new.h index f667e41..742cc42 100644 --- a/drivers/ide_new.h +++ b/drivers/ide_new.h @@ -77,6 +77,7 @@ * atapi sense keys */ #define ATAPI_SENSE_NOT_READY 0x02 +#define ATAPI_SENSE_UNIT_ATTENTION 0x06 /* * supported device types @@ -142,8 +143,7 @@ struct atapi_command { unsigned char data_direction; unsigned char stat; - unsigned char sense_valid; - struct request_sense sense; + struct request_sense *sense; unsigned char old_cdb; }; @@ -199,7 +199,7 @@ struct ide_channel { */ struct ata_command ata_cmd; struct atapi_command atapi_cmd; - + struct request_sense atapi_sense; }; enum {

1 0

Patch merged into coreboot/master: 3e0c664 libpayload: Remove orphaned delay from USB mass storage
by gerrit＠coreboot.org 01 Jun '12

01 Jun '12

the following patch was just integrated into master: commit 3e0c6641a94dbcb652f9283b4b769c56c0d9829a Author: Nico Huber <nico.huber(a)secunet.com> Date: Mon May 21 14:01:52 2012 +0200 libpayload: Remove orphaned delay from USB mass storage This removes a synthetic delay of 10ms from every mass storage command. A delay here seems to be of no use and first tests have only shown a huge speed increase. Change-Id: Ida7423229373ec521d4326c5467a3f518b76149c Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-By: Patrick Georgi <patrick(a)georgi-clan.de> at Fri Jun 1 12:26:30 2012, giving +2 See http://review.coreboot.org/1071 for details. -gerrit

1 0

Patch merged into coreboot/master: b2718de Enable CONFIG_GFXUMA for roda/rk886ex
by gerrit＠coreboot.org 01 Jun '12

01 Jun '12

the following patch was just integrated into master: commit b2718de7d2812aa391e6c845ef344ed8a35b8f3a Author: Nico Huber <nico.huber(a)secunet.com> Date: Wed May 30 14:50:53 2012 +0200 Enable CONFIG_GFXUMA for roda/rk886ex Without GFXUMA beeing set, MTRR initialization runs out of variable MTRRs. Change-Id: I5d1aa0d5fa2d72f17a0d88cae3fad880b489828c Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-By: Patrick Georgi <patrick(a)georgi-clan.de> at Fri Jun 1 12:25:13 2012, giving +2 See http://review.coreboot.org/1086 for details. -gerrit

1 0

Patch merged into coreboot/master: 6ce9398 libpayload: Disable some buggy debugging code
by gerrit＠coreboot.org 01 Jun '12

01 Jun '12

the following patch was just integrated into master: commit 6ce9398ed02e1fd55b1f329de89e4eff5e1c8c2a Author: Nico Huber <nico.huber(a)secunet.com> Date: Mon May 21 14:38:08 2012 +0200 libpayload: Disable some buggy debugging code This disables some debugging code in the OHCI USB driver which causes reboots under rare circumstances. Change-Id: Ic274c162846137ee00638ffbc59ccf1d8130586f Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-By: Patrick Georgi <patrick(a)georgi-clan.de> at Fri Jun 1 12:24:16 2012, giving +2 See http://review.coreboot.org/1074 for details. -gerrit

1 0

Patch merged into coreboot/master: f1f1cbb libpayload: fix OHCI IN commands
by gerrit＠coreboot.org 01 Jun '12

01 Jun '12

the following patch was just integrated into master: commit f1f1cbb111ff7a5b70d6a9baf1c75d28f70bffc1 Author: Mathias Krause <mathias.krause(a)secunet.com> Date: Tue May 29 14:28:26 2012 +0200 libpayload: fix OHCI IN commands Due to operator precedence incomming USB commands were missing some flags. Change-Id: I87ef51590c9db7a6cbc7304e1ccac29895f8a51e Signed-off-by: Mathias Krause <mathias.krause(a)secunet.com> Reviewed-By: Patrick Georgi <patrick(a)georgi-clan.de> at Fri Jun 1 12:22:59 2012, giving +2 See http://review.coreboot.org/1084 for details. -gerrit

1 0

Patch merged into coreboot/master: fe83010 libpayload: fix UHCI timeout
by gerrit＠coreboot.org 01 Jun '12

01 Jun '12

the following patch was just integrated into master: commit fe83010951dbf184cb5665ed375851360073d02b Author: Mathias Krause <mathias.krause(a)secunet.com> Date: Tue May 29 16:19:19 2012 +0200 libpayload: fix UHCI timeout UHCI commands should have a timeout of 30ms, not 30s! Change-Id: Iebcf338317164eb1e683e1de850ffab5022ca3a1 Signed-off-by: Mathias Krause <mathias.krause(a)secunet.com> Reviewed-By: Patrick Georgi <patrick(a)georgi-clan.de> at Fri Jun 1 12:22:38 2012, giving +2 See http://review.coreboot.org/1085 for details. -gerrit

1 0

Patch merged into coreboot/master: 8141a15 libpayload: Correct port power settings for EHCI root hub
by gerrit＠coreboot.org 01 Jun '12

01 Jun '12

the following patch was just integrated into master: commit 8141a15f6aa96be6c388421fa190950cfbaf9f55 Author: Nico Huber <nico.huber(a)secunet.com> Date: Mon May 21 14:56:21 2012 +0200 libpayload: Correct port power settings for EHCI root hub Enable power on EHCI root hub ports only if the controller supports it. Wait 20ms for the power to become stable. Change-Id: I8897756ed2bfcb88408fe5e9f9e3f8af5dd900ac Signed-off-by: Nico Huber <nico.huber(a)secunet.com> Reviewed-By: Patrick Georgi <patrick(a)georgi-clan.de> at Fri Jun 1 12:22:08 2012, giving +2 See http://review.coreboot.org/1078 for details. -gerrit

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

coreboot June 2012