Re: [PATCH] checkpatch: Make '$min_conf_desc_length' default value to 1

[Leo Yan]

On Sat, Nov 18, 2017 at 04:48:42PM +0800, Leo Yan wrote:
> Make '$min_conf_desc_length' default value from 4 to 1; so can give more
> relax check for Kconfig description has at least one line rather than
> four lines.
> 
> User can still pass parameter "--min-conf-desc-length=X" to set more
> strict check.

Gentle ping.

> Signed-off-by: Leo Yan 
> ---
>  scripts/checkpatch.pl | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 6050323..1dd5366 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -51,7 +51,7 @@ my $configuration_file = ".checkpatch.conf";
>  my $max_line_length = 80;
>  my $ignore_perl_version = 0;
>  my $minimum_perl_version = 5.10.0;
> -my $min_conf_desc_length = 4;
> +my $min_conf_desc_length = 1;
>  my $spelling_file = "$D/spelling.txt";
>  my $codespell = 0;
>  my $codespellfile = "/usr/share/codespell/dictionary.txt";
> -- 
> 2.7.4
> 

Re: [PATCH] checkpatch: Make '$min_conf_desc_length' default value to 1

[Leo Yan]

On Sat, Nov 18, 2017 at 04:48:42PM +0800, Leo Yan wrote:
> Make '$min_conf_desc_length' default value from 4 to 1; so can give more
> relax check for Kconfig description has at least one line rather than
> four lines.
> 
> User can still pass parameter "--min-conf-desc-length=X" to set more
> strict check.

Gentle ping.

> Signed-off-by: Leo Yan 
> ---
>  scripts/checkpatch.pl | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
> index 6050323..1dd5366 100755
> --- a/scripts/checkpatch.pl
> +++ b/scripts/checkpatch.pl
> @@ -51,7 +51,7 @@ my $configuration_file = ".checkpatch.conf";
>  my $max_line_length = 80;
>  my $ignore_perl_version = 0;
>  my $minimum_perl_version = 5.10.0;
> -my $min_conf_desc_length = 4;
> +my $min_conf_desc_length = 1;
>  my $spelling_file = "$D/spelling.txt";
>  my $codespell = 0;
>  my $codespellfile = "/usr/share/codespell/dictionary.txt";
> -- 
> 2.7.4
> 

[PATCH] checkpatch: Make '$min_conf_desc_length' default value to 1

[Leo Yan]

Make '$min_conf_desc_length' default value from 4 to 1; so can give more
relax check for Kconfig description has at least one line rather than
four lines.

User can still pass parameter "--min-conf-desc-length=X" to set more
strict check.

Signed-off-by: Leo Yan 
---
 scripts/checkpatch.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 6050323..1dd5366 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -51,7 +51,7 @@ my $configuration_file = ".checkpatch.conf";
 my $max_line_length = 80;
 my $ignore_perl_version = 0;
 my $minimum_perl_version = 5.10.0;
-my $min_conf_desc_length = 4;
+my $min_conf_desc_length = 1;
 my $spelling_file = "$D/spelling.txt";
 my $codespell = 0;
 my $codespellfile = "/usr/share/codespell/dictionary.txt";
-- 
2.7.4

[PATCH] checkpatch: Make '$min_conf_desc_length' default value to 1

[Leo Yan]

Make '$min_conf_desc_length' default value from 4 to 1; so can give more
relax check for Kconfig description has at least one line rather than
four lines.

User can still pass parameter "--min-conf-desc-length=X" to set more
strict check.

Signed-off-by: Leo Yan 
---
 scripts/checkpatch.pl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 6050323..1dd5366 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -51,7 +51,7 @@ my $configuration_file = ".checkpatch.conf";
 my $max_line_length = 80;
 my $ignore_perl_version = 0;
 my $minimum_perl_version = 5.10.0;
-my $min_conf_desc_length = 4;
+my $min_conf_desc_length = 1;
 my $spelling_file = "$D/spelling.txt";
 my $codespell = 0;
 my $codespellfile = "/usr/share/codespell/dictionary.txt";
-- 
2.7.4