NACK/Cmnt: [SRU v1][F:linux-bluefield][Pull Request] Sync up gpio interrupt handling with upstreamed version

Stefan Bader stefan.bader at canonical.com
Tue Mar 22 07:45:49 UTC 2022


On 16.03.22 15:32, Asmaa Mnebhi wrote:
> This pull request (19 commits) includes reverting 6 gpio-mlxbf2 commits in
> linux-bluefield. Then it cherry-picks the commits made to gpio-mlxbf2 in the
> linux master branch. commit c8382b7415913d30f22cdfc28badc5e5de812f45
> introduces proper IRQ handling in the gpio-mlxbf2 driver.
> On top of the above, there are 2 commits:
> one which adds the gpio-mlxbf2 driver version and the second one creates
> the pwr-mlxbf driver which is responsible of handling the low power mode
> and reset interrupts.
> 
> BugLink: https://bugs.launchpad.net/bugs/1922393
> 
> This pull request page can be found at:
> 
> https://code.launchpad.net/~asmaam/ubuntu/+source/linux-bluefield/+git/version-seeds/+merge/416960
> 
> Git repo can be found at:
> 
> git+ssh://asmaam@git.launchpad.net/~asmaam/ubuntu/+source/linux-bluefield master-next-gpio-interrupts
> 
> for you to fetch changes starting from commit 281a55483e5033dd5f6a55580bdee24e4c264758
> 
>    Revert "UBUNTU: SAUCE: gpio-mlxbf2.c: Fix setting the gpio direction
>      to output"
> 
> Signed-off-by: Asmaa Mnebhi <asmaa at nvidia.com>
> 
> Andy Shevchenko (4):
>    gpio: mlxbf2: Convert to device PM ops
>    gpio: mlxbf2: Drop wrong use of ACPI_PTR()
>    gpio: mlxbf2: Use devm_platform_ioremap_resource()
>    gpio: mlxbf2: Use DEFINE_RES_MEM_NAMED() helper macro
> 
> Asmaa Mnebhi (10):
>    Revert "UBUNTU: SAUCE: gpio-mlxbf2.c: Fix setting the gpio direction
>      to output"
>    Revert "UBUNTU: SAUCE: gpio-mlxbf2.c: remove phy interrupt"
>    Revert "UBUNTU: SAUCE: gpio-mlxbf2: Cleanup and use generic
>      gpio_irq_chip struct"
>    Revert "UBUNTU: SAUCE: gpio-mlxbf2.c: Support soft reset gpio
>      interrupt"
>    Revert "UBUNTU: SAUCE: gpio-mlxbf2.c: fix spinlock bug and using
>      uninitialized work"
>    Revert "UBUNTU: SAUCE: gpio: Add irq support for gpio-mlxbf2"
>    gpio: mlxbf2.c: Add check for bgpio_init failure
>    gpio: mlxbf2: Introduce IRQ support
>    UBUNTU: SAUCE: Add driver version and fix SPDX-License_Identifier
>    UBUNTU: SAUCE: Add power driver to handle reset interrupt and low
>      power mode interrupt
> 
> Axel Lin (1):
>    gpio: mlxbf2: Fix sleeping while holding spinlock
> 
> Lee Jones (2):
>    gpio: gpio-mlxbf2: Tell the compiler that ACPI functions may not be
>      use
>    gpio: gpio-mlxbf2.c: Provide __releases() annotation to stop confusing
>      Sparse
> 
> Wei Yongjun (1):
>    gpio: mlxbf2: fix return value check in mlxbf2_gpio_get_lock_res()
> 
> YueHaibing (1):
>    gpio: mlxbf2: remove unused including <linux/version.h>
> 
>   drivers/gpio/gpio-mlxbf2.c      | 380 ++++++++------------------------
>   drivers/power/reset/Kconfig     |   6 +
>   drivers/power/reset/Makefile    |   1 +
>   drivers/power/reset/pwr-mlxbf.c | 101 +++++++++
>   4 files changed, 206 insertions(+), 282 deletions(-)
>   create mode 100644 drivers/power/reset/pwr-mlxbf.c
> 

Patch #5 (Revert "UBUNTU: SAUCE: gpio: Add irq support for gpio-mlxbf2") does 
not apply to current master-next (also can not be cherry-picked).
Also for v2 you should fix the bug reference in the cover email to be
https://bugs.launchpad.net/bugs/1965017.

-Stefan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20220322/baaf6283/attachment.sig>


More information about the kernel-team mailing list