https://bugzilla.kernel.org/show_bug.cgi?id=59911

--- Comment #57 from barton xu <tank.xu...@gmail.com> ---
I have shown my change in comment 39, copy to here again


static void advance_transaction(struct acpi_ec *ec, u8 status)
{
    unsigned long flags;
    struct transaction *t = ec->curr;

    spin_lock_irqsave(&ec->lock, flags);
    if (!t)
        goto unlock;
    if (t->wlen > t->wi) {
//bxudbg>>>
        if (t->wi == 0) {
            if (((status & ACPI_EC_FLAG_IBF) == 0) && ((status &
ACPI_EC_FLAG_CMD) == 0x08))
                acpi_ec_write_data(ec,t->wdata[t->wi++]);
            else
                goto err;
        }
        else {
            if (((status & ACPI_EC_FLAG_IBF) == 0) && ((status &
ACPI_EC_FLAG_CMD) == 0))
                acpi_ec_write_data(ec,t->wdata[t->wi++]);
            else
                goto err;
        }
//bxudbg<<<
//        if ((status & ACPI_EC_FLAG_IBF) == 0)
//            acpi_ec_write_data(ec,
//                t->wdata[t->wi++]);
//        else
//            goto err;



Hi tianyu,

     You patch in comment 49 can lower the fail rate, but still fail.

-- 
You are receiving this mail because:
You are watching the assignee of the bug.

------------------------------------------------------------------------------
"Accelerate Dev Cycles with Automated Cross-Browser Testing - For FREE
Instantly run your Selenium tests across 300+ browser/OS combos.
Get unparalleled scalability from the best Selenium testing platform available
Simple to use. Nothing to install. Get started now for free."
http://p.sf.net/sfu/SauceLabs
_______________________________________________
acpi-bugzilla mailing list
acpi-bugzilla@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acpi-bugzilla

Reply via email to