Re: [PATCH] clk: bindings: Fix assigned-clock-rates property name
From: Stephen Boyd <hidden>
Date: 2015-05-06 07:45:26
Also in:
linux-clk, lkml
On 05/06, Krzysztof Kozlowski wrote:
In documentation the "assigned-clock-parents" name was used when describing the "assigned-clock-rates" property. Signed-off-by: Krzysztof Kozlowski <redacted> ---
Looks right. The whole sentence could use some work though.
quoted hunk ↗ jump to hunk
Documentation/devicetree/bindings/clock/clock-bindings.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)diff --git a/Documentation/devicetree/bindings/clock/clock-bindings.txt b/Documentation/devicetree/bindings/clock/clock-bindings.txt index 06fc6d541c89..bf5b0414dc17 100644 --- a/Documentation/devicetree/bindings/clock/clock-bindings.txt +++ b/Documentation/devicetree/bindings/clock/clock-bindings.txt@@ -138,7 +138,7 @@ Some platforms may require initial configuration of default parent clocks and clock frequencies. Such a configuration can be specified in a device tree node through assigned-clocks, assigned-clock-parents and assigned-clock-rates properties. The assigned-clock-parents property should contain a list of parent -clocks in form of phandle and clock specifier pairs, the assigned-clock-parents +clocks in form of phandle and clock specifier pairs, the assigned-clock-rates property the list of assigned clock frequency values - corresponding to clocks
This half-sentence: The assigned-clock-ratse property the list of assigned clock frequency values doesn't read correctly. Maybe the whole thing should be rewritten as: The assigned-clock-parents property should contain a list of parent clocks in the form of a phandle and clock specifier pair and the assigned-clock-rates property should contain a list of frequencies in Hz. Both these properties should correspond to the clocks listed in the assigned-clocks property.
listed in the assigned-clocks property.
-- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project