On 05.01.2022 09:57:47, Marc Kleine-Budde wrote:
> The patch
>
> | 98c90e5ea34e sh: remove __iounmap
>
> removed the __iounmap macro for the NOMMU case, but also converted the
> static inline no-op iounmap() to a macro, resulting in lots of unused
> variable warnings.
>
> This patch coverts the macro into a static inline function, similar to
> previous patches in the sh arch:
>
> | 4580ba4ad2e6 sh: Convert iounmap() macros to inline functions
> | 733f0025f0fb sh: prevent warnings when using iounmap
The problem still exists in linus/master, can someone take this patch?
regards,
Marc
> Fixes: 98c90e5ea34e ("sh: remove __iounmap")
> Cc: Kuninori Morimoto <[email protected]>
> Cc: Sam Ravnborg <[email protected]>
> Cc: Geert Uytterhoeven <[email protected]>
> Cc: Rich Felker <[email protected]>
> Cc: Yoshinori Sato <[email protected]>
> Signed-off-by: Marc Kleine-Budde <[email protected]>
> ---
> arch/sh/include/asm/io.h | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/arch/sh/include/asm/io.h b/arch/sh/include/asm/io.h
> index cf9a3ec32406..4960b8ff1ad4 100644
> --- a/arch/sh/include/asm/io.h
> +++ b/arch/sh/include/asm/io.h
> @@ -271,7 +271,9 @@ static inline void __iomem *ioremap_prot(phys_addr_t offset, unsigned long size,
> #endif /* CONFIG_HAVE_IOREMAP_PROT */
>
> #else /* CONFIG_MMU */
> -#define iounmap(addr) do { } while (0)
> +static inline void iounmap(void __iomem *addr)
> +{
> +}
> #define ioremap(offset, size) ((void __iomem *)(unsigned long)(offset))
> #endif /* CONFIG_MMU */
>
> --
> 2.34.1
--
Pengutronix e.K. | Marc Kleine-Budde |
Embedded Linux | https://www.pengutronix.de |
Vertretung West/Dortmund | Phone: +49-231-2826-924 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |