On 4 May 2011 17:20, Nicolas Pitre <nicolas.pitre@xxxxxxxxxx> wrote: > On Wed, 4 May 2011, Per Forlin wrote: > >> On 4 May 2011 05:40, Nicolas Pitre <nicolas.pitre@xxxxxxxxxx> wrote: >> > On Tue, 3 May 2011, Per Forlin wrote: >> > >> >> From: Stefan Nilsson XK <stefan.xk.nilsson@xxxxxxxxxxxxxx> >> >> >> >> If there is only 1 function registered, and IRQ:s are supported and >> >> currently enabled, call the callback handler directly >> >> without checking the CCCR registers. >> >> >> >> Signed-off-by: Stefan Nilsson XK <stefan.xk.nilsson@xxxxxxxxxxxxxx> >> >> Signed-off-by: Per Forlin <per.forlin@xxxxxxxxxx> >> > >> > Acked-by: Nicolas Pitre <nicolas.pitre@xxxxxxxxxx> >> > >> I am working o a patch version 2 after offline discussion with Ulf Hansson. >> Instead of adding this code here. >> Add sdio_single_func member in mmc_card. Set and reset this function >> in sdio_claim_irq and sdio_release_irq. >> process_sdio_pending_irqs would only check if sdio_single_func is != >> null and call it. > > Yes, that's what I was about to propose after thinking about it some > more. > >> This will result in a bigger patch overall but the new code in >> process_sdio_pending_irqs will be minimal. > > Something like this (untested) ? > What I had in mind is similar. Please let me know what you think. I am about to post "patch v2" Regards, Per -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
