Re: ARM_ATAG_DTB_COMPAT fails on fdt_ro.c compiling

On Tue, Oct 09, 2012 at 12:25:44AM -0400, Nicolas Pitre wrote:

On Tue, 9 Oct 2012, Shawn Guo wrote:

quoted

I have seen this error on mainline for a couple days when
ARM_ATAG_DTB_COMPAT is enabled.

  CC      arch/arm/boot/compressed/fdt_ro.o
arch/arm/boot/compressed/fdt_ro.c: In function ‘fdt_path_offset’:
arch/arm/boot/compressed/fdt_ro.c:172:3: error: implicit declaration of function ‘fdt_get_alias_namelen’ [-Werror=implicit-function-declaration]
arch/arm/boot/compressed/fdt_ro.c:172:5: warning: assignment makes pointer from integer without a cast [enabled by default]
arch/arm/boot/compressed/fdt_ro.c: At top level:
arch/arm/boot/compressed/fdt_ro.c:340:13: error: conflicting types for ‘fdt_get_alias_namelen’
arch/arm/boot/compressed/fdt_ro.c:172:7: note: previous implicit declaration of ‘fdt_get_alias_namelen’ was here
cc1: some warnings being treated as errors

I just tried building mainline now.  It is at commit f5a246eab9.

The build still fails for me at this commit.

  [...]
  CC      arch/arm/boot/compressed/misc.o
  CC      arch/arm/boot/compressed/decompress.o
  CC      arch/arm/boot/compressed/string.o
  SHIPPED arch/arm/boot/compressed/fdt_rw.c
  SHIPPED arch/arm/boot/compressed/fdt.h
  SHIPPED arch/arm/boot/compressed/libfdt.h
  SHIPPED arch/arm/boot/compressed/libfdt_internal.h
  CC      arch/arm/boot/compressed/fdt_rw.o
  SHIPPED arch/arm/boot/compressed/fdt_ro.c
  CC      arch/arm/boot/compressed/fdt_ro.o
  SHIPPED arch/arm/boot/compressed/fdt_wip.c
  CC      arch/arm/boot/compressed/fdt_wip.o
  SHIPPED arch/arm/boot/compressed/fdt.c
  CC      arch/arm/boot/compressed/fdt.o
  CC      arch/arm/boot/compressed/atags_to_fdt.o
  SHIPPED arch/arm/boot/compressed/lib1funcs.S
  AS      arch/arm/boot/compressed/lib1funcs.o
  SHIPPED arch/arm/boot/compressed/ashldi3.S
  AS      arch/arm/boot/compressed/ashldi3.o
  LD      arch/arm/boot/compressed/vmlinux
  OBJCOPY arch/arm/boot/zImage
  Kernel: arch/arm/boot/zImage is ready

What config do you use?

My config is attached.  It's just imx_v6_v7_defconfig with
ARM_ATAG_DTB_COMPAT enabled.

Shawn