Re: [PATCH][5.15.y] gpio: pca953x: fix pca953x_irq_bus_sync_unlock race

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



[ Sasha's backport helper bot ]

Hi,

The upstream commit SHA1 provided is correct: bfc6444b57dc7186b6acc964705d7516cbaf3904

WARNING: Author mismatch between patch and upstream commit:
Backport author: guocai.he.cn@xxxxxxxxxxxxx
Commit author: Ian Ray <ian.ray@xxxxxxxxxxxxxxxx>


Status in newer kernel trees:
6.12.y | Present (exact SHA1)
6.6.y | Present (different SHA1: e2ecdddca80d)
6.1.y | Present (different SHA1: 58a5c93bd1a6)
5.15.y | Not found

Note: The patch differs from the upstream commit:
---
1:  bfc6444b57dc7 ! 1:  3812c0bc93e5e gpio: pca953x: fix pca953x_irq_bus_sync_unlock race
    @@ Metadata
      ## Commit message ##
         gpio: pca953x: fix pca953x_irq_bus_sync_unlock race
     
    +    [ Upstream commit bfc6444b57dc7186b6acc964705d7516cbaf3904 ]
    +
         Ensure that `i2c_lock' is held when setting interrupt latch and mask in
         pca953x_irq_bus_sync_unlock() in order to avoid races.
     
    @@ Commit message
         Signed-off-by: Ian Ray <ian.ray@xxxxxxxxxxxxxxxx>
         Link: https://lore.kernel.org/r/20240620042915.2173-1-ian.ray@xxxxxxxxxxxxxxxx
         Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
    +    Signed-off-by: Guocai He <guocai.he.cn@xxxxxxxxxxxxx>
     
      ## drivers/gpio/gpio-pca953x.c ##
     @@ drivers/gpio/gpio-pca953x.c: static void pca953x_irq_bus_sync_unlock(struct irq_data *d)
---

Results of testing on various branches:

| Branch                    | Patch Apply | Build Test |
|---------------------------|-------------|------------|
| stable/linux-5.15.y       |  Success    |  Failed    |

Build Errors:
Build error for stable/linux-5.15.y:
    drivers/gpio/gpio-pca953x.c: In function 'pca953x_irq_bus_sync_unlock':
    drivers/gpio/gpio-pca953x.c:675:17: error: implicit declaration of function 'guard' [-Werror=implicit-function-declaration]
      675 |                 guard(mutex)(&chip->i2c_lock);
          |                 ^~~~~
    drivers/gpio/gpio-pca953x.c:675:23: error: 'mutex' undeclared (first use in this function)
      675 |                 guard(mutex)(&chip->i2c_lock);
          |                       ^~~~~
    drivers/gpio/gpio-pca953x.c:675:23: note: each undeclared identifier is reported only once for each function it appears in
    cc1: some warnings being treated as errors
    make[2]: *** [scripts/Makefile.build:289: drivers/gpio/gpio-pca953x.o] Error 1
    make[2]: Target '__build' not remade because of errors.
    make[1]: *** [scripts/Makefile.build:552: drivers/gpio] Error 2
    make[1]: Target '__build' not remade because of errors.
    make: *** [Makefile:1906: drivers] Error 2
    make: Target '__all' not remade because of errors.




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux