mbox series

[v4,00/16] soc: mediatek: pm-domains: Add new driver for SCPSYS power domains controller

Message ID 20201030113622.201188-1-enric.balletbo@collabora.com (mailing list archive)
Headers show
Series soc: mediatek: pm-domains: Add new driver for SCPSYS power domains controller | expand

Message

Enric Balletbo i Serra Oct. 30, 2020, 11:36 a.m. UTC
Dear all,

This is a new driver with the aim to deprecate the mtk-scpsys driver.
The problem with that driver is that, in order to support more Mediatek
SoCs you need to add some logic to handle properly the power-up
sequence of newer Mediatek SoCs, doesn't handle parent-child power
domains and need to hardcode all the clocks in the driver itself. The
result is that the driver is getting bigger and bigger every time a
new SoC needs to be supported.

All this information can be getted from a properly defined binding, so
can be cleaner and smaller, hence, we implemented a new driver. For
now, only MT8173 and MT8183 is supported but should be fairly easy to
add support for new SoCs.

Three important notes:

1. This patch depends now on [1] to build correctly.

2. Support for MT8183 is not ready to land yet because has some
   dependencies, i.e mmsys support is still missing.

3. Support for MT8192. I picked the patches [2] from Weiyi Lu and
   adapted to this new series. I posted only for reference due that this
   new version has some changes that affects that patchset.

Only patches from 1 to 9 are ready, the others are provided for reference and test.

[1] https://lore.kernel.org/patchwork/patch/1328096/
[2] https://patchwork.kernel.org/project/linux-mediatek/list/?series=368821

Best regards,
  Enric

Enric Balletbo i Serra (5):
  dt-bindings: power: Add bindings for the Mediatek SCPSYS power domains
    controller
  soc: mediatek: Add MediaTek SCPSYS power domains
  arm64: dts: mediatek: Add mt8173 power domain controller
  dt-bindings: power: Add MT8183 power domains
  arm64: dts: mediatek: Add smi_common node for MT8183

Matthias Brugger (8):
  soc: mediatek: pm-domains: Add bus protection protocol
  soc: mediatek: pm_domains: Make bus protection generic
  soc: mediatek: pm-domains: Add SMI block as bus protection block
  soc: mediatek: pm-domains: Add extra sram control
  soc: mediatek: pm-domains: Add subsystem clocks
  soc: mediatek: pm-domains: Allow bus protection to ignore clear ack
  soc: mediatek: pm-domains: Add support for mt8183
  arm64: dts: mediatek: Add mt8183 power domains controller

Weiyi Lu (3):
  dt-bindings: power: Add MT8192 power domains
  soc: mediatek: pm-domains: Add default power off flag
  soc: mediatek: pm-domains: Add support for mt8192

 .../power/mediatek,power-controller.yaml      | 293 +++++++++
 arch/arm64/boot/dts/mediatek/mt8173.dtsi      | 164 +++--
 arch/arm64/boot/dts/mediatek/mt8183.dtsi      | 172 +++++
 drivers/soc/mediatek/Kconfig                  |  12 +
 drivers/soc/mediatek/Makefile                 |   1 +
 drivers/soc/mediatek/mt8173-pm-domains.h      |  94 +++
 drivers/soc/mediatek/mt8183-pm-domains.h      | 221 +++++++
 drivers/soc/mediatek/mt8192-pm-domains.h      | 292 +++++++++
 drivers/soc/mediatek/mtk-infracfg.c           |   5 -
 drivers/soc/mediatek/mtk-pm-domains.c         | 614 ++++++++++++++++++
 drivers/soc/mediatek/mtk-pm-domains.h         | 102 +++
 include/dt-bindings/power/mt8183-power.h      |  26 +
 include/dt-bindings/power/mt8192-power.h      |  32 +
 include/linux/soc/mediatek/infracfg.h         | 107 +++
 14 files changed, 2081 insertions(+), 54 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
 create mode 100644 drivers/soc/mediatek/mt8173-pm-domains.h
 create mode 100644 drivers/soc/mediatek/mt8183-pm-domains.h
 create mode 100644 drivers/soc/mediatek/mt8192-pm-domains.h
 create mode 100644 drivers/soc/mediatek/mtk-pm-domains.c
 create mode 100644 drivers/soc/mediatek/mtk-pm-domains.h
 create mode 100644 include/dt-bindings/power/mt8183-power.h
 create mode 100644 include/dt-bindings/power/mt8192-power.h

Comments

