staging: axis-fifo: align arguments to open parenthesis in axis-fifo.c

[PATCH] staging: axis-fifo: align arguments to open parenthesis in axis-fifo.c

Posted by Shresth Prasad 1 year, 10 months ago

Align some function arguments to the opening parenthesis of the
function as reported by checkpatch:

CHECK: Alignment should match open parenthesis

Signed-off-by: Shresth Prasad <shresthprasad7@gmail.com>
---
 drivers/staging/axis-fifo/axis-fifo.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/axis-fifo/axis-fifo.c b/drivers/staging/axis-fifo/axis-fifo.c
index c51818c56dd2..a70af76be7fb 100644
--- a/drivers/staging/axis-fifo/axis-fifo.c
+++ b/drivers/staging/axis-fifo/axis-fifo.c
@@ -376,8 +376,8 @@ static ssize_t axis_fifo_read(struct file *f, char __user *buf,
 		 */
 		mutex_lock(&fifo->read_lock);
 		ret = wait_event_interruptible_timeout(fifo->read_queue,
-			ioread32(fifo->base_addr + XLLF_RDFO_OFFSET),
-			read_timeout);
+						       ioread32(fifo->base_addr + XLLF_RDFO_OFFSET),
+						       read_timeout);
 
 		if (ret <= 0) {
 			if (ret == 0) {
@@ -517,9 +517,9 @@ static ssize_t axis_fifo_write(struct file *f, const char __user *buf,
 		 */
 		mutex_lock(&fifo->write_lock);
 		ret = wait_event_interruptible_timeout(fifo->write_queue,
-			ioread32(fifo->base_addr + XLLF_TDFV_OFFSET)
-				 >= words_to_write,
-			write_timeout);
+						       ioread32(fifo->base_addr + XLLF_TDFV_OFFSET)
+						       >= words_to_write,
+						       write_timeout);
 
 		if (ret <= 0) {
 			if (ret == 0) {
-- 
2.44.0

Re: [PATCH] staging: axis-fifo: align arguments to open parenthesis in axis-fifo.c

Posted by Uri Arev 1 year, 10 months ago

> Align some function arguments to the opening parenthesis of the
> function as reported by checkpatch:
Hey, I sent a similar patch around a month ago and got no reply.
Good luck with that :)

                 Uri Arev

Re: [PATCH] staging: axis-fifo: align arguments to open parenthesis in axis-fifo.c

Posted by Shresth Prasad 1 year, 10 months ago

Oh, that's too bad. Thanks for the heads up!

Regards,
Shresth Prasad

Re: [PATCH] staging: axis-fifo: align arguments to open parenthesis in axis-fifo.c

Posted by Uri Arev 1 year, 10 months ago

> Oh, that's too bad. Thanks for the heads up!
No worries, I wouldn't mind it if your patch got accepted and not mine
:p
I just don't think they accept patches to axis-fifo anymore, especially
not basic syntax and checkpatch patches which are not as important as
actual bug-fixes.

                 Uri Arev

Re: [PATCH] staging: axis-fifo: align arguments to open parenthesis in axis-fifo.c

Posted by Dan Carpenter 1 year, 10 months ago

On Sun, Mar 31, 2024 at 09:33:55PM +0300, Uri Arev wrote:
> > Oh, that's too bad. Thanks for the heads up!
> No worries, I wouldn't mind it if your patch got accepted and not mine
> :p
> I just don't think they accept patches to axis-fifo anymore, especially
> not basic syntax and checkpatch patches which are not as important as
> actual bug-fixes.

No, we're always taking patches.  You just had bad luck/timing where you
sent the patch 2 days before Greg went on vacation.

Greg is back now, but I imagine he has a long list of stuff to catch up
with.  It's possible that he also accidentally dropped your patch
because of the vacation...  I could imagine a situation where that
happened.

regards,
dan carpenter

Re: [PATCH] staging: axis-fifo: align arguments to open parenthesis in axis-fifo.c

Posted by eretmochelys imbricata 1 year, 10 months ago

>No, we're always taking patches.  You just had bad luck/timing where you
>sent the patch 2 days before Greg went on vacation.

>Greg is back now, but I imagine he has a long list of stuff to catch up
>with.  It's possible that he also accidentally dropped your patch
>because of the vacation...  I could imagine a situation where that
>happened.

That's quite unfortunate. It's good to know that you guys are still
taking patches tho!

Regards,
Shresth Prasad