Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Occasional ENXIO on gpiod_to_irq #31

Open
PandorasFox opened this issue Jan 4, 2020 · 1 comment
Open

Occasional ENXIO on gpiod_to_irq #31

PandorasFox opened this issue Jan 4, 2020 · 1 comment
Labels
C: bug (upstream) Upstream bug D: Laptop 3 (AMD) Device: Surface Laptop 3 (AMD) P: medium Priority: Medium priority

Comments

@PandorasFox
Copy link

Occasionally (say, 1 in every 10 boots) my SL3A will get ENXIO returned on the gpiod_to_irq in the ssh module.

Using my usb keyboard to unload/reload the module has it init properly later, so I'm assuming that it's just a weird race condition.

@qzed
Copy link
Member

qzed commented Jan 5, 2020

That's a bit weird. Looking at https://elixir.bootlin.com/linux/v5.4.8/source/drivers/gpio/gpiolib.c#L3811 would suggest that the gpiochip might not be fully set up yet, which would be collaborated by your finding that it's a race condition. It's interesting because I had a conversation with Hans de Goede regarding the button driver, who mentioned that in x86 GPIOs should be fully set up once "normal" modules load (https://lkml.org/lkml/2019/10/5/104, although that's related to gpiod_get in that case). Even more so because the surface_sam_ssh module is loaded via late_initcall, so even later due to some DMA setup issues we had before. I guess that would kind of suggest something going wrong in the GPIO driver again. I'm not quite sure yet though.

@qzed qzed added D: Laptop 3 (AMD) Device: Surface Laptop 3 (AMD) C: bug Something isn't working P: medium Priority: Medium priority labels Jan 5, 2020
@qzed qzed added C: bug (upstream) Upstream bug and removed C: bug Something isn't working labels Jul 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: bug (upstream) Upstream bug D: Laptop 3 (AMD) Device: Surface Laptop 3 (AMD) P: medium Priority: Medium priority
Projects
None yet
Development

No branches or pull requests

2 participants