diff mbox

checkpatch: do not warn on Camel case for printf PRIx macro

Message ID 1433257380-10817-1-git-send-email-maxim.uvarov@linaro.org
State Accepted
Commit d6c09414e6220320a8ec229e219568c92aad0008
Headers show

Commit Message

Maxim Uvarov June 2, 2015, 3:03 p.m. UTC 
Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
---
 scripts/checkpatch.pl | 2 ++
 1 file changed, 2 insertions(+)

Comments

Mike Holmes June 2, 2015, 7:23 p.m. UTC | #1 
On 2 June 2015 at 11:03, Maxim Uvarov <maxim.uvarov@linaro.org> wrote:

> Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org>
>

Reviewed-by: Mike Holmes <mike.holmes@linaro.org>



> ---
>  scripts/checkpatch.pl | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index bc13831..ca60292 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -4255,6 +4255,8 @@ sub process {
>                             $var !~
> /^(?:[a-z_]*?)_?[a-z][A-Z](?:_[a-z_]+)?$/ &&
>  #Ignore some three character SI units explicitly, like MiB and KHz
>                             $var !~ /\bCU_/ &&
> +                           $var !~ /\bPRI[diux]32/ &&
> +                           $var !~ /\bPRI[diux]64/ &&
>                             $var !~
> /^(?:[a-z_]*?)_?(?:[KMGT]iB|[KMGT]?Hz)(?:_[a-z_]+)?$/) {
>                                 while ($var =~ m{($Ident)}g) {
>                                         my $word = $1;
> --
> 1.9.1
>
> _______________________________________________
> lng-odp mailing list
> lng-odp@lists.linaro.org
> https://lists.linaro.org/mailman/listinfo/lng-odp
>
Maxim Uvarov June 4, 2015, 10:21 a.m. UTC | #2 
Merged,
Maxim.

On 06/02/15 22:23, Mike Holmes wrote:
>
>
> On 2 June 2015 at 11:03, Maxim Uvarov <maxim.uvarov@linaro.org 
> <mailto:maxim.uvarov@linaro.org>> wrote:
>
>     Signed-off-by: Maxim Uvarov <maxim.uvarov@linaro.org
>     <mailto:maxim.uvarov@linaro.org>>
>
>
> Reviewed-by: Mike Holmes <mike.holmes@linaro.org 
> <mailto:mike.holmes@linaro.org>>
>
>     ---
>      scripts/checkpatch.pl <http://checkpatch.pl> | 2 ++
>      1 file changed, 2 insertions(+)
>
>     diff --git a/scripts/checkpatch.pl <http://checkpatch.pl>
>     b/scripts/checkpatch.pl <http://checkpatch.pl>
>     index bc13831..ca60292 100755
>     --- a/scripts/checkpatch.pl <http://checkpatch.pl>
>     +++ b/scripts/checkpatch.pl <http://checkpatch.pl>
>     @@ -4255,6 +4255,8 @@ sub process {
>                                 $var !~
>     /^(?:[a-z_]*?)_?[a-z][A-Z](?:_[a-z_]+)?$/ &&
>      #Ignore some three character SI units explicitly, like MiB and KHz
>                                 $var !~ /\bCU_/ &&
>     +                           $var !~ /\bPRI[diux]32/ &&
>     +                           $var !~ /\bPRI[diux]64/ &&
>                                 $var !~
>     /^(?:[a-z_]*?)_?(?:[KMGT]iB|[KMGT]?Hz)(?:_[a-z_]+)?$/) {
>                                     while ($var =~ m{($Ident)}g) {
>                                             my $word = $1;
>     --
>     1.9.1
>
>     _______________________________________________
>     lng-odp mailing list
>     lng-odp@lists.linaro.org <mailto:lng-odp@lists.linaro.org>
>     https://lists.linaro.org/mailman/listinfo/lng-odp
>
>
>
>
> -- 
> Mike Holmes
> Technical Manager - Linaro Networking Group
> Linaro.org <http://www.linaro.org/>***│ *Open source software for ARM SoCs
>
diff mbox

Patch

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index bc13831..ca60292 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -4255,6 +4255,8 @@  sub process {
 			    $var !~ /^(?:[a-z_]*?)_?[a-z][A-Z](?:_[a-z_]+)?$/ &&
 #Ignore some three character SI units explicitly, like MiB and KHz
 			    $var !~ /\bCU_/ &&
+			    $var !~ /\bPRI[diux]32/ &&
+			    $var !~ /\bPRI[diux]64/ &&
 			    $var !~ /^(?:[a-z_]*?)_?(?:[KMGT]iB|[KMGT]?Hz)(?:_[a-z_]+)?$/) {
 				while ($var =~ m{($Ident)}g) {
 					my $word = $1;