Djam/u-boot
1
0
Fork 0
mirror of https://github.com/u-boot/u-boot.git synced 2025-04-26 07:17:10 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

env: Drop env_get_char_spec() and old, unused .get_char() implementations

Browse source 
Commit b2cdef4861 ("env: restore old env_get_char() behaviour")
dropped the .get_char() method from struct env_driver, but left the two
existing implementations (eeprom and nvram) in case someone would use
them by overwriting weak function env_get_char_spec().

Since this was never done in the 3.5 years, let's drop these methods and
simplify the code.

Signed-off-by: Marek Behún <marek.behun@nic.cz>
Reviewed-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Marek Behún 2021-10-17 17:36:27 +02:00 • committed by Simon Glass
parent 6dc1e2f10c
commit dd1c5a7f80
3 changed files with 1 additions and 38 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
env/eeprom.c vendored
View file

@ -64,24 +64,6 @@ static int eeprom_bus_write(unsigned dev_addr, unsigned offset,
return rcode; return rcode;
} }
/** Call this function from overridden env_get_char_spec() if you need
* this functionality.
*/
int env_eeprom_get_char(int index)
{
uchar c;
unsigned int off = CONFIG_ENV_OFFSET;
#ifdef CONFIG_ENV_OFFSET_REDUND
if (gd->env_valid == ENV_REDUND)
off = CONFIG_ENV_OFFSET_REDUND;
#endif
eeprom_bus_read(CONFIG_SYS_I2C_EEPROM_ADDR,
off + index + offsetof(env_t, data), &c, 1);
return c;
}
static int env_eeprom_load(void) static int env_eeprom_load(void)
{ {
char buf_env[CONFIG_ENV_SIZE]; char buf_env[CONFIG_ENV_SIZE];

7
env/env.c vendored
View file

@ -166,17 +166,12 @@ static struct env_driver *env_driver_lookup(enum env_operation op, int prio)
return drv; return drv;
} }
__weak int env_get_char_spec(int index)
{
return *(uchar *)(gd->env_addr + index);
}
int env_get_char(int index) int env_get_char(int index)
{ {
if (gd->env_valid == ENV_INVALID) if (gd->env_valid == ENV_INVALID)
return default_environment[index]; return default_environment[index];
else else
return env_get_char_spec(index); return *(uchar *)(gd->env_addr + index);
} }
int env_load(void) int env_load(void)

14
env/nvram.c vendored
View file

@ -42,20 +42,6 @@ extern void nvram_write(long dest, const void *src, size_t count);
static env_t *env_ptr = (env_t *)CONFIG_ENV_ADDR; static env_t *env_ptr = (env_t *)CONFIG_ENV_ADDR;
#endif #endif
#ifdef CONFIG_SYS_NVRAM_ACCESS_ROUTINE
/** Call this function from overridden env_get_char_spec() if you need
* this functionality.
*/
int env_nvram_get_char(int index)
{
uchar c;
nvram_read(&c, CONFIG_ENV_ADDR + index, 1);
return c;
}
#endif
static int env_nvram_load(void) static int env_nvram_load(void)
{ {
char buf[CONFIG_ENV_SIZE]; char buf[CONFIG_ENV_SIZE];