Re: [PATCH v2] serial/8250_dw: use platform_get_irq() instead of platform_get_resource()

[Alexey Brodkin <Alexey.Brodkin@synopsys.com>]

Hi Andy,

On Tue, 2015-03-03 at 17:26 +0200, Andy Shevchenko wrote:
> On Tue, 2015-03-03 at 18:11 +0300, Alexey Brodkin wrote:
> > It is not recommened to use platform_get_resource(pdev, IORESOURCE_IRQ)
> > for requesting IRQ's resources any more, as they can be not ready yet in
> > case of DT-booting.
> > 
> > platform_get_irq() instead is a recommended way for getting IRQ even if
> > it was not retrieved earlier.
> > 
> > It also makes code simpler because we're getting "int" value right away
> > and no conversion from resource to int is required.
> > 
> >  Changes in v2:
> >   * Suppress error message if platform_get_irq() returns -EPROBE_DEFER
> 
> Do we really need that message at all?

IMHO it makes sense.

For example it was useful for me when debugging stacked interrupt
controllers setup - I got explicitly notified why this particular device
failed on probe.

Note that IRQ is a pretty specific resource due to the fact of INTC
stacking and situations when each and every INTC gets finally probed.

Still if you believe we may drop this message with no loss of usability
- I'm fine with that as well.

-Alexey