From: Ulf Hansson <ulf.hansson@linaro.org>
To: Jan Kiszka <jan.kiszka@web.de>
Cc: John Stultz <john.stultz@linaro.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
Wei Xu <xuwei5@hisilicon.com>, Ryan Grachek <ryan@edited.us>,
linux-wireless@vger.kernel.org, "Reizer, Eyal" <eyalr@ti.com>,
Valentin Schneider <valentin.schneider@arm.com>,
Tony Lindgren <tony@atomide.com>,
Kalle Valo <kvalo@codeaurora.org>,
KISHON VIJAY ABRAHAM <kishon@ti.com>, "Mishol, Guy" <guym@ti.com>,
Anders Roxell <anders.roxell@linaro.org>,
Ricardo Salveti <rsalveti@rsalveti.net>
Subject: Re: [PATCH] arm64: dts: hikey: Give wifi some time after power-on
Date: Tue, 15 Jan 2019 09:13:03 +0100 [thread overview]
Message-ID: <CAPDyKFpmFGu6R4PkBe_8z7NcwewZrJ_Cv0ywj0+jwO7_WLZgrQ@mail.gmail.com> (raw)
In-Reply-To: <0a6e6ca5-3ea0-4f6a-63f1-96187700a4bc@web.de>
On Tue, 15 Jan 2019 at 09:10, Jan Kiszka <jan.kiszka@web.de> wrote:
>
> On 06.01.19 10:53, Jan Kiszka wrote:
> > On 03.01.19 22:32, Ulf Hansson wrote:
> >> On Wed, 2 Jan 2019 at 23:21, John Stultz <john.stultz@linaro.org> wrote:
> >>>
> >>> Adding a few folks to cc from the thread here:
> >>> https://patchwork.kernel.org/patch/10734021/
> >>>
> >>> As this sounds like a very similar issue.
> >>> thanks
> >>> -john
> >>
> >> John, thanks for looping me in.
> >>
> >>>
> >>> On Sun, Dec 30, 2018 at 3:38 AM Jan Kiszka <jan.kiszka@web.de> wrote:
> >>>>
> >>>> From: Jan Kiszka <jan.kiszka@siemens.com>
> >>>>
> >>>> Somewhere along recent changes to power control of the wl1835, power-on
> >>>> became very unreliable on the hikey, failing like this:
> >>>>
> >>>> wl1271_sdio: probe of mmc2:0001:1 failed with error -16
> >>>> wl1271_sdio: probe of mmc2:0001:2 failed with error -16
> >>>>
> >>>> After playing with some dt parameters and comparing to other users of
> >>>> this chip, it turned out we need some power-on delay to make things
> >>>> stable again. In contrast to those other users which define 200 ms, the
> >>>> hikey is already very happy with 1 ms.
> >>
> >> According to the discussions we had so far, this doesn't sound like
> >> the correct solution.
> >>
> >> I am intending to post patch soon, however, I am not sure exactly what
> >> solution to pick yet. I will keep you on cc - and of course I
> >> appreciate if you could help to test.
> >>
> >
> > FWIW, the Ultra96 is also affected. Here I worked around it by using a 10 ms
> > power-on delay. The original workaround from the other thread did not help.
> > Looking forward to the proper fix now!
> >
>
> Any news on this regression?
I will post a (or two) patch during the day.
Kind regards
Uffe
prev parent reply other threads:[~2019-01-15 8:13 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-30 11:38 [PATCH] arm64: dts: hikey: Give wifi some time after power-on Jan Kiszka
2019-01-02 22:21 ` John Stultz
2019-01-03 21:32 ` Ulf Hansson
2019-01-06 9:53 ` Jan Kiszka
2019-01-15 8:10 ` Jan Kiszka
2019-01-15 8:13 ` Ulf Hansson [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAPDyKFpmFGu6R4PkBe_8z7NcwewZrJ_Cv0ywj0+jwO7_WLZgrQ@mail.gmail.com \
--to=ulf.hansson@linaro.org \
--cc=anders.roxell@linaro.org \
--cc=eyalr@ti.com \
--cc=guym@ti.com \
--cc=jan.kiszka@web.de \
--cc=john.stultz@linaro.org \
--cc=kishon@ti.com \
--cc=kvalo@codeaurora.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=rsalveti@rsalveti.net \
--cc=ryan@edited.us \
--cc=tony@atomide.com \
--cc=valentin.schneider@arm.com \
--cc=xuwei5@hisilicon.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).