All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fabian Wittenberg <Fabian.Wittenberg@sophos.com>
To: Michal Kazior <michal.kazior@tieto.com>
Cc: "ath10k@lists.infradead.org" <ath10k@lists.infradead.org>
Subject: Re: ath10k + INTEL_IDLE aka. cstates == firmware crash
Date: Mon, 23 Feb 2015 15:41:52 +0100	[thread overview]
Message-ID: <54EB3C30.60302@sophos.com> (raw)
In-Reply-To: <CA+BoTQm01wMzdB1-1V6OE=oNRuCcrrAoRU5LtxYiQt+TmSEfxA@mail.gmail.com>

Hi Michal,

I already did this approach. This works fine and is the current
workaround to get the product out, but I would like to know what the
basic problem is.
The power consumption increases by ~1.25W on idle devices if you disable
cstates. This is not a real problem but a low mem corruption is one.
So I assume a bug in the ath10k-driver/firmware.

Regards,
Fabian

Am 23.02.2015 um 15:20 schrieb Michal Kazior:
> On 23 February 2015 at 14:44, Fabian Wittenberg
> <Fabian.Wittenberg@sophos.com> wrote:
>> Hi Michal,
>>
>> I used firmware version 10.1 and 10.2 from here:
>> https://github.com/kvalo/ath10k-firmware. Both show the same behavior.
>>
>> You are right. There are some BIOS that do strange handling of this
>> cstate stuff but we have no influence on the BIOS as this is done by
>> our hardware vendor. We experimented a lot with the MSI masking bit of
>> the pci-e root bridge where the ac-card is connected to.
>> There were no remarkable improvements playing around with this bit.
> If you don't have BIOS/UEFI upgrade available you can try appending
> `intel_idle.max_cstate=0` to kernel boot parameters. Keep in mind this
> will disable CPU power management.
>
>
>> We have tested the same boards with cards that need ath9k as well. They
>> are working just fine. With and without enabled INTEL_IDLE...
> I run with my laptops (i5-3320M and  i5-2520M) with INTEL_IDLE and QCA988x fine.
>
> I'm not really an expert on this stuff but since C-state alter some
> voltages perhaps the Atom SoC has some deviations/instabilities in
> voltages which prevent some quirky devices like QCA988x from working
> reliably.
>
>
> Michał


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

  reply	other threads:[~2015-02-23 14:42 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-02-23 13:08 ath10k + INTEL_IDLE aka. cstates == firmware crash Fabian Wittenberg
2015-02-23 13:32 ` Michal Kazior
2015-02-23 13:44   ` Fabian Wittenberg
2015-02-23 14:20     ` Michal Kazior
2015-02-23 14:41       ` Fabian Wittenberg [this message]
2015-03-02 12:20         ` Michal Kazior
2015-03-19  9:20           ` Fabian Wittenberg
2015-03-19 15:44             ` Adrian Chadd
2015-03-19 15:57               ` Fabian Wittenberg
2015-03-19 16:05                 ` Adrian Chadd
2015-03-19 16:18                   ` Fabian Wittenberg
2015-03-19 16:23                     ` Adrian Chadd
2015-03-20 10:46                   ` Fabian Wittenberg
2015-02-23 16:58 ` Ben Greear
2015-03-08 13:45 ` Jeremias Blendin
2015-03-08 18:27   ` Ben Greear

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=54EB3C30.60302@sophos.com \
    --to=fabian.wittenberg@sophos.com \
    --cc=ath10k@lists.infradead.org \
    --cc=michal.kazior@tieto.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.