staging: sm750fb: Fix alignment in sm750_hw_cursor_set_size call

[PATCH] staging: sm750fb: Fix alignment in sm750_hw_cursor_set_size call

Posted by Karthikeya 2 months ago

Align the arguments of the sm750_hw_cursor_set_size function call
with the opening parenthesis.

This fixes a checkpatch.pl CHECK warning:
'Alignment should match open parenthesis'

Signed-off-by: Karthikeya <garagakarthikeya2007@gmail.com>
---
 drivers/staging/sm750fb/sm750.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm750.c
index 3659af7e5..9740f2705 100644
--- a/drivers/staging/sm750fb/sm750.c
+++ b/drivers/staging/sm750fb/sm750.c
@@ -121,8 +121,8 @@ static int lynxfb_ops_cursor(struct fb_info *info, struct fb_cursor *fbcursor)
 	sm750_hw_cursor_disable(cursor);
 	if (fbcursor->set & FB_CUR_SETSIZE)
 		sm750_hw_cursor_set_size(cursor,
-					fbcursor->image.width,
-					fbcursor->image.height);
+					 fbcursor->image.width,
+					 fbcursor->image.height);
 
 	if (fbcursor->set & FB_CUR_SETPOS)
 		sm750_hw_cursor_set_pos(cursor,
-- 
2.52.0

Re: [PATCH] staging: sm750fb: Fix alignment in sm750_hw_cursor_set_size call

Posted by Greg KH 1 month, 3 weeks ago

On Thu, Dec 04, 2025 at 08:31:26PM +0530, Karthikeya wrote:
> Align the arguments of the sm750_hw_cursor_set_size function call
> with the opening parenthesis.
> 
> This fixes a checkpatch.pl CHECK warning:
> 'Alignment should match open parenthesis'
> 
> Signed-off-by: Karthikeya <garagakarthikeya2007@gmail.com>
> ---
>  drivers/staging/sm750fb/sm750.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/staging/sm750fb/sm750.c b/drivers/staging/sm750fb/sm750.c
> index 3659af7e5..9740f2705 100644
> --- a/drivers/staging/sm750fb/sm750.c
> +++ b/drivers/staging/sm750fb/sm750.c
> @@ -121,8 +121,8 @@ static int lynxfb_ops_cursor(struct fb_info *info, struct fb_cursor *fbcursor)
>  	sm750_hw_cursor_disable(cursor);
>  	if (fbcursor->set & FB_CUR_SETSIZE)
>  		sm750_hw_cursor_set_size(cursor,
> -					fbcursor->image.width,
> -					fbcursor->image.height);
> +					 fbcursor->image.width,
> +					 fbcursor->image.height);
>  
>  	if (fbcursor->set & FB_CUR_SETPOS)
>  		sm750_hw_cursor_set_pos(cursor,
> -- 
> 2.52.0
> 

Does not apply at all to the latest kernel tree :(