* [PATCH 0/2] Allwinner power domain support @ 2022-12-31 16:04 Samuel Holland 2022-12-31 16:04 ` [PATCH 1/2] dt-bindings: power: Add Allwinner D1 PPU Samuel Holland ` (2 more replies) 0 siblings, 3 replies; 8+ messages in thread From: Samuel Holland @ 2022-12-31 16:04 UTC (permalink / raw) To: Chen-Yu Tsai, Jernej Skrabec, Krzysztof Kozlowski, Rob Herring Cc: Samuel Holland, Philipp Zabel, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi This series adds support for the power controller found in D1 and other recent Allwinner SoCs. There is no first-party documentation, but there are a couple of vendor drivers for different hardware revisions[1][2], and the register definitions were easy to verify empirically. I have tested this driver on D1 with the video engine. There is no DT update patch here to avoid dependencies between series. The example in the binding is what will go in the D1 DT. [1]: https://github.com/mangopi-sbc/tina-linux-5.4/blob/main/drivers/soc/sunxi/gpu_domain.c [1]: https://github.com/mangopi-sbc/tina-linux-5.4/blob/main/drivers/soc/sunxi/pm_domains.c Samuel Holland (2): dt-bindings: power: Add Allwinner D1 PPU soc: sunxi: Add Allwinner D1 PPU driver .../power/allwinner,sun20i-d1-ppu.yaml | 54 +++++ drivers/soc/sunxi/Kconfig | 9 + drivers/soc/sunxi/Makefile | 1 + drivers/soc/sunxi/sun20i-ppu.c | 207 ++++++++++++++++++ .../power/allwinner,sun20i-d1-ppu.h | 10 + 5 files changed, 281 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml create mode 100644 drivers/soc/sunxi/sun20i-ppu.c create mode 100644 include/dt-bindings/power/allwinner,sun20i-d1-ppu.h -- 2.37.4 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/2] dt-bindings: power: Add Allwinner D1 PPU 2022-12-31 16:04 [PATCH 0/2] Allwinner power domain support Samuel Holland @ 2022-12-31 16:04 ` Samuel Holland 2022-12-31 18:58 ` Krzysztof Kozlowski 2022-12-31 16:04 ` [PATCH 2/2] soc: sunxi: Add Allwinner D1 PPU driver Samuel Holland 2023-01-05 16:34 ` [PATCH 0/2] Allwinner power domain support Jernej Škrabec 2 siblings, 1 reply; 8+ messages in thread From: Samuel Holland @ 2022-12-31 16:04 UTC (permalink / raw) To: Chen-Yu Tsai, Jernej Skrabec, Krzysztof Kozlowski, Rob Herring Cc: Samuel Holland, Philipp Zabel, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi The Allwinner D1 family of SoCs contain a "PPU" power domain controller separate from the PRCM. It can power down the video engine and DSP, and it contains special logic for hardware-assisted CPU idle. Other recent Allwinner SoCs (e.g. TV303) have a PPU with a different set of domains. Signed-off-by: Samuel Holland <samuel@sholland.org> --- .../power/allwinner,sun20i-d1-ppu.yaml | 54 +++++++++++++++++++ .../power/allwinner,sun20i-d1-ppu.h | 10 ++++ 2 files changed, 64 insertions(+) create mode 100644 Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml create mode 100644 include/dt-bindings/power/allwinner,sun20i-d1-ppu.h diff --git a/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml b/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml new file mode 100644 index 000000000000..64c9a9f398a2 --- /dev/null +++ b/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml @@ -0,0 +1,54 @@ +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/power/allwinner,sun20i-d1-ppu.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Allwinner SoCs "PPU" power domain controller + +maintainers: + - Samuel Holland <samuel@sholland.org> + +description: + D1 and related SoCs contain a power domain controller for the CPUs, GPU, and + video-related hardware. + +properties: + compatible: + enum: + - allwinner,sun20i-d1-ppu + + reg: + maxItems: 1 + + clocks: + description: Bus Clock + maxItems: 1 + + resets: + maxItems: 1 + + '#power-domain-cells': + const: 1 + +required: + - compatible + - reg + - clocks + - resets + - '#power-domain-cells' + +additionalProperties: false + +examples: + - | + #include <dt-bindings/clock/sun20i-d1-r-ccu.h> + #include <dt-bindings/reset/sun20i-d1-r-ccu.h> + + ppu: power-controller@7001000 { + compatible = "allwinner,sun20i-d1-ppu"; + reg = <0x7001000 0x1000>; + clocks = <&r_ccu CLK_BUS_R_PPU>; + resets = <&r_ccu RST_BUS_R_PPU>; + #power-domain-cells = <1>; + }; diff --git a/include/dt-bindings/power/allwinner,sun20i-d1-ppu.h b/include/dt-bindings/power/allwinner,sun20i-d1-ppu.h new file mode 100644 index 000000000000..23cfb57256d6 --- /dev/null +++ b/include/dt-bindings/power/allwinner,sun20i-d1-ppu.h @@ -0,0 +1,10 @@ +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */ + +#ifndef _DT_BINDINGS_POWER_SUN20I_D1_PPU_H_ +#define _DT_BINDINGS_POWER_SUN20I_D1_PPU_H_ + +#define PD_CPU 0 +#define PD_VE 1 +#define PD_DSP 2 + +#endif /* _DT_BINDINGS_POWER_SUN20I_D1_PPU_H_ */ -- 2.37.4 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: power: Add Allwinner D1 PPU 2022-12-31 16:04 ` [PATCH 1/2] dt-bindings: power: Add Allwinner D1 PPU Samuel Holland @ 2022-12-31 18:58 ` Krzysztof Kozlowski 2022-12-31 19:12 ` Samuel Holland 0 siblings, 1 reply; 8+ messages in thread From: Krzysztof Kozlowski @ 2022-12-31 18:58 UTC (permalink / raw) To: Samuel Holland, Chen-Yu Tsai, Jernej Skrabec, Krzysztof Kozlowski, Rob Herring Cc: Philipp Zabel, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi On 31/12/2022 17:04, Samuel Holland wrote: > The Allwinner D1 family of SoCs contain a "PPU" power domain controller > separate from the PRCM. It can power down the video engine and DSP, and > it contains special logic for hardware-assisted CPU idle. Other recent > Allwinner SoCs (e.g. TV303) have a PPU with a different set of domains. > > Signed-off-by: Samuel Holland <samuel@sholland.org> > --- > > .../power/allwinner,sun20i-d1-ppu.yaml | 54 +++++++++++++++++++ > .../power/allwinner,sun20i-d1-ppu.h | 10 ++++ > 2 files changed, 64 insertions(+) > create mode 100644 Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml > create mode 100644 include/dt-bindings/power/allwinner,sun20i-d1-ppu.h > > diff --git a/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml b/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml > new file mode 100644 > index 000000000000..64c9a9f398a2 > --- /dev/null > +++ b/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml > @@ -0,0 +1,54 @@ > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/power/allwinner,sun20i-d1-ppu.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Allwinner SoCs "PPU" power domain controller Drop quotes over "PPU" - it's not some nickname, alias, but acronym. Explain the acronym in description. > + > +maintainers: > + - Samuel Holland <samuel@sholland.org> > + > +description: > + D1 and related SoCs contain a power domain controller for the CPUs, GPU, and > + video-related hardware. > + > +properties: > + compatible: > + enum: > + - allwinner,sun20i-d1-ppu > + > + reg: > + maxItems: 1 > + > + clocks: > + description: Bus Clock > + maxItems: 1 > + > + resets: > + maxItems: 1 > + > + '#power-domain-cells': > + const: 1 > + > +required: > + - compatible > + - reg > + - clocks > + - resets > + - '#power-domain-cells' > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/clock/sun20i-d1-r-ccu.h> > + #include <dt-bindings/reset/sun20i-d1-r-ccu.h> > + > + ppu: power-controller@7001000 { > + compatible = "allwinner,sun20i-d1-ppu"; Use 4 spaces for example indentation. With above: Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 1/2] dt-bindings: power: Add Allwinner D1 PPU 2022-12-31 18:58 ` Krzysztof Kozlowski @ 2022-12-31 19:12 ` Samuel Holland 0 siblings, 0 replies; 8+ messages in thread From: Samuel Holland @ 2022-12-31 19:12 UTC (permalink / raw) To: Krzysztof Kozlowski, Chen-Yu Tsai, Jernej Skrabec, Krzysztof Kozlowski, Rob Herring Cc: Philipp Zabel, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi On 12/31/22 12:58, Krzysztof Kozlowski wrote: > On 31/12/2022 17:04, Samuel Holland wrote: >> The Allwinner D1 family of SoCs contain a "PPU" power domain controller >> separate from the PRCM. It can power down the video engine and DSP, and >> it contains special logic for hardware-assisted CPU idle. Other recent >> Allwinner SoCs (e.g. TV303) have a PPU with a different set of domains. >> >> Signed-off-by: Samuel Holland <samuel@sholland.org> >> --- >> >> .../power/allwinner,sun20i-d1-ppu.yaml | 54 +++++++++++++++++++ >> .../power/allwinner,sun20i-d1-ppu.h | 10 ++++ >> 2 files changed, 64 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml >> create mode 100644 include/dt-bindings/power/allwinner,sun20i-d1-ppu.h >> >> diff --git a/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml b/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml >> new file mode 100644 >> index 000000000000..64c9a9f398a2 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml >> @@ -0,0 +1,54 @@ >> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/power/allwinner,sun20i-d1-ppu.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: Allwinner SoCs "PPU" power domain controller > > Drop quotes over "PPU" - it's not some nickname, alias, but acronym. > Explain the acronym in description. I don't know what the acronym stands for; it is never spelled out in the vendor code. I will drop the quotes in v2, but I can't give an explanation. Regards, Samuel ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/2] soc: sunxi: Add Allwinner D1 PPU driver 2022-12-31 16:04 [PATCH 0/2] Allwinner power domain support Samuel Holland 2022-12-31 16:04 ` [PATCH 1/2] dt-bindings: power: Add Allwinner D1 PPU Samuel Holland @ 2022-12-31 16:04 ` Samuel Holland 2023-01-05 16:38 ` Jernej Škrabec 2023-01-05 16:34 ` [PATCH 0/2] Allwinner power domain support Jernej Škrabec 2 siblings, 1 reply; 8+ messages in thread From: Samuel Holland @ 2022-12-31 16:04 UTC (permalink / raw) To: Chen-Yu Tsai, Jernej Skrabec, Krzysztof Kozlowski, Rob Herring Cc: Samuel Holland, Philipp Zabel, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi The PPU contains a series of identical MMIO register ranges, one for each power domain. Each range contains control/status bits for a clock gate, reset line, output gates, and a power switch. (The clock and reset are separate from, and in addition to, the bits in the CCU.) It also contains a hardware power sequence engine to control the other bits. Signed-off-by: Samuel Holland <samuel@sholland.org> --- drivers/soc/sunxi/Kconfig | 9 ++ drivers/soc/sunxi/Makefile | 1 + drivers/soc/sunxi/sun20i-ppu.c | 207 +++++++++++++++++++++++++++++++++ 3 files changed, 217 insertions(+) create mode 100644 drivers/soc/sunxi/sun20i-ppu.c diff --git a/drivers/soc/sunxi/Kconfig b/drivers/soc/sunxi/Kconfig index 8aecbc9b1976..5e84cf4b8510 100644 --- a/drivers/soc/sunxi/Kconfig +++ b/drivers/soc/sunxi/Kconfig @@ -19,3 +19,12 @@ config SUNXI_SRAM Say y here to enable the SRAM controller support. This device is responsible on mapping the SRAM in the sunXi SoCs whether to the CPU/DMA, or to the devices. + +config SUN20I_PPU + bool "Allwinner D1 PPU power domain driver" + depends on ARCH_SUNXI || COMPILE_TEST + select PM_GENERIC_DOMAINS + help + Say y to enable the PPU power domain driver. This saves power + when certain peripherals, such as the video engine, are idle. + All power domains are on by default, so this is optional. diff --git a/drivers/soc/sunxi/Makefile b/drivers/soc/sunxi/Makefile index 549159571d4f..90ff2ebe7655 100644 --- a/drivers/soc/sunxi/Makefile +++ b/drivers/soc/sunxi/Makefile @@ -1,3 +1,4 @@ # SPDX-License-Identifier: GPL-2.0-only obj-$(CONFIG_SUNXI_MBUS) += sunxi_mbus.o obj-$(CONFIG_SUNXI_SRAM) += sunxi_sram.o +obj-$(CONFIG_SUN20I_PPU) += sun20i-ppu.o diff --git a/drivers/soc/sunxi/sun20i-ppu.c b/drivers/soc/sunxi/sun20i-ppu.c new file mode 100644 index 000000000000..98cb41d36560 --- /dev/null +++ b/drivers/soc/sunxi/sun20i-ppu.c @@ -0,0 +1,207 @@ +// SPDX-License-Identifier: GPL-2.0-only + +#include <linux/bitfield.h> +#include <linux/clk.h> +#include <linux/io.h> +#include <linux/iopoll.h> +#include <linux/module.h> +#include <linux/of_device.h> +#include <linux/platform_device.h> +#include <linux/pm_domain.h> +#include <linux/reset.h> + +#define PD_STATE_ON 1 +#define PD_STATE_OFF 2 + +#define PD_RSTN_REG 0x00 +#define PD_CLK_GATE_REG 0x04 +#define PD_PWROFF_GATE_REG 0x08 +#define PD_PSW_ON_REG 0x0c +#define PD_PSW_OFF_REG 0x10 +#define PD_PSW_DELAY_REG 0x14 +#define PD_OFF_DELAY_REG 0x18 +#define PD_ON_DELAY_REG 0x1c +#define PD_COMMAND_REG 0x20 +#define PD_STATUS_REG 0x24 +#define PD_STATUS_COMPLETE BIT(1) +#define PD_STATUS_BUSY BIT(3) +#define PD_STATUS_STATE GENMASK(17, 16) +#define PD_ACTIVE_CTRL_REG 0x2c +#define PD_GATE_STATUS_REG 0x30 +#define PD_RSTN_STATUS BIT(0) +#define PD_CLK_GATE_STATUS BIT(1) +#define PD_PWROFF_GATE_STATUS BIT(2) +#define PD_PSW_STATUS_REG 0x34 + +#define PD_REGS_SIZE 0x80 + +struct sun20i_ppu_desc { + const char *const *names; + unsigned int num_domains; +}; + +struct sun20i_ppu_pd { + struct generic_pm_domain genpd; + void __iomem *base; +}; + +#define to_sun20i_ppu_pd(_genpd) \ + container_of(_genpd, struct sun20i_ppu_pd, genpd) + +static bool sun20i_ppu_pd_is_on(const struct sun20i_ppu_pd *pd) +{ + u32 status = readl(pd->base + PD_STATUS_REG); + + return FIELD_GET(PD_STATUS_STATE, status) == PD_STATE_ON; +} + +static int sun20i_ppu_pd_set_power(const struct sun20i_ppu_pd *pd, bool power_on) +{ + u32 state, status; + int ret; + + if (sun20i_ppu_pd_is_on(pd) == power_on) + return 0; + + /* Wait for the power controller to be idle. */ + ret = readl_poll_timeout(pd->base + PD_STATUS_REG, status, + !(status & PD_STATUS_BUSY), 100, 1000); + if (ret) + return ret; + + state = power_on ? PD_STATE_ON : PD_STATE_OFF; + writel(state, pd->base + PD_COMMAND_REG); + + /* Wait for the state transition to complete. */ + ret = readl_poll_timeout(pd->base + PD_STATUS_REG, status, + FIELD_GET(PD_STATUS_STATE, status) == state && + (status & PD_STATUS_COMPLETE), 100, 1000); + if (ret) + return ret; + + /* Clear the completion flag. */ + writel(status, pd->base + PD_STATUS_REG); + + return 0; +} + +static int sun20i_ppu_pd_power_on(struct generic_pm_domain *genpd) +{ + const struct sun20i_ppu_pd *pd = to_sun20i_ppu_pd(genpd); + + return sun20i_ppu_pd_set_power(pd, true); +} + +static int sun20i_ppu_pd_power_off(struct generic_pm_domain *genpd) +{ + const struct sun20i_ppu_pd *pd = to_sun20i_ppu_pd(genpd); + + return sun20i_ppu_pd_set_power(pd, false); +} + +static int sun20i_ppu_probe(struct platform_device *pdev) +{ + const struct sun20i_ppu_desc *desc; + struct device *dev = &pdev->dev; + struct genpd_onecell_data *ppu; + struct sun20i_ppu_pd *pds; + struct reset_control *rst; + void __iomem *base; + struct clk *clk; + int ret; + + desc = of_device_get_match_data(dev); + if (!desc) + return -EINVAL; + + pds = devm_kcalloc(dev, desc->num_domains, sizeof(*pds), GFP_KERNEL); + if (!pds) + return -ENOMEM; + + ppu = devm_kzalloc(dev, sizeof(*ppu), GFP_KERNEL); + if (!ppu) + return -ENOMEM; + + ppu->domains = devm_kcalloc(dev, desc->num_domains, + sizeof(*ppu->domains), GFP_KERNEL); + if (!ppu->domains) + return -ENOMEM; + + ppu->num_domains = desc->num_domains; + platform_set_drvdata(pdev, ppu); + + base = devm_platform_ioremap_resource(pdev, 0); + if (IS_ERR(base)) + return PTR_ERR(base); + + clk = devm_clk_get_enabled(dev, NULL); + if (IS_ERR(clk)) + return PTR_ERR(clk); + + rst = devm_reset_control_get_exclusive(dev, NULL); + if (IS_ERR(rst)) + return PTR_ERR(rst); + + ret = reset_control_deassert(rst); + if (ret) + return ret; + + for (unsigned int i = 0; i < ppu->num_domains; ++i) { + struct sun20i_ppu_pd *pd = &pds[i]; + + pd->genpd.name = desc->names[i]; + pd->genpd.power_off = sun20i_ppu_pd_power_off; + pd->genpd.power_on = sun20i_ppu_pd_power_on; + pd->base = base + PD_REGS_SIZE * i; + + ret = pm_genpd_init(&pd->genpd, NULL, sun20i_ppu_pd_is_on(pd)); + if (ret) { + dev_warn(dev, "Failed to add '%s' domain: %d\n", + pd->genpd.name, ret); + continue; + } + + ppu->domains[i] = &pd->genpd; + } + + ret = of_genpd_add_provider_onecell(dev->of_node, ppu); + if (ret) + dev_warn(dev, "Failed to add provider: %d\n", ret); + + return 0; +} + +static const char *const sun20i_d1_ppu_pd_names[] = { + "CPU", + "VE", + "DSP", +}; + +static const struct sun20i_ppu_desc sun20i_d1_ppu_desc = { + .names = sun20i_d1_ppu_pd_names, + .num_domains = ARRAY_SIZE(sun20i_d1_ppu_pd_names), +}; + +static const struct of_device_id sun20i_ppu_of_match[] = { + { + .compatible = "allwinner,sun20i-d1-ppu", + .data = &sun20i_d1_ppu_desc, + }, + { } +}; +MODULE_DEVICE_TABLE(of, sun20i_ppu_of_match); + +static struct platform_driver sun20i_ppu_driver = { + .probe = sun20i_ppu_probe, + .driver = { + .name = "sun20i-ppu", + .of_match_table = sun20i_ppu_of_match, + /* Power domains cannot be removed while they are in use. */ + .suppress_bind_attrs = true, + }, +}; +module_platform_driver(sun20i_ppu_driver); + +MODULE_AUTHOR("Samuel Holland <samuel@sholland.org>"); +MODULE_DESCRIPTION("Allwinner D1 PPU power domain driver"); +MODULE_LICENSE("GPL"); -- 2.37.4 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 2/2] soc: sunxi: Add Allwinner D1 PPU driver 2022-12-31 16:04 ` [PATCH 2/2] soc: sunxi: Add Allwinner D1 PPU driver Samuel Holland @ 2023-01-05 16:38 ` Jernej Škrabec 0 siblings, 0 replies; 8+ messages in thread From: Jernej Škrabec @ 2023-01-05 16:38 UTC (permalink / raw) To: Chen-Yu Tsai, Krzysztof Kozlowski, Rob Herring, Samuel Holland Cc: Samuel Holland, Philipp Zabel, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi Dne sobota, 31. december 2022 ob 17:04:02 CET je Samuel Holland napisal(a): > The PPU contains a series of identical MMIO register ranges, one for > each power domain. Each range contains control/status bits for a clock > gate, reset line, output gates, and a power switch. (The clock and reset > are separate from, and in addition to, the bits in the CCU.) It also > contains a hardware power sequence engine to control the other bits. > > Signed-off-by: Samuel Holland <samuel@sholland.org> > --- > > drivers/soc/sunxi/Kconfig | 9 ++ > drivers/soc/sunxi/Makefile | 1 + > drivers/soc/sunxi/sun20i-ppu.c | 207 +++++++++++++++++++++++++++++++++ > 3 files changed, 217 insertions(+) > create mode 100644 drivers/soc/sunxi/sun20i-ppu.c > > diff --git a/drivers/soc/sunxi/Kconfig b/drivers/soc/sunxi/Kconfig > index 8aecbc9b1976..5e84cf4b8510 100644 > --- a/drivers/soc/sunxi/Kconfig > +++ b/drivers/soc/sunxi/Kconfig > @@ -19,3 +19,12 @@ config SUNXI_SRAM > Say y here to enable the SRAM controller support. This > device is responsible on mapping the SRAM in the sunXi SoCs > whether to the CPU/DMA, or to the devices. > + > +config SUN20I_PPU > + bool "Allwinner D1 PPU power domain driver" > + depends on ARCH_SUNXI || COMPILE_TEST > + select PM_GENERIC_DOMAINS > + help > + Say y to enable the PPU power domain driver. This saves power > + when certain peripherals, such as the video engine, are idle. > + All power domains are on by default, so this is optional. If this driver will gain support for other SoCs, above statement might not be true anymore. One such example is GPU on H616 (if PPU is compatible). Other than that, driver looks good: Acked-by: Jernej Skrabec <jernej.skrabec@gmail.com> Best regards, Jernej > diff --git a/drivers/soc/sunxi/Makefile b/drivers/soc/sunxi/Makefile > index 549159571d4f..90ff2ebe7655 100644 > --- a/drivers/soc/sunxi/Makefile > +++ b/drivers/soc/sunxi/Makefile > @@ -1,3 +1,4 @@ > # SPDX-License-Identifier: GPL-2.0-only > obj-$(CONFIG_SUNXI_MBUS) += sunxi_mbus.o > obj-$(CONFIG_SUNXI_SRAM) += sunxi_sram.o > +obj-$(CONFIG_SUN20I_PPU) += sun20i-ppu.o > diff --git a/drivers/soc/sunxi/sun20i-ppu.c b/drivers/soc/sunxi/sun20i-ppu.c > new file mode 100644 > index 000000000000..98cb41d36560 > --- /dev/null > +++ b/drivers/soc/sunxi/sun20i-ppu.c > @@ -0,0 +1,207 @@ > +// SPDX-License-Identifier: GPL-2.0-only > + > +#include <linux/bitfield.h> > +#include <linux/clk.h> > +#include <linux/io.h> > +#include <linux/iopoll.h> > +#include <linux/module.h> > +#include <linux/of_device.h> > +#include <linux/platform_device.h> > +#include <linux/pm_domain.h> > +#include <linux/reset.h> > + > +#define PD_STATE_ON 1 > +#define PD_STATE_OFF 2 > + > +#define PD_RSTN_REG 0x00 > +#define PD_CLK_GATE_REG 0x04 > +#define PD_PWROFF_GATE_REG 0x08 > +#define PD_PSW_ON_REG 0x0c > +#define PD_PSW_OFF_REG 0x10 > +#define PD_PSW_DELAY_REG 0x14 > +#define PD_OFF_DELAY_REG 0x18 > +#define PD_ON_DELAY_REG 0x1c > +#define PD_COMMAND_REG 0x20 > +#define PD_STATUS_REG 0x24 > +#define PD_STATUS_COMPLETE BIT(1) > +#define PD_STATUS_BUSY BIT(3) > +#define PD_STATUS_STATE GENMASK(17, 16) > +#define PD_ACTIVE_CTRL_REG 0x2c > +#define PD_GATE_STATUS_REG 0x30 > +#define PD_RSTN_STATUS BIT(0) > +#define PD_CLK_GATE_STATUS BIT(1) > +#define PD_PWROFF_GATE_STATUS BIT(2) > +#define PD_PSW_STATUS_REG 0x34 > + > +#define PD_REGS_SIZE 0x80 > + > +struct sun20i_ppu_desc { > + const char *const *names; > + unsigned int num_domains; > +}; > + > +struct sun20i_ppu_pd { > + struct generic_pm_domain genpd; > + void __iomem *base; > +}; > + > +#define to_sun20i_ppu_pd(_genpd) \ > + container_of(_genpd, struct sun20i_ppu_pd, genpd) > + > +static bool sun20i_ppu_pd_is_on(const struct sun20i_ppu_pd *pd) > +{ > + u32 status = readl(pd->base + PD_STATUS_REG); > + > + return FIELD_GET(PD_STATUS_STATE, status) == PD_STATE_ON; > +} > + > +static int sun20i_ppu_pd_set_power(const struct sun20i_ppu_pd *pd, bool > power_on) +{ > + u32 state, status; > + int ret; > + > + if (sun20i_ppu_pd_is_on(pd) == power_on) > + return 0; > + > + /* Wait for the power controller to be idle. */ > + ret = readl_poll_timeout(pd->base + PD_STATUS_REG, status, > + !(status & PD_STATUS_BUSY), 100, 1000); > + if (ret) > + return ret; > + > + state = power_on ? PD_STATE_ON : PD_STATE_OFF; > + writel(state, pd->base + PD_COMMAND_REG); > + > + /* Wait for the state transition to complete. */ > + ret = readl_poll_timeout(pd->base + PD_STATUS_REG, status, > + FIELD_GET(PD_STATUS_STATE, status) == state && > + (status & PD_STATUS_COMPLETE), 100, 1000); > + if (ret) > + return ret; > + > + /* Clear the completion flag. */ > + writel(status, pd->base + PD_STATUS_REG); > + > + return 0; > +} > + > +static int sun20i_ppu_pd_power_on(struct generic_pm_domain *genpd) > +{ > + const struct sun20i_ppu_pd *pd = to_sun20i_ppu_pd(genpd); > + > + return sun20i_ppu_pd_set_power(pd, true); > +} > + > +static int sun20i_ppu_pd_power_off(struct generic_pm_domain *genpd) > +{ > + const struct sun20i_ppu_pd *pd = to_sun20i_ppu_pd(genpd); > + > + return sun20i_ppu_pd_set_power(pd, false); > +} > + > +static int sun20i_ppu_probe(struct platform_device *pdev) > +{ > + const struct sun20i_ppu_desc *desc; > + struct device *dev = &pdev->dev; > + struct genpd_onecell_data *ppu; > + struct sun20i_ppu_pd *pds; > + struct reset_control *rst; > + void __iomem *base; > + struct clk *clk; > + int ret; > + > + desc = of_device_get_match_data(dev); > + if (!desc) > + return -EINVAL; > + > + pds = devm_kcalloc(dev, desc->num_domains, sizeof(*pds), GFP_KERNEL); > + if (!pds) > + return -ENOMEM; > + > + ppu = devm_kzalloc(dev, sizeof(*ppu), GFP_KERNEL); > + if (!ppu) > + return -ENOMEM; > + > + ppu->domains = devm_kcalloc(dev, desc->num_domains, > + sizeof(*ppu->domains), GFP_KERNEL); > + if (!ppu->domains) > + return -ENOMEM; > + > + ppu->num_domains = desc->num_domains; > + platform_set_drvdata(pdev, ppu); > + > + base = devm_platform_ioremap_resource(pdev, 0); > + if (IS_ERR(base)) > + return PTR_ERR(base); > + > + clk = devm_clk_get_enabled(dev, NULL); > + if (IS_ERR(clk)) > + return PTR_ERR(clk); > + > + rst = devm_reset_control_get_exclusive(dev, NULL); > + if (IS_ERR(rst)) > + return PTR_ERR(rst); > + > + ret = reset_control_deassert(rst); > + if (ret) > + return ret; > + > + for (unsigned int i = 0; i < ppu->num_domains; ++i) { > + struct sun20i_ppu_pd *pd = &pds[i]; > + > + pd->genpd.name = desc->names[i]; > + pd->genpd.power_off = sun20i_ppu_pd_power_off; > + pd->genpd.power_on = sun20i_ppu_pd_power_on; > + pd->base = base + PD_REGS_SIZE * i; > + > + ret = pm_genpd_init(&pd->genpd, NULL, sun20i_ppu_pd_is_on(pd)); > + if (ret) { > + dev_warn(dev, "Failed to add '%s' domain: %d\n", > + pd->genpd.name, ret); > + continue; > + } > + > + ppu->domains[i] = &pd->genpd; > + } > + > + ret = of_genpd_add_provider_onecell(dev->of_node, ppu); > + if (ret) > + dev_warn(dev, "Failed to add provider: %d\n", ret); > + > + return 0; > +} > + > +static const char *const sun20i_d1_ppu_pd_names[] = { > + "CPU", > + "VE", > + "DSP", > +}; > + > +static const struct sun20i_ppu_desc sun20i_d1_ppu_desc = { > + .names = sun20i_d1_ppu_pd_names, > + .num_domains = ARRAY_SIZE(sun20i_d1_ppu_pd_names), > +}; > + > +static const struct of_device_id sun20i_ppu_of_match[] = { > + { > + .compatible = "allwinner,sun20i-d1-ppu", > + .data = &sun20i_d1_ppu_desc, > + }, > + { } > +}; > +MODULE_DEVICE_TABLE(of, sun20i_ppu_of_match); > + > +static struct platform_driver sun20i_ppu_driver = { > + .probe = sun20i_ppu_probe, > + .driver = { > + .name = "sun20i-ppu", > + .of_match_table = sun20i_ppu_of_match, > + /* Power domains cannot be removed while they are in use. */ > + .suppress_bind_attrs = true, > + }, > +}; > +module_platform_driver(sun20i_ppu_driver); > + > +MODULE_AUTHOR("Samuel Holland <samuel@sholland.org>"); > +MODULE_DESCRIPTION("Allwinner D1 PPU power domain driver"); > +MODULE_LICENSE("GPL"); ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] Allwinner power domain support 2022-12-31 16:04 [PATCH 0/2] Allwinner power domain support Samuel Holland 2022-12-31 16:04 ` [PATCH 1/2] dt-bindings: power: Add Allwinner D1 PPU Samuel Holland 2022-12-31 16:04 ` [PATCH 2/2] soc: sunxi: Add Allwinner D1 PPU driver Samuel Holland @ 2023-01-05 16:34 ` Jernej Škrabec 2023-01-26 6:23 ` Samuel Holland 2 siblings, 1 reply; 8+ messages in thread From: Jernej Škrabec @ 2023-01-05 16:34 UTC (permalink / raw) To: Chen-Yu Tsai, Krzysztof Kozlowski, Rob Herring, Samuel Holland Cc: Samuel Holland, Philipp Zabel, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi Dne sobota, 31. december 2022 ob 17:04:00 CET je Samuel Holland napisal(a): > This series adds support for the power controller found in D1 and other > recent Allwinner SoCs. There is no first-party documentation, but there > are a couple of vendor drivers for different hardware revisions[1][2], > and the register definitions were easy to verify empirically. > > I have tested this driver on D1 with the video engine. There is no DT > update patch here to avoid dependencies between series. The example in > the binding is what will go in the D1 DT. So such driver is needed for H616 for GPU? Or is power domain handling different there? Best regards, Jernej > > [1]: > https://github.com/mangopi-sbc/tina-linux-5.4/blob/main/drivers/soc/sunxi/g > pu_domain.c [1]: > https://github.com/mangopi-sbc/tina-linux-5.4/blob/main/drivers/soc/sunxi/p > m_domains.c > > > Samuel Holland (2): > dt-bindings: power: Add Allwinner D1 PPU > soc: sunxi: Add Allwinner D1 PPU driver > > .../power/allwinner,sun20i-d1-ppu.yaml | 54 +++++ > drivers/soc/sunxi/Kconfig | 9 + > drivers/soc/sunxi/Makefile | 1 + > drivers/soc/sunxi/sun20i-ppu.c | 207 ++++++++++++++++++ > .../power/allwinner,sun20i-d1-ppu.h | 10 + > 5 files changed, 281 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/power/allwinner,sun20i-d1-ppu.yaml create > mode 100644 drivers/soc/sunxi/sun20i-ppu.c > create mode 100644 include/dt-bindings/power/allwinner,sun20i-d1-ppu.h ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 0/2] Allwinner power domain support 2023-01-05 16:34 ` [PATCH 0/2] Allwinner power domain support Jernej Škrabec @ 2023-01-26 6:23 ` Samuel Holland 0 siblings, 0 replies; 8+ messages in thread From: Samuel Holland @ 2023-01-26 6:23 UTC (permalink / raw) To: Jernej Škrabec, Chen-Yu Tsai Cc: Philipp Zabel, devicetree, linux-arm-kernel, linux-kernel, linux-sunxi, Krzysztof Kozlowski, Rob Herring Hi Jernej, On 1/5/23 10:34, Jernej Škrabec wrote: > Dne sobota, 31. december 2022 ob 17:04:00 CET je Samuel Holland napisal(a): >> This series adds support for the power controller found in D1 and other >> recent Allwinner SoCs. There is no first-party documentation, but there >> are a couple of vendor drivers for different hardware revisions[1][2], >> and the register definitions were easy to verify empirically. >> >> I have tested this driver on D1 with the video engine. There is no DT >> update patch here to avoid dependencies between series. The example in >> the binding is what will go in the D1 DT. > > So such driver is needed for H616 for GPU? Or is power domain handling > different there? H616 does not appear to have a PPU. The PRCM gates otherwise match A100, but there are no settable gate/reset bits at 0x17c, and the registers at 0x7001000 read as zero, even after being written. I believe H616 uses only GPU_PWROFF_GATING_REG in the PRCM. Regards, Samuel ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-01-26 6:23 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-12-31 16:04 [PATCH 0/2] Allwinner power domain support Samuel Holland 2022-12-31 16:04 ` [PATCH 1/2] dt-bindings: power: Add Allwinner D1 PPU Samuel Holland 2022-12-31 18:58 ` Krzysztof Kozlowski 2022-12-31 19:12 ` Samuel Holland 2022-12-31 16:04 ` [PATCH 2/2] soc: sunxi: Add Allwinner D1 PPU driver Samuel Holland 2023-01-05 16:38 ` Jernej Škrabec 2023-01-05 16:34 ` [PATCH 0/2] Allwinner power domain support Jernej Škrabec 2023-01-26 6:23 ` Samuel Holland
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).