[RFC PATCH 01/11] ARM: NOMMU: define stubs for fixup

Russell King - ARM Linux linux at armlinux.org.uk
Tue Nov 22 01:48:51 PST 2016


On Tue, Nov 22, 2016 at 09:25:58AM +0000, Vladimir Murzin wrote:
> When build for NOMMU following errors show ups:
> 
> arch/arm/kernel/patch.c: In function 'patch_map':
> arch/arm/kernel/patch.c:39:2: error: implicit declaration of function 'set_fixmap' [-Werror=implicit-function-declaration]
>   set_fixmap(fixmap, page_to_phys(page));
>   ^
> arch/arm/kernel/patch.c:41:2: error: implicit declaration of function '__fix_to_virt' [-Werror=implicit-function-declaration]
>   return (void *) (__fix_to_virt(fixmap) + (uintaddr & ~PAGE_MASK));
>   ^
> arch/arm/kernel/patch.c: In function 'patch_unmap':
> arch/arm/kernel/patch.c:47:2: error: implicit declaration of function 'clear_fixmap' [-Werror=implicit-function-declaration]
>   clear_fixmap(fixmap);
>   ^
> cc1: some warnings being treated as errors
> 
> Fixup does not make much sense in NOMMU configurations, so provide
> stub definitions.
> 
> Cc: Russell King <linux at armlinux.org.uk>

I think there should be a Fixes: line for the commit which introduced
this?

> Signed-off-by: Vladimir Murzin <vladimir.murzin at arm.com>
> ---
>  arch/arm/include/asm/fixmap.h |    6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/arch/arm/include/asm/fixmap.h b/arch/arm/include/asm/fixmap.h
> index 5c17d2d..0bfc2e3 100644
> --- a/arch/arm/include/asm/fixmap.h
> +++ b/arch/arm/include/asm/fixmap.h
> @@ -59,6 +59,12 @@ enum fixed_addresses {
>  
>  #else
>  
> +#define set_fixmap(idx, phys)
> +#define clear_fixmap(idx)
> +
> +#define __fix_to_virt(x)	(x)
> +#define __virt_to_fix(x)	(x)
> +
>  static inline void early_fixmap_init(void) { }
>  
>  #endif
> -- 
> 1.7.9.5
> 

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.



More information about the linux-arm-kernel mailing list