[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