[U-Boot] [PATCH] biosemu: include <asm/io.h> header

Albert ARIBAUD albert.u.boot at aribaud.net
Tue Apr 2 10:56:58 CEST 2013


Hi Linus,

On Tue,  2 Apr 2013 10:14:14 +0200, Linus Walleij
<linus.walleij at linaro.org> wrote:

> This makes sure we have inline functions such as inb/outb that
> are used in these two files by including the arch-specific
> <asm/io.h> header. However the ARM version does not provide the
> accessors unless the config symbol __io is also defined so add
> that in front of the include.
> 
> After this the bios emulator will compile on ARM systems.
> 
> Signed-off-by: Linus Walleij <linus.walleij at linaro.org>
> ---
>  drivers/bios_emulator/besys.c | 2 ++
>  drivers/bios_emulator/bios.c  | 2 ++
>  2 files changed, 4 insertions(+)
> 
> diff --git a/drivers/bios_emulator/besys.c b/drivers/bios_emulator/besys.c
> index 84724b7..ad88a53 100644
> --- a/drivers/bios_emulator/besys.c
> +++ b/drivers/bios_emulator/besys.c
> @@ -47,6 +47,8 @@
>  *
>  ****************************************************************************/
>  
> +#define __io
> +#include <asm/io.h>
>  #include <common.h>
>  #include "biosemui.h"
>  
> diff --git a/drivers/bios_emulator/bios.c b/drivers/bios_emulator/bios.c
> index 7cf4879..bcc192f 100644
> --- a/drivers/bios_emulator/bios.c
> +++ b/drivers/bios_emulator/bios.c
> @@ -41,6 +41,8 @@
>  *
>  ****************************************************************************/
>  
> +#define __io
> +#include <asm/io.h>
>  #include <common.h>
>  #include "biosemui.h"

NAK -- no ARM target needs bios emulation, so basing the #define on ARM
requirements is incorrect.

Actually, ARM targets build drivers/bios_emulator/libatibiosemu.o as
the result of an overlook in ./Makefile where this object is compiled 
unconditionally.

A git grep CONFIG_BIOSEMU seems to indicate only a handful of PowerPC
targets need bios emulation; I suggest doing a V2 of this patch
where the object is built only for PowerPC, and the #define is removed.

Amicalement,
-- 
Albert.


More information about the U-Boot mailing list