On Mon, Jul 29, 2019 at 01:49:54PM -0700, Brian Norris wrote:
Commit daaef255dc96 ("driver: platform: Support parsing GpioInt 0 in platform_get_irq()") broke the Embedded Controller driver on most LPC Chromebooks (i.e., most x86 Chromebooks), because cros_ec_lpc expects platform_get_irq() to return -ENXIO for non-existent IRQs. Unfortunately, acpi_dev_gpio_irq_get() doesn't follow this convention and returns -ENOENT instead. So we get this error from cros_ec_lpc:
couldn't retrieve IRQ number (-2)
I see a variety of drivers that treat -ENXIO specially, so rather than fix all of them, let's fix up the API to restore its previous behavior.
I reported this on v2 of this patch:
https://lore.kernel.org/lkml/20190220180538.GA42642@google.com/
but apparently the patch had already been merged before v3 got sent out:
https://lore.kernel.org/lkml/20190221193429.161300-1-egranata@chromium.org/
and the result is that the bug landed and remains unfixed.
I differ from the v3 patch by:
- allowing for ret==0, even though acpi_dev_gpio_irq_get() specifically documents (and enforces) that 0 is not a valid return value (noted on the v3 review)
- adding a small comment
Reported-by: Brian Norris briannorris@chromium.org Reported-by: Salvatore Bellizzi salvatore.bellizzi@linux.seppia.net Cc: Enrico Granata egranata@chromium.org Cc: stable@vger.kernel.org Fixes: daaef255dc96 ("driver: platform: Support parsing GpioInt 0 in platform_get_irq()") Signed-off-by: Brian Norris briannorris@chromium.org Reviewed-by: Andy Shevchenko andriy.shevchenko@linux.intel.com Acked-by: Enrico Granata egranata@google.com
Side note: it might have helped alleviate some of this pain if there were email notifications to the mailing list when a patch gets applied. I didn't realize (and I'm not sure if Enrico did) that v2 was already merged by the time I noted its mistakes. If I had known, I would have suggested a follow-up patch, not a v3.
I know some maintainers' "tip bots" do this, but not all apparently.
We can't drown out mailing list traffic with a ton of "this patch was applied" emails. We send them directly to the people involved in it.
Note, you can always set up your own "watch" for stuff like this by pulling linux-next every day and sending yourself any new patches that get applied for any specific files/directories you are concerned about.
thanks,
greg k-h