Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1689059 > unrolled thread

[PATCH resend] checkpatch: Replace __init_refok by __refconst and __refdata

Started byGeert Uytterhoeven <geert+renesas@glider.be>
First post2017-07-17 15:50 +0200
Last post2017-07-17 17:10 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH resend] checkpatch: Replace __init_refok by __refconst and __refdata Geert Uytterhoeven <geert+renesas@glider.be> - 2017-07-17 15:50 +0200
    Re: [PATCH resend] checkpatch: Replace __init_refok by __refconst  and __refdata Joe Perches <joe@perches.com> - 2017-07-17 17:10 +0200

#1689059 — [PATCH resend] checkpatch: Replace __init_refok by __refconst and __refdata

FromGeert Uytterhoeven <geert+renesas@glider.be>
Date2017-07-17 15:50 +0200
Subject[PATCH resend] checkpatch: Replace __init_refok by __refconst and __refdata
Message-ID<u4ezo-5Kj-31@gated-at.bofh.it>
As of commit bd721ea73e1f9655 ("treewide: replace obsolete _refok by
__ref"), __init_refok no longer exists, so it can be removed.
While at it, add the modern variants that were still missing.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
 scripts/checkpatch.pl | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 2287a0bca863bf57..b88740bfe9d14e41 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -343,9 +343,10 @@ our $Sparse	= qr{
 			__force|
 			__iomem|
 			__must_check|
-			__init_refok|
 			__kprobes|
 			__ref|
+			__refconst|
+			__refdata|
 			__rcu|
 			__private
 		}x;
-- 
2.7.4

[toc] | [next] | [standalone]


#1689164 — Re: [PATCH resend] checkpatch: Replace __init_refok by __refconst and __refdata

FromJoe Perches <joe@perches.com>
Date2017-07-17 17:10 +0200
SubjectRe: [PATCH resend] checkpatch: Replace __init_refok by __refconst and __refdata
Message-ID<u4fOO-6GU-31@gated-at.bofh.it>
In reply to#1689059
On Mon, 2017-07-17 at 15:42 +0200, Geert Uytterhoeven wrote:
> As of commit bd721ea73e1f9655 ("treewide: replace obsolete _refok by
> __ref"), __init_refok no longer exists, so it can be removed.
> While at it, add the modern variants that were still missing.

Maybe nicer to add some phrase like "Update sparse keywords"
in the patch title

> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
>  scripts/checkpatch.pl | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 2287a0bca863bf57..b88740bfe9d14e41 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -343,9 +343,10 @@ our $Sparse	= qr{
>  			__force|
>  			__iomem|
>  			__must_check|
> -			__init_refok|
>  			__kprobes|
>  			__ref|
> +			__refconst|
> +			__refdata|
>  			__rcu|
>  			__private
>  		}x;

Anyway:

Acked-by: Joe Perches <joe@perches.com>

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web