diff options
| author | Tom Rini <trini@konsulko.com> | 2020-08-20 08:37:49 -0400 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2020-08-27 11:26:58 -0400 |
| commit | 12178b51c22d07229375f25e39acb0ab7735ff81 (patch) | |
| tree | 3319d2ac0a63c9201983c5dcf61e1aa4ebc34d7b | |
| parent | 076617847a2cfa5a7b16fd2b729091c5bf2d5b78 (diff) | |
checkpatch.pl: Make fdt / initrd relocation disabling an error
Entirely disabling relocation of the device tree or initrd is almost
never the right answer. Doing this by default leads to hard to diagnose
run-time failures.
Signed-off-by: Tom Rini <trini@konsulko.com>
| -rwxr-xr-x | scripts/checkpatch.pl | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 3932362dba..4bed2b0cdc 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -2359,6 +2359,12 @@ sub u_boot_line { ERROR("BARRED_INCLUDE_IN_HDR", "Avoid including common.h and dm.h in header files\n" . $herecurr); } + + # Do not disable fdt / initrd relocation + if ($rawline =~ /.*(fdt|initrd)_high=0xffffffff/) { + ERROR("DISABLE_FDT_OR_INITRD_RELOC", + "fdt or initrd relocation disabled at boot time\n" . $herecurr); + } } sub process { |