Change in coreboot[master]: util/lint/checkpatch.pl: Update TYPECAST_INT_CONSTANT lines

[Martin L Roth (Code Review)]

Martin L Roth has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63579 )

Change subject: util/lint/checkpatch.pl: Update TYPECAST_INT_CONSTANT lines
......................................................................

util/lint/checkpatch.pl: Update TYPECAST_INT_CONSTANT lines

Update to v5.18-2 version.

Signed-off-by: Elyes Haouas 
Change-Id: I8ed89e53f647b1b071abff33a434fb3b8dbb1de1
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63579
Tested-by: build bot (Jenkins) 
Reviewed-by: Martin L Roth 
---
M util/lint/checkpatch.pl
1 file changed, 10 insertions(+), 10 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Martin L Roth: Looks good to me, approved

diff --git a/util/lint/checkpatch.pl b/util/lint/checkpatch.pl
index d75d655..b1cb524 100755
--- a/util/lint/checkpatch.pl
+++ b/util/lint/checkpatch.pl
@@ -6080,18 +6080,18 @@
    	if ($line =~ /((\s*$C90_int_types\s*)\s*)($Constant)\b/) {
    		my $cast = $1;
    		my $const = $2;
+			my $suffix = "";
+			my $newconst = $const;
+			$newconst =~ s/${Int_type}$//;
+			$suffix .= 'U' if ($cast =~ /\bunsigned\b/);
+			if ($cast =~ /\blong\s+long\b/) {
+			    $suffix .= 'LL';
+			} elsif ($cast =~ /\blong\b/) {
+			    $suffix .= 'L';
+			}
    		if (WARN("TYPECAST_INT_CONSTANT",
-				 "Unnecessary typecast of c90 int constant\n" . $herecurr) &&
+				 "Unnecessary typecast of c90 int constant - '$cast$const' could be '$const$suffix'\n" . $herecurr) &&
    		    $fix) {
-				my $suffix = "";
-				my $newconst = $const;
-				$newconst =~ s/${Int_type}$//;
-				$suffix .= 'U' if ($cast =~ /\bunsigned\b/);
-				if ($cast =~ /\blong\s+long\b/) {
-					$suffix .= 'LL';
-				} elsif ($cast =~ /\blong\b/) {
-					$suffix .= 'L';
-				}
    			$fixed[$fixlinenr] =~ s/\Q$cast\E$const\b/$newconst$suffix/;
    		}
    	}