mbox series

[v2,0/2] Fix allmodconfig build with bare-metal toolchain

Message ID 20180829170015.7729-1-semen.protsenko@linaro.org
Headers show
Series Fix allmodconfig build with bare-metal toolchain | expand

Message

Sam Protsenko Aug. 29, 2018, 5 p.m. UTC
Attempt to build allmodconfig (for ARM platforms) using bare-metal
toolchain fails with errors related to __linux__ definition. It's only
defined on Linux toolchains, but not defined on bare-metal toolchains.
This patch series fixes that.

Some similar patches (for different files) were applied previously:
  - commit 4c82456eeb4d ("fuse: fix type definitions in uapi header")
  - commit b6330548383c ("drm.h: Fix DRM compilation with bare-metal
toolchain.")

Sam Protsenko (2):
  codafs: Fix build using bare-metal toolchain
  scsi: aic7xxx: Fix build using bare-metal toolchain

 drivers/scsi/aic7xxx/aic7770.c                |  6 ---
 drivers/scsi/aic7xxx/aic79xx.h                |  6 ---
 drivers/scsi/aic7xxx/aic79xx_core.c           | 43 -------------------
 drivers/scsi/aic7xxx/aic79xx_pci.c            |  6 ---
 drivers/scsi/aic7xxx/aic7xxx.h                |  6 ---
 drivers/scsi/aic7xxx/aic7xxx_93cx6.c          |  6 ---
 drivers/scsi/aic7xxx/aic7xxx_core.c           | 40 +----------------
 drivers/scsi/aic7xxx/aic7xxx_pci.c            |  7 ---
 drivers/scsi/aic7xxx/aicasm/aicasm.h          |  4 --
 drivers/scsi/aic7xxx/aicasm/aicasm_gram.y     |  4 --
 .../scsi/aic7xxx/aicasm/aicasm_macro_gram.y   |  4 --
 .../scsi/aic7xxx/aicasm/aicasm_macro_scan.l   |  4 --
 drivers/scsi/aic7xxx/aicasm/aicasm_scan.l     |  4 --
 drivers/scsi/aic7xxx/aicasm/aicasm_symbol.c   |  4 --
 drivers/scsi/aic7xxx/aicasm/aicasm_symbol.h   |  4 --
 include/linux/coda.h                          |  3 +-
 16 files changed, 2 insertions(+), 149 deletions(-)

-- 
2.18.0

Comments

Sam Protsenko Sept. 11, 2018, 4:53 p.m. UTC | #1
On Wed, Aug 29, 2018 at 8:00 PM, Sam Protsenko
<semen.protsenko@linaro.org> wrote:
> The kernel is self-contained project and can be built with bare-metal

> toolchain. But bare-metal toolchain doesn't define __linux__. Because of

> this u_quad_t type is not defined when using bare-metal toolchain and

> codafs build fails. This patch fixes it by defining u_quad_t type

> unconditionally.

>

> Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org>

> ---

>  include/linux/coda.h | 3 +--

>  1 file changed, 1 insertion(+), 2 deletions(-)

>

> diff --git a/include/linux/coda.h b/include/linux/coda.h

> index d30209b9cef8..0ca0c83fdb1c 100644

> --- a/include/linux/coda.h

> +++ b/include/linux/coda.h

> @@ -58,8 +58,7 @@ Mellon the rights to redistribute these changes without encumbrance.

>  #ifndef _CODA_HEADER_

>  #define _CODA_HEADER_

>

> -#if defined(__linux__)

>  typedef unsigned long long u_quad_t;

> -#endif

> +

>  #include <uapi/linux/coda.h>

>  #endif

> --

> 2.18.0

>


Hi guys,

It's been a while since I sent this patch. Can we please merge it if
there is no objections?

Thanks.