[PATCH v2 07/13] env: Inline env_get_char() into it's only user
Simon Glass
sjg at chromium.org
Thu Oct 14 17:11:13 CEST 2021
On Wed, 13 Oct 2021 at 09:46, Marek Behún <kabel at kernel.org> wrote:
>
> From: Marek Behún <marek.behun at nic.cz>
>
> This function is a relic from the past when environment was read from
> underlying device one character at a time.
>
> It is used only in the case when getting an environemnt variable prior
> relocation, and the function is simple enough to be inlined there.
>
> Since env_get_char() is being changed to simple access to an array, we
> can drop the failing cases and simplify the code (this could have been
> done before, since env_get_char() did not fail even before).
>
> Signed-off-by: Marek Behún <marek.behun at nic.cz>
> ---
> cmd/nvedit.c | 28 ++++++++++++++--------------
> env/env.c | 8 --------
> env/nowhere.c | 5 ++---
> include/env.h | 10 ----------
> 4 files changed, 16 insertions(+), 35 deletions(-)
>
in subject: typo it's should be its
Reviewed-by: Simon Glass <sjg at chromium.org>
More information about the U-Boot
mailing list