Matthias Brugger Nov. 27, 2020, 11:19 a.m. UTC | #1
On 30/10/2020 12:36, Enric Balletbo i Serra wrote:
> Dear all,
> 
> This is a new driver with the aim to deprecate the mtk-scpsys driver.
> The problem with that driver is that, in order to support more Mediatek
> SoCs you need to add some logic to handle properly the power-up
> sequence of newer Mediatek SoCs, doesn't handle parent-child power
> domains and need to hardcode all the clocks in the driver itself. The
> result is that the driver is getting bigger and bigger every time a
> new SoC needs to be supported.
> 
> All this information can be getted from a properly defined binding, so
> can be cleaner and smaller, hence, we implemented a new driver. For
> now, only MT8173 and MT8183 is supported but should be fairly easy to
> add support for new SoCs.
> 

All patches are pushed:
DT-bindings went to v5.10-next/pm-domains-stable as they are needed by both soc 
and dts64 branch.

Regards,
Matthias

> Three important notes:
> 
> 1. This patch depends now on [1] to build correctly.
> 
> 2. Support for MT8183 is not ready to land yet because has some
>     dependencies, i.e mmsys support is still missing.
> 
> 3. Support for MT8192. I picked the patches [2] from Weiyi Lu and
>     adapted to this new series. I posted only for reference due that this
>     new version has some changes that affects that patchset.
> 
> Only patches from 1 to 9 are ready, the others are provided for reference and test.
> 
> [1] https://lore.kernel.org/patchwork/patch/1328096/
> [2] https://patchwork.kernel.org/project/linux-mediatek/list/?series=368821
> 
> Best regards,
>    Enric
> 
> Enric Balletbo i Serra (5):
>    dt-bindings: power: Add bindings for the Mediatek SCPSYS power domains
>      controller
>    soc: mediatek: Add MediaTek SCPSYS power domains
>    arm64: dts: mediatek: Add mt8173 power domain controller
>    dt-bindings: power: Add MT8183 power domains
>    arm64: dts: mediatek: Add smi_common node for MT8183
> 
> Matthias Brugger (8):
>    soc: mediatek: pm-domains: Add bus protection protocol
>    soc: mediatek: pm_domains: Make bus protection generic
>    soc: mediatek: pm-domains: Add SMI block as bus protection block
>    soc: mediatek: pm-domains: Add extra sram control
>    soc: mediatek: pm-domains: Add subsystem clocks
>    soc: mediatek: pm-domains: Allow bus protection to ignore clear ack
>    soc: mediatek: pm-domains: Add support for mt8183
>    arm64: dts: mediatek: Add mt8183 power domains controller
> 
> Weiyi Lu (3):
>    dt-bindings: power: Add MT8192 power domains
>    soc: mediatek: pm-domains: Add default power off flag
>    soc: mediatek: pm-domains: Add support for mt8192
> 
>   .../power/mediatek,power-controller.yaml      | 293 +++++++++
>   arch/arm64/boot/dts/mediatek/mt8173.dtsi      | 164 +++--
>   arch/arm64/boot/dts/mediatek/mt8183.dtsi      | 172 +++++
>   drivers/soc/mediatek/Kconfig                  |  12 +
>   drivers/soc/mediatek/Makefile                 |   1 +
>   drivers/soc/mediatek/mt8173-pm-domains.h      |  94 +++
>   drivers/soc/mediatek/mt8183-pm-domains.h      | 221 +++++++
>   drivers/soc/mediatek/mt8192-pm-domains.h      | 292 +++++++++
>   drivers/soc/mediatek/mtk-infracfg.c           |   5 -
>   drivers/soc/mediatek/mtk-pm-domains.c         | 614 ++++++++++++++++++
>   drivers/soc/mediatek/mtk-pm-domains.h         | 102 +++
>   include/dt-bindings/power/mt8183-power.h      |  26 +
>   include/dt-bindings/power/mt8192-power.h      |  32 +
>   include/linux/soc/mediatek/infracfg.h         | 107 +++
>   14 files changed, 2081 insertions(+), 54 deletions(-)
>   create mode 100644 Documentation/devicetree/bindings/power/mediatek,power-controller.yaml
>   create mode 100644 drivers/soc/mediatek/mt8173-pm-domains.h
>   create mode 100644 drivers/soc/mediatek/mt8183-pm-domains.h
>   create mode 100644 drivers/soc/mediatek/mt8192-pm-domains.h
>   create mode 100644 drivers/soc/mediatek/mtk-pm-domains.c
>   create mode 100644 drivers/soc/mediatek/mtk-pm-domains.h
>   create mode 100644 include/dt-bindings/power/mt8183-power.h
>   create mode 100644 include/dt-bindings/power/mt8192-power.h
>