[U-Boot] [PATCH] dm: core: Widen the dump tree to show more of the driver's name.

Simon Glass sjg at chromium.org
Tue Oct 2 11:22:25 UTC 2018


Hi Liviu,

On 2 October 2018 at 03:46, Liviu Dudau <Liviu.Dudau at foss.arm.com> wrote:
> On Fri, Sep 28, 2018 at 04:14:12PM -0600, Simon Glass wrote:
>> Hi,
>
> Hi Simon,
>
>>
>> On 17 September 2018 at 20:06, Simon Glass <sjg at chromium.org> wrote:
>> > On 17 September 2018 at 10:57, Liviu Dudau <Liviu.Dudau at foss.arm.com> wrote:
>> >> With drivers that have prefix names that are quite long (like
>> >> 'versatile_') it is useful to have a wider column for the driver's
>> >> name when dumping the device driver tree.
>> >>
>> >> Signed-off-by: Liviu Dudau <liviu.dudau at foss.arm.com>
>> >> ---
>> >>  drivers/core/dump.c | 8 ++++----
>> >>  1 file changed, 4 insertions(+), 4 deletions(-)
>> >
>> > Reviewed-by: Simon Glass <sjg at chromium.org>
>>
>> Unfortunately this seems to break some tests (make tests). Please can
>> you take a look?
>
> I've had a go at trying to run 'make tests' on my machine but things
> fail miserably for me. Attached is the output of the command. U-Boot is
> v2018.11-rc1 plus my remaining patches that have not yet been merged.
>
> It looks to me like the 'sandbox' board (or defconfig) needs some
> updating? Please advise!

I cannot explain your build log - it looks completely wrong.

Can you try:

make O=sandbox sandbox_defconfig
make O=sandbox

?

Maybe try 'make mrproper' first?

But still, the errors seems to make no sense. E.g.
CONFIG_SANDBOX_SPI_MAX_BUS is defined in a config file.

Also what gcc version are you using on your computer?

Regards,
Simon


More information about the U-Boot mailing list