On Wed, 17 Aug 2011, Simon Horman wrote: > 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. No, wrong again. This would break existing single-IRQ configurations, which only use IRQ resource #0, which is CARD_DETECT. Guennadi --- Guennadi Liakhovetski, Ph.D. Freelance Open-Source Software Developer http://www.open-technology.de/ -- 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