[coreboot-gerrit] Patch merged into coreboot/master: qualcomm/ipq40xx: add vector operation method to SPI

gerrit at coreboot.org gerrit at coreboot.org
Wed Feb 22 17:03:11 CET 2017


the following patch was just integrated into master:
commit 292d4e120efe46deee1c06989e3acb2ddfd6d060
Author: Yuji Sasaki <sasakiy at google.com>
Date:   Thu Jan 26 10:17:45 2017 -0800

    qualcomm/ipq40xx: add vector operation method to SPI
    
    Adding spi_xfer_two_vectors as .xfer_vector for ipq40xx spi_ctrlr.
    Commit c2973d196d1 ("UPSTREAM: spi: Get rid of SPI_ATOMIC_SEQUENCING")
    has added a new driver method xfer_vector to support combined write-read
    operation within a single CS cycle. The method is wrapped in the
    spi_xfer_vector() API. When spi_ctrlr structure does not have
    xfer_vector method, API calls write and read operations sequentially.
    However the QCA40xx SPI driver has "forced" CS activation-inactivation
    in xfer method, so individual operation will break CS after write
    operation, making combined write-read cycle broken.
    Adding xfer_vector method to spi_ctrlr is a simple fix to prevent this.
    
    BUG=None
    BRANCH=none
    TEST=built and run on Gale
    
    Change-Id: I2258e563d0793bcacd626f78b8e96b3649a8e4a4
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 88a8824951cef4fe293dfa6e3a1a837ae07b6156
    Original-Change-Id: I031e85ce5b847353cb1084f6f68b2af8c6f702e1
    Original-Signed-off-by: Yuji Sasaki <sasakiy at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/433439
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Reviewed-by: Kan Yan <kyan at google.com>
    Reviewed-on: https://review.coreboot.org/18297
    Tested-by: build bot (Jenkins)
    Reviewed-by: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
    Reviewed-by: Martin Roth <martinroth at google.com>


See https://review.coreboot.org/18297 for details.

-gerrit



More information about the coreboot-gerrit mailing list