* [PATCH] checkpatch: update $allowed_asm_includes macros, add reboot.h and time.h
@ 2014-08-21 7:43 Sergey Ryazanov
2014-08-21 15:01 ` Joe Perches
0 siblings, 1 reply; 2+ messages in thread
From: Sergey Ryazanov @ 2014-08-21 7:43 UTC (permalink / raw)
To: Andy Whitcroft, Joe Perches; +Cc: linux-kernel, Andrew Morton
Several architectures (e.g. x86, MIPS, Blackfin) have asm/reboot.h and
asm/time.h header files, which are not included in linux/reboot.h and
linux/time.h headers. This lead to generation of false positive errors.
Signed-off-by: Sergey Ryazanov <ryazanov.s.a@gmail.com>
---
scripts/checkpatch.pl | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 31a731e..29d6b56 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -425,7 +425,9 @@ foreach my $entry (@mode_permission_funcs) {
our $allowed_asm_includes = qr{(?x:
irq|
- memory
+ memory|
+ time|
+ reboot
)};
# memory.h: ARM has a custom one
--
1.8.1.5
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] checkpatch: update $allowed_asm_includes macros, add reboot.h and time.h
2014-08-21 7:43 [PATCH] checkpatch: update $allowed_asm_includes macros, add reboot.h and time.h Sergey Ryazanov
@ 2014-08-21 15:01 ` Joe Perches
0 siblings, 0 replies; 2+ messages in thread
From: Joe Perches @ 2014-08-21 15:01 UTC (permalink / raw)
To: Sergey Ryazanov; +Cc: Andy Whitcroft, linux-kernel, Andrew Morton
On Thu, 2014-08-21 at 11:43 +0400, Sergey Ryazanov wrote:
> Several architectures (e.g. x86, MIPS, Blackfin) have asm/reboot.h and
> asm/time.h header files, which are not included in linux/reboot.h and
> linux/time.h headers. This lead to generation of false positive errors.
I've no objection.
> Signed-off-by: Sergey Ryazanov <ryazanov.s.a@gmail.com>
> ---
> scripts/checkpatch.pl | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 31a731e..29d6b56 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -425,7 +425,9 @@ foreach my $entry (@mode_permission_funcs) {
>
> our $allowed_asm_includes = qr{(?x:
> irq|
> - memory
> + memory|
> + time|
> + reboot
> )};
> # memory.h: ARM has a custom one
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-08-21 15:02 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-08-21 7:43 [PATCH] checkpatch: update $allowed_asm_includes macros, add reboot.h and time.h Sergey Ryazanov
2014-08-21 15:01 ` Joe Perches
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.