[PATCH] spi: ti_qspi: Fix "spi-max-frequency" error path in ti_qspi_ofdata_to_platdata

Lokesh Vutla lokeshvutla at ti.com
Wed Dec 23 09:15:25 CET 2020



On 28/11/20 1:41 pm, Ovidiu Panait wrote:
> struct ti_qspi_priv->max_hz is declared as unsigned int, so the following
> error path check will always be false, even when "spi-max-frequency"
> property is invalid/missing:
>   priv->max_hz = fdtdec_get_int(blob, node, "spi-max-frequency", -1);
>   if (priv->max_hz < 0) {
>     ...
>   }
> 
> Replace the fdtdec call with dev_read_u32_default() and use 0 as the
> default value. Error out if max_hz is zero.
> 
> Signed-off-by: Ovidiu Panait <ovidiu.panait at windriver.com>


Applied to u-boot-ti/for-next

Thanks and regards,
Lokesh



More information about the U-Boot mailing list