Message ID | 20220131072022.440711-1-ovidiu.panait@windriver.com |
---|---|
State | Accepted |
Commit | 6303b275a3424dc1b6c79e98ac575ab2c88659eb |
Delegated to: | Tom Rini |
Headers | show |
Series | [1/2] powerpc: mpc8xx: drop CONFIG_SYS_RESET_ADDRESS | expand |
On 1/31/22 08:20, Ovidiu Panait wrote: > There are no boards that define CONFIG_SYS_RESET_ADDRESS, so drop the > associated mpc8xx code that checks for it. > > Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan > --- > > arch/powerpc/cpu/mpc8xx/cpu.c | 11 ++--------- > scripts/config_whitelist.txt | 1 - > 2 files changed, 2 insertions(+), 10 deletions(-) > > diff --git a/arch/powerpc/cpu/mpc8xx/cpu.c b/arch/powerpc/cpu/mpc8xx/cpu.c > index 893aecef21..6d16ed084e 100644 > --- a/arch/powerpc/cpu/mpc8xx/cpu.c > +++ b/arch/powerpc/cpu/mpc8xx/cpu.c > @@ -215,19 +215,12 @@ int do_reset(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) > /* > * Trying to execute the next instruction at a non-existing address > * should cause a machine check, resulting in reset > - */ > -#ifdef CONFIG_SYS_RESET_ADDRESS > - addr = CONFIG_SYS_RESET_ADDRESS; > -#else > - /* > + * > * note: when CONFIG_SYS_MONITOR_BASE points to a RAM address, > * CONFIG_SYS_MONITOR_BASE - sizeof (ulong) is usually a valid address. > - * Better pick an address known to be invalid on your system and assign > - * it to CONFIG_SYS_RESET_ADDRESS. > - * "(ulong)-1" used to be a good choice for many systems... > */ > addr = CONFIG_SYS_MONITOR_BASE - sizeof(ulong); > -#endif > + > ((void (*)(void)) addr)(); > return 1; > } > diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt > index 7b45d578a3..84e46e3763 100644 > --- a/scripts/config_whitelist.txt > +++ b/scripts/config_whitelist.txt > @@ -1837,7 +1837,6 @@ CONFIG_SYS_RCAR_I2C1_BASE > CONFIG_SYS_RCAR_I2C2_BASE > CONFIG_SYS_RCAR_I2C3_BASE > CONFIG_SYS_RESET_ADDR > -CONFIG_SYS_RESET_ADDRESS > CONFIG_SYS_RFD > CONFIG_SYS_RGMII1_PHY_ADDR > CONFIG_SYS_RGMII2_PHY_ADDR Viele Grüße, Stefan Roese
Le 31/01/2022 à 08:20, Ovidiu Panait a écrit : > There are no boards that define CONFIG_SYS_RESET_ADDRESS, so drop the > associated mpc8xx code that checks for it. > > Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> Acked-by: Christophe Leroy <christophe.leroy@csgroup.eu> > --- > > arch/powerpc/cpu/mpc8xx/cpu.c | 11 ++--------- > scripts/config_whitelist.txt | 1 - > 2 files changed, 2 insertions(+), 10 deletions(-) > > diff --git a/arch/powerpc/cpu/mpc8xx/cpu.c b/arch/powerpc/cpu/mpc8xx/cpu.c > index 893aecef21..6d16ed084e 100644 > --- a/arch/powerpc/cpu/mpc8xx/cpu.c > +++ b/arch/powerpc/cpu/mpc8xx/cpu.c > @@ -215,19 +215,12 @@ int do_reset(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) > /* > * Trying to execute the next instruction at a non-existing address > * should cause a machine check, resulting in reset > - */ > -#ifdef CONFIG_SYS_RESET_ADDRESS > - addr = CONFIG_SYS_RESET_ADDRESS; > -#else > - /* > + * > * note: when CONFIG_SYS_MONITOR_BASE points to a RAM address, > * CONFIG_SYS_MONITOR_BASE - sizeof (ulong) is usually a valid address. > - * Better pick an address known to be invalid on your system and assign > - * it to CONFIG_SYS_RESET_ADDRESS. > - * "(ulong)-1" used to be a good choice for many systems... > */ > addr = CONFIG_SYS_MONITOR_BASE - sizeof(ulong); > -#endif > + > ((void (*)(void)) addr)(); > return 1; > } > diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt > index 7b45d578a3..84e46e3763 100644 > --- a/scripts/config_whitelist.txt > +++ b/scripts/config_whitelist.txt > @@ -1837,7 +1837,6 @@ CONFIG_SYS_RCAR_I2C1_BASE > CONFIG_SYS_RCAR_I2C2_BASE > CONFIG_SYS_RCAR_I2C3_BASE > CONFIG_SYS_RESET_ADDR > -CONFIG_SYS_RESET_ADDRESS > CONFIG_SYS_RFD > CONFIG_SYS_RGMII1_PHY_ADDR > CONFIG_SYS_RGMII2_PHY_ADDR
On Mon, Jan 31, 2022 at 09:20:21AM +0200, Ovidiu Panait wrote: > There are no boards that define CONFIG_SYS_RESET_ADDRESS, so drop the > associated mpc8xx code that checks for it. > > Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> > Reviewed-by: Stefan Roese <sr@denx.de> > Acked-by: Christophe Leroy <christophe.leroy@csgroup.eu> Applied to u-boot/master, thanks!
diff --git a/arch/powerpc/cpu/mpc8xx/cpu.c b/arch/powerpc/cpu/mpc8xx/cpu.c index 893aecef21..6d16ed084e 100644 --- a/arch/powerpc/cpu/mpc8xx/cpu.c +++ b/arch/powerpc/cpu/mpc8xx/cpu.c @@ -215,19 +215,12 @@ int do_reset(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[]) /* * Trying to execute the next instruction at a non-existing address * should cause a machine check, resulting in reset - */ -#ifdef CONFIG_SYS_RESET_ADDRESS - addr = CONFIG_SYS_RESET_ADDRESS; -#else - /* + * * note: when CONFIG_SYS_MONITOR_BASE points to a RAM address, * CONFIG_SYS_MONITOR_BASE - sizeof (ulong) is usually a valid address. - * Better pick an address known to be invalid on your system and assign - * it to CONFIG_SYS_RESET_ADDRESS. - * "(ulong)-1" used to be a good choice for many systems... */ addr = CONFIG_SYS_MONITOR_BASE - sizeof(ulong); -#endif + ((void (*)(void)) addr)(); return 1; } diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt index 7b45d578a3..84e46e3763 100644 --- a/scripts/config_whitelist.txt +++ b/scripts/config_whitelist.txt @@ -1837,7 +1837,6 @@ CONFIG_SYS_RCAR_I2C1_BASE CONFIG_SYS_RCAR_I2C2_BASE CONFIG_SYS_RCAR_I2C3_BASE CONFIG_SYS_RESET_ADDR -CONFIG_SYS_RESET_ADDRESS CONFIG_SYS_RFD CONFIG_SYS_RGMII1_PHY_ADDR CONFIG_SYS_RGMII2_PHY_ADDR
There are no boards that define CONFIG_SYS_RESET_ADDRESS, so drop the associated mpc8xx code that checks for it. Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> --- arch/powerpc/cpu/mpc8xx/cpu.c | 11 ++--------- scripts/config_whitelist.txt | 1 - 2 files changed, 2 insertions(+), 10 deletions(-)