[PATCH 4/5] fdt: Fix 'system' command

Tom Warren TWarren at nvidia.com
Tue Mar 17 18:46:04 CET 2020

-----Original Message-----
From: Stephen Warren <swarren at wwwdotorg.org> 
Sent: Tuesday, March 17, 2020 10:33 AM
To: Tom Warren <TWarren at nvidia.com>
Cc: u-boot at lists.denx.de
Subject: Re: [PATCH 4/5] fdt: Fix 'system' command

External email: Use caution opening links or attachments

On 3/16/20 1:40 PM, twarren at nvidia.com wrote:
> From: Tom Warren <twarren at nvidia.com>
> 'fdt systemsetup' wasn't working, due to the fact that the 'set' 
> command was being parsed in do_fdt() by only testing for the leading 
> 's' instead of "se", which kept the "sys" test further down from 
> executing. Changed to test for "se" instead, now 'fdt systemsetup' 
> works (to test the ft_system_setup proc w/o having to boot a kernel).

Reviewed-by: Stephen Warren <swarren at nvidia.com>

(Although I wonder if this shouldn't just test the entire string to avoid any possible future ambiguity if more sub-commands are added later. Or doesn't U-Boot already have some sub-command table infra-structure this could use?)
[Tom] I usually take the path of least rework approach, so I just fixed the systemsetup command, which is what I was interested in. Let's see if any other Denxians chime in w/their opinion, and I can rework it if needed.

This email message is for the sole use of the intended recipient(s) and may contain
confidential information.  Any unauthorized review, use, disclosure or distribution
is prohibited.  If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.

More information about the U-Boot mailing list