[coreboot-gerrit] Patch merged into coreboot/master: 68ec2fc ipq806x: Break apart large transfers in spi_xfer()

gerrit at coreboot.org gerrit at coreboot.org
Tue Mar 17 16:54:31 CET 2015


the following patch was just integrated into master:
commit 68ec2fce2bcf0f9d8f02d80feb4d6bc4385e9047
Author: David Hendricks <dhendrix at chromium.org>
Date:   Fri Jul 25 12:59:48 2014 -0700

    ipq806x: Break apart large transfers in spi_xfer()
    
    The current spi_xfer() function sets the count in hardware and then
    loops while waiting for the requested number of bytes to be sent or
    received. However, the number of bytes to be transferred may exceed
    the maximum count that can be programmed into the controller.
    
    This patch re-factors spi_xfer() to split the low-level FIFO handling
    portions for transmit/receive into their own functions to be called
    by loops in spi_xfer() which will break large transfers into smaller
    ones.
    
    BUG=chrome-os-partner:30904
    BRANCH=storm
    TEST=built and booted with a >64KB payload on Storm
    
    Original-Change-Id: I70743487996cf08cfc602449f2181a7fcd99bfa4
    Original-Signed-off-by: David Hendricks <dhendrix at chromium.org>
    Original-Signed-off-by: Vadim Bendebury <vbendeb at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/209838
    Original-Reviewed-by: Trevor Bourget <tbourget at codeaurora.org>
    Original-Tested-by: Trevor Bourget <tbourget at codeaurora.org>
    (cherry picked from commit 5ec28de11f12c2438356f45ce978a17fbb603bf7)
    Signed-off-by: Marc Jones <marc.jones at se-eng.com>
    
    Change-Id: I0033e0dd96006cfd30a7a4f5e5a052f677e05108
    Reviewed-on: http://review.coreboot.org/8676
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list