Re: [PATCH v4 30/47] serial: ucc_uart: factor out soft_uart initialization

From: Timur Tabi
Date: Thu Nov 14 2019 - 00:15:30 EST


On Fri, Nov 8, 2019 at 7:03 AM Rasmus Villemoes
<linux@xxxxxxxxxxxxxxxxxx> wrote:
>
> - /*
> - * Determine if we need Soft-UART mode
> - */
> if (of_find_property(np, "soft-uart", NULL)) {
> dev_dbg(&ofdev->dev, "using Soft-UART mode\n");
> soft_uart = 1;
> + } else {
> + return 0;
> }

How about:

if (!of_find_property(np, "soft-uart", NULL))
return 0;

And I think you should be able to get rid of the "soft_uart" variable.