Re: [PATCH v4] checkpatch: add command-line option for TAB size

2019-05-09 Thread Antonio Borneo
On Thu, May 9, 2019 at 10:03 AM Joe Perches wrote: > > On Thu, 2019-05-09 at 09:21 +0200, Antonio Borneo wrote: > > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl > [] > > @@ -2224,7 +2229,7 @@ sub string_find_replace { > > sub tabify { > > my ($leading) = @_; > > > > - my $

Re: [PATCH v4] checkpatch: add command-line option for TAB size

2019-05-09 Thread Joe Perches
On Thu, 2019-05-09 at 09:21 +0200, Antonio Borneo wrote: > diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl [] > @@ -2224,7 +2229,7 @@ sub string_find_replace { > sub tabify { > my ($leading) = @_; > > - my $source_indent = 8; > + my $source_indent = $tabsize; > my

[PATCH v4] checkpatch: add command-line option for TAB size

2019-05-09 Thread Antonio Borneo
Linux kernel coding style requires a size of 8 characters for both TAB and indentation, and such value is embedded as magic value allover the checkpatch script. This makes hard to reuse the script by other projects with different requirements in their coding style (e.g. OpenOCD [1] requires TAB siz