-----Original Message----- From: Stephen Warren swarren@wwwdotorg.org Sent: Tuesday, March 17, 2020 10:33 AM To: Tom Warren TWarren@nvidia.com 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, twarren@nvidia.com wrote:
From: Tom Warren twarren@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@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. -----------------------------------------------------------------------------------