[coreboot] Patch merged into coreboot/master: 2354ef8 exynos/snow: get rid of board-specific arbitration code

gerrit at coreboot.org gerrit at coreboot.org
Mon Feb 4 06:08:21 CET 2013


the following patch was just integrated into master:
commit 2354ef8869a287cf4c7de792c17485ad82b026cf
Author: David Hendricks <dhendrix at chromium.org>
Date:   Sun Feb 3 20:47:38 2013 -0800

    exynos/snow: get rid of board-specific arbitration code
    
    Snow's AP, EC, PMU, and smarty battery share a bus. Both the AP and
    EC can act as a master, so to avoid conflicts an arbitration
    mechanism consisting of two GPIOs is used.
    
    By default, the AP "owns" the bus unless it is off (in which case
    the EC doesn't monitor the arbitration pins). This means the boot
    firmware does not need to worry about these lines. The payload may
    if it needs to communicate with the EC, though.
    
    In any case, board-specific bus arbitration logic does not belong
    in a low-level driver that is supposed to be generic for an entire
    CPU family. If the payload needs to talk to the EC, we'll deal with
    it there.
    
    Change-Id: I0774d4592af2b21b6ad668441532c5ceab988404
    Signed-off-by: David Hendricks <dhendrix at chromium.org>
    Reviewed-on: http://review.coreboot.org/2272
    Tested-by: build bot (Jenkins)
    Reviewed-by: Ronald G. Minnich <rminnich at gmail.com>

Build-Tested: build bot (Jenkins) at Mon Feb  4 06:03:56 2013, giving +1
Reviewed-By: Ronald G. Minnich <rminnich at gmail.com> at Mon Feb  4 06:08:20 2013, giving +2
See http://review.coreboot.org/2272 for details.

-gerrit



More information about the coreboot mailing list