Re: [PATCH] resource: Remove unneeded semicolon

2021-01-19 Thread Rafael J. Wysocki

On 1/18/2021 2:50 AM, Chengyang Fan wrote:

Remove a superfluous semicolon after function definition.

Signed-off-by: Chengyang Fan 
---
  include/linux/ioport.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/ioport.h b/include/linux/ioport.h
index fe48b7840665..803c10f6a6df 100644
--- a/include/linux/ioport.h
+++ b/include/linux/ioport.h
@@ -337,7 +337,7 @@ static inline void irqresource_disabled(struct resource 
*res, u32 irq)
  #ifdef CONFIG_IO_STRICT_DEVMEM
  void revoke_devmem(struct resource *res);
  #else
-static inline void revoke_devmem(struct resource *res) { };
+static inline void revoke_devmem(struct resource *res) { }
  #endif
  
  #endif /* __ASSEMBLY__ */


The patch is fine AFAICS, so

Reviewed-by: Rafael J. Wysocki 

but I'm not sure why it has been sent to me.

It would be better to send it to Greg instead.

Thanks!




[PATCH] resource: Remove unneeded semicolon

2021-01-17 Thread Chengyang Fan
Remove a superfluous semicolon after function definition.

Signed-off-by: Chengyang Fan 
---
 include/linux/ioport.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/ioport.h b/include/linux/ioport.h
index fe48b7840665..803c10f6a6df 100644
--- a/include/linux/ioport.h
+++ b/include/linux/ioport.h
@@ -337,7 +337,7 @@ static inline void irqresource_disabled(struct resource 
*res, u32 irq)
 #ifdef CONFIG_IO_STRICT_DEVMEM
 void revoke_devmem(struct resource *res);
 #else
-static inline void revoke_devmem(struct resource *res) { };
+static inline void revoke_devmem(struct resource *res) { }
 #endif
 
 #endif /* __ASSEMBLY__ */
-- 
2.25.1