Re: [PATCH/RFT 07/12] USB: ohci-da8xx: Request gpios and handle interrupt in the driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 10/07/2016 11:42 AM, ahaslam@xxxxxxxxxxxx wrote:
From: Axel Haslam <ahaslam@xxxxxxxxxxxx>

Currently requesting the vbus and overcurrent gpio is handled on
the board specific file. But this does not play well moving to
device tree.

In preparation to migrate to a device tree boot, handle requesting
gpios and overcurrent interrupt on the usb driver itself, thus avoiding
callbacks to arch/mach*


Instead of using gpios, it seems like it would be better to use a regulator here. I don't know of any real-life cases, but who is to say someone will not design a board that uses a regulator controlled by I2C instead of gpios or something like that.

Then, boards that don't have gpios can just use a fixed regulator (or you can make the regulator optional). Using a regulator would also allow users to decide how to respond to overcurrent events (by supplying their own regulator driver) instead of the behavior being dictated by the ohci driver.

In my particular area of interest (LEGO MINDSTORMS EV3), the 5V (hardware) regulator for VBUS does use gpios, but the 5V is also shared with the LEGO input and output ports. So what I would ultimately like to be able to do is have userspace notified of an overcurrent event and let userspace decided when to turn the vbus back on. For example, someone might plug something into one of the LEGO input or output ports that causes a short circuit. I would like to display a notification to the user and wait for them to correct the problem and then press a button to turn the power back on.

This will require some modifications to the regulator subsystem though. I actually started work on this a while back, but haven't had the time to pursue it any farther.

Here are my WIP patches in case there is any interest:
* https://github.com/dlech/ev3dev-kernel/commit/541a42b3b8ed639e95bbc835df3292f80190c789 * https://github.com/dlech/ev3dev-kernel/commit/2ba99b1ad6a06c944dd33a073f54044e71b75ae6 * https://github.com/dlech/ev3dev-kernel/commit/cdb03caa50e64931d4f2836c648739aa4385ed3b * https://github.com/dlech/ev3dev-kernel/commit/9d6b50cde34b51309c74d97c26b1430c7ff6aa0f
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux