mbox series

[v21,0/5] add power control in i2c

Message ID 20210507131406.2224177-1-hsinyi@chromium.org (mailing list archive)
Headers show
Series add power control in i2c | expand

Message

Hsin-Yi Wang May 7, 2021, 1:14 p.m. UTC
Although in the most platforms, the power of eeprom
and i2c are alway on, some platforms disable the
eeprom and i2c power in order to meet low power request.

This patch add the pm_runtime ops to control power to
support all platforms.

Changes since v20:
 - fix regulator check logic in suspend/resume.

Changes since v19:
 - resend v19 with fix tag added.

Changes since v18:
 - Fix a function name conflict with drivers/gpu/drm/i915/selftests/i915_gem.c

Changes since v17:
 - Add a patch to fix unbalanced regulator disabling.
 - Add dts patch.

Changes since v16:
 - request regulator in device instead of in the core.
 - control regulator only if it's provided.

Changes since v15:
 - Squash the fix[1] for v15.
[1] https://patchwork.ozlabs.org/project/linux-i2c/patch/20200522101327.13456-1-m.szyprowski@samsung.com/

Changes since v14:
 - change the return value in normal condition
 - access the variable after NULL pointer checking
 - add ack tag

Changes since v13:
 - fixup some logic error

Changes since v12:
 - rebase onto v5.7-rc1
 - change the property description in binding

Changes since v11:
 - use suspend_late/resume_early instead of suspend/resume
 - rebase onto v5.6-rc1

Changes since v10:
 - fixup some worng codes

Changes since v9:
 - fixup build error
 - remove redundant code

Changes since v8:
 - fixup some wrong code
 - remove redundant message

[... snip ...]


Bibby Hsieh (1):
  i2c: core: support bus regulator controlling in adapter

Hsin-Yi Wang (4):
  dt-binding: i2c: mt65xx: add vbus-supply property
  i2c: mediatek: mt65xx: add optional vbus-supply
  misc: eeprom: at24: check suspend status before disable regulator
  arm64: dts: mt8183: add supply name for eeprom

 .../devicetree/bindings/i2c/i2c-mt65xx.txt    |  1 +
 .../dts/mediatek/mt8183-kukui-kakadu.dtsi     |  4 +
 .../dts/mediatek/mt8183-kukui-kodama.dtsi     |  4 +
 .../boot/dts/mediatek/mt8183-kukui-krane.dtsi |  4 +
 drivers/i2c/busses/i2c-mt65xx.c               |  7 ++
 drivers/i2c/i2c-core-base.c                   | 95 +++++++++++++++++++
 drivers/misc/eeprom/at24.c                    |  6 +-
 include/linux/i2c.h                           |  2 +
 8 files changed, 121 insertions(+), 2 deletions(-)

Comments

Hsin-Yi Wang May 20, 2021, 10:56 a.m. UTC | #1
On Fri, May 7, 2021 at 9:14 PM Hsin-Yi Wang <hsinyi@chromium.org> wrote:
>
> Although in the most platforms, the power of eeprom
> and i2c are alway on, some platforms disable the
> eeprom and i2c power in order to meet low power request.
>
> This patch add the pm_runtime ops to control power to
> support all platforms.
>

Hi maintainers,

Gentle ping on the thread. (the 4th patch is already picked) Thanks.

> Changes since v20:
>  - fix regulator check logic in suspend/resume.
>
> Changes since v19:
>  - resend v19 with fix tag added.
>
> Changes since v18:
>  - Fix a function name conflict with drivers/gpu/drm/i915/selftests/i915_gem.c
>
> Changes since v17:
>  - Add a patch to fix unbalanced regulator disabling.
>  - Add dts patch.
>
> Changes since v16:
>  - request regulator in device instead of in the core.
>  - control regulator only if it's provided.
>
> Changes since v15:
>  - Squash the fix[1] for v15.
> [1] https://patchwork.ozlabs.org/project/linux-i2c/patch/20200522101327.13456-1-m.szyprowski@samsung.com/
>
> Changes since v14:
>  - change the return value in normal condition
>  - access the variable after NULL pointer checking
>  - add ack tag
>
> Changes since v13:
>  - fixup some logic error
>
> Changes since v12:
>  - rebase onto v5.7-rc1
>  - change the property description in binding
>
> Changes since v11:
>  - use suspend_late/resume_early instead of suspend/resume
>  - rebase onto v5.6-rc1
>
> Changes since v10:
>  - fixup some worng codes
>
> Changes since v9:
>  - fixup build error
>  - remove redundant code
>
> Changes since v8:
>  - fixup some wrong code
>  - remove redundant message
>
> [... snip ...]
>
>
> Bibby Hsieh (1):
>   i2c: core: support bus regulator controlling in adapter
>
> Hsin-Yi Wang (4):
>   dt-binding: i2c: mt65xx: add vbus-supply property
>   i2c: mediatek: mt65xx: add optional vbus-supply
>   misc: eeprom: at24: check suspend status before disable regulator
>   arm64: dts: mt8183: add supply name for eeprom
>
>  .../devicetree/bindings/i2c/i2c-mt65xx.txt    |  1 +
>  .../dts/mediatek/mt8183-kukui-kakadu.dtsi     |  4 +
>  .../dts/mediatek/mt8183-kukui-kodama.dtsi     |  4 +
>  .../boot/dts/mediatek/mt8183-kukui-krane.dtsi |  4 +
>  drivers/i2c/busses/i2c-mt65xx.c               |  7 ++
>  drivers/i2c/i2c-core-base.c                   | 95 +++++++++++++++++++
>  drivers/misc/eeprom/at24.c                    |  6 +-
>  include/linux/i2c.h                           |  2 +
>  8 files changed, 121 insertions(+), 2 deletions(-)
>
> --
> 2.31.1.607.g51e8a6a459-goog
>