SeaBIOS

seabios@seabios.org

3783 discussions

Re: [SeaBIOS] Failure to detect high-capacity SD card
by Kevin O'Connor 28 Oct '17

28 Oct '17

On Thu, Oct 26, 2017 at 09:04:57PM -0500, Matt DeVillier wrote: > On Thu, Oct 26, 2017 at 11:26 AM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > > > On Thu, Oct 26, 2017 at 11:14:53AM -0400, Matt DeVillier wrote: > > > On Thu, Oct 26, 2017 at 10:59 AM, Kevin O'Connor <kevin(a)koconnor.net> > > wrote: > > > > > > with the patch, USB devices are not available from the boot menu either > > (it > > > appears to hang?), and therefore device cannot be booted to get said log > > > > That sounds like a bad compile then - nothing in the patch should have > > impacted usb. > > > > In any case, the patch below tests the same thing and it should work > > on eMMC also. > > still timing out on detection: https://paste.ubuntu.com/25826387/ Can you try the two patches below? (Go back to master, apply the first, gather the log, go back to master, apply the second, gather the log.) -Kevin ====================== patch 1 ========================= --- a/src/hw/sdcard.c +++ b/src/hw/sdcard.c @@ -181,7 +181,7 @@ static int sdcard_pio(struct sdhci_s *regs, int cmd, u32 *param) { u32 state = readl(&regs->present_state); - dprintf(9, "sdcard_pio cmd %x %x %x\n", cmd, *param, state); + dprintf(1, "sdcard_pio cmd %x %x %x\n", cmd, *param, state); if ((state & SP_CMD_INHIBIT) || ((cmd & 0x03) == 0x03 && state & SP_DAT_INHIBIT)) { dprintf(1, "sdcard_pio not ready %x\n", state); @@ -203,7 +203,7 @@ sdcard_pio(struct sdhci_s *regs, int cmd, u32 *param) writew(&regs->irq_status, SI_CMD_COMPLETE); // Read response memcpy(param, regs->response, sizeof(regs->response)); - dprintf(9, "sdcard cmd %x response %x %x %x %x\n" + dprintf(1, "sdcard cmd %x response %x %x %x %x\n" , cmd, param[0], param[1], param[2], param[3]); return 0; } @@ -403,7 +403,7 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, int prio) param[0] = vrange; ret = sdcard_pio(regs, SC_SEND_IF_COND, param); if (!ret && param[0] == vrange) - hcs = (1<<30); + hcs = (1<<30) | (1<<28); // Verify SD card (instead of MMC or SDIO) param[0] = 0x00; ret = sdcard_pio_app(regs, SC_APP_SEND_OP_COND, param); ====================== patch 2 ========================= --- a/src/hw/sdcard.c +++ b/src/hw/sdcard.c @@ -123,6 +123,7 @@ struct sdhci_s { #define SRF_DATA 0x04 // SDHCI result flags +#define SR_OCR_S18R (1<<24) #define SR_OCR_CCS (1<<30) #define SR_OCR_NOTBUSY (1<<31) @@ -181,7 +182,7 @@ static int sdcard_pio(struct sdhci_s *regs, int cmd, u32 *param) { u32 state = readl(&regs->present_state); - dprintf(9, "sdcard_pio cmd %x %x %x\n", cmd, *param, state); + dprintf(1, "sdcard_pio cmd %x %x %x\n", cmd, *param, state); if ((state & SP_CMD_INHIBIT) || ((cmd & 0x03) == 0x03 && state & SP_DAT_INHIBIT)) { dprintf(1, "sdcard_pio not ready %x\n", state); @@ -203,7 +204,7 @@ sdcard_pio(struct sdhci_s *regs, int cmd, u32 *param) writew(&regs->irq_status, SI_CMD_COMPLETE); // Read response memcpy(param, regs->response, sizeof(regs->response)); - dprintf(9, "sdcard cmd %x response %x %x %x %x\n" + dprintf(1, "sdcard cmd %x response %x %x %x %x\n" , cmd, param[0], param[1], param[2], param[3]); return 0; } @@ -416,6 +417,9 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, int prio) drive->card_type |= SF_MMC; hcs = (1<<30); } + // XXX + if (volt == (1<<20) && hcs && readl(&regs->cap_lo) & SD_CAPLO_V18) + hcs |= SR_OCR_S18R; // Init card u32 end = timer_calc(SDHCI_POWERUP_TIMEOUT); for (;;) { @@ -434,6 +438,10 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, int prio) } msleep(5); // Avoid flooding log when debugging } + if (param[0] & SR_OCR_S18R) { + dprintf(1, "Card requests 1.8 volt\n"); + return -1; + } drive->card_type |= (param[0] & SR_OCR_CCS) ? SF_HIGHCAPACITY : 0; // Select card (get cid, set rca, get csd, select card) param[0] = 0x00;

