Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-28 Thread B, Ravi
Masihiro-san >See, the following hunk >> diff --git a/arch/arm/cpu/armv7/omap5/Kconfig >> b/arch/arm/cpu/armv7/omap5/Kconfig >> index 4fb5ef9..a8600b1 100644 >> --- a/arch/arm/cpu/armv7/omap5/Kconfig >> +++ b/arch/arm/cpu/armv7/omap5/Kconfig >> @@ -13,6 +13,7 @@ config TARGET_OMAP5_UEVM config

Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-28 Thread Masahiro Yamada
2016-07-28 17:18 GMT+09:00 B, Ravi : > Masahiro-san > >>#if !defined(CONFIG_PPC_P5020) && !defined(CONFIG_PPC_P5040) #error Must >>call Cyrus CONFIG with a specific CPU enabled. >>#endif >>diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h >>index 0d51aeb..a11b692 100644 >>

Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-28 Thread B, Ravi
Masahiro-san >>>#if !defined(CONFIG_PPC_P5020) && !defined(CONFIG_PPC_P5040) #error Must >>>call Cyrus CONFIG with a specific CPU enabled. >>>#endif >>>diff --git a/include/configs/dra7xx_evm.h >>>b/include/configs/dra7xx_evm.h index 0d51aeb..a11b692 100644 >>>--- a/include/configs/dra7xx_evm.h

Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-28 Thread B, Ravi
Masahiro-san >>#if !defined(CONFIG_PPC_P5020) && !defined(CONFIG_PPC_P5040) #error Must >>call Cyrus CONFIG with a specific CPU enabled. >>#endif >>diff --git a/include/configs/dra7xx_evm.h >>b/include/configs/dra7xx_evm.h index 0d51aeb..a11b692 100644 >>--- a/include/configs/dra7xx_evm.h >>+++

Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-28 Thread B, Ravi
Masahiro-san >#if !defined(CONFIG_PPC_P5020) && !defined(CONFIG_PPC_P5040) #error Must call >Cyrus CONFIG with a specific CPU enabled. >#endif >diff --git a/include/configs/dra7xx_evm.h b/include/configs/dra7xx_evm.h index >0d51aeb..a11b692 100644 >--- a/include/configs/dra7xx_evm.h >+++ b/incl

Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-25 Thread york sun
On 07/25/2016 09:07 PM, Masahiro Yamada wrote: > Hi York, > > > 2016-07-26 2:19 GMT+09:00 york sun : > >> Masahiro, >> >> You added CONFIG_PHYS_64BIT=y to most defconfig for powerpc, but not >> C29XPCI. Did you just leave them out on purpose, or just missed? >> >> York > > > Because all of C29XPCIE

Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-25 Thread Masahiro Yamada
Hi York, 2016-07-26 2:19 GMT+09:00 york sun : > Masahiro, > > You added CONFIG_PHYS_64BIT=y to most defconfig for powerpc, but not > C29XPCI. Did you just leave them out on purpose, or just missed? > > York Because all of C29XPCIE_*_defconfig enabled this options, so I decided to add "select P

Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-25 Thread york sun
On 07/25/2016 03:58 AM, Masahiro Yamada wrote: > We need to ensure that CONFIG_PHYS_64BIT is configured via Kconfig so > that it is always available to the build system. Otherwise we can run > into cases where we have inconsistent sizes of certain attributes. > > Ravi Babu reported offset mismatch

Re: [U-Boot] [PATCH v2] treewide: move CONFIG_PHYS_64BIT to Kconfig

2016-07-25 Thread Tom Rini
On Mon, Jul 25, 2016 at 07:56:03PM +0900, Masahiro Yamada wrote: > We need to ensure that CONFIG_PHYS_64BIT is configured via Kconfig so > that it is always available to the build system. Otherwise we can run > into cases where we have inconsistent sizes of certain attributes. > > Ravi Babu repo