From mboxrd@z Thu Jan  1 00:00:00 1970
From: Tom Rini <trini@konsulko.com>
Date: Wed, 26 Jul 2017 12:13:56 -0400
Subject: [U-Boot] [PATCH v2 16/34] env: common: Factor out the common
 env_valid check
In-Reply-To: <20170724032009.43994-17-sjg@chromium.org>
References: <20170724032009.43994-1-sjg@chromium.org>
 <20170724032009.43994-17-sjg@chromium.org>
Message-ID: <20170726161356.GO26163@bill-the-cat>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On Sun, Jul 23, 2017 at 09:19:51PM -0600, Simon Glass wrote:

> The check for gd->env_valid is used in both the 'if' and 'else' part of
> env_get_char(). Move it into that function instead for simplicity. Drop
> that code from the two leaf functions.
> 
> Signed-off-by: Simon Glass <sjg@chromium.org>

Reviewed-by: Tom Rini <trini@konsulko.com>

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20170726/b114329f/attachment.sig>