[coreboot-gerrit] Patch merged into coreboot/master: intel/strago: BCRD2: Enable Realtek Audio codec on I2C4

gerrit at coreboot.org gerrit at coreboot.org
Thu Jul 23 16:33:12 CEST 2015


the following patch was just integrated into master:
commit 0a754021cf7eecf385bc6e9885ce57c5b9a33c0d
Author: Jenny TC <jenny.tc at intel.com>
Date:   Thu Jun 18 16:06:57 2015 +0530

    intel/strago: BCRD2: Enable Realtek Audio codec on I2C4
    
    In BCRD2, RTEK audio codec is connected to I2C4.
    Create a RTEK device entry on I2C4 to enable Audio
    on BCRD2. In BCRD1, RTEK device is connected to I2C2.
    Having two devices with same HID breaks the Audio
    on BCRD2 even if I2C2.RTEK._STA returns 0. The Audio
    codec driver in kernel is hard coded to use first
    instance of the device (:00). When two devices are present
    with same HID, first device gets an instance number :00
    even though _STA returns 0. Second device which is on I2C4
    and POR for BCRD2 assigned with instance number :01. The
    device with :01 is not getting enabled since the Audio codec
    driver supports only :00. This need a proper fix in kernel
    which is in the pipeline. Audio on non BCRD2 platforms on
    Strago build would be disabled since RTEK device is not present
    on I2C2.
    
    BRANCH=None
    BUG=None
    TEST=Build and boot the system
    
    Change-Id: Ia97d011c951275e6179c8b79a22c496b8169356b
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: d71a41ee703e6f60299b9e31a408af2ca06d8e24
    Original-Change-Id: I4b032e930e46da77474f8f5969e95f9560b3e905
    Original-Signed-off-by: Jenny TC <jenny.tc at intel.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/285193
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Commit-Queue: Divagar Mohandass <divagar.mohandass at intel.com>
    Reviewed-on: http://review.coreboot.org/11003
    Tested-by: build bot (Jenkins)
    Reviewed-by: Stefan Reinauer <stefan.reinauer at coreboot.org>


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

-gerrit



More information about the coreboot-gerrit mailing list