[U-Boot] [PATCH v2 1/1] efi_selftest: make tests easier to read

Simon Glass sjg at chromium.org
Mon Oct 9 04:41:56 UTC 2017


Hi Heinrich,

On 4 October 2017 at 07:31, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
> Rename counter to more illustrative names.
> Update notification function description.
> Simplify notification function.
> Add comment for arbitrary non-zero value.
> Document @return.
> Use constants for return values of setup, execute, teardown.
>
> Reported-by: Simon Glass <sjg at chromium.org>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
> v2:
>         Remove superfluous blank in a message text.
> ---
>  include/efi_selftest.h                           |  3 +
>  lib/efi_selftest/efi_selftest.c                  | 15 +++--
>  lib/efi_selftest/efi_selftest_events.c           | 80 ++++++++++++----------
>  lib/efi_selftest/efi_selftest_exitbootservices.c | 46 +++++++------
>  lib/efi_selftest/efi_selftest_tpl.c              | 85 +++++++++++++-----------
>  5 files changed, 129 insertions(+), 100 deletions(-)

Reviewed-by: Simon Glass <sjg at chromium.org>

But I am not that keen on returning SUCCESS instead of 0. I think 0 is
well understood and easier on the eye.

Regards,
Simon


More information about the U-Boot mailing list