On 06/06/2014 11:34 PM, Tom Rini wrote:
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.


Yes..I will repost new patch.
Thanks!

--
Regards,
Ivan Khoronzhuk

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

Reply via email to