Re: [PATCH] MIPS: BCM47XX: drop unneeded semicolon

On Sat, Aug 01, 2026 at 09:10:02PM +0200, Julia Lawall wrote:
> When a function-like macro expands to an expression, that expression
> doesn't need a semicolon after it.  All uses have been verified to
> have their own semicolons.
> 
> This was found using the following Coccinelle semantic patch:
> 
> @r@
> identifier i : script:ocaml() { String.lowercase_ascii i = i };
> expression e;
> @@
> 
> *#define i(...) e;
> 
> Signed-off-by: Julia Lawall <[email protected]>
> 
> ---
>  arch/mips/bcm47xx/buttons.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/mips/bcm47xx/buttons.c b/arch/mips/bcm47xx/buttons.c
> index 46994f9bb..e76ca34c8 100644
> --- a/arch/mips/bcm47xx/buttons.c
> +++ b/arch/mips/bcm47xx/buttons.c
> @@ -512,7 +512,7 @@ static int __init bcm47xx_buttons_copy(const struct gpio_keys_button *buttons,
>  }
>  
>  #define bcm47xx_copy_bdata(dev_buttons)					\
> -	bcm47xx_buttons_copy(dev_buttons, ARRAY_SIZE(dev_buttons));
> +	bcm47xx_buttons_copy(dev_buttons, ARRAY_SIZE(dev_buttons))

not applicable to mips-next, because commit 27c61f8565a4 ("MIPS: BCM47XX:
 Convert buttons to software nodes") removed it.

Thomas.

-- 
Crap can work. Given enough thrust pigs will fly, but it's not necessarily a
good idea.                                                [ RFC1925, 2.3 ]