2 1

Time for a seabios 1.11 release?
by Gerd Hoffmann 28 Oct '17

28 Oct '17

Hi, Quite a few changes accumulated in master. Time to plan a new release I think, so we can pick up the improvements in qemu 2.11. Comments? Anything pending for qemu which needs firmware support? cheers, Gerd

2 3

Re: [SeaBIOS] Failure to detect high-capacity SD card
by Kevin O'Connor 27 Oct '17

27 Oct '17

On Thu, Oct 26, 2017 at 11:14:53AM -0400, Matt DeVillier wrote: > On Thu, Oct 26, 2017 at 10:59 AM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > > > On Thu, Oct 26, 2017 at 10:46:06AM -0400, Matt DeVillier wrote: > > > On Thu, Oct 26, 2017 at 12:56 AM, Kevin O'Connor <kevin(a)koconnor.net> > > wrote: > > > > > > > Does the patch below help? > > > > > > > > -Kevin > > > > > > > > > > > > --- a/src/hw/sdcard.c > > > > +++ b/src/hw/sdcard.c > > > > @@ -405,6 +405,7 @@ sdcard_card_setup(struct sddrive_s *drive, int > > volt, > > > > int prio) > > > > if (!ret && param[0] == vrange) > > > > hcs = (1<<30); > > > > // Verify SD card (instead of MMC or SDIO) > > > > +#if 0 > > > > param[0] = 0x00; > > > > ret = sdcard_pio_app(regs, SC_APP_SEND_OP_COND, param); > > > > if (ret) { > > > > @@ -416,6 +417,7 @@ sdcard_card_setup(struct sddrive_s *drive, int > > volt, > > > > int prio) > > > > drive->card_type |= SF_MMC; > > > > hcs = (1<<30); > > > > } > > > > +#endif > > > > // Init card > > > > u32 end = timer_calc(SDHCI_POWERUP_TIMEOUT); > > > > for (;;) { > > > > > > > > > > with the patch applied, neither the internal eMMC nor the large-capacity > > > external SD card are detected > > > > Yeah - this disables MMC. Any chance to get the log from this run? > > (Let me know if eMMC is needed to get the log.) > > > > -Kevin > > > > with the patch, USB devices are not available from the boot menu either (it > appears to hang?), and therefore device cannot be booted to get said log That sounds like a bad compile then - nothing in the patch should have impacted usb. In any case, the patch below tests the same thing and it should work on eMMC also. -Kevin --- a/src/hw/sdcard.c +++ b/src/hw/sdcard.c @@ -404,19 +404,8 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, int prio) ret = sdcard_pio(regs, SC_SEND_IF_COND, param); if (!ret && param[0] == vrange) hcs = (1<<30); - // Verify SD card (instead of MMC or SDIO) - param[0] = 0x00; - ret = sdcard_pio_app(regs, SC_APP_SEND_OP_COND, param); - if (ret) { - // Check for MMC card - param[0] = 0x00; - ret = sdcard_pio(regs, SC_SEND_OP_COND, param); - if (ret) - return ret; - drive->card_type |= SF_MMC; - hcs = (1<<30); - } // Init card + int firstloop = 1; u32 end = timer_calc(SDHCI_POWERUP_TIMEOUT); for (;;) { param[0] = hcs | volt; // high-capacity support and voltage level @@ -424,8 +413,18 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, int prio) ret = sdcard_pio(regs, SC_SEND_OP_COND, param); else ret = sdcard_pio_app(regs, SC_APP_SEND_OP_COND, param); - if (ret) + if (ret) { + if (firstloop) { + // Check for MMC card + firstloop = 0; + drive->card_type |= SF_MMC; + hcs = (1<<30); + end = timer_calc(SDHCI_POWERUP_TIMEOUT); + continue; + } return ret; + } + firstloop = 0; if (param[0] & SR_OCR_NOTBUSY) break; if (timer_check(end)) {

2 1

Re: [SeaBIOS] Failure to detect high-capacity SD card
by Kevin O'Connor 27 Oct '17

27 Oct '17

On Thu, Oct 26, 2017 at 10:46:06AM -0400, Matt DeVillier wrote: > On Thu, Oct 26, 2017 at 12:56 AM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > > > Does the patch below help? > > > > -Kevin > > > > > > --- a/src/hw/sdcard.c > > +++ b/src/hw/sdcard.c > > @@ -405,6 +405,7 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, > > int prio) > > if (!ret && param[0] == vrange) > > hcs = (1<<30); > > // Verify SD card (instead of MMC or SDIO) > > +#if 0 > > param[0] = 0x00; > > ret = sdcard_pio_app(regs, SC_APP_SEND_OP_COND, param); > > if (ret) { > > @@ -416,6 +417,7 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, > > int prio) > > drive->card_type |= SF_MMC; > > hcs = (1<<30); > > } > > +#endif > > // Init card > > u32 end = timer_calc(SDHCI_POWERUP_TIMEOUT); > > for (;;) { > > > > with the patch applied, neither the internal eMMC nor the large-capacity > external SD card are detected Yeah - this disables MMC. Any chance to get the log from this run? (Let me know if eMMC is needed to get the log.) -Kevin

