Re: [Xen-devel] [PATCH] xen: char: Remove unnecessary (uart->irq > 0) check

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 28.04.18 at 11:08, <amittomer25@xxxxxxxxx> wrote:
> While working on MVEBU uart driver, Julien pointed out that (uart->irq > 0)
> check is unnecessary during irq set up.if ever there is an invalid irq, driver
> initialization itself would be bailed out from platform_get_irq.
> 
> This patch would remove similar check for other uart drivers present in XEN.

At the example of the changes to ns16550.c you do, this is not correct. I
can't judge about the various ARM specific drivers, but the 16550 can well
be run in polling mode, and hence failure to set up an interrupt is not fatal
to overall driver initialization.

> Signed-off-by: Amit Singh Tomar <amittomer25@xxxxxxxxx>
> ---
>     * This patch is only compiled tested.

In which case this should be marked RFC imo.

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel