[7/7] arm: vexpress64: Remove unneeded CONFIG_ check

Message ID 20200325144702.16288-8-andre.przywara@arm.com
State Accepted
Commit af6d4c0567f71fd24879fdb26b712b59b6a25160
Headers show
Series
  • Arm Juno board OF_CONTROL upgrade
Related show

Commit Message

Andre Przywara March 25, 2020, 2:47 p.m.
CONFIG_SEMIHOSTING is selected for the VFP target by the means of
Kconfig already, there is no need to check this in the header file.

Signed-off-by: Andre Przywara <andre.przywara at arm.com>
---
 include/configs/vexpress_aemv8a.h | 6 ------
 1 file changed, 6 deletions(-)

Comments

Linus Walleij March 27, 2020, 9:26 p.m. | #1
On Wed, Mar 25, 2020 at 3:47 PM Andre Przywara <andre.przywara at arm.com> wrote:

> CONFIG_SEMIHOSTING is selected for the VFP target by the means of
> Kconfig already, there is no need to check this in the header file.
>
> Signed-off-by: Andre Przywara <andre.przywara at arm.com>

Reviewed-by: Linus Walleij <linus.walleij at linaro.org>

Yours,
Linus Walleij

Patch

diff --git a/include/configs/vexpress_aemv8a.h b/include/configs/vexpress_aemv8a.h
index b02caae96c..adbe886ae5 100644
--- a/include/configs/vexpress_aemv8a.h
+++ b/include/configs/vexpress_aemv8a.h
@@ -7,12 +7,6 @@ 
 #ifndef __VEXPRESS_AEMV8A_H
 #define __VEXPRESS_AEMV8A_H
 
-#ifdef CONFIG_TARGET_VEXPRESS64_BASE_FVP
-#ifndef CONFIG_SEMIHOSTING
-#error CONFIG_TARGET_VEXPRESS64_BASE_FVP requires CONFIG_SEMIHOSTING
-#endif
-#endif
-
 #define CONFIG_REMAKE_ELF
 
 /* Link Definitions */