[U-Boot] [WORKING PATCH 1/2] sandbox: Use defconfig to enable features

Simon Glass sjg at chromium.org
Thu May 7 00:00:28 CEST 2015


Hi Joe,

On 6 May 2015 at 11:54, Joe Hershberger <joe.hershberger at ni.com> wrote:
> Stop using the sandbox arch Kconfig to override defaults for config
> options. This is a bit of abuse and may be causing build problems.
>
> Signed-off-by: Joe Hershberger <joe.hershberger at ni.com>
> ---
> The errors you are seeing are the result of "CONFIG_UNIT_TEST" not being
> defined. That should not be at all related to that test/dm/eth.c patch.
> This is changing the way that CONFIG_UNIT_TEST is enabled to be more
> typical in case there is some bad behavior with all this override of
> defaults. I have seen some non-ideal behavior, so I want to move away
> from using it.
> Please apply this on top of u-boot-dm/env-working and try it.
>
>  arch/sandbox/Kconfig      | 18 ------------------
>  configs/sandbox_defconfig |  5 +++++
>  2 files changed, 5 insertions(+), 18 deletions(-)

I should have mentioned that the errors are fixed by a later patch:

61: test: dm: Move the dm tests over to the ut command

So should I merge this change into one of your existing patches for testing?

BTW this series does have some driver model things but most of it is
not, so perhaps Tom plans to apply it?

Regards,
Simon


More information about the U-Boot mailing list