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@xxxxxxxxxxxx> >> --- >> >> .../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