On Thu, May 25, 2023 at 09:05:35AM +0530, Anshuman Khandual wrote: > > @@ -127,6 +127,8 @@ extern void copy_to_user_page(struct vm_area_struct *, struct page *, > > */ > > #define ARCH_IMPLEMENTS_FLUSH_DCACHE_PAGE 1 > > extern void flush_dcache_page(struct page *); > > +void flush_dcache_folio(struct folio *); > > This is giving a checkpatch.pl warning > > WARNING: function definition argument 'struct folio *' should also have an identifier name > #36: FILE: arch/arm64/include/asm/cacheflush.h:130: > +void flush_dcache_folio(struct folio *); Yes, but checkpatch is *stupid*. Don't just follow tools blindly. How is naming the parameter here helping anyone?
