[coreboot-gerrit] Patch merged into coreboot/master: d9af30b broadcom/cygnus: Implement I2C driver

gerrit at coreboot.org gerrit at coreboot.org
Wed Apr 22 08:57:42 CEST 2015


the following patch was just integrated into master:
commit d9af30bec1c5d47f51dcdfbe540ad2fcebb2fdc4
Author: Anatol Pomazau <anatol at google.com>
Date:   Wed Feb 4 10:09:27 2015 -0800

    broadcom/cygnus: Implement I2C driver
    
    BUG=chrome-os-partner:35810
    BRANCH=purin
    TEST=Enable I2C1, reset devboard codec, read a register.
         Here is the code that demonstrates how I2C works:
    
    	i2c_init(1, 100*KHz);
    	mdelay(50);
    
    	int rc = i2c_writeb(1, 0x18, 1, 0x80); // reset codec
    	printk(BIOS_INFO, "I2C reset rc=%d\n", rc);
    	mdelay(50);
    	uint8_t data = 0;
    	rc = i2c_readb(1, 0x18, 43, &data);
    	printk(BIOS_INFO, "I2C read rc=%d data=%x\n", rc, data); // data == 0x80
    
    Change-Id: I0d202f8b0375b5ccd9f71b23fb0cadd5a70ae779
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 6bbe9afe3dccd104f39c2c286d3765a28ea20141
    Original-Signed-off-by: Anatol Pomazau <anatol at google.com>
    Original-Reviewed-on: https://chrome-internal-review.googlesource.com/195706
    Original-Reviewed-by: Daisuke Nojiri <dnojiri at google.com>
    Original-Reviewed-by: Anatol Pomazau <anatol at google.com>
    Original-Commit-Queue: Anatol Pomazau <anatol at google.com>
    Original-Tested-by: Anatol Pomazau <anatol at google.com>
    Original-Change-Id: I178acef9de18fa854983294edcd2c05886795e2a
    Original-Reviewed-on: https://chromium-review.googlesource.com/263496
    Original-Commit-Queue: Daisuke Nojiri <dnojiri at chromium.org>
    Original-Trybot-Ready: Daisuke Nojiri <dnojiri at chromium.org>
    Original-Tested-by: Daisuke Nojiri <dnojiri at chromium.org>
    Original-Reviewed-by: Daisuke Nojiri <dnojiri at chromium.org>
    Reviewed-on: http://review.coreboot.org/9908
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list