[U-Boot] [PATCH v2] dm: clk: Remove simple version of clk_get_by_index/name()

[U-Boot] [PATCH v2] dm: clk: Remove simple version of clk_get_by_index/name()

[Michal Simek]

Simple version of clk_get_by_index() added by:
"dm: clk: Add a simple version of clk_get_by_index()"
(sha1: a4b10c088c4f6ef2e2bba33e8cfea369bcbbce44)
is only working for #clock-cells=<1> but not for
any other values. Fixed clocks is using #clock-cells=<0>
which requires full implementation.

Remove simplified versions of clk_get_by_index() and use full version.
Also remove empty clk_get_by_name() which is failing when it is called
which is useless.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

Changes in v2:
- Also use full version of clk_get_by_name()
- Subject change from "Remove simple version of clk_get_by_index()"

---
 drivers/clk/clk-uclass.c | 26 --------------------------
 1 file changed, 26 deletions(-)

diff --git a/drivers/clk/clk-uclass.c b/drivers/clk/clk-uclass.c
index 6e4d67220a16..f6fd4cbf57b6 100644
--- a/drivers/clk/clk-uclass.c
+++ b/drivers/clk/clk-uclass.c
@@ -20,31 +20,6 @@ static inline struct clk_ops *clk_dev_ops(struct udevice *dev)
 }
 
 #if CONFIG_IS_ENABLED(OF_CONTROL)
-#ifdef CONFIG_SPL_BUILD
-int clk_get_by_index(struct udevice *dev, int index, struct clk *clk)
-{
-	int ret;
-	u32 cell[2];
-
-	if (index != 0)
-		return -ENOSYS;
-	assert(clk);
-	ret = uclass_get_device(UCLASS_CLK, 0, &clk->dev);
-	if (ret)
-		return ret;
-	ret = fdtdec_get_int_array(gd->fdt_blob, dev->of_offset, "clocks",
-				   cell, 2);
-	if (ret)
-		return ret;
-	clk->id = cell[1];
-	return 0;
-}
-
-int clk_get_by_name(struct udevice *dev, const char *name, struct clk *clk)
-{
-	return -ENOSYS;
-}
-#else
 static int clk_of_xlate_default(struct clk *clk,
 				struct fdtdec_phandle_args *args)
 {
@@ -118,7 +93,6 @@ int clk_get_by_name(struct udevice *dev, const char *name, struct clk *clk)
 	return clk_get_by_index(dev, index, clk);
 }
 #endif
