Re: linux-next: build warning after merge of the swiotlb tree

From: Ashish Kalra
Date: Thu Dec 10 2020 - 21:49:10 EST


Posted a fix for this.

Thanks,
Ashish

On Fri, Dec 11, 2020 at 01:02:08PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> After merging the swiotlb tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
>
> In file included from arch/arm/mm/init.c:24:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from include/linux/dma-direct.h:13,
> from arch/arm/mm/dma-mapping.c:17:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from include/linux/dma-direct.h:13,
> from kernel/dma/direct.h:10,
> from kernel/dma/mapping.c:17:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from include/linux/dma-direct.h:13,
> from kernel/dma/coherent.c:10:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from include/linux/dma-direct.h:13,
> from kernel/dma/direct.h:10,
> from kernel/dma/direct.c:16:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from include/linux/dma-direct.h:13,
> from drivers/of/device.c:8:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from include/linux/dma-direct.h:13,
> from drivers/remoteproc/remoteproc_core.c:27:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from include/linux/dma-direct.h:13,
> from drivers/of/address.c:16:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from drivers/mmc/host/sdhci.c:23:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
> In file included from drivers/gpu/drm/nouveau/nouveau_bo.c:31:
> include/linux/swiotlb.h:108:13: warning: 'swiotlb_adjust_size' defined but not used [-Wunused-function]
> 108 | static void swiotlb_adjust_size(unsigned long new_size)
> | ^~~~~~~~~~~~~~~~~~~
>
> Introduced by commit
>
> 79f748d1bdb6 ("x86,swiotlb: Adjust SWIOTLB bounce buffer size for SEV guests")
>
> Forgot the "inline" :-(
> --
> Cheers,
> Stephen Rothwell