2021-10-29 13:30:42

by Ajith P V

[permalink] [raw]
Subject: [PATCH] staging: fieldbus: anybus: reframe comment to avoid warning

host.c file comment produce warning with checkpatch as below:
WARNING: Possible repeated word: 'interrupt'
Reframe the comment into bullet points or steps avoid this warning.

Signed-off-by: Ajith P V <[email protected]>
---
drivers/staging/fieldbus/anybuss/host.c | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/fieldbus/anybuss/host.c b/drivers/staging/fieldbus/anybuss/host.c
index 8a75f6642c78..a344410e48fe 100644
--- a/drivers/staging/fieldbus/anybuss/host.c
+++ b/drivers/staging/fieldbus/anybuss/host.c
@@ -1318,11 +1318,11 @@ anybuss_host_common_probe(struct device *dev,
}
/*
* startup sequence:
- * perform dummy IND_AB read to prevent false 'init done' irq
+ * a) perform dummy IND_AB read to prevent false 'init done' irq
* (already done by test_dpram() above)
- * release reset
- * wait for first interrupt
- * interrupt came in: ready to go !
+ * b) release reset
+ * c) wait for first interrupt
+ * d) interrupt came in: ready to go !
*/
reset_deassert(cd);
if (!wait_for_completion_timeout(&cd->card_boot, TIMEOUT)) {
--
2.17.1


2021-10-31 15:25:34

by Sven Van Asbroeck

[permalink] [raw]
Subject: Re: [PATCH] staging: fieldbus: anybus: reframe comment to avoid warning

On Fri, Oct 29, 2021 at 9:28 AM Ajith P V <[email protected]> wrote:
>
> host.c file comment produce warning with checkpatch as below:
> WARNING: Possible repeated word: 'interrupt'
> Reframe the comment into bullet points or steps avoid this warning.
>

Looks good to me.

Acked-by: Sven Van Asbroeck <[email protected]>