[coreboot-gerrit] Patch merged into coreboot/master: a967f41 libpayload: usb mass storage detect empty media

gerrit at coreboot.org gerrit at coreboot.org
Mon Nov 25 23:56:15 CET 2013


the following patch was just integrated into master:
commit a967f414dfb1f85097ba7ca2f00fdc7f415da776
Author: Aaron Durbin <adurbin at chromium.org>
Date:   Thu Jun 6 16:14:21 2013 -0500

    libpayload: usb mass storage detect empty media
    
    There is currently a hard-coded 30 sec delay in the mass storage
    driver while waiting for each device to become ready. However, mass
    storage card readers that are empty return an error code on the
    TEST UNIT READY command. A REQUEST SENSE command then needs to be
    issued and interrogate the data to determine if no media is present.
    If no media determination is found to be true the USB device is no
    longer considered a candidate to be a disk.
    
    This code does lead to the fact that the media card reader needs to be
    populated at enumeration time. I suspect this is not an issue as it
    appears the storage stack in libpayload can't handle removable media
    coming online later.
    
    Booted recovery and dev modes. Noted that removable mass storage
    devices with no media were ignored without any boot delay.
    
    Change-Id: Ida7a45614d97c6e6fbfc9bb099765aad4df550fd
    Signed-off-by: Aaron Durbin <adurbin at chromium.org>
    Reviewed-on: https://gerrit.chromium.org/gerrit/57828
    Reviewed-by: Duncan Laurie <dlaurie at chromium.org>
    Reviewed-on: http://review.coreboot.org/4225
    Tested-by: build bot (Jenkins)
    Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>


See http://review.coreboot.org/4225 for details.

-gerrit



More information about the coreboot-gerrit mailing list