usb: ohci-da8xx: disable the regulator if the overcurrent irq fired
authorBartosz Golaszewski <bgolaszewski@baylibre.com>
Fri, 12 Apr 2019 12:36:37 +0000 (14:36 +0200)
committerSekhar Nori <nsekhar@ti.com>
Fri, 12 Apr 2019 14:15:52 +0000 (19:45 +0530)
commitd327330185f192411be80563a3c8398f4538cdb2
treed631cb45bc97e7ca0354d734c38270701d4a9e6a
parent8e2f5eae24351d6f536c602eccdc4ad12b675c28
usb: ohci-da8xx: disable the regulator if the overcurrent irq fired

Historically the power supply management in this driver has been handled
in two separate places in parallel. Device-tree users simply defined an
appropriate regulator, while two boards with no DT support (da830-evm and
omapl138-hawk) passed functions defined in their respective board files
over platform data. These functions simply used legacy GPIO calls to
watch the oc GPIO for interrupts and disable the vbus GPIO when the irq
fires.

Commit d193abf1c913 ("usb: ohci-da8xx: add vbus and overcurrent gpios")
updated these GPIO calls to the modern API and moved them inside the
driver.

This however is not the optimal solution for the vbus GPIO which should
be modeled as a fixed regulator that can be controlled with a GPIO.

In order to keep the overcurrent protection available once we move the
board files to using fixed regulators we need to disable the enable_reg
regulator when the overcurrent indicator interrupt fires. Since we
cannot call regulator_disable() from interrupt context, we need to
switch to using a oneshot threaded interrupt.

Acked-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
Signed-off-by: Sekhar Nori <nsekhar@ti.com>
drivers/usb/host/ohci-da8xx.c