Skip to contentSkip to conversation

staging: vme_user: add the spaces around the "*"

PATCH New Matt Jan 1 participant 3 messages Dec 9, 2022

Subsystems (2)
Staging VME
lore
Share & tools
lore

Matt Jan

author
#0 lore raw

PATCH v3 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]>

---

Changelog
v2->v3: Resend the patch set in properly threaded and move the patch
changelog to correct position, noted by Greg Kroah-Hartman <[email protected]>
v1->v2: Resend the correct patch set, noted by Greg Kroah-Hartman
<[email protected]>

 drivers/staging/vme_user/vme_bridge.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/vme_user/vme_bridge.h +1 -1
index 0bbefe9851d7..e17a8f7de165 100644
@@ -4,7 +4,7 @@
 
 #include "vme.h"
 
-#define VME_CRCSR_BUF_SIZE (508*1024)
+#define VME_CRCSR_BUF_SIZE (508 * 1024)
 /*
  * Resource structures
  */
Signature · 1 line
2.25.1
author
#1 lore raw

PATCH v3 2/3 staging: vme_user: remove unnecessary spaces

Remove unnecessary spaces before function pointer arguments in vme_bridge.h. This is done with command ':%s/) (/)('.

Signed-off-by Matt Jan <[email protected]>

---

Changelog
v2->v3: Resend the patch set in properly threaded and move the patch
changelog to correct position, noted by Greg Kroah-Hartman <[email protected]>
v1->v2: Resend the correct patch set, noted by Greg Kroah-Hartman
<[email protected]>

 drivers/staging/vme_user/vme_bridge.h | 32 +++++++++++++--------------
 1 file changed, 16 insertions(+), 16 deletions(-)
drivers/staging/vme_user/vme_bridge.h +16 -16
index e17a8f7de165..5da2643e20d5 100644
@@ -128,45 +128,45 @@ struct vme_bridge {
 	struct mutex irq_mtx;
 
 	/* Slave Functions */
-	int (*slave_get) (struct vme_slave_resource *, int *,
+	int (*slave_get)(struct vme_slave_resource *, int *,
 		unsigned long long *, unsigned long long *, dma_addr_t *,
 		u32 *, u32 *);
-	int (*slave_set) (struct vme_slave_resource *, int, unsigned long long,
+	int (*slave_set)(struct vme_slave_resource *, int, unsigned long long,
 		unsigned long long, dma_addr_t, u32, u32);
 
 	/* Master Functions */
-	int (*master_get) (struct vme_master_resource *, int *,
+	int (*master_get)(struct vme_master_resource *, int *,
 		unsigned long long *, unsigned long long *, u32 *, u32 *,
 		u32 *);
-	int (*master_set) (struct vme_master_resource *, int,
+	int (*master_set)(struct vme_master_resource *, int,
 		unsigned long long, unsigned long long,  u32, u32, u32);
-	ssize_t (*master_read) (struct vme_master_resource *, void *, size_t,
+	ssize_t (*master_read)(struct vme_master_resource *, void *, size_t,
 		loff_t);
-	ssize_t (*master_write) (struct vme_master_resource *, void *, size_t,
+	ssize_t (*master_write)(struct vme_master_resource *, void *, size_t,
 		loff_t);
-	unsigned int (*master_rmw) (struct vme_master_resource *, unsigned int,
+	unsigned int (*master_rmw)(struct vme_master_resource *, unsigned int,
 		unsigned int, unsigned int, loff_t);
 
 	/* DMA Functions */
-	int (*dma_list_add) (struct vme_dma_list *, struct vme_dma_attr *,
+	int (*dma_list_add)(struct vme_dma_list *, struct vme_dma_attr *,
 		struct vme_dma_attr *, size_t);
-	int (*dma_list_exec) (struct vme_dma_list *);
-	int (*dma_list_empty) (struct vme_dma_list *);
+	int (*dma_list_exec)(struct vme_dma_list *);
+	int (*dma_list_empty)(struct vme_dma_list *);
 
 	/* Interrupt Functions */
-	void (*irq_set) (struct vme_bridge *, int, int, int);
-	int (*irq_generate) (struct vme_bridge *, int, int);
+	void (*irq_set)(struct vme_bridge *, int, int, int);
+	int (*irq_generate)(struct vme_bridge *, int, int);
 
 	/* Location monitor functions */
-	int (*lm_set) (struct vme_lm_resource *, unsigned long long, u32, u32);
-	int (*lm_get) (struct vme_lm_resource *, unsigned long long *, u32 *,
+	int (*lm_set)(struct vme_lm_resource *, unsigned long long, u32, u32);
+	int (*lm_get)(struct vme_lm_resource *, unsigned long long *, u32 *,
 		u32 *);
 	int (*lm_attach)(struct vme_lm_resource *, int,
 			 void (*callback)(void *), void *);
-	int (*lm_detach) (struct vme_lm_resource *, int);
+	int (*lm_detach)(struct vme_lm_resource *, int);
 
 	/* CR/CSR space functions */
-	int (*slot_get) (struct vme_bridge *);
+	int (*slot_get)(struct vme_bridge *);
 
 	/* Bridge parent interface */
 	void *(*alloc_consistent)(struct device *dev, size_t size,
Signature · 1 line
2.25.1
author
#2 lore raw

PATCH v3 3/3 staging: vme_user: replace 'unsigned' with 'unsigned int'

In reply to Matt Jan

Update the bare use of "unsigned" to the preferred "unsigned int".

Signed-off-by Matt Jan <[email protected]>

---

Changelog
v2->v3: Resend the patch set in properly threaded and move the patch
changelog to correct position, noted by Greg Kroah-Hartman <[email protected]>
v1->v2: Resend the correct patch set, noted by Greg Kroah-Hartman

		<[email protected]>
		Align the comment correctly, noted by Dan Carpenter
		<[email protected]>
 drivers/staging/vme_user/vme_bridge.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/staging/vme_user/vme_bridge.h +1 -1
index 5da2643e20d5..11df0a5e7f7b 100644
@@ -84,7 +84,7 @@ struct vme_error_handler {
 	unsigned long long end;		/* End of error window */
 	unsigned long long first_error;	/* Address of the first error */
 	u32 aspace;			/* Address space of error window*/
-	unsigned num_errors;		/* Number of errors */
+	unsigned int num_errors;	/* Number of errors */
 };
 
 struct vme_callback {
Signature · 1 line
2.25.1