diff mbox

[U-Boot,v2,13/15] dm: am335x: Remove serial options from CONFIG_SYS_EXTRA_OPTIONS

Message ID 1414035435-17897-14-git-send-email-sjg@chromium.org
State Accepted
Delegated to: Simon Glass
Headers show

Commit Message

Simon Glass Oct. 23, 2014, 3:37 a.m. UTC
With these options in place it is not possible to change the serial port
using 'make menuconfig' or similar. It seems to result in duplicate
defines.

For example:

In file included from include/linux/kconfig.h:4:0,
                 from <command-line>:0:
include/generated/autoconf.h:20:0: note: this is the location of the previous definition
 #define CONFIG_CONS_INDEX 2
 ^

The default option seems to be 1 anyway, in board/ti/am335x/Kconfig.

Remove the options so that we can adjust the serial port if required.

Signed-off-by: Simon Glass <sjg@chromium.org>
Acked-by: Tom Rini <trini@ti.com>
---

Changes in v2: None

 configs/am335x_boneblack_defconfig       | 2 +-
 configs/am335x_boneblack_vboot_defconfig | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Simon Glass Oct. 24, 2014, 3:52 a.m. UTC | #1
On 22 October 2014 21:37, Simon Glass <sjg@chromium.org> wrote:
> With these options in place it is not possible to change the serial port
> using 'make menuconfig' or similar. It seems to result in duplicate
> defines.
>
> For example:
>
> In file included from include/linux/kconfig.h:4:0,
>                  from <command-line>:0:
> include/generated/autoconf.h:20:0: note: this is the location of the previous definition
>  #define CONFIG_CONS_INDEX 2
>  ^
>
> The default option seems to be 1 anyway, in board/ti/am335x/Kconfig.
>
> Remove the options so that we can adjust the serial port if required.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> Acked-by: Tom Rini <trini@ti.com>
> ---
>
> Changes in v2: None
>
>  configs/am335x_boneblack_defconfig       | 2 +-
>  configs/am335x_boneblack_vboot_defconfig | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)

Applied to u-boot-dm/master
diff mbox

Patch

diff --git a/configs/am335x_boneblack_defconfig b/configs/am335x_boneblack_defconfig
index 38450c0..b631c41 100644
--- a/configs/am335x_boneblack_defconfig
+++ b/configs/am335x_boneblack_defconfig
@@ -1,4 +1,4 @@ 
 CONFIG_SPL=y
-CONFIG_SYS_EXTRA_OPTIONS="SERIAL1,CONS_INDEX=1,EMMC_BOOT"
+CONFIG_SYS_EXTRA_OPTIONS="EMMC_BOOT"
 +S:CONFIG_ARM=y
 +S:CONFIG_TARGET_AM335X_EVM=y
diff --git a/configs/am335x_boneblack_vboot_defconfig b/configs/am335x_boneblack_vboot_defconfig
index e257143..5837a0a 100644
--- a/configs/am335x_boneblack_vboot_defconfig
+++ b/configs/am335x_boneblack_vboot_defconfig
@@ -1,5 +1,5 @@ 
 CONFIG_SPL=y
-CONFIG_SYS_EXTRA_OPTIONS="SERIAL1,CONS_INDEX=1,EMMC_BOOT,ENABLE_VBOOT"
+CONFIG_SYS_EXTRA_OPTIONS="EMMC_BOOT,ENABLE_VBOOT"
 +S:CONFIG_ARM=y
 +S:CONFIG_TARGET_AM335X_EVM=y
 CONFIG_OF_CONTROL=y