RE: [PATCH V2 1/7] dt-bindings: soc: add i.MX93 SRC

[Peng Fan]

> Subject: Re: [PATCH V2 1/7] dt-bindings: soc: add i.MX93 SRC
> 
> On 26/05/2022 14:34, Peng Fan (OSS) wrote:
> > From: Peng Fan <peng.fan@xxxxxxx>
> >
> > Add bindings for i.MX93 System Reset Controller(SRC). SRC supports
> > resets and power gating for mixes.
> >
> > Signed-off-by: Peng Fan <peng.fan@xxxxxxx>
> > ---
> >  .../bindings/soc/imx/fsl,imx93-src.yaml       | 96
> +++++++++++++++++++
> 
> Still wrong location of bindings.

Although it is called SRC(system reset controller), but actually
it not functions only as reset controller, it also supports power
gating of each slice including slice's memory low power control.
It also includes some system general configuration.

So follow your suggestion in V1, I rename it as system-controller,
but I not find a directory for system-controller.

> 
> >  include/dt-bindings/power/fsl,imx93-power.h   | 11 +++
> >  2 files changed, 107 insertions(+)
> >  create mode 100644
> > Documentation/devicetree/bindings/soc/imx/fsl,imx93-src.yaml
> >  create mode 100644 include/dt-bindings/power/fsl,imx93-power.h
> >
> > diff --git
> > a/Documentation/devicetree/bindings/soc/imx/fsl,imx93-src.yaml
> > b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-src.yaml
> > new file mode 100644
> > index 000000000000..c20f0bb692fe
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/soc/imx/fsl,imx93-src.yaml
> > @@ -0,0 +1,96 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
> > +
> > +title: NXP i.MX9 System Reset Controller
> > +
> > +maintainers:
> > +  - Peng Fan <peng.fan@xxxxxxx>
> > +
> > +description: |
> > +  The System Reset Controller (SRC) is responsible for the generation
> > +of
> > +  all the system reset signals and boot argument latching.
> > +
> > +  Its main functions are as follows,
> > +  - Deals with all global system reset sources from other modules,
> > +    and generates global system reset.
> > +  - Responsible for power gating of MIXs (Slices) and their memory
> > +    low power control.
> > +
> > +properties:
> > +  compatible:
> > +    items:
> > +      - const: fsl,imx93-src
> > +      - const: syscon
> > +
> > +  reg:
> > +    maxItems: 1
> > +
> > +  slice:
> > +    type: object
> > +    description: list of power domains provided by this controller.
> 
> Continuing our discussion, I think I see the point of having the node grouping
> subnodes, just the name confused me. Please make it plural, so "slices".

Sure, fix in V3.

Thanks,
Peng.

> 
> 
> Best regards,
> Krzysztof