Ревизии для проверки звука из колонок

Материал из Toshiba AC100 wiki
Перейти к навигации Перейти к поиску

Сейчас не работают колонки в последних ревизиях. В ревизии f23feedc0 колонки работали. Соответственно нужно найти ревизию, в которой поломалось. Ревизии, выбираемые бисектом зачастую сложно скомпилить, т.к. большинство из них будут попадать в середину мерджей.

ВАЖНО Проверять поочереди: снизу, потом сверху. ВАЖНО Если нижняя ревизия работает, то пинаете Stuw'а на канале #ac100-ru или #ac100 (анг.) на irc.freenode.net. ВАЖНО Перед тестами надо применять патчи - https://dl.dropboxusercontent.com/u/40761340/www/ac100/patches/sound-pinmuxes.patch и https://dl.dropboxusercontent.com/u/40761340/www/ac100/patches/sound-kcontrol.patch ВАЖНО Если берете ревизию на проверку, делаете коментарий на этой странице, чтобы никто не взял ту же самую ревизию. ВАЖНО Конфиг алсы лучше взять отсюда - конфиг марвина, на котором работают колонки. Применять вроде командой alsactl restore


Как настроить звук из колонок. - включаем музыку, в алсамиксере убираем мьют у всех элементов и меняем значнеие SpealerOut Mux на Speaker Mix если не помогло - берем ревизию ebcf023034bf5f81078c639962cfbc000acfa999 и применяем патч https://dl.dropboxusercontent.com/u/40761340/www/ac100/patches/sound-test.patch - включаем звук и играемся с алсамиксером :)


stuw - работает - ebcf023034bf5f81078c639962cfbc000acfa999

commit ebcf023034bf5f81078c639962cfbc000acfa999
Author: Marc Dietrich <[email protected]>
Date:   Sun Aug 14 22:31:51 2011 +0200

    nvec: use init from i2c-tegra

    This patch uses the I2C controller initialization from i2c-tegra.
    Tested, but there may be still some glitches...

    I had to increase the sleep timers in nvec_power again to get rid
    of returning nvec init errors. How I hate this!


stuw - работает - b5d4ab75afce84f25879ba72b028f08f67861079

commit b5d4ab75afce84f25879ba72b028f08f67861079
Merge: fb9a0ee 0f5d8be
Author: Marc Dietrich <[email protected]>
Date:   Thu Aug 25 23:14:25 2011 +0200

    Merge branch 'chromeos-ac100-2.6.38' of gitorious.org:~marvin24/ac100/marvin24s-kernel into chromeos-ac100-2.6.38


leonro - работает - ebe0f9874341305206a586a54859bb882af6d80a

commit ebe0f9874341305206a586a54859bb882af6d80a
Author: Marc Dietrich <[email protected]>
Date:   Sat Aug 27 15:26:10 2011 +0200

    Revert "adds initial wake from lp0 support"

    This reverts commit 22e30c586e0d2addbf7ccdb56d08535557034759.

stuw - работает - 3710d073184d5c3010df5e58233b85547b0b723a

commit 3710d073184d5c3010df5e58233b85547b0b723a
Merge: f95593d ec53b53
Author: Marc Dietrich <[email protected]>
Date:   Sat Aug 27 16:47:40 2011 +0200

    Merge branch 'chromeos-2.6.38' of http://git.chromium.org/chromiumos/third_party/kernel into chromeos-ac100-2.6.38

    Conflicts:
        arch/arm/mach-tegra/usb_phy.c
        drivers/usb/host/ehci-tegra.c
        net/mac80211/mlme.c


stuw - spk event:2 in dmesg, клава не пашет - 65c7568406d14ff4f213c23a76e040aa6c0b6e46 leonro - работает - 65c7568406d14ff4f213c23a76e040aa6c0b6e46

commit 65c7568406d14ff4f213c23a76e040aa6c0b6e46
Author: Marc Dietrich <[email protected]>
Date:   Sat Aug 27 22:31:20 2011 +0200

    paz00: panel: autodetect framebuffer address

    Gets the framebuffer address from atags (same way as lp0_vec). This
    should fix problems with framebuffer during boot on 2.1 bootloaders.

тут больше нет коммитов. Похоже проблема в коммите 2c83404c75ae14d4f4440cb45386824eb883fd6c

leonro - не работает - 2c83404c75ae14d4f4440cb45386824eb883fd6c stuw - не могу проверить, не пашет клава - 2c83404c75ae14d4f4440cb45386824eb883fd6c

commit 2c83404c75ae14d4f4440cb45386824eb883fd6c
Author: Marc Dietrich <[email protected]>
Date:   Sun Aug 28 16:21:23 2011 +0200

    nvec: don't enable the clock in i2c-tegra for slave mode

    It also seems we can read out the battery info safely without crashing
    the system now. This should bring battery statistic graphs back.


такой патч не помогает (частичный реверт):

diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c
index 44e37d0..fec9306 100644
--- a/drivers/staging/nvec/nvec.c
+++ b/drivers/staging/nvec/nvec.c
@@ -311,6 +311,7 @@ static void nvec_request_master(struct work_struct *work)

                if (err == 0) {
                        dev_warn(nvec->dev, "timeout waiting for ec transfer\n");
+                       writel(I2C_SL_NEWSL | I2C_SL_NACK, nvec->base + I2C_SL_CNFG);
                        nvec_gpio_set_value(nvec, 1);
                        msg->pos = 0;
                }

нужно проверить вот такой патч (тоже частичный реверт)

diff --git a/drivers/i2c/busses/i2c-tegra.c b/drivers/i2c/busses/i2c-tegra.c
index 029b479..f8c827c 100755
--- a/drivers/i2c/busses/i2c-tegra.c
+++ b/drivers/i2c/busses/i2c-tegra.c
@@ -689,8 +689,8 @@ static int tegra_i2c_probe(struct platform_device *pdev)
                        dev_err(&pdev->dev, "Failed to request irq %i\n", i2c_dev->irq);
                        goto err_free;
                }
-               clk_enable(i2c_dev->i2c_clk);
        }
+       clk_enable(i2c_dev->i2c_clk);

        for (i = 0; i < nbus; i++) {
                struct tegra_i2c_bus *i2c_bus = &i2c_dev->busses[i];