On 07/03/2022 13:21, Tim Chang wrote:
> 1. add cci property.
> 2. add example of MT8186.
One logical change at a time. Are these related? Why entirely new
example just for "cci" node? Maybe this should be part of existing example?
>
> Signed-off-by: Jia-Wei Chang <[email protected]>
> ---
> .../bindings/cpufreq/cpufreq-mediatek.yaml | 41 +++++++++++++++++++
> 1 file changed, 41 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.yaml b/Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.yaml
> index 584946eb3790..d3ce17fd8fcf 100644
> --- a/Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.yaml
> +++ b/Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.yaml
> @@ -48,6 +48,10 @@ properties:
> When absent, the voltage scaling flow is handled by hardware, hence no
> software "voltage tracking" is needed.
>
> + cci:
> + description:
> + Phandle of the cci to be linked with the phandle of CPU if present.
This does not look like a standard type, so you need type.
Best regards,
Krzysztof