Message ID | f0ab5a90fc459633aa3da098c272f52d532abbe3.1449245152.git.michal.simek@xilinx.com |
---|---|
State | Accepted |
Delegated to: | Simon Glass |
Headers | show |
On 4 December 2015 at 09:05, Michal Simek <michal.simek@xilinx.com> wrote: > Subcommands contain pointers to functions which are not updated when > MANUAL_REALOC is enabled. This patch fix it. > > Signed-off-by: Michal Simek <michal.simek@xilinx.com> > --- > > common/cmd_i2c.c | 16 ++++++++++++---- > 1 file changed, 12 insertions(+), 4 deletions(-) Reviewed-by: Simon Glass <sjg@chromium.org>
Hello Michal, Am 04.12.2015 um 17:05 schrieb Michal Simek: > Subcommands contain pointers to functions which are not updated when > MANUAL_REALOC is enabled. This patch fix it. > > Signed-off-by: Michal Simek <michal.simek@xilinx.com> > --- > > common/cmd_i2c.c | 16 ++++++++++++---- > 1 file changed, 12 insertions(+), 4 deletions(-) Acked-by: Heiko Schocher <hs@denx.de> bye, Heiko > > diff --git a/common/cmd_i2c.c b/common/cmd_i2c.c > index 3d0de81c32f8..552c875f62dd 100644 > --- a/common/cmd_i2c.c > +++ b/common/cmd_i2c.c > @@ -1944,11 +1944,15 @@ static cmd_tbl_t cmd_i2c_sub[] = { > U_BOOT_CMD_MKENT(speed, 1, 1, do_i2c_bus_speed, "", ""), > }; > > -#ifdef CONFIG_NEEDS_MANUAL_RELOC > -void i2c_reloc(void) { > - fixup_cmdtable(cmd_i2c_sub, ARRAY_SIZE(cmd_i2c_sub)); > +static __maybe_unused void i2c_reloc(void) > +{ > + static int relocated; > + > + if (!relocated) { > + fixup_cmdtable(cmd_i2c_sub, ARRAY_SIZE(cmd_i2c_sub)); > + relocated = 1; > + }; > } > -#endif > > /** > * do_i2c() - Handle the "i2c" command-line command > @@ -1964,6 +1968,10 @@ static int do_i2c(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) > { > cmd_tbl_t *c; > > +#ifdef CONFIG_NEEDS_MANUAL_RELOC > + i2c_reloc(); > +#endif > + > if (argc < 2) > return CMD_RET_USAGE; > >
diff --git a/common/cmd_i2c.c b/common/cmd_i2c.c index 3d0de81c32f8..552c875f62dd 100644 --- a/common/cmd_i2c.c +++ b/common/cmd_i2c.c @@ -1944,11 +1944,15 @@ static cmd_tbl_t cmd_i2c_sub[] = { U_BOOT_CMD_MKENT(speed, 1, 1, do_i2c_bus_speed, "", ""), }; -#ifdef CONFIG_NEEDS_MANUAL_RELOC -void i2c_reloc(void) { - fixup_cmdtable(cmd_i2c_sub, ARRAY_SIZE(cmd_i2c_sub)); +static __maybe_unused void i2c_reloc(void) +{ + static int relocated; + + if (!relocated) { + fixup_cmdtable(cmd_i2c_sub, ARRAY_SIZE(cmd_i2c_sub)); + relocated = 1; + }; } -#endif /** * do_i2c() - Handle the "i2c" command-line command @@ -1964,6 +1968,10 @@ static int do_i2c(cmd_tbl_t * cmdtp, int flag, int argc, char * const argv[]) { cmd_tbl_t *c; +#ifdef CONFIG_NEEDS_MANUAL_RELOC + i2c_reloc(); +#endif + if (argc < 2) return CMD_RET_USAGE;
Subcommands contain pointers to functions which are not updated when MANUAL_REALOC is enabled. This patch fix it. Signed-off-by: Michal Simek <michal.simek@xilinx.com> --- common/cmd_i2c.c | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-)