2022-12-08 09:32:22

by Matt Jan

[permalink] [raw]
Subject: [PATCH 1/3] staging: vme_user: add the spaces around the "*"

Add a space around the "*" at vme_bridge.h to clear up a checkpatch
check.

Signed-off-by: Matt Jan <[email protected]>
---
drivers/staging/vme_user/vme_bridge.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/vme_user/vme_bridge.h b/drivers/staging/vme_user/vme_bridge.h
index 0bbefe9851d7..e17a8f7de165 100644
--- a/drivers/staging/vme_user/vme_bridge.h
+++ b/drivers/staging/vme_user/vme_bridge.h
@@ -4,7 +4,7 @@

#include "vme.h"

-#define VME_CRCSR_BUF_SIZE (508*1024)
+#define VME_CRCSR_BUF_SIZE (508 * 1024)
/*
* Resource structures
*/
--
2.25.1


2022-12-08 10:42:33

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: [PATCH 1/3] staging: vme_user: add the spaces around the "*"

On Thu, Dec 08, 2022 at 05:11:00PM +0800, Matt Jan wrote:
> Add a space around the "*" at vme_bridge.h to clear up a checkpatch
> check.
>
> Signed-off-by: Matt Jan <[email protected]>
> ---
> drivers/staging/vme_user/vme_bridge.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/vme_user/vme_bridge.h b/drivers/staging/vme_user/vme_bridge.h
> index 0bbefe9851d7..e17a8f7de165 100644
> --- a/drivers/staging/vme_user/vme_bridge.h
> +++ b/drivers/staging/vme_user/vme_bridge.h
> @@ -4,7 +4,7 @@
>
> #include "vme.h"
>
> -#define VME_CRCSR_BUF_SIZE (508*1024)
> +#define VME_CRCSR_BUF_SIZE (508 * 1024)
> /*
> * Resource structures
> */
> --
> 2.25.1
>
>

I see 2 copies of this patch sent to us, one properly threaded with
others, and one (this one), not. Which one is correct? As I have no
way of guessing, I'll drop both sets :(

Please fix up and resend a v2 set only once.

thanks,

greg k-h