On 18/11/25 09:58, Djordje Todorovic wrote:
Developers should specify endianess explicitly.
Signed-off-by: Djordje Todorovic <[email protected]>
---
scripts/checkpatch.pl | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl
index 40b6955c69..dc88fa4da4 100755
--- a/scripts/checkpatch.pl
+++ b/scripts/checkpatch.pl
@@ -3206,6 +3206,10 @@ sub process {
if ($realfile =~ /.*\/hw\/.*/ && $line =~ /\baio_bh_new\s*\(/) {
ERROR("use aio_bh_new_guarded() instead of aio_bh_new() to
avoid reentrancy problems\n" . $herecurr);
}
+# check for DEVICE_NATIVE_ENDIAN, use explicit endianness instead
+ if ($line =~ /\bDEVICE_NATIVE_ENDIAN\b/) {
+ ERROR("DEVICE_NATIVE_ENDIAN is not allowed, use
DEVICE_LITTLE_ENDIAN or DEVICE_BIG_ENDIAN instead\n" . $herecurr);
+ }
# check for module_init(), use category-specific init macros explicitly please
if ($line =~ /^module_init\s*\(/) {
ERROR("please use block_init(), type_init() etc. instead of
module_init()\n" . $herecurr);
Thank you for this patch :)
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
