[PATCH net-next v2] tun: Fix code style issues in <linux/if_tun.h>

Yunjian Wang posted 1 patch 2 years ago
include/linux/if_tun.h | 16 +++++++++++++---
1 file changed, 13 insertions(+), 3 deletions(-)
[PATCH net-next v2] tun: Fix code style issues in <linux/if_tun.h>
Posted by Yunjian Wang 2 years ago
This fixes the following code style problem:
- WARNING: please, no spaces at the start of a line
- CHECK: Please use a blank line after
         function/struct/union/enum declarations

Signed-off-by: Yunjian Wang <wangyunjian@huawei.com>
---
 include/linux/if_tun.h | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/include/linux/if_tun.h b/include/linux/if_tun.h
index 2a7660843444..043d442994b0 100644
--- a/include/linux/if_tun.h
+++ b/include/linux/if_tun.h
@@ -27,44 +27,54 @@ struct tun_xdp_hdr {
 #if defined(CONFIG_TUN) || defined(CONFIG_TUN_MODULE)
 struct socket *tun_get_socket(struct file *);
 struct ptr_ring *tun_get_tx_ring(struct file *file);
+
 static inline bool tun_is_xdp_frame(void *ptr)
 {
-       return (unsigned long)ptr & TUN_XDP_FLAG;
+	return (unsigned long)ptr & TUN_XDP_FLAG;
 }
+
 static inline void *tun_xdp_to_ptr(struct xdp_frame *xdp)
 {
-       return (void *)((unsigned long)xdp | TUN_XDP_FLAG);
+	return (void *)((unsigned long)xdp | TUN_XDP_FLAG);
 }
+
 static inline struct xdp_frame *tun_ptr_to_xdp(void *ptr)
 {
-       return (void *)((unsigned long)ptr & ~TUN_XDP_FLAG);
+	return (void *)((unsigned long)ptr & ~TUN_XDP_FLAG);
 }
+
 void tun_ptr_free(void *ptr);
 #else
 #include <linux/err.h>
 #include <linux/errno.h>
 struct file;
 struct socket;
+
 static inline struct socket *tun_get_socket(struct file *f)
 {
 	return ERR_PTR(-EINVAL);
 }
+
 static inline struct ptr_ring *tun_get_tx_ring(struct file *f)
 {
 	return ERR_PTR(-EINVAL);
 }
+
 static inline bool tun_is_xdp_frame(void *ptr)
 {
 	return false;
 }
+
 static inline void *tun_xdp_to_ptr(struct xdp_frame *xdp)
 {
 	return NULL;
 }
+
 static inline struct xdp_frame *tun_ptr_to_xdp(void *ptr)
 {
 	return NULL;
 }
+
 static inline void tun_ptr_free(void *ptr)
 {
 }
-- 
2.33.0
Re: [PATCH net-next v2] tun: Fix code style issues in <linux/if_tun.h>
Posted by Jason Wang 2 years ago
On Fri, Feb 2, 2024 at 3:26 PM Yunjian Wang <wangyunjian@huawei.com> wrote:
>
> This fixes the following code style problem:
> - WARNING: please, no spaces at the start of a line
> - CHECK: Please use a blank line after
>          function/struct/union/enum declarations
>
> Signed-off-by: Yunjian Wang <wangyunjian@huawei.com>
> ---

Acked-by: Jason Wang <jasowang@redhat.com>

Thanks
Re: [PATCH net-next v2] tun: Fix code style issues in <linux/if_tun.h>
Posted by Jiri Pirko 2 years ago
Fri, Feb 02, 2024 at 08:25:55AM CET, wangyunjian@huawei.com wrote:
>This fixes the following code style problem:
>- WARNING: please, no spaces at the start of a line
>- CHECK: Please use a blank line after
>         function/struct/union/enum declarations
>
>Signed-off-by: Yunjian Wang <wangyunjian@huawei.com>

Reviewed-by: Jiri Pirko <jiri@nvidia.com>
Re: [PATCH net-next v2] tun: Fix code style issues in <linux/if_tun.h>
Posted by Willem de Bruijn 2 years ago
Jiri Pirko wrote:
> Fri, Feb 02, 2024 at 08:25:55AM CET, wangyunjian@huawei.com wrote:
> >This fixes the following code style problem:
> >- WARNING: please, no spaces at the start of a line
> >- CHECK: Please use a blank line after
> >         function/struct/union/enum declarations
> >
> >Signed-off-by: Yunjian Wang <wangyunjian@huawei.com>
> 
> Reviewed-by: Jiri Pirko <jiri@nvidia.com>

Reviewed-by: Willem de Bruijn <willemb@google.com>