evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

checkpatch: accept qca.qualcomm email addresses

Browse source 
@qca.qualcomm.com identities are allowed, hence
change the signed-off and author email validation to
accept @qca.qualcomm.com email addresses.

Change-Id: I8757226f53826c0e5e2825ebe89e80e10a2f18bb
Signed-off-by: Maya Erez <merez@codeaurora.org>
This commit is contained in:
Maya Erez 2017-02-22 16:21:43 +02:00
parent 2aa89ab3ff
commit fc30deb005
1 changed files with 19 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
scripts/checkpatch.pl
View file

@ -60,6 +60,8 @@ my $spelling_file = "$D/spelling.txt";
my $codespell = 0;
my $codespellfile = "/usr/share/codespell/dictionary.txt";
my $color = 1;
my $qca_sign_off = 0;
my $codeaurora_sign_off = 0;
sub help {
my ($exitcode) = @_;
@ -2429,10 +2431,16 @@ sub process {
"email address '$email' might be better as '$suggested_email$comment'\n" . $herecurr);
}
}
if ($chk_author && $line =~ /^\s*signed-off-by:.*(quicinc|qualcomm)\.com/i) {
WARN("BAD_SIGN_OFF",
"invalid Signed-off-by identity\n" . $line );
}
if ($chk_author) {
if ($line =~ /^\s*signed-off-by:.*qca\.qualcomm\.com/i) {
$qca_sign_off = 1;
} elsif ($line =~ /^\s*signed-off-by:.*codeaurora\.org/i) {
$codeaurora_sign_off = 1;
} elsif ($line =~ /^\s*signed-off-by:.*(quicinc|qualcomm)\.com/i) {
WARN("BAD_SIGN_OFF",
"invalid Signed-off-by identity\n" . $line );
}
}
# Check for duplicate signatures
my $sig_nospace = $line;
@ -2558,7 +2566,8 @@ sub process {
}
#check the patch for invalid author credentials
if ($chk_author && $line =~ /^From:.*(quicinc|qualcomm)\.com/) {
if ($chk_author && !($line =~ /^From:.*qca\.qualcomm\.com/) &&
$line =~ /^From:.*(quicinc|qualcomm)\.com/) {
WARN("BAD_AUTHOR", "invalid author identity\n" . $line );
}
@ -6042,6 +6051,11 @@ sub process {
}
}
if ($chk_author && $qca_sign_off && !$codeaurora_sign_off) {
WARN("BAD_SIGN_OFF",
"QCA Signed-off-by requires CODEAURORA Signed-off-by\n" . $line );
}
# If we have no input at all, then there is nothing to report on
# so just keep quiet.
if ($#rawlines == -1) {