2 1

Re: [SeaBIOS] Failure to detect high-capacity SD card
by Kevin O'Connor 26 Oct '17

26 Oct '17

On Wed, Oct 25, 2017 at 11:02:50PM -0500, Matt DeVillier wrote: > On Mon, Oct 23, 2017 at 12:04 PM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > > > On Mon, Oct 23, 2017 at 11:59:16AM -0500, Matt DeVillier wrote: > > > On Mon, Oct 23, 2017 at 11:34 AM, Kevin O'Connor <kevin(a)koconnor.net> > > wrote: > > > > On Mon, Oct 23, 2017 at 10:55:57AM -0500, Matt DeVillier wrote: > > > > > On Mon, Oct 23, 2017 at 10:50 AM, Kevin O'Connor <kevin(a)koconnor.net > > > > > > > > > On Mon, Oct 23, 2017 at 10:39:03AM -0400, Kevin O'Connor wrote: > > > > > > Unfortunately, I can't see anything wrong in the log - the card > > isn't > > > > > > coming out of its initialization phase. I can't see any reason why > > > > > > that would be. Has the user verified the card works under Linux? > > > > > card is detected/usable under Linux, no problem there > > > > Can you grab the syslog/dmesg output from Linux as it detects and > > > > initializes the card? > > > sure, any special kernel params for debugging? > > > > The more info on the sdcard the better. I'm not hopeful, but maybe > > something Linux does to the sdcard will help point out why seabios > > isn't able to init the card. > > > > -Kevin > > > > dmesg: https://paste.ubuntu.com/25820588/ > cat /sys/kernel/debug/mmc1/ios : https://paste.ubuntu.com/25821138/ Does the patch below help? -Kevin --- a/src/hw/sdcard.c +++ b/src/hw/sdcard.c @@ -405,6 +405,7 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, int prio) if (!ret && param[0] == vrange) hcs = (1<<30); // Verify SD card (instead of MMC or SDIO) +#if 0 param[0] = 0x00; ret = sdcard_pio_app(regs, SC_APP_SEND_OP_COND, param); if (ret) { @@ -416,6 +417,7 @@ sdcard_card_setup(struct sddrive_s *drive, int volt, int prio) drive->card_type |= SF_MMC; hcs = (1<<30); } +#endif // Init card u32 end = timer_calc(SDHCI_POWERUP_TIMEOUT); for (;;) {

2 1

Re: [SeaBIOS] Failure to detect high-capacity SD card
by Kevin O'Connor 26 Oct '17

26 Oct '17

On Mon, Oct 23, 2017 at 11:59:16AM -0500, Matt DeVillier wrote: > On Mon, Oct 23, 2017 at 11:34 AM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > > On Mon, Oct 23, 2017 at 10:55:57AM -0500, Matt DeVillier wrote: > > > On Mon, Oct 23, 2017 at 10:50 AM, Kevin O'Connor <kevin(a)koconnor.net> > > > > On Mon, Oct 23, 2017 at 10:39:03AM -0400, Kevin O'Connor wrote: > > > > Unfortunately, I can't see anything wrong in the log - the card isn't > > > > coming out of its initialization phase. I can't see any reason why > > > > that would be. Has the user verified the card works under Linux? > > > card is detected/usable under Linux, no problem there > > Can you grab the syslog/dmesg output from Linux as it detects and > > initializes the card? > sure, any special kernel params for debugging? The more info on the sdcard the better. I'm not hopeful, but maybe something Linux does to the sdcard will help point out why seabios isn't able to init the card. -Kevin

2 1

答复: Welcome to the "SeaBIOS" mailing list (Digest mode)
by 李勇 24 Oct '17

24 Oct '17

