DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 701202086D
MIME-Version: 1.0
References: <CALAqxLUVajBw_PkyhA0s=FhQQHWu1FBEMX+4oDTStRXr6L5Txw@mail.gmail.com>
 <20181211201221.GY39861@atomide.com> <CAHYQr0qffa_G2ZVnSsDfjqSgxuBBZB+8e6ZODpAWV_TMhqzFYg@mail.gmail.com>
 <CAHYQr0qhqce2QcwMLRMWnqhZTL4KzHKJLzt6tfCNLHUcEt9pyQ@mail.gmail.com>
 <20181212014556.GC39861@atomide.com> <c8625c42f04a4721b7efb7d7b6b3931c@ti.com>
 <20181212183116.GH39861@atomide.com> <CAHYQr0ojkf3CvFb_zuEK4Vuz1w-rm9Sb=+dk5BkqYkWoxy8+aQ@mail.gmail.com>
 <9060b4655e064665a1812d9fae00f1ec@ti.com> <CAHYQr0r4eQNSqZEr-vbCiVN6G_O1CXxX9cxemANSRY-jqDMZ=A@mail.gmail.com>
 <20181213144522.GO39861@atomide.com> <12a609f677aa48f8b2aadba69edf342a@ti.com>
In-Reply-To: <12a609f677aa48f8b2aadba69edf342a@ti.com>
From:   Ricardo Salveti <rsalveti@rsalveti.net>
Date:   Thu, 13 Dec 2018 12:55:12 -0200
Message-ID: <CAHYQr0onjCz4YqrWoqLYu+omYmn_e2+uwWGCBFvkNf+kupYucQ@mail.gmail.com>
Subject: Re: [EXTERNAL] Re: wlcore getting stuck on hikey after the runtime PM
 autosuspend support change
To:     eyalr@ti.com
Cc:     Tony Lindgren <tony@atomide.com>,
        John Stultz <john.stultz@linaro.org>,
        linux-wireless@vger.kernel.org, anders.roxell@linaro.org
Content-Type: text/plain; charset="UTF-8"
Sender: linux-wireless-owner@vger.kernel.org
Precedence: bulk

On Thu, Dec 13, 2018 at 12:53 PM Reizer, Eyal <eyalr@ti.com> wrote:
> > > I tried just increasing WL1271_PRE_POWER_ON_SLEEP from 20ms to
> > 50ms
> > > (which gets used before calling wl12xx_sdio_power_on), and that was
> > > already enough to fix my issues on both hikey and beaglebone black
> > > wireless.
> >
> > OK good to hear that helps :)
> >
> > > Should we define another pre power on sleep specifically for the sdio
> > > case (and use directly in wl12xx_sdio_power_on)?
> >
> > I'd probably prefer to just increase WL1271_PRE_POWER_ON_SLEEP,
> > chances are the same delay is needed in all cases.
> >
> > Up to Eyeal to decide though as far as I'm concerned.
> >
>
> I tend to agree with Tony. Probably better to just increase the value.

Cool, will do a bit more stress testing and send the patch then.

Thanks,
-- 
Ricardo Salveti de Araujo