[PATCH] spi: exynos: Remove extra term from test
Andrew Goodbody
andrew.goodbody at linaro.org
Mon Sep 1 17:08:48 CEST 2025
On 25/08/2025 03:17, Minkyu Kang wrote:
> Hi,
>
>
> 2025년 8월 12일 (화) 21:01, Andrew Goodbody <andrew.goodbody at linaro.org
> <mailto:andrew.goodbody at linaro.org>>님이 작성:
>
> In spi_rx_tx there comes a test for execution of a code block that
> allows execution if rxp is not NULL or stopping is true. However all the
> code in this block relies on rxp being valid so allowing entry just if
> stopping is true does not make sense. So remove this from the test
> expression leaving just a NULL check for rxp.
>
> This issue was found by Smatch.
>
> Signed-off-by: Andrew Goodbody <andrew.goodbody at linaro.org
> <mailto:andrew.goodbody at linaro.org>>
> ---
> drivers/spi/exynos_spi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/spi/exynos_spi.c b/drivers/spi/exynos_spi.c
> index 1b9bf004b7c..d11952afe5d 100644
> --- a/drivers/spi/exynos_spi.c
> +++ b/drivers/spi/exynos_spi.c
> @@ -166,7 +166,7 @@ static int spi_rx_tx(struct exynos_spi_priv
> *priv, int todo,
> stopping = 0;
> }
> } else {
> - if (rxp || stopping) {
> + if (rxp) {
>
>
> then, isn't the stopping variable no longer necessary?
Yes indeed, thanks for spotting that. V2 coming.
Andrew
>
> if (step == 4)
> *(uint32_t
> *)rxp = temp;
> else
>
> ---
> base-commit: 7807ed921314cd7af83fd88162d0b8c6fb20a9ca
> change-id: 20250812-exynos_spi-8172ecb31a83
>
> Best regards,
> --
> Andrew Goodbody <andrew.goodbody at linaro.org
> <mailto:andrew.goodbody at linaro.org>>
>
>
> Thanks.
> Minkyu Kang.
More information about the U-Boot
mailing list