-----邮件原件----- 发件人: SeaBIOS [mailto:seabios-bounces@seabios.org] 代表 seabios-request(a)seabios.org 发送时间: 2017年10月20日 7:35 收件人: edwardli(a)dl.cn 主题: Welcome to the "SeaBIOS" mailing list (Digest mode) Welcome to the SeaBIOS(a)seabios.org mailing list! To post to this list, send your message to: seabios(a)seabios.org General information about the mailing list is at: https://mail.coreboot.org/mailman/listinfo/seabios If you ever want to unsubscribe or change your options (eg, switch to or from digest mode, change your password, etc.), visit your subscription page at: https://mail.coreboot.org/mailman/options/seabios/edwardli%40dl.cn You can also make such adjustments via email by sending a message to: SeaBIOS-request(a)seabios.org with the word `help' in the subject or body (don't include the quotes), and you will get back a message with instructions. You must know your password to change your options (including changing the password, itself) or to unsubscribe without confirmation. It is: nLixZhot Normally, Mailman will remind you of your seabios.org mailing list passwords once every month, although you can disable this if you prefer. This reminder will also include instructions on how to unsubscribe or change your account options. There is also a button on your options page that will email your current password to you.

1 0

Re: [SeaBIOS] Failure to detect high-capacity SD card
by Kevin O'Connor 24 Oct '17

24 Oct '17

On Mon, Oct 23, 2017 at 10:55:57AM -0500, Matt DeVillier wrote: > On Mon, Oct 23, 2017 at 10:50 AM, Kevin O'Connor <kevin(a)koconnor.net> wrote: > > > On Mon, Oct 23, 2017 at 10:39:03AM -0400, Kevin O'Connor wrote: > > > Hi Matt, > > > > > > Can you gzip and attach the full log? (At a minimum, we need to see > > > the log extracts around the failure.) > > > > Ah, nevermind - I see the timeout is generated in the wait for the > > card to be in a non-busy state. > > > > Unfortunately, I can't see anything wrong in the log - the card isn't > > coming out of its initialization phase. I can't see any reason why > > that would be. Has the user verified the card works under Linux? > > > > card is detected/usable under Linux, no problem there Can you grab the syslog/dmesg output from Linux as it detects and initializes the card? The failure is pretty early in the card detection process and the card isn't responding with a particular failure - it's just not going into a ready state. -Kevin

2 1

Re: [SeaBIOS] Failure to detect high-capacity SD card
by Kevin O'Connor 24 Oct '17

24 Oct '17

On Thu, Oct 19, 2017 at 09:16:13PM -0500, Matt DeVillier wrote: > I have a user running one of my SeaBIOS builds on a Asus Chromebook C302sa > (google/cave) who is having trouble with high-capacity SD cards - they > aren't detected at all due to timing out during init. I've tried adjusting > the various timeouts, and the only change is longer log files. > > The SD card in question is a Samsung MB-MC128GA U3 Micro SDXC 128GB EVO > Plus. Trimmed log below. The function start indicators were inserted by me > as part of debugging. > > thanks in advance, > Matt > > SeaBIOS (version rel-1.10.0-81-g0fff851-MrChromebox-2017.10.19) > /7a9fe000\ Start thread > |7a9fe000| Searching bootorder for: /pci@i0cf8/*@1e,6 > |7a9fe000| sdcard_pci_setup() start > |7a9fe000| sdcard_controller_setup() start > |7a9fe000| sdhci@0xd1138000 ver=1002 cap=7568c881 807 > |7a9fe000| sdcard_card_setup() start > |7a9fe000| sdcard_set_frequency 200 400 fa00 > |7a9fe000| sdcard_pio cmd 0 0 1ff0000 > |7a9fe000| sdcard cmd 0 response 0 0 0 0 > |7a9fe000| sdcard_pio cmd 81a 1aa 1ff0000 > |7a9fe000| sdcard cmd 81a response 1aa 0 0 0 > |7a9fe000| sdcard_pio cmd 371a 0 1ff0000 > |7a9fe000| sdcard cmd 371a response 120 0 0 0 > |7a9fe000| sdcard_pio cmd 2902 0 1ff0000 > |7a9fe000| sdcard cmd 2902 response ff8000 0 0 0 > |7a9fe000| sdcard_pio cmd 371a 0 1ff0000 > |7a9fe000| sdcard cmd 371a response 120 0 0 0 > |7a9fe000| sdcard_pio cmd 2902 40100000 1ff0000 > |7a9fe000| sdcard cmd 2902 response ff8000 0 0 0 > |7a9fe000| sdcard_pio cmd 371a 0 1ff0000 > |7a9fe000| sdcard cmd 371a response 120 0 0 0 > |7a9fe000| sdcard_pio cmd 2902 40100000 1ff0000 > |7a9fe000| sdcard cmd 2902 response ff8000 0 0 0 > <snip ~1s of 371a/2902 sequence repeating> > |7a9fe000| WARNING - Timeout at sdcard_card_setup:432! > \7a9fe000/ End thread > All threads complete. Hi Matt, Can you gzip and attach the full log? (At a minimum, we need to see the log extracts around the failure.) -Kevin

2 3

答复: Welcome to the "SeaBIOS" mailing list (Digest mode)
by 李勇 21 Oct '17

21 Oct '17

1 0

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

SeaBIOS