On Wed, Aug 17, 2011 at 01:24:58PM +0200, Guennadi Liakhovetski wrote: > On Wed, 17 Aug 2011, Simon Horman wrote: > > > Make use of per-source irq handles if the > > platform (data) has multiple irq sources. > > > > Also, as suggested by Guennadi Liakhovetski, > > add and use defines the index or irqs in platform data. > > > > Cc: Guennadi Liakhovetski <g.liakhovetski@xxxxxx> > > Cc: Magnus Damm <magnus.damm@xxxxxxxxx> > > Signed-off-by: Simon Horman <horms@xxxxxxxxxxxx> > > > > --- > > > > v5 > > * As suggested by Guennadi Liakhovetski: > > - Allow only SH_MOBILE_SDHI_IRQ_SDCARD and > > SH_MOBILE_SDHI_IRQ_SDIO to be specified in platform data > > This means, CARD_DETECT is optional in a multi-irq configuration, agree. [snip] > > + ret = irq = platform_get_irq(pdev, SH_MOBILE_SDHI_IRQ_SDCARD); > > + if (irq >= 0) { > > + multi_irq = true; > > + ret = request_irq(irq, tmio_mmc_sdcard_irq, 0, > > + dev_name(&pdev->dev), host); > > + if (ret) > > + goto eirq_sdcard; > > } > > This means SDCARD is optional in both single- and multi-irq > configurations. > > > + > > + ret = irq = platform_get_irq(pdev, SH_MOBILE_SDHI_IRQ_CARD_DETECT); > > + if (irq < 0) > > + goto eirq_card_detect; > > This means, CARD_DETECT is compulsory in all configurations. > > One of us must be speaking klingonish today. Sorry, I messed that up a bit. I will make CARD_DETECT optional in multi-irq configurations. Likewise SDIO will be optional in multi-irq configurations. But SDCARD will always be required. -- 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