mbox series

[v4,0/5] add PM functionality for act8945a PMIC

Message ID 1544703110-27161-1-git-send-email-claudiu.beznea@microchip.com (mailing list archive)
Headers show
Series add PM functionality for act8945a PMIC | expand

Message

Claudiu Beznea Dec. 13, 2018, 12:49 p.m. UTC
From: Claudiu Beznea <claudiu.beznea@microchip.com>

This series implements PM functionality for act8945a PMIC and use that support
on SAMA5D2 Xplained board.

Changes in v4:
- move dt-bindings/regulator/active-semi,8945a-regulator.h in documentation
  patch; driver uses macros included in this file; due to this documentation
  patch and drivers patches should go through the same tree
- make documentation patch, patch 2 of this series

Changes in v3:
- add patch "unlock expert registers"; it was wrongly missed in previous version

Changes in v2:
- split patch 1/1 from previous series in 3 patches: one adding regmap,
  one adding pm functionality, one adding shutdown functionality
- use dev_pm_ops suspend function to set register specific to PWRHLD line: this
  is specific to PMIC itself and not to regulators
- add mode support for LDOs
- add constants for mode in dt-bindings/regulator/active-semi,8945a-regulator.h
- document modes and regulator suspend states
- remove rdevs from struct act8945a_pmic since were not used anywhere
- add op_mode field in struct act8945a_pmic, update it when setting mode and
  return it when reading mode (avoiding one extra register read while
  reading mode)
- add a patch to fix a checkpatch.pl warning
- in shutdown function use directly act8945a->regmap instead of an intermediary
  struct regmap variable

Boris Brezillon (2):
  regulator: act8945a-regulator: Implement PM functionalities
  ARM: dts: at91: sama5d2_xplained: Add proper regulator states for
    suspend-to-mem

Claudiu Beznea (3):
  regulator: act8945a-regulator: fix line over 80 chars warning
  regulator: act8945a-regulator: add shutdown function
  regulator: add documentation for regulator modes and suspend states

 .../bindings/regulator/act8945a-regulator.txt      |  34 ++++
 arch/arm/boot/dts/at91-sama5d2_xplained.dts        |  54 ++++++
 drivers/regulator/act8945a-regulator.c             | 200 ++++++++++++++++++++-
 .../regulator/active-semi,8945a-regulator.h        |  30 ++++
 4 files changed, 312 insertions(+), 6 deletions(-)
 create mode 100644 include/dt-bindings/regulator/active-semi,8945a-regulator.h

Comments

Claudiu Beznea Dec. 13, 2018, 12:59 p.m. UTC | #1
Hi Mark,

I saw that previous version of this series was applied to your tree just
after I've send this new one. The only difference b/w them is in the
documentation patch where I moved the new header. No functionality change.

Sorry for the noise,
Claudiu Beznea

On 13.12.2018 14:49, Claudiu Beznea - M18063 wrote:
> From: Claudiu Beznea <claudiu.beznea@microchip.com>
> 
> This series implements PM functionality for act8945a PMIC and use that support
> on SAMA5D2 Xplained board.
> 
> Changes in v4:
> - move dt-bindings/regulator/active-semi,8945a-regulator.h in documentation
>   patch; driver uses macros included in this file; due to this documentation
>   patch and drivers patches should go through the same tree
> - make documentation patch, patch 2 of this series
> 
> Changes in v3:
> - add patch "unlock expert registers"; it was wrongly missed in previous version
> 
> Changes in v2:
> - split patch 1/1 from previous series in 3 patches: one adding regmap,
>   one adding pm functionality, one adding shutdown functionality
> - use dev_pm_ops suspend function to set register specific to PWRHLD line: this
>   is specific to PMIC itself and not to regulators
> - add mode support for LDOs
> - add constants for mode in dt-bindings/regulator/active-semi,8945a-regulator.h
> - document modes and regulator suspend states
> - remove rdevs from struct act8945a_pmic since were not used anywhere
> - add op_mode field in struct act8945a_pmic, update it when setting mode and
>   return it when reading mode (avoiding one extra register read while
>   reading mode)
> - add a patch to fix a checkpatch.pl warning
> - in shutdown function use directly act8945a->regmap instead of an intermediary
>   struct regmap variable
> 
> Boris Brezillon (2):
>   regulator: act8945a-regulator: Implement PM functionalities
>   ARM: dts: at91: sama5d2_xplained: Add proper regulator states for
>     suspend-to-mem
> 
> Claudiu Beznea (3):
>   regulator: act8945a-regulator: fix line over 80 chars warning
>   regulator: act8945a-regulator: add shutdown function
>   regulator: add documentation for regulator modes and suspend states
> 
>  .../bindings/regulator/act8945a-regulator.txt      |  34 ++++
>  arch/arm/boot/dts/at91-sama5d2_xplained.dts        |  54 ++++++
>  drivers/regulator/act8945a-regulator.c             | 200 ++++++++++++++++++++-
>  .../regulator/active-semi,8945a-regulator.h        |  30 ++++
>  4 files changed, 312 insertions(+), 6 deletions(-)
>  create mode 100644 include/dt-bindings/regulator/active-semi,8945a-regulator.h
>
Mark Brown Dec. 13, 2018, 3:19 p.m. UTC | #2
On Thu, Dec 13, 2018 at 12:59:00PM +0000, Claudiu.Beznea@microchip.com wrote:
> Hi Mark,
> 
> I saw that previous version of this series was applied to your tree just
> after I've send this new one. The only difference b/w them is in the
> documentation patch where I moved the new header. No functionality change.
> 
> Sorry for the noise,

No problem.