On Thu, Feb 26, 2026 at 07:04:15PM +0800, Peng Fan (OSS) wrote:
> From: Peng Fan <peng.fan@nxp.com>
>
> Add missing #clock-cells to make it work as a clock controller.
Yes, that is what adding #clock-cells would do, but why is it correct to
do? That's what your commit message needs to explain.
>
> Signed-off-by: Peng Fan <peng.fan@nxp.com>
> ---
> Documentation/devicetree/bindings/arm/freescale/fsl,imx7ulp-pm.yaml | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/arm/freescale/fsl,imx7ulp-pm.yaml b/Documentation/devicetree/bindings/arm/freescale/fsl,imx7ulp-pm.yaml
> index 9d377e193c123c7de0ec4db4d4a649ed966b2d9a..7ad470260c0d08bd1e7146ef49e5f60dd6c6d4d7 100644
> --- a/Documentation/devicetree/bindings/arm/freescale/fsl,imx7ulp-pm.yaml
> +++ b/Documentation/devicetree/bindings/arm/freescale/fsl,imx7ulp-pm.yaml
> @@ -28,6 +28,9 @@ properties:
> reg:
> maxItems: 1
>
> + '#clock-cells':
> + const: 1
> +
> clocks:
> maxItems: 2
>
> @@ -39,6 +42,7 @@ properties:
> required:
> - compatible
> - reg
> + - '#clock-cells'
>
> additionalProperties: false
>
> @@ -47,4 +51,5 @@ examples:
> smc1@40410000 {
> compatible = "fsl,imx7ulp-smc1";
> reg = <0x40410000 0x1000>;
> + #clock-cells = <1>;
> };
>
> --
> 2.37.1
>