linux-kernel - [PATCH/RFC v2 0/3] regulator: handling of error conditions for usb drivers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Thu,  3 Nov 2016 12:11:41 +0100
From:   Axel Haslam <ahaslam@...libre.com>
To:     broonie@...nel.org, lgirdwood@...il.com, khilman@...libre.com,
        nsekhar@...com, david@...hnology.com, robh+dt@...nel.org
Cc:     linux-kernel@...r.kernel.org, Axel Haslam <ahaslam@...libre.com>
Subject: [PATCH/RFC v2 0/3] regulator: handling of error conditions for usb drivers

Some usb drivers rely on external power switches/regulators
to for the port vbus. Some of these drivers are using
a plain gpio for the enable pin and also the over current
indicator pin.

To make these drivers more generic, we can use a regulator
to handle vbus, and send and over current event, but we are
missing a way to transmit the over current pin status, which
the usb layer may poll at any time.

We would like to move these drivers to use a regulator, this
would make the usb driver generic allowing to use any type
of regulator. Also, it would help removing code, making DT
migration simpler and avoiding new DT bindings for each driver.

These patches do 2 things:
* Add a new API, that consumers can use to poll the regulator
  error status.
* Extends the fixed regulator driver to handle an optional
  over current gpio pin.

Changes v1->v2
* add new API to get error status instead of extending events (Mark)
* use gpiod for fixed regulator: This spears us extra platform
  data and bindings

Axel Haslam (3):
  regulator: core: Add new API to poll for error conditions
  regulator: fixed: dt: Allow an optional over current pin
  regulator: fixed: Handle optional overcurrent pin

 .../bindings/regulator/fixed-regulator.txt         |  2 +
 drivers/regulator/core.c                           | 33 ++++++++++++
 drivers/regulator/fixed.c                          | 60 ++++++++++++++++++++++
 include/linux/regulator/consumer.h                 | 26 ++++++++++
 include/linux/regulator/driver.h                   |  4 ++
 5 files changed, 125 insertions(+)

-- 
2.10.1