u-boot.lists.denx.de archive mirror
 help / color / mirror / Atom feed
From: Stefan Roese <sr@denx.de>
To: "Marek Behún" <kabel@kernel.org>
Cc: u-boot@lists.denx.de, pali@kernel.org,
	"Marek Behún" <marek.behun@nic.cz>
Subject: Re: [PATCH u-boot-marvell 04/12] arm: kirkwood, mvebu: Remove CONFIG_SYS_RESET_ADDRESS option
Date: Thu, 21 Oct 2021 07:31:44 +0200	[thread overview]
Message-ID: <ac04ff2c-bd95-23f4-4b22-ab711e782543@denx.de> (raw)
In-Reply-To: <20211009173346.7859-5-kabel@kernel.org>

On 09.10.21 19:33, Marek Behún wrote:
> From: Marek Behún <marek.behun@nic.cz>
> 
> This option is not used anywhere.
> 
> Signed-off-by: Marek Behún <marek.behun@nic.cz>

Reviewed-by: Stefan Roese <sr@denx.de>

Thanks,
Stefan

> ---
>   include/configs/SBx81LIFKW.h        | 5 -----
>   include/configs/SBx81LIFXCAT.h      | 5 -----
>   include/configs/edminiv2.h          | 6 ------
>   include/configs/mv-common.h         | 1 -
>   include/configs/mvebu_armada-37xx.h | 1 -
>   include/configs/mvebu_armada-8k.h   | 1 -
>   include/configs/turris_mox.h        | 1 -
>   include/configs/x530.h              | 5 -----
>   8 files changed, 25 deletions(-)
> 
> diff --git a/include/configs/SBx81LIFKW.h b/include/configs/SBx81LIFKW.h
> index fc6167cf96..462619ff69 100644
> --- a/include/configs/SBx81LIFKW.h
> +++ b/include/configs/SBx81LIFKW.h
> @@ -59,11 +59,6 @@
>   /* There is no PHY directly connected so don't ask it for link status */
>   #undef CONFIG_SYS_FAULT_ECHO_LINK_DOWN
>   
> -/*
> - * Other required minimal configurations
> - */
> -#define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
> -
>   /*
>    * Ethernet Driver configuration
>    */
> diff --git a/include/configs/SBx81LIFXCAT.h b/include/configs/SBx81LIFXCAT.h
> index 06be63e242..a271567f85 100644
> --- a/include/configs/SBx81LIFXCAT.h
> +++ b/include/configs/SBx81LIFXCAT.h
> @@ -64,11 +64,6 @@
>   /* There is no PHY directly connected so don't ask it for link status */
>   #undef CONFIG_SYS_FAULT_ECHO_LINK_DOWN
>   
> -/*
> - * Other required minimal configurations
> - */
> -#define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
> -
>   /*
>    * Ethernet Driver configuration
>    */
> diff --git a/include/configs/edminiv2.h b/include/configs/edminiv2.h
> index fbe468010b..664d6d1f34 100644
> --- a/include/configs/edminiv2.h
> +++ b/include/configs/edminiv2.h
> @@ -151,12 +151,6 @@
>    *  Environment variables configurations
>    */
>   
> -/*
> - * Other required minimal configurations
> - */
> -
> -#define CONFIG_SYS_RESET_ADDRESS	0xffff0000
> -
>   /* Enable command line editing */
>   
>   /* provide extensive help */
> diff --git a/include/configs/mv-common.h b/include/configs/mv-common.h
> index e460f69a08..cc3b597f28 100644
> --- a/include/configs/mv-common.h
> +++ b/include/configs/mv-common.h
> @@ -55,7 +55,6 @@
>   /*
>    * Other required minimal configurations
>    */
> -#define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
>   #define CONFIG_SYS_MAXARGS	32	/* max number of command args */
>   
>   /* ====> Include platform Common Definitions */
> diff --git a/include/configs/mvebu_armada-37xx.h b/include/configs/mvebu_armada-37xx.h
> index 8c315eb563..e7f7e772fc 100644
> --- a/include/configs/mvebu_armada-37xx.h
> +++ b/include/configs/mvebu_armada-37xx.h
> @@ -30,7 +30,6 @@
>   /*
>    * Other required minimal configurations
>    */
> -#define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
>   #define CONFIG_SYS_MAXARGS	32	/* max number of command args */
>   
>   /* End of 16M scrubbed by training in bootrom */
> diff --git a/include/configs/mvebu_armada-8k.h b/include/configs/mvebu_armada-8k.h
> index 2f8be2ee49..886f44c903 100644
> --- a/include/configs/mvebu_armada-8k.h
> +++ b/include/configs/mvebu_armada-8k.h
> @@ -24,7 +24,6 @@
>   /*
>    * Other required minimal configurations
>    */
> -#define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
>   #define CONFIG_SYS_MAXARGS	32	/* max number of command args */
>   
>   /* End of 16M scrubbed by training in bootrom */
> diff --git a/include/configs/turris_mox.h b/include/configs/turris_mox.h
> index ab7931d35b..070abe38db 100644
> --- a/include/configs/turris_mox.h
> +++ b/include/configs/turris_mox.h
> @@ -28,7 +28,6 @@
>   /*
>    * Other required minimal configurations
>    */
> -#define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
>   #define CONFIG_SYS_MAXARGS	32	/* max number of command args */
>   
>   /* End of 16M scrubbed by training in bootrom */
> diff --git a/include/configs/x530.h b/include/configs/x530.h
> index d6aec6d7f2..699deffbe0 100644
> --- a/include/configs/x530.h
> +++ b/include/configs/x530.h
> @@ -61,11 +61,6 @@
>   
>   #include <asm/arch/config.h>
>   
> -/*
> - * Other required minimal configurations
> - */
> -#define CONFIG_SYS_RESET_ADDRESS 0xffff0000	/* Rst Vector Adr */
> -
>   /* Keep device tree and initrd in low memory so the kernel can access them */
>   #define CONFIG_EXTRA_ENV_SETTINGS	\
>   	"fdt_high=0x10000000\0"		\
> 


Viele Grüße,
Stefan

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de

  reply	other threads:[~2021-10-21  5:32 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-09 17:33 [PATCH u-boot-marvell 00/12] Small Turris MOX and Omnia changes Marek Behún
2021-10-09 17:33 ` [PATCH u-boot-marvell 01/12] arm: mvebu: turris_mox: Drop SF_DEFAULT_MODE from defconfig Marek Behún
2021-10-21  5:30   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 02/12] arm: mvebu: turris_mox: Move options to defconfig Marek Behún
2021-10-21  5:31   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 03/12] arm: mvebu: a3720: Create Kconfig option for I2C_MV Marek Behún
2021-10-21  5:31   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 04/12] arm: kirkwood, mvebu: Remove CONFIG_SYS_RESET_ADDRESS option Marek Behún
2021-10-21  5:31   ` Stefan Roese [this message]
2021-10-09 17:33 ` [PATCH u-boot-marvell 05/12] arm: mvebu: turris_mox: Cosmetic update for board config header Marek Behún
2021-10-21  5:32   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 06/12] arm: mvebu: turris_mox: Use show_board_info() Marek Behún
2021-10-21  5:32   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 07/12] arm: mvebu: turris_mox: Always handle reset button Marek Behún
2021-10-21  5:32   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 08/12] arm: mvebu: turris_mox: Better check for valid ethernet addresses in env Marek Behún
2021-10-21  5:33   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 09/12] arm: mvebu: turris_omnia: Overwrite ethaddr only if invalid Marek Behún
2021-10-21  5:33   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 10/12] arm: mvebu: turris_omnia: Use show_board_info() Marek Behún
2021-10-21  5:34   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 11/12] arm: mvebu: turris_omnia: Move SPL's SYS_MALLOC_SIMPLE to Kconfig Marek Behún
2021-10-21  5:34   ` Stefan Roese
2021-10-09 17:33 ` [PATCH u-boot-marvell 12/12] arm: mvebu: turris_omnia: Move CONFIG_SPL_DRIVERS_MISC " Marek Behún
2021-10-21  5:34   ` Stefan Roese
2021-10-21  7:47 ` [PATCH u-boot-marvell 00/12] Small Turris MOX and Omnia changes Stefan Roese

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ac04ff2c-bd95-23f4-4b22-ab711e782543@denx.de \
    --to=sr@denx.de \
    --cc=kabel@kernel.org \
    --cc=marek.behun@nic.cz \
    --cc=pali@kernel.org \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).