On Thu, May 29, 2014 at 08:46:22PM +0300, Ivan Khoronzhuk wrote:

> The definitions inside emif_defs.h concern davinci nand driver and
> should be in it's header. So create header file for davinci nand
> driver and move definitions from emif_defs.h to it.
> 
> Acked-by: Vitaly Andrianov <vita...@ti.com>
> Signed-off-by: Ivan Khoronzhuk <ivan.khoronz...@ti.com>
> ---
>  arch/arm/cpu/armv7/keystone/aemif.c            |  2 +-
>  arch/arm/include/asm/arch-keystone/emif_defs.h | 73 
> --------------------------
>  arch/arm/include/asm/arch-keystone/nand_defs.h | 23 --------
>  board/ti/k2hk_evm/board.c                      |  1 -
>  drivers/mtd/nand/davinci_nand.c                |  3 +-
>  drivers/mtd/nand/davinci_nand.h                | 66 +++++++++++++++++++++++
>  6 files changed, 68 insertions(+), 100 deletions(-)
>  delete mode 100644 arch/arm/include/asm/arch-keystone/emif_defs.h
>  delete mode 100644 arch/arm/include/asm/arch-keystone/nand_defs.h
>  create mode 100644 drivers/mtd/nand/davinci_nand.h

This breaks building a bunch of davinci platforms.  Further, we should
put the new header file under arch/arm/include/asm/ti-common/ not
drivers/mtd/nand/.  Can you please update things and reconcile
arch/arm/include/asm/arch-davinci/emif_defs.h with these new changes?
Thanks.

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to