Re: [PATCH 1/3] clk: Add new function of_clk_is_provider()

2016-06-21 Thread Ricardo Ribalda Delgado
Hi Stephen On Tue, Jun 21, 2016 at 3:30 AM, Stephen Boyd wrote: > (Please don't top post) Sorry about that > > I was thinking that of_clk_init() would mark any nodes that > matched and provided clk providers as OF_POPULATED. That way, > when of_platform_populate() ran, it would *not* add platfo

Re: [PATCH 1/3] clk: Add new function of_clk_is_provider()

2016-06-20 Thread Stephen Boyd
(Please don't top post) On 06/20, Ricardo Ribalda Delgado wrote: > Hi Stephen > > When the device tree is populated or when an overlay is added, all its > nodes have the flag OF_POPULATED set. The flag is enabled recursively > in > of_platform_bus_create->of_platform_device_create_pdata() > So we

Re: [PATCH 1/3] clk: Add new function of_clk_is_provider()

2016-06-20 Thread Ricardo Ribalda Delgado
Hi Stephen When the device tree is populated or when an overlay is added, all its nodes have the flag OF_POPULATED set. The flag is enabled recursively in of_platform_bus_create->of_platform_device_create_pdata() So we cannot use that flag to mark what is enabled and what is not. The other issue

Re: [PATCH 1/3] clk: Add new function of_clk_is_provider()

2016-06-15 Thread Stephen Boyd
On 06/08, Ricardo Ribalda Delgado wrote: > of_clk_is_provider() checks if a device_node has already been added to > the clk provider list. This can be used to avoid adding the same clock > provider twice. > > Signed-off-by: Ricardo Ribalda Delgado While I don't disagree with the concept, I'd lik

[PATCH 1/3] clk: Add new function of_clk_is_provider()

2016-06-08 Thread Ricardo Ribalda Delgado
of_clk_is_provider() checks if a device_node has already been added to the clk provider list. This can be used to avoid adding the same clock provider twice. Signed-off-by: Ricardo Ribalda Delgado --- drivers/clk/clk.c| 20 include/linux/clk-provider.h | 5 +