diff mbox

[12/25] arm64: compat: change config dependences to aarch32

Message ID 1459894127-17698-13-git-send-email-ynorov@caviumnetworks.com (mailing list archive)
State New, archived
Headers show

Commit Message

Yury Norov April 5, 2016, 10:08 p.m. UTC 
From: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>

With the patches of ILP32, COMPAT is not equivalent to AARCH32 in EL0.
This patch fix this by updating the dependency from COMPAT to
AARCH32_EL0 for ARMV8_DEPRECATED and ARM64_ERRATUM_845719.

Signed-off-by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
Signed-off-by: Yury Norov <ynorov@caviumnetworks.com>
---
 arch/arm64/Kconfig | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Catalin Marinas April 22, 2016, 3:58 p.m. UTC | #1 
On Wed, Apr 06, 2016 at 01:08:34AM +0300, Yury Norov wrote:
> From: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
> 
> With the patches of ILP32, COMPAT is not equivalent to AARCH32 in EL0.
> This patch fix this by updating the dependency from COMPAT to
> AARCH32_EL0 for ARMV8_DEPRECATED and ARM64_ERRATUM_845719.
> 
> Signed-off-by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
> Signed-off-by: Yury Norov <ynorov@caviumnetworks.com>

Can you not merge patches 10-12 into a single one? They all deal with
the s/COMPAT/AARCH32_EL0/ replacement. I'm not even sure the series is
bisectable after patch 10.
Yury Norov April 22, 2016, 9:59 p.m. UTC | #2 
On Fri, Apr 22, 2016 at 04:58:18PM +0100, Catalin Marinas wrote:
> On Wed, Apr 06, 2016 at 01:08:34AM +0300, Yury Norov wrote:
> > From: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
> > 
> > With the patches of ILP32, COMPAT is not equivalent to AARCH32 in EL0.
> > This patch fix this by updating the dependency from COMPAT to
> > AARCH32_EL0 for ARMV8_DEPRECATED and ARM64_ERRATUM_845719.
> > 
> > Signed-off-by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
> > Signed-off-by: Yury Norov <ynorov@caviumnetworks.com>
> 
> Can you not merge patches 10-12 into a single one? They all deal with
> the s/COMPAT/AARCH32_EL0/ replacement. I'm not even sure the series is
> bisectable after patch 10.
> 
> -- 
> Catalin

 OK, if Bamvoor is not objecting
zhangjian April 23, 2016, 2:33 a.m. UTC | #3 
Hi, Yury

On 2016/4/23 5:59, Yury Norov wrote:
> On Fri, Apr 22, 2016 at 04:58:18PM +0100, Catalin Marinas wrote:
>> On Wed, Apr 06, 2016 at 01:08:34AM +0300, Yury Norov wrote:
>>> From: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
>>>
>>> With the patches of ILP32, COMPAT is not equivalent to AARCH32 in EL0.
>>> This patch fix this by updating the dependency from COMPAT to
>>> AARCH32_EL0 for ARMV8_DEPRECATED and ARM64_ERRATUM_845719.
>>>
>>> Signed-off-by: Bamvor Jian Zhang <bamvor.zhangjian@linaro.org>
>>> Signed-off-by: Yury Norov <ynorov@caviumnetworks.com>
>>
>> Can you not merge patches 10-12 into a single one? They all deal with
>> the s/COMPAT/AARCH32_EL0/ replacement. I'm not even sure the series is
>> bisectable after patch 10.
>>
>> --
>> Catalin
>
>   OK, if Bamvoor is not objecting
It is ok.

Bamvor
>
>
diff mbox

Patch

diff --git a/arch/arm64/Kconfig b/arch/arm64/Kconfig
index f923687..ebaf38a 100644
--- a/arch/arm64/Kconfig
+++ b/arch/arm64/Kconfig
@@ -370,7 +370,7 @@  config ARM64_ERRATUM_834220
 
 config ARM64_ERRATUM_845719
 	bool "Cortex-A53: 845719: a load might read incorrect data"
-	depends on COMPAT
+	depends on AARCH32_EL0
 	default y
 	help
 	  This option adds an alternative code sequence to work around ARM
@@ -655,7 +655,7 @@  config FORCE_MAX_ZONEORDER
 
 menuconfig ARMV8_DEPRECATED
 	bool "Emulate deprecated/obsolete ARMv8 instructions"
-	depends on COMPAT
+	depends on AARCH32_EL0
 	help
 	  Legacy software support may require certain instructions
 	  that have been deprecated or obsoleted in the architecture.