Re: [PATCH 3/3] clk: split of_clk_get_parent_name() into two functions

From: Stephen Boyd
Date: Fri Nov 20 2015 - 19:37:39 EST

On 11/20, Masahiro Yamada wrote:
> Currently, there is no function to get the clock name of the given
> node. Create a new helper function, of_clk_get_name(). This is
> useful to get the clock name where "clock-indices" property is used.
> of_clk_get_name(): get the clock name in the given node
> of_clk_get_parent_name(): get the name of the parent clock
> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
> ---
> I want to use of_clk_get_name() for my clk drivers for my SoCs,
> which I will submit later.
> I found this helper function is useful.

I don't see how this is useful. Is the new driver so generic it
doesn't know what clocks it's outputting? We've been trying to
move people away from using clock-output-names, so most likely
this sort of information should be conveyed from DT via the
compatible string and a table in the driver that matches up the
compatible string with the list of clock names.

> @@ -3102,7 +3114,6 @@ const char *of_clk_get_parent_name(struct device_node *np, int index)
> }
> }
> -

Noise. Please remove.

> of_node_put(;
> return clk_name;
> }

Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at