<div dir="ltr"><div>Hi Gao,</div><div><br></div><div>I will rework the patch.<br></div><div>Will take care of the patch subject & '80 characters line limit'</div><div><br></div><div>Thanks,</div><div>Pratik.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jul 15, 2019 at 7:54 AM Gao Xiang <<a href="mailto:gaoxiang25@huawei.com">gaoxiang25@huawei.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Pratik,<br>
<br>
On 2019/7/15 3:30, Pratik Shinde wrote:<br>
> Fixing checkpath warnings : converting all 'unsigned' to 'unsigned int'<br>
> <br>
> Signed-off-by: Pratik Shinde <<a href="mailto:pratikshinde320@gmail.com" target="_blank">pratikshinde320@gmail.com</a>><br>
> ---<br>
>  drivers/staging/erofs/internal.h      |  6 +++---<br>
>  drivers/staging/erofs/unzip_pagevec.h | 10 +++++-----<br>
>  drivers/staging/erofs/unzip_vle.h     |  8 ++++----<br>
>  drivers/staging/erofs/xattr.h         | 10 +++++-----<br>
>  4 files changed, 17 insertions(+), 17 deletions(-)<br>
> <br>
> diff --git a/drivers/staging/erofs/internal.h b/drivers/staging/erofs/internal.h<br>
> index 963cc1b..daae90b 100644<br>
> --- a/drivers/staging/erofs/internal.h<br>
> +++ b/drivers/staging/erofs/internal.h<br>
> @@ -359,8 +359,8 @@ struct erofs_vnode {<br>
>       unsigned char inode_isize;<br>
>       unsigned short xattr_isize;<br>
>  <br>
> -     unsigned xattr_shared_count;<br>
> -     unsigned *xattr_shared_xattrs;<br>
> +     unsigned int xattr_shared_count;<br>
> +     unsigned int *xattr_shared_xattrs;<br>
>  <br>
>       union {<br>
>               erofs_blk_t raw_blkaddr;<br>
> @@ -510,7 +510,7 @@ erofs_grab_bio(struct super_block *sb,<br>
>       return bio;<br>
>  }<br>
>  <br>
> -static inline void __submit_bio(struct bio *bio, unsigned op, unsigned op_flags)<br>
> +static inline void __submit_bio(struct bio *bio, unsigned int op, unsigned int op_flags)<br>
<br>
The subject line could be better as "staging: erofs: converting all 'unsigned' to 'unsigned int' "<br>
and three new checkpatch warnings occurs after this patch...<br>
<br>
<br>
WARNING: line over 80 characters<br>
#86: FILE: drivers/staging/erofs/internal.h:513:<br>
+static inline void __submit_bio(struct bio *bio, unsigned int op, unsigned int op_flags)<br>
<br>
WARNING: line over 80 characters<br>
#122: FILE: drivers/staging/erofs/unzip_pagevec.h:95:<br>
+                                            erofs_vtptr_t *pages, unsigned int i)<br>
<br>
WARNING: line over 80 characters<br>
#203: FILE: drivers/staging/erofs/xattr.h:52:<br>
+static inline const struct xattr_handler *erofs_xattr_handler(unsigned int index)<br>
<br>
Thanks,<br>
Gao Xiang<br>
<br>
>  {<br>
>       bio_set_op_attrs(bio, op, op_flags);<br>
>       submit_bio(bio);<br>
> diff --git a/drivers/staging/erofs/unzip_pagevec.h b/drivers/staging/erofs/unzip_pagevec.h<br>
> index 7af0ba8..198b556 100644<br>
> --- a/drivers/staging/erofs/unzip_pagevec.h<br>
> +++ b/drivers/staging/erofs/unzip_pagevec.h<br>
> @@ -54,9 +54,9 @@ static inline void z_erofs_pagevec_ctor_exit(struct z_erofs_pagevec_ctor *ctor,<br>
>  <br>
>  static inline struct page *<br>
>  z_erofs_pagevec_ctor_next_page(struct z_erofs_pagevec_ctor *ctor,<br>
> -                            unsigned nr)<br>
> +                            unsigned int nr)<br>
>  {<br>
> -     unsigned index;<br>
> +     unsigned int index;<br>
>  <br>
>       /* keep away from occupied pages */<br>
>       if (ctor->next)<br>
> @@ -64,7 +64,7 @@ z_erofs_pagevec_ctor_next_page(struct z_erofs_pagevec_ctor *ctor,<br>
>  <br>
>       for (index = 0; index < nr; ++index) {<br>
>               const erofs_vtptr_t t = ctor->pages[index];<br>
> -             const unsigned tags = tagptr_unfold_tags(t);<br>
> +             const unsigned int tags = tagptr_unfold_tags(t);<br>
>  <br>
>               if (tags == Z_EROFS_PAGE_TYPE_EXCLUSIVE)<br>
>                       return tagptr_unfold_ptr(t);<br>
> @@ -91,8 +91,8 @@ z_erofs_pagevec_ctor_pagedown(struct z_erofs_pagevec_ctor *ctor,<br>
>  }<br>
>  <br>
>  static inline void z_erofs_pagevec_ctor_init(struct z_erofs_pagevec_ctor *ctor,<br>
> -                                          unsigned nr,<br>
> -                                          erofs_vtptr_t *pages, unsigned i)<br>
> +                                          unsigned int nr,<br>
> +                                          erofs_vtptr_t *pages, unsigned int i)<br>
>  {<br>
>       ctor->nr = nr;<br>
>       ctor->curr = ctor->next = NULL;<br>
> diff --git a/drivers/staging/erofs/unzip_vle.h b/drivers/staging/erofs/unzip_vle.h<br>
> index ab509d75..df91ad1 100644<br>
> --- a/drivers/staging/erofs/unzip_vle.h<br>
> +++ b/drivers/staging/erofs/unzip_vle.h<br>
> @@ -34,7 +34,7 @@ struct z_erofs_vle_work {<br>
>       unsigned short nr_pages;<br>
>  <br>
>       /* L: queued pages in pagevec[] */<br>
> -     unsigned vcnt;<br>
> +     unsigned int vcnt;<br>
>  <br>
>       union {<br>
>               /* L: pagevec */<br>
> @@ -124,7 +124,7 @@ union z_erofs_onlinepage_converter {<br>
>       unsigned long *v;<br>
>  };<br>
>  <br>
> -static inline unsigned z_erofs_onlinepage_index(struct page *page)<br>
> +static inline unsigned int z_erofs_onlinepage_index(struct page *page)<br>
>  {<br>
>       union z_erofs_onlinepage_converter u;<br>
>  <br>
> @@ -164,7 +164,7 @@ static inline void z_erofs_onlinepage_fixup(struct page *page,<br>
>       }<br>
>  <br>
>       v = (index << Z_EROFS_ONLINEPAGE_INDEX_SHIFT) |<br>
> -             ((o & Z_EROFS_ONLINEPAGE_COUNT_MASK) + (unsigned)down);<br>
> +             ((o & Z_EROFS_ONLINEPAGE_COUNT_MASK) + (unsigned int)down);<br>
>       if (cmpxchg(p, o, v) != o)<br>
>               goto repeat;<br>
>  }<br>
> @@ -172,7 +172,7 @@ static inline void z_erofs_onlinepage_fixup(struct page *page,<br>
>  static inline void z_erofs_onlinepage_endio(struct page *page)<br>
>  {<br>
>       union z_erofs_onlinepage_converter u;<br>
> -     unsigned v;<br>
> +     unsigned int v;<br>
>  <br>
>       DBG_BUGON(!PagePrivate(page));<br>
>       u.v = &page_private(page);<br>
> diff --git a/drivers/staging/erofs/xattr.h b/drivers/staging/erofs/xattr.h<br>
> index 35ba5ac..2fc9b43 100644<br>
> --- a/drivers/staging/erofs/xattr.h<br>
> +++ b/drivers/staging/erofs/xattr.h<br>
> @@ -20,14 +20,14 @@<br>
>  /* Attribute not found */<br>
>  #define ENOATTR         ENODATA<br>
>  <br>
> -static inline unsigned inlinexattr_header_size(struct inode *inode)<br>
> +static inline unsigned int inlinexattr_header_size(struct inode *inode)<br>
>  {<br>
>       return sizeof(struct erofs_xattr_ibody_header)<br>
>               + sizeof(u32) * EROFS_V(inode)->xattr_shared_count;<br>
>  }<br>
>  <br>
>  static inline erofs_blk_t<br>
> -xattrblock_addr(struct erofs_sb_info *sbi, unsigned xattr_id)<br>
> +xattrblock_addr(struct erofs_sb_info *sbi, unsigned int xattr_id)<br>
>  {<br>
>  #ifdef CONFIG_EROFS_FS_XATTR<br>
>       return sbi->xattr_blkaddr +<br>
> @@ -37,8 +37,8 @@ xattrblock_addr(struct erofs_sb_info *sbi, unsigned xattr_id)<br>
>  #endif<br>
>  }<br>
>  <br>
> -static inline unsigned<br>
> -xattrblock_offset(struct erofs_sb_info *sbi, unsigned xattr_id)<br>
> +static inline unsigned int<br>
> +xattrblock_offset(struct erofs_sb_info *sbi, unsigned int xattr_id)<br>
>  {<br>
>       return (xattr_id * sizeof(__u32)) % EROFS_BLKSIZ;<br>
>  }<br>
> @@ -49,7 +49,7 @@ extern const struct xattr_handler erofs_xattr_trusted_handler;<br>
>  extern const struct xattr_handler erofs_xattr_security_handler;<br>
>  #endif<br>
>  <br>
> -static inline const struct xattr_handler *erofs_xattr_handler(unsigned index)<br>
> +static inline const struct xattr_handler *erofs_xattr_handler(unsigned int index)<br>
>  {<br>
>  static const struct xattr_handler *xattr_handler_map[] = {<br>
>       [EROFS_XATTR_INDEX_USER] = &erofs_xattr_user_handler,<br>
> <br>
</blockquote></div>