On Tue, May 09, 2023 at 06:53:07PM -0700, Changhuang Liang wrote: > Add AON PMU for StarFive JH7110 SoC, it can be used to turn on/off DPHY > rx/tx power switch, and it don't need the property of interrupts. It > also can use syscon operation. > > Signed-off-by: Changhuang Liang <changhuang.liang@xxxxxxxxxxxxxxxx> > --- > .../bindings/power/starfive,jh7110-pmu.yaml | 28 +++++++++++++++++-- > .../dt-bindings/power/starfive,jh7110-pmu.h | 5 +++- > 2 files changed, 29 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/power/starfive,jh7110-pmu.yaml b/Documentation/devicetree/bindings/power/starfive,jh7110-pmu.yaml > index 98eb8b4110e7..0591a4e9db6c 100644 > --- a/Documentation/devicetree/bindings/power/starfive,jh7110-pmu.yaml > +++ b/Documentation/devicetree/bindings/power/starfive,jh7110-pmu.yaml > @@ -8,6 +8,7 @@ title: StarFive JH7110 Power Management Unit > > maintainers: > - Walker Chen <walker.chen@xxxxxxxxxxxxxxxx> > + - Changhuang Liang <changhuang.liang@xxxxxxxxxxxxxxxx> > > description: | > StarFive JH7110 SoC includes support for multiple power domains which can be > @@ -15,8 +16,13 @@ description: | > > properties: > compatible: > - enum: > - - starfive,jh7110-pmu > + oneOf: > + - enum: > + - starfive,jh7110-pmu > + - items: > + - enum: > + - starfive,jh7110-aon-syscon > + - const: syscon Unfortunately, this is not what was wanted. This syscon, of which power domain control is just one of the things that it can do, should be documented in Documentation/devicetree/bindings/soc/starfive/starfive,jh7110-syscon.yaml alongside it's brethern "starfive,jh7110-sys-syscon" & "starfive,jh7110-stg-syscon". Thanks, Conor.
Attachment:
signature.asc
Description: PGP signature