-#endif
 
 int clk_request(struct udevice *dev, struct clk *clk)
 {
-- 
1.9.1

[U-Boot] [PATCH v2] dm: clk: Remove simple version of clk_get_by_index/name()

[Stephen Warren]

On 07/15/2016 01:02 AM, Michal Simek wrote:
> Simple version of clk_get_by_index() added by:
> "dm: clk: Add a simple version of clk_get_by_index()"
> (sha1: a4b10c088c4f6ef2e2bba33e8cfea369bcbbce44)
> is only working for #clock-cells=<1> but not for
> any other values. Fixed clocks is using #clock-cells=<0>
> which requires full implementation.
>
> Remove simplified versions of clk_get_by_index() and use full version.
> Also remove empty clk_get_by_name() which is failing when it is called
> which is useless.

This seems reasonable to me,
Acked-by: Stephen Warren <swarren@nvidia.com>

[U-Boot] [PATCH v2] dm: clk: Remove simple version of clk_get_by_index/name()

[Michal Simek]

Hi Simon,

On 15.7.2016 17:43, Stephen Warren wrote:
> On 07/15/2016 01:02 AM, Michal Simek wrote:
>> Simple version of clk_get_by_index() added by:
>> "dm: clk: Add a simple version of clk_get_by_index()"
>> (sha1: a4b10c088c4f6ef2e2bba33e8cfea369bcbbce44)
>> is only working for #clock-cells=<1> but not for
>> any other values. Fixed clocks is using #clock-cells=<0>
>> which requires full implementation.
>>
>> Remove simplified versions of clk_get_by_index() and use full version.
>> Also remove empty clk_get_by_name() which is failing when it is called
>> which is useless.
> 
> This seems reasonable to me,
> Acked-by: Stephen Warren <swarren@nvidia.com>

I see your patch
"dm: clk: Add support for of-platdata"
(sha1: 	7423daa60eb30b6613dfc19a51c55de23fd4d703)
has been merged to the tree and it is in conflict with this patch.
Is there any problem about using full version and just remove the whole
SPL part?

Thanks,
Michal

[U-Boot] [PATCH v2] dm: clk: Remove simple version of clk_get_by_index/name()

[Simon Glass]

Hi Michal,

On 18 July 2016 at 00:57, Michal Simek <michal.simek@xilinx.com> wrote:
> Hi Simon,
>
> On 15.7.2016 17:43, Stephen Warren wrote:
>> On 07/15/2016 01:02 AM, Michal Simek wrote:
>>> Simple version of clk_get_by_index() added by:
>>> "dm: clk: Add a simple version of clk_get_by_index()"
>>> (sha1: a4b10c088c4f6ef2e2bba33e8cfea369bcbbce44)
>>> is only working for #clock-cells=<1> but not for
>>> any other values. Fixed clocks is using #clock-cells=<0>
>>> which requires full implementation.
>>>
>>> Remove simplified versions of clk_get_by_index() and use full version.
>>> Also remove empty clk_get_by_name() which is failing when it is called
>>> which is useless.
>>
>> This seems reasonable to me,
>> Acked-by: Stephen Warren <swarren@nvidia.com>
>
> I see your patch
> "dm: clk: Add support for of-platdata"
> (sha1:  7423daa60eb30b6613dfc19a51c55de23fd4d703)
> has been merged to the tree and it is in conflict with this patch.
> Is there any problem about using full version and just remove the whole
> SPL part?

Yes I was going to mention that. Unfortunately we need the of-platdata
version since in that case there is no device tree (it has been
converted to a C struct).

Regards,
Simon

[U-Boot] [PATCH v2] dm: clk: Remove simple version of clk_get_by_index/name()

[Michal Simek]

Hi Simon,

On 18.7.2016 13:20, Simon Glass wrote:
> Hi Michal,
> 
> On 18 July 2016 at 00:57, Michal Simek <michal.simek@xilinx.com> wrote:
>> Hi Simon,
>>
>> On 15.7.2016 17:43, Stephen Warren wrote:
>>> On 07/15/2016 01:02 AM, Michal Simek wrote:
>>>> Simple version of clk_get_by_index() added by:
>>>> "dm: clk: Add a simple version of clk_get_by_index()"
>>>> (sha1: a4b10c088c4f6ef2e2bba33e8cfea369bcbbce44)
>>>> is only working for #clock-cells=<1> but not for
>>>> any other values. Fixed clocks is using #clock-cells=<0>
>>>> which requires full implementation.
>>>>
>>>> Remove simplified versions of clk_get_by_index() and use full version.
>>>> Also remove empty clk_get_by_name() which is failing when it is called
>>>> which is useless.
>>>
>>> This seems reasonable to me,
>>> Acked-by: Stephen Warren <swarren@nvidia.com>
>>
>> I see your patch
>> "dm: clk: Add support for of-platdata"
>> (sha1:  7423daa60eb30b6613dfc19a51c55de23fd4d703)
>> has been merged to the tree and it is in conflict with this patch.
>> Is there any problem about using full version and just remove the whole
>> SPL part?
> 
> Yes I was going to mention that. Unfortunately we need the of-platdata
> version since in that case there is no device tree (it has been
> converted to a C struct).

v3 sent which address this.

Thanks,
Michal