Re: [PATCH v4 1/4] dt-bindings: clock: add Qualcomm IPQ9650 GCC

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] The binding description claims the module provides power domains, but the schema explicitly disables this capability.
--

commit 4db89a6dba2bb07dd7133c52ce96e3e38a472691
Author: Kathiravan Thirumoorthy <kathiravan.thirumoorthy@oss.qualcomm.com>

dt-bindings: clock: add Qualcomm IPQ9650 GCC

This commit adds the device tree binding for the Qualcomm IPQ9650 Global
Clock Controller. It includes the YAML schema and header files for the 
clocks and resets provided by the module.

> diff --git a/Documentation/devicetree/bindings/clock/qcom,ipq9650-gcc.yaml b/Documentation/devicetree/bindings/clock/qcom,ipq9650-gcc.yaml
> new file mode 100644
> index 0000000000000..f33105217a062
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/clock/qcom,ipq9650-gcc.yaml
[ ... ]
> +description: |
> +  Qualcomm global clock control module provides the clocks, resets and power
> +  domains on IPQ9650

This isn't a bug, but the description mentions that the module provides power
domains.

[ ... ]
> +  '#power-domain-cells': false

Does this controller actually provide power domains? The schema explicitly
disables them here, which suggests the mention of power domains in the
description might be a leftover from copying another binding.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260512-ipq9650_boot_to_shell-v4-0-1e87c8326e59@oss.qualcomm.com?part=1