public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: maxime.ripard@free-electrons.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [linux-sunxi] [PATCH 7/7] ARM: dts: sun7i: Enable cap-sdio-irq for bananapro sdio wifi
Date: Sun, 8 Mar 2015 18:26:38 +0100	[thread overview]
Message-ID: <20150308172638.GH5085@lukather> (raw)
In-Reply-To: <54FC3239.5030200@redhat.com>

On Sun, Mar 08, 2015 at 12:27:53PM +0100, Hans de Goede wrote:
> Hi,
> 
> On 08-03-15 03:42, Chen-Yu Tsai wrote:
> >On Sun, Mar 8, 2015 at 3:01 AM, Hans de Goede <hdegoede@redhat.com> wrote:
> >>The sdio wifi on the Banana Pro does not appear to have an oob irq hooked
> >>up, so enable sdio-irq support to avoid unnecessary polling.
> >
> >I wonder if this is stable enough to put back into the mmc driver, i.e.
> >declare the capability in the driver itself? IIRC at one point it did.
> 
> That is a good question, I believe strongly that the sdio wifi issues we've
> been seeing which were the reason to disable cap-sdio-irq in the driver are
> actually unrelated to the irq delivery mechanism. So IMHO yes we can just enable
> this for all devices in the driver, and yes that would be a better solution.
> 
> Maxime, if I send a patch to enable cal-sdio-irq at the driver level ( as it
> really is a hardware feature not a board feature), will you ack it ?

You know a lot more than I do on this, so I fully trust you. If you
feel like it's the best solution, go ahead.

Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20150308/a70e052d/attachment-0001.sig>

  reply	other threads:[~2015-03-08 17:26 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-07 19:01 [PATCH 1/7] ARM: dts: sun4i: Enable tablet keys on Chuwi V7 CW0825 Hans de Goede
2015-03-07 19:01 ` [PATCH 2/7] ARM: dts: sun5i: Stop using different compatibles for ehci/ohci on a13 vs a10s Hans de Goede
2015-03-08 17:19   ` Maxime Ripard
2015-03-07 19:01 ` [PATCH 3/7] ARM: dts: sun5i: Add mmc2 pinmux settings Hans de Goede
2015-03-08 17:19   ` Maxime Ripard
2015-03-07 19:01 ` [PATCH 4/7] ARM: dts: sun5i: Add new Auxtek-t004 board Hans de Goede
2015-03-08 17:21   ` Maxime Ripard
2015-03-08 19:15     ` Hans de Goede
2015-03-09 14:53       ` Maxime Ripard
2015-03-07 19:01 ` [PATCH 5/7] ARM: dts: sun5i: Add new Utoo P66 board Hans de Goede
2015-03-08 17:22   ` Maxime Ripard
2015-03-07 19:01 ` [PATCH 6/7] ARM: dts: sun7i: Add OOB irq support to boards with broadcom sdio wifi Hans de Goede
2015-03-08  2:40   ` [linux-sunxi] " Chen-Yu Tsai
2015-03-08  3:12   ` Chen-Yu Tsai
2015-03-08 11:25     ` Hans de Goede
2015-03-08 17:25   ` Maxime Ripard
2015-03-08 19:19     ` Hans de Goede
2015-03-07 19:01 ` [PATCH 7/7] ARM: dts: sun7i: Enable cap-sdio-irq for bananapro " Hans de Goede
2015-03-08  2:42   ` [linux-sunxi] " Chen-Yu Tsai
2015-03-08 11:27     ` Hans de Goede
2015-03-08 17:26       ` Maxime Ripard [this message]
2015-03-08 17:18 ` [PATCH 1/7] ARM: dts: sun4i: Enable tablet keys on Chuwi V7 CW0825 Maxime Ripard

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=20150308172638.GH5085@lukather \
    --to=maxime.ripard@free-electrons.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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