On 01/11/2013 08:59 AM, Marc Dietrich wrote:

On Friday 11 January 2013 13:40:03 Prashant Gaikwad wrote:

quoted

On Thursday 10 January 2013 02:14 AM, Stephen Warren wrote:

quoted

On 01/09/2013 10:34 AM, Stephen Warren wrote:

...

quoted

quoted

Your changes don't actually cause the driver to break though, since it
abuses clk_get_sys() to retrieve clocks under a different driver name,
which matches what the clock driver provides. However, I think you
should also include the following patch at the end of your series to fix
this up, so the clock looking happens through device tree:

quoted

diff --git a/drivers/staging/nvec/nvec.c b/drivers/staging/nvec/nvec.c
index d8826ed..6d44076 100644
--- a/drivers/staging/nvec/nvec.c
+++ b/drivers/staging/nvec/nvec.c

@@ -770,7 +770,7 @@ static int tegra_nvec_probe(struct platform_device

*pdev)>> 
                 return -ENODEV;
         
         }

-       i2c_clk = clk_get_sys("tegra-i2c.2", "div-clk");
+       i2c_clk = clk_get(&pdev->dev, "div-clk");

         if (IS_ERR(i2c_clk)) {
         
                 dev_err(nvec->dev, "failed to get controller clock\n");
                 return -ENODEV;

Included in the latest patches sent.

em, not yet in V4.

It's in V2 of the other series Prashant posted which sits on top of the
CCF rework series.

Maybe you can also adjust the TODO (2nd entry) file now 
that this issue is fixed.

I'll try to remember to repost an updated version of the patch which
does that...