Re: [PATCH v2 4/4] net: dsa: vsc73xx: Assert reset if iCPU is enabled

[Florian Fainelli]

On 7/3/2019 10:19 AM, Pawel Dembicki wrote:
> Driver allow to use devices with disabled iCPU only.
> 
> Some devices have pre-initialised iCPU by bootloader.
> That state make switch unmanaged. This patch force reset
> if device is in unmanaged state. In the result chip lost
> internal firmware from RAM and it can be managed.
> 
> Signed-off-by: Pawel Dembicki <paweldembi...@gmail.com>
> ---

[snip]

> @@ -1158,6 +1143,19 @@ int vsc73xx_probe(struct vsc73xx *vsc)
>               msleep(20);
>  
>       ret = vsc73xx_detect(vsc);
> +     if (ret == -EAGAIN) {
> +             dev_err(vsc->dev,
> +                     "Chip seams to be out of control. Assert reset and try 
> again.\n");
> +             gpiod_set_value_cansleep(vsc->reset, 1);

s/seams/seems/

With that fixed:

Reviewed-by: Florian Fainelli <f.faine...@gmail.com>
-- 
Florian