Message ID | 20220131072022.440711-2-ovidiu.panait@windriver.com |
---|---|
State | Accepted |
Commit | 1ca890d789b45ddcda73454fe2b54f5fdb10ee3b |
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_ADDR, so drop the > remaining comments referencing it and also the config_whitelist.txt entry. > > Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan > --- > > include/configs/10m50_devboard.h | 9 --------- > include/configs/3c120_devboard.h | 9 --------- > scripts/config_whitelist.txt | 1 - > 3 files changed, 19 deletions(-) > > diff --git a/include/configs/10m50_devboard.h b/include/configs/10m50_devboard.h > index 04ce88c9dd..3b4d1fd626 100644 > --- a/include/configs/10m50_devboard.h > +++ b/include/configs/10m50_devboard.h > @@ -47,13 +47,4 @@ > CONFIG_SYS_SDRAM_SIZE - \ > CONFIG_SYS_MONITOR_LEN) > > -/* > - * ENVIRONMENT -- Put environment in sector CONFIG_SYS_MONITOR_LEN above > - * CONFIG_SYS_RESET_ADDR, since we assume the monitor is stored at the > - * reset address, no? This will keep the environment in user region > - * of flash. NOTE: the monitor length must be multiple of sector size > - * (which is common practice). > - */ > - > - > #endif /* __CONFIG_H */ > diff --git a/include/configs/3c120_devboard.h b/include/configs/3c120_devboard.h > index e12e54fe4f..763cb8db7c 100644 > --- a/include/configs/3c120_devboard.h > +++ b/include/configs/3c120_devboard.h > @@ -47,13 +47,4 @@ > CONFIG_SYS_SDRAM_SIZE - \ > CONFIG_SYS_MONITOR_LEN) > > -/* > - * ENVIRONMENT -- Put environment in sector CONFIG_SYS_MONITOR_LEN above > - * CONFIG_SYS_RESET_ADDR, since we assume the monitor is stored at the > - * reset address, no? This will keep the environment in user region > - * of flash. NOTE: the monitor length must be multiple of sector size > - * (which is common practice). > - */ > - > - > #endif /* __CONFIG_H */ > diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt > index 84e46e3763..4bd298694a 100644 > --- a/scripts/config_whitelist.txt > +++ b/scripts/config_whitelist.txt > @@ -1836,7 +1836,6 @@ CONFIG_SYS_RCAR_I2C0_BASE > CONFIG_SYS_RCAR_I2C1_BASE > CONFIG_SYS_RCAR_I2C2_BASE > CONFIG_SYS_RCAR_I2C3_BASE > -CONFIG_SYS_RESET_ADDR > CONFIG_SYS_RFD > CONFIG_SYS_RGMII1_PHY_ADDR > CONFIG_SYS_RGMII2_PHY_ADDR Viele Grüße, Stefan Roese
On 2022-01-31 07:20, Ovidiu Panait wrote: > There are no boards that define CONFIG_SYS_RESET_ADDR, so drop the > remaining comments referencing it and also the config_whitelist.txt > entry. > > Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> > --- > > include/configs/10m50_devboard.h | 9 --------- > include/configs/3c120_devboard.h | 9 --------- > scripts/config_whitelist.txt | 1 - > 3 files changed, 19 deletions(-) > Acked-by: thomas@wytron.com.tw Thanks, Thomas Chou > diff --git a/include/configs/10m50_devboard.h > b/include/configs/10m50_devboard.h > index 04ce88c9dd..3b4d1fd626 100644 > --- a/include/configs/10m50_devboard.h > +++ b/include/configs/10m50_devboard.h > @@ -47,13 +47,4 @@ > CONFIG_SYS_SDRAM_SIZE - \ > CONFIG_SYS_MONITOR_LEN) > > -/* > - * ENVIRONMENT -- Put environment in sector CONFIG_SYS_MONITOR_LEN > above > - * CONFIG_SYS_RESET_ADDR, since we assume the monitor is stored at the > - * reset address, no? This will keep the environment in user region > - * of flash. NOTE: the monitor length must be multiple of sector size > - * (which is common practice). > - */ > - > - > #endif /* __CONFIG_H */ > diff --git a/include/configs/3c120_devboard.h > b/include/configs/3c120_devboard.h > index e12e54fe4f..763cb8db7c 100644 > --- a/include/configs/3c120_devboard.h > +++ b/include/configs/3c120_devboard.h > @@ -47,13 +47,4 @@ > CONFIG_SYS_SDRAM_SIZE - \ > CONFIG_SYS_MONITOR_LEN) > > -/* > - * ENVIRONMENT -- Put environment in sector CONFIG_SYS_MONITOR_LEN > above > - * CONFIG_SYS_RESET_ADDR, since we assume the monitor is stored at the > - * reset address, no? This will keep the environment in user region > - * of flash. NOTE: the monitor length must be multiple of sector size > - * (which is common practice). > - */ > - > - > #endif /* __CONFIG_H */ > diff --git a/scripts/config_whitelist.txt > b/scripts/config_whitelist.txt > index 84e46e3763..4bd298694a 100644 > --- a/scripts/config_whitelist.txt > +++ b/scripts/config_whitelist.txt > @@ -1836,7 +1836,6 @@ CONFIG_SYS_RCAR_I2C0_BASE > CONFIG_SYS_RCAR_I2C1_BASE > CONFIG_SYS_RCAR_I2C2_BASE > CONFIG_SYS_RCAR_I2C3_BASE > -CONFIG_SYS_RESET_ADDR > CONFIG_SYS_RFD > CONFIG_SYS_RGMII1_PHY_ADDR > CONFIG_SYS_RGMII2_PHY_ADDR
On Mon, Jan 31, 2022 at 09:20:22AM +0200, Ovidiu Panait wrote: > There are no boards that define CONFIG_SYS_RESET_ADDR, so drop the > remaining comments referencing it and also the config_whitelist.txt entry. > > Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> > Reviewed-by: Stefan Roese <sr@denx.de> > Acked-by: thomas@wytron.com.tw Applied to u-boot/master, thanks!
diff --git a/include/configs/10m50_devboard.h b/include/configs/10m50_devboard.h index 04ce88c9dd..3b4d1fd626 100644 --- a/include/configs/10m50_devboard.h +++ b/include/configs/10m50_devboard.h @@ -47,13 +47,4 @@ CONFIG_SYS_SDRAM_SIZE - \ CONFIG_SYS_MONITOR_LEN) -/* - * ENVIRONMENT -- Put environment in sector CONFIG_SYS_MONITOR_LEN above - * CONFIG_SYS_RESET_ADDR, since we assume the monitor is stored at the - * reset address, no? This will keep the environment in user region - * of flash. NOTE: the monitor length must be multiple of sector size - * (which is common practice). - */ - - #endif /* __CONFIG_H */ diff --git a/include/configs/3c120_devboard.h b/include/configs/3c120_devboard.h index e12e54fe4f..763cb8db7c 100644 --- a/include/configs/3c120_devboard.h +++ b/include/configs/3c120_devboard.h @@ -47,13 +47,4 @@ CONFIG_SYS_SDRAM_SIZE - \ CONFIG_SYS_MONITOR_LEN) -/* - * ENVIRONMENT -- Put environment in sector CONFIG_SYS_MONITOR_LEN above - * CONFIG_SYS_RESET_ADDR, since we assume the monitor is stored at the - * reset address, no? This will keep the environment in user region - * of flash. NOTE: the monitor length must be multiple of sector size - * (which is common practice). - */ - - #endif /* __CONFIG_H */ diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt index 84e46e3763..4bd298694a 100644 --- a/scripts/config_whitelist.txt +++ b/scripts/config_whitelist.txt @@ -1836,7 +1836,6 @@ CONFIG_SYS_RCAR_I2C0_BASE CONFIG_SYS_RCAR_I2C1_BASE CONFIG_SYS_RCAR_I2C2_BASE CONFIG_SYS_RCAR_I2C3_BASE -CONFIG_SYS_RESET_ADDR CONFIG_SYS_RFD CONFIG_SYS_RGMII1_PHY_ADDR CONFIG_SYS_RGMII2_PHY_ADDR
There are no boards that define CONFIG_SYS_RESET_ADDR, so drop the remaining comments referencing it and also the config_whitelist.txt entry. Signed-off-by: Ovidiu Panait <ovidiu.panait@windriver.com> --- include/configs/10m50_devboard.h | 9 --------- include/configs/3c120_devboard.h | 9 --------- scripts/config_whitelist.txt | 1 - 3 files changed, 19 deletions(-)