[coreboot-gerrit] Patch merged into coreboot/master: intel/fsp_baytrail: Use read32() and write32() in i2c.c

gerrit at coreboot.org gerrit at coreboot.org
Fri Mar 25 18:24:36 CET 2016


the following patch was just integrated into master:
commit 5aecd0e533fe8e7dd5fee0a3c10ee8aabe6c8803
Author: Ben Gardner <gardner.ben at gmail.com>
Date:   Wed Mar 23 10:11:24 2016 -0500

    intel/fsp_baytrail: Use read32() and write32() in i2c.c
    
    i2c.c uses "*(volatile unsigned int *)" constructs where it could use
    read32() and write32().
    Switch to using read32() and write32().
    
    The remaining instances in wait_tx_fifo() and wait_rx_fifo() are fixed
    in https://review.coreboot.org/#/c/14160/
    Change-Id: I39e4ff4206587267b6fceef58f4a567bf162fbbe
    (intel/fsp_baytrail: Fix I2C abort logic)
    
    I also fixed a few minor white space issues.
    
    Change-Id: I587551272ac171ef1f42c7eb26daf877dc56646b
    Signed-off-by: Ben Gardner <gardner.ben at gmail.com>
    Reviewed-on: https://review.coreboot.org/14162
    Tested-by: build bot (Jenkins)
    Reviewed-by: Martin Roth <martinroth at google.com>
    Reviewed-by: Werner Zeh <werner.zeh at siemens.com>


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

-gerrit



More information about the coreboot-gerrit mailing list