Hi, Ruoyao, On Wed, Oct 5, 2022 at 10:14 PM Xi Ruoyao <xry111@xxxxxxxxxxx> wrote: > > Hi Huacai, > > Try: > > #define iounmap(addr) ((void)(addr)) This can solve the problem, and arm use the same way: #define tlb_flush(tlb) ((void) tlb) but I don't know whether this is the correct way. Huacai > > This is how we normally silence an unused variable or parameter warning. > Not sure if the kernel uses the same convention though. > > On Wed, 2022-10-05 at 13:54 +0100, Sudip Mukherjee wrote: > > On Wed, Oct 5, 2022 at 1:30 PM Huacai Chen <chenhuacai@xxxxxxxxxx> > > wrote: > > > > > > Hi, Sudip, > > > > > > This seems a false positive. When the implementation of iounmap() is > > > empty (but this is completely legal I think), these drivers will > > > cause > > > "unused variable error". > > > > And that error will fail the build as allmodconfig will have > > CONFIG_WERROR=y. > > > > -- > Xi Ruoyao <xry111@xxxxxxxxxxx> > School of Aerospace Science and Technology, Xidian University >
