Gentoo Archives: gentoo-commits

From: Mike Pagano <mpagano@g.o>
To: gentoo-commits@l.g.o
Subject: [gentoo-commits] proj/linux-patches:5.15 commit in: /
Date: Wed, 27 Apr 2022 13:14:07
Message-Id: 1651065226.822de2a279bf448ce0ff3be7937fbdb8eca43122.mpagano@gentoo
1 commit: 822de2a279bf448ce0ff3be7937fbdb8eca43122
2 Author: Mike Pagano <mpagano <AT> gentoo <DOT> org>
3 AuthorDate: Wed Apr 27 13:13:46 2022 +0000
4 Commit: Mike Pagano <mpagano <AT> gentoo <DOT> org>
5 CommitDate: Wed Apr 27 13:13:46 2022 +0000
6 URL: https://gitweb.gentoo.org/proj/linux-patches.git/commit/?id=822de2a2
7
8 Remove redundant patch
9
10 Removed:
11 2800_gpio-Request-interrupts-after-IRQ-is-initialized.patch
12
13 Signed-off-by: Mike Pagano <mpagano <AT> gentoo.org>
14
15 0000_README | 4 --
16 ...quest-interrupts-after-IRQ-is-initialized.patch | 75 ----------------------
17 2 files changed, 79 deletions(-)
18
19 diff --git a/0000_README b/0000_README
20 index f44dec35..0f44e39b 100644
21 --- a/0000_README
22 +++ b/0000_README
23 @@ -199,10 +199,6 @@ Patch: 2000_BT-Check-key-sizes-only-if-Secure-Simple-Pairing-enabled.patch
24 From: https://lore.kernel.org/linux-bluetooth/20190522070540.48895-1-marcel@××××××××.org/raw
25 Desc: Bluetooth: Check key sizes only when Secure Simple Pairing is enabled. See bug #686758
26
27 -Path: 2800_gpio-Request-interrupts-after-IRQ-is-initialized.patch
28 -From: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/patch/drivers/gpio?id=06fb4ecfeac7e00d6704fa5ed19299f2fefb3cc9
29 -Desc: gpio: Request interrupts after IRQ is initialized
30 -
31 Patch: 2900_tmp513-Fix-build-issue-by-selecting-CONFIG_REG.patch
32 From: https://bugs.gentoo.org/710790
33 Desc: tmp513 requies REGMAP_I2C to build. Select it by default in Kconfig. See bug #710790. Thanks to Phil Stracchino
34
35 diff --git a/2800_gpio-Request-interrupts-after-IRQ-is-initialized.patch b/2800_gpio-Request-interrupts-after-IRQ-is-initialized.patch
36 deleted file mode 100644
37 index 0a1d4624..00000000
38 --- a/2800_gpio-Request-interrupts-after-IRQ-is-initialized.patch
39 +++ /dev/null
40 @@ -1,75 +0,0 @@
41 -From 06fb4ecfeac7e00d6704fa5ed19299f2fefb3cc9 Mon Sep 17 00:00:00 2001
42 -From: Mario Limonciello <mario.limonciello@×××.com>
43 -Date: Fri, 22 Apr 2022 08:14:52 -0500
44 -Subject: gpio: Request interrupts after IRQ is initialized
45 -MIME-Version: 1.0
46 -Content-Type: text/plain; charset=UTF-8
47 -Content-Transfer-Encoding: 8bit
48 -
49 -Commit 5467801f1fcb ("gpio: Restrict usage of GPIO chip irq members
50 -before initialization") attempted to fix a race condition that lead to a
51 -NULL pointer, but in the process caused a regression for _AEI/_EVT
52 -declared GPIOs.
53 -
54 -This manifests in messages showing deferred probing while trying to
55 -allocate IRQs like so:
56 -
57 - amd_gpio AMDI0030:00: Failed to translate GPIO pin 0x0000 to IRQ, err -517
58 - amd_gpio AMDI0030:00: Failed to translate GPIO pin 0x002C to IRQ, err -517
59 - amd_gpio AMDI0030:00: Failed to translate GPIO pin 0x003D to IRQ, err -517
60 - [ .. more of the same .. ]
61 -
62 -The code for walking _AEI doesn't handle deferred probing and so this
63 -leads to non-functional GPIO interrupts.
64 -
65 -Fix this issue by moving the call to `acpi_gpiochip_request_interrupts`
66 -to occur after gc->irc.initialized is set.
67 -
68 -Fixes: 5467801f1fcb ("gpio: Restrict usage of GPIO chip irq members before initialization")
69 -Link: https://lore.kernel.org/linux-gpio/BL1PR12MB51577A77F000A008AA694675E2EF9@×××××××××××××××××××××××××××××××××××.com/
70 -Link: https://bugzilla.suse.com/show_bug.cgi?id=1198697
71 -Link: https://bugzilla.kernel.org/show_bug.cgi?id=215850
72 -Link: https://gitlab.freedesktop.org/drm/amd/-/issues/1979
73 -Link: https://gitlab.freedesktop.org/drm/amd/-/issues/1976
74 -Reported-by: Mario Limonciello <mario.limonciello@×××.com>
75 -Signed-off-by: Mario Limonciello <mario.limonciello@×××.com>
76 -Reviewed-by: Shreeya Patel <shreeya.patel@×××××××××.com>
77 -Tested-By: Samuel Čavoj <samuel@×××××.net>
78 -Tested-By: lukeluk498@×××××.com Link:
79 -Reviewed-by: Andy Shevchenko <andy.shevchenko@×××××.com>
80 -Acked-by: Linus Walleij <linus.walleij@××××××.org>
81 -Reviewed-and-tested-by: Takashi Iwai <tiwai@××××.de>
82 -Cc: Shreeya Patel <shreeya.patel@×××××××××.com>
83 -Cc: stable@×××××××××××.org
84 -Signed-off-by: Linus Torvalds <torvalds@××××××××××××××××.org>
85 ----
86 - drivers/gpio/gpiolib.c | 4 ++--
87 - 1 file changed, 2 insertions(+), 2 deletions(-)
88 -
89 -(limited to 'drivers/gpio')
90 -
91 -diff --git a/drivers/gpio/gpiolib.c b/drivers/gpio/gpiolib.c
92 -index 085348e089860..b7694171655cf 100644
93 ---- a/drivers/gpio/gpiolib.c
94 -+++ b/drivers/gpio/gpiolib.c
95 -@@ -1601,8 +1601,6 @@ static int gpiochip_add_irqchip(struct gpio_chip *gc,
96 -
97 - gpiochip_set_irq_hooks(gc);
98 -
99 -- acpi_gpiochip_request_interrupts(gc);
100 --
101 - /*
102 - * Using barrier() here to prevent compiler from reordering
103 - * gc->irq.initialized before initialization of above
104 -@@ -1612,6 +1610,8 @@ static int gpiochip_add_irqchip(struct gpio_chip *gc,
105 -
106 - gc->irq.initialized = true;
107 -
108 -+ acpi_gpiochip_request_interrupts(gc);
109 -+
110 - return 0;
111 - }
112 -
113 ---
114 -cgit 1.2.3-1.el7
115 -