[PATCH V3 15/15] kmap: Consolidate kmap_prot definitions

Andrew Morton akpm at linux-foundation.org
Thu May 7 20:53:07 UTC 2020

On Thu,  7 May 2020 08:00:03 -0700 ira.weiny at intel.com wrote:

> From: Ira Weiny <ira.weiny at intel.com>
> Most architectures define kmap_prot to be PAGE_KERNEL.
> Let sparc and xtensa define there own and define PAGE_KERNEL as the
> default if not overridden.

checkpatch considered useful ;)

From: Andrew Morton <akpm at linux-foundation.org>
Subject: kmap-consolidate-kmap_prot-definitions-checkpatch-fixes

WARNING: macros should not use a trailing semicolon
#134: FILE: arch/sparc/include/asm/highmem.h:33:
+#define kmap_prot __pgprot(SRMMU_ET_PTE | SRMMU_PRIV | SRMMU_CACHE);

total: 0 errors, 1 warnings, 100 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

./patches/kmap-consolidate-kmap_prot-definitions.patch has style problems, please review.

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.

Please run checkpatch prior to sending patches

Cc: Ira Weiny <ira.weiny at intel.com>
Signed-off-by: Andrew Morton <akpm at linux-foundation.org>

 arch/sparc/include/asm/highmem.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/arch/sparc/include/asm/highmem.h~kmap-consolidate-kmap_prot-definitions-checkpatch-fixes
+++ a/arch/sparc/include/asm/highmem.h
@@ -30,7 +30,7 @@
 /* declarations for highmem.c */
 extern unsigned long highstart_pfn, highend_pfn;
-#define kmap_prot __pgprot(SRMMU_ET_PTE | SRMMU_PRIV | SRMMU_CACHE);
+#define kmap_prot __pgprot(SRMMU_ET_PTE | SRMMU_PRIV | SRMMU_CACHE)
 extern pte_t *pkmap_page_table;
 void kmap_init(void) __init;

More information about the dri-devel mailing list