Re: [PATCH v4 2/3] staging: axis-fifo: Fix alignment of wait_event_interruptible arguments

From: Jonathan Cameron

Date: Sat May 16 2026 - 07:20:39 EST


On Fri, 15 May 2026 03:57:35 -0400
Pramod Maurya <pramod.nexgen@xxxxxxxxx> wrote:

> The second argument to wait_event_interruptible() was indented with a
> single tab instead of being aligned to the opening parenthesis. Since
> the fully-aligned form exceeds 80 columns, break the condition at the
> comparison operator and align the continuation line to the opening
> parenthesis.
>
> Fixes the following checkpatch.pl warning:
> CHECK: Alignment should match open parenthesis
>
> Signed-off-by: Pramod Maurya <pramod.nexgen@xxxxxxxxx>

It's not a related cleanup as nothing to do with DT - so break it out of the series
and send it separately.

Jonathan

> ---
> drivers/staging/axis-fifo/axis-fifo.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/axis-fifo/axis-fifo.c b/drivers/staging/axis-fifo/axis-fifo.c
> index 3aa2aa870ea9..1c34de020cf8 100644
> --- a/drivers/staging/axis-fifo/axis-fifo.c
> +++ b/drivers/staging/axis-fifo/axis-fifo.c
> @@ -246,7 +246,8 @@ static ssize_t axis_fifo_write(struct file *f, const char __user *buf,
> mutex_lock(&fifo->write_lock);
>
> ret = wait_event_interruptible(fifo->write_queue,
> - ioread32(fifo->base_addr + XLLF_TDFV_OFFSET) >= words_to_write);
> + ioread32(fifo->base_addr + XLLF_TDFV_OFFSET) >=
> + words_to_write);
> if (ret)
> goto end_unlock;
> }