
23 Dec
2020
23 Dec
'20
9:15 a.m.
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@windriver.com
Applied to u-boot-ti/for-next
Thanks and regards, Lokesh