* [PATCH] staging: octeon: correct parameter alignment
@ 2026-09-12 14:51 Matthew Lavine
2026-10-01 5:33 ` Greg Kroah-Hartman
0 siblings, 1 reply; 2+ messages in thread
From: Matthew Lavine @ 2026-09-12 14:51 UTC (permalink / raw)
To: Greg Kroah-Hartman; +Cc: linux-staging, Matthew Lavine
Correct parameter alignment in cvmx_pko_send_packet_finish() while
abiding by the column limit
This addresses the "Alignment should match open parenthesis" check
reported by checkpatch.pl
Signed-off-by: Matthew Lavine <matt@matthewlavine.net>
---
drivers/staging/octeon/octeon-stubs.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/octeon/octeon-stubs.h b/drivers/staging/octeon/octeon-stubs.h
index 9c1968b7e2d1..935ce6495f3b 100644
--- a/drivers/staging/octeon/octeon-stubs.h
+++ b/drivers/staging/octeon/octeon-stubs.h
@@ -1386,9 +1386,9 @@ static inline void cvmx_pko_send_packet_prepare(u64 port, u64 queue,
enum cvmx_pko_lock use_locking)
{ }
-static inline enum cvmx_pko_status cvmx_pko_send_packet_finish(u64 port,
- u64 queue, union cvmx_pko_command_word0 pko_command,
- union cvmx_buf_ptr packet, enum cvmx_pko_lock use_locking)
+static inline enum cvmx_pko_status
+cvmx_pko_send_packet_finish(u64 port, u64 queue, union cvmx_pko_command_word0 pko_command,
+ union cvmx_buf_ptr packet, enum cvmx_pko_lock use_locking)
{
return 0;
}
--
2.55.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] staging: octeon: correct parameter alignment
2026-09-12 14:51 [PATCH] staging: octeon: correct parameter alignment Matthew Lavine
@ 2026-10-01 5:33 ` Greg Kroah-Hartman
0 siblings, 0 replies; 2+ messages in thread
From: Greg Kroah-Hartman @ 2026-10-01 5:33 UTC (permalink / raw)
To: Matthew Lavine; +Cc: linux-staging
On Sat, Sep 12, 2026 at 10:51:26AM -0400, Matthew Lavine wrote:
> Correct parameter alignment in cvmx_pko_send_packet_finish() while
> abiding by the column limit
>
> This addresses the "Alignment should match open parenthesis" check
> reported by checkpatch.pl
>
> Signed-off-by: Matthew Lavine <matt@matthewlavine.net>
> ---
> drivers/staging/octeon/octeon-stubs.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/staging/octeon/octeon-stubs.h b/drivers/staging/octeon/octeon-stubs.h
> index 9c1968b7e2d1..935ce6495f3b 100644
> --- a/drivers/staging/octeon/octeon-stubs.h
> +++ b/drivers/staging/octeon/octeon-stubs.h
> @@ -1386,9 +1386,9 @@ static inline void cvmx_pko_send_packet_prepare(u64 port, u64 queue,
> enum cvmx_pko_lock use_locking)
> { }
>
> -static inline enum cvmx_pko_status cvmx_pko_send_packet_finish(u64 port,
> - u64 queue, union cvmx_pko_command_word0 pko_command,
> - union cvmx_buf_ptr packet, enum cvmx_pko_lock use_locking)
> +static inline enum cvmx_pko_status
> +cvmx_pko_send_packet_finish(u64 port, u64 queue, union cvmx_pko_command_word0 pko_command,
> + union cvmx_buf_ptr packet, enum cvmx_pko_lock use_locking)
> {
> return 0;
> }
> --
> 2.55.0
>
>
No, this warning can be igured as the function name and structures just
are "too big".
thanks,
greg k-h
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2026-10-01 5:33 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-12 14:51 [PATCH] staging: octeon: correct parameter alignment Matthew Lavine
2026-10-01 5:33 ` Greg Kroah-Hartman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox