[PATCH] cmd: sf: Fix sf probe crash

[Heinrich Schuchardt]

On 04.03.24 15:47, Weizhao Ouyang wrote:
> Gentle ping. Not merged yet.
>
> BR,
> Weizhao

Hello Dario,

that patch is in your patchwork review queue. Could you, please, have a
look.

Best regards

Heinrich

>
> On Thu, Jan 4, 2024 at 7:46 PM Weizhao Ouyang <o451686892 at gmail.com> wrote:
>>
>> Handle the return value of spi_flash_probe_bus_cs() to avoid sf probe
>> crashes.
>>
>> Signed-off-by: Weizhao Ouyang <o451686892 at gmail.com>
>> ---
>>   cmd/sf.c | 5 +++--
>>   1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/cmd/sf.c b/cmd/sf.c
>> index 730996c02b..e3866899f6 100644
>> --- a/cmd/sf.c
>> +++ b/cmd/sf.c
>> @@ -135,8 +135,9 @@ static int do_spi_flash_probe(int argc, char *const argv[])
>>          }
>>          flash = NULL;
>>          if (use_dt) {
>> -               spi_flash_probe_bus_cs(bus, cs, &new);
>> -               flash = dev_get_uclass_priv(new);
>> +               ret = spi_flash_probe_bus_cs(bus, cs, &new);
>> +               if (!ret)
>> +                       flash = dev_get_uclass_priv(new);
>>          } else {
>>                  flash = spi_flash_probe(bus, cs, speed, mode);
>>          }
>> --
>> 2.39.2
>>