-Original Message-
From: Stephen Warren
Sent: Tuesday, March 17, 2020 10:33 AM
To: Tom Warren
Cc: u-boot@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, twar...@nvidia.com wrote:
> From: Tom Warren
>
> '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
(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.
---
