Skip to content

pmbus: ltc2978: add regulator support

[email protected] linux-devicetree lkml lm-sensors 7 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v6 Latest4 patches 9 repliesIn review
  2. v54 patches 18 repliesSuperseded
  3. v4 posting 2/21 of 4 patches 0 repliesSuperseded
Earlier versions · 4
VersionPostedPatchesRepliesStatus
v4 posting 1/23 of 4 patches7 repliesSuperseded
v33 patches10 repliesSuperseded
v22 patches10 repliesSuperseded
v12 patches6 repliesSuperseded

Patches in v6 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4hwmon: ltc2978: device tree bindings documentationDocumentation/devicetree/bindings/hwmon/ltc2978.txt+39 -01 reply
2/4pmbus: core: add helpers for byte write and read modify writedrivers/hwmon/pmbus/pmbus.h drivers/hwmon/pmbus/pmbus_core.c+35 -00 replies
3/4pmbus: add regulator supportdrivers/hwmon/pmbus/pmbus.h drivers/hwmon/pmbus/pmbus_core.c include/linux/i2c/pmbus.h+117 -06 replies
4/4pmbus: ltc2978: add regulator supportdrivers/hwmon/pmbus/Kconfig drivers/hwmon/pmbus/ltc2978.c+47 -30 replies

Reviews trailers per patch

1 of 4 patches has a Reviewed-by or Acked-by.

PatchMark Brown
1/4
2/4
3/4R-b
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v6 thread

Cover letter [PATCH v6 0/4] pmbus: ltc2978: add regulator support

From: Alan Tull <[email protected]>

Each output has individual on/off control.

New in v6:
* Much cleanup of the bindings document
* s/vout_en/vout/g

Hopefully this is getting closer. We will still have the potential problem of repeated node names on boards on boards that have many regulators of the same kind. If this can be handled in the regulator layer, that might be great. If we handle it here, whatever we come up with will have to keep the same name over reboots and be predictable enough to serve as regulator node names in the DT. I'm open for suggestions here.

This patchset now uses "regulator: of: Provide simplified DT parsing method" which are in the next-20141014 tag of linux-next.

Quoted 1 line

From PMBus_Specification_Part_II_Rev_1-3_20140318.pdf:

12.1.1. OPERATION Command Bit [7]
Bit [7] controls whether the PMBus device output is on or off.
If bit [7] is cleared (equals 0), then the output is off. If bit [7] is set (equals 1), then the output is on.

Patch 1: document device tree bindings for ltc2978

Patch 2: add two helper functions for byte pmbus byte operations
* byte write and byte read/modify/write

Patch 3: regulator support added in pmbus_core.c and pmbus.h
* regulator_ops functions (is_enabled, enable, and disable)
* gets regulator init data from device tree or platform data
* registers the regulators
* header has a macro for chip drivers to build their regulator_desc data

Patch 4: changes for ltc2978.c
* Add Kconfig to enable/disable ltc2978 regulator functionality.
* Update list of parts supported in Kconfig.
* add regulator_desc info.
* use same structs for all parts; set num_regulators appropriately.

Alan Tull (4):
hwmon: ltc2978: device tree bindings documentation
pmbus: core: add helpers for byte write and read modify write
pmbus: add regulator support
pmbus: ltc2978: add regulator support

 .../devicetree/bindings/hwmon/ltc2978.txt          |   39 +++++++
 drivers/hwmon/pmbus/Kconfig                        |   11 +-
 drivers/hwmon/pmbus/ltc2978.c                      |   39 ++++++-
 drivers/hwmon/pmbus/pmbus.h                        |   30 +++++
 drivers/hwmon/pmbus/pmbus_core.c                   |  118 ++++++++++++++++++++
 include/linux/i2c/pmbus.h                          |    4 +
 6 files changed, 238 insertions(+), 3 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/hwmon/ltc2978.txt
Signature · 1 line
1.7.9.5