On Thu, 2009-01-08 at 15:33 +0200, Tony Lindgren wrote:
> * stanley.miao <stanley.m...@windriver.com> [081107 15:47]:
> > This solution keeps the virtual clock in place and enable the child
> > clocks before enable the virtual clock. So, any comments ?
> 
> What if we just removed the custom clock and had a struct **clk
> in struct omap_mcbsp that contains the clocks for each instance?

It works. This is what I did in my first patch. 

The difference is I add two struct *clk in struct omap_mcbsp.

struct omap_mcbsp {
@@ -365,7 +366,8 @@ struct omap_mcbsp {
        /* Protect the field .free, while checking if the mcbsp is in
use */
        spinlock_t lock;
        struct omap_mcbsp_platform_data *pdata;
-       struct clk *clk;
+       struct clk *ick;
+       struct clk *fck;


If one struct **clk is better, I will resend the patch later.

Stanley
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to