On 4/22/2024 5:29 PM, Rob Herring wrote: > On Thu, Apr 18, 2024 at 11:37:19PM +0300, Laurentiu Mihalcea wrote: >> From: Laurentiu Mihalcea <laurentiu.mihalcea@xxxxxxx> >> >> Add schema for i.MX8ULP's AVD-SIM module. >> >> Signed-off-by: Liu Ying <victor.liu@xxxxxxx> >> Signed-off-by: Laurentiu Mihalcea <laurentiu.mihalcea@xxxxxxx> >> --- >> .../bindings/mfd/fsl,imx8ulp-avd-sim.yaml | 42 +++++++++++++++++++ >> 1 file changed, 42 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/mfd/fsl,imx8ulp-avd-sim.yaml >> >> diff --git a/Documentation/devicetree/bindings/mfd/fsl,imx8ulp-avd-sim.yaml b/Documentation/devicetree/bindings/mfd/fsl,imx8ulp-avd-sim.yaml >> new file mode 100644 >> index 000000000000..4020c6e37f80 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/mfd/fsl,imx8ulp-avd-sim.yaml >> @@ -0,0 +1,42 @@ >> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) >> +%YAML 1.2 >> +--- >> +$id: http://devicetree.org/schemas/mfd/fsl,imx8ulp-avd-sim.yaml# >> +$schema: http://devicetree.org/meta-schemas/core.yaml# >> + >> +title: NXP i.MX8ULP Audio-Video Domain System Integration Module >> + >> +maintainers: >> + - Liu Ying <victor.liu@xxxxxxx> >> + >> +description: | >> + The AVD-SIM module provides configuration options for components of AVD. >> + >> +properties: >> + compatible: >> + items: >> + - const: fsl,imx8ulp-avd-sim >> + - const: syscon >> + - const: simple-mfd > 'simple-mfd' means you have child nodes, but you have none defined. Thanks for the clarification! These were intentionally omitted since they're not needed for the DSP. > >> + >> + reg: >> + maxItems: 1 >> + >> + clocks: >> + maxItems: 1 > If you do add child nodes, 'simple-mfd' also means there is not any > dependency on the parent node such as needing this clock to be enabled > for the child nodes. > > Rob The plan was to add the children later on as required. Given the children would in fact depend on the parent's clock being enabled, I wonder if just removing 'simple-mfd' from the compatible list would do the trick?
