On Thu, Oct 18, 2012 at 10:27 AM, Philipp Zabel <[email protected]> wrote: > On some platforms the SRAM needs a clock to be enabled explicitly.
Since this is a file that you've just created in the previous commit, I don't see why this needs to exist as a standalone commit, vs just being folded back into the previous commit? P. -- > > Signed-off-by: Philipp Zabel <[email protected]> > --- > drivers/misc/sram.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/drivers/misc/sram.c b/drivers/misc/sram.c > index 7a363f2..0cc2e75 100644 > --- a/drivers/misc/sram.c > +++ b/drivers/misc/sram.c > @@ -21,6 +21,8 @@ > #include <linux/kernel.h> > #include <linux/module.h> > #include <linux/init.h> > +#include <linux/clk.h> > +#include <linux/err.h> > #include <linux/io.h> > #include <linux/of.h> > #include <linux/platform_device.h> > @@ -29,6 +31,7 @@ > > struct sram_dev { > struct gen_pool *pool; > + struct clk *clk; > }; > > static int __devinit sram_probe(struct platform_device *pdev) > @@ -53,6 +56,10 @@ static int __devinit sram_probe(struct platform_device > *pdev) > if (!sram) > return -ENOMEM; > > + sram->clk = devm_clk_get(&pdev->dev, NULL); > + if (!IS_ERR(sram->clk)) > + clk_prepare_enable(sram->clk); > + > sram->pool = gen_pool_create(PAGE_SHIFT, -1); > if (!sram->pool) > return -ENOMEM; > @@ -80,6 +87,9 @@ static int __devexit sram_remove(struct platform_device > *pdev) > > gen_pool_destroy(sram->pool); > > + if (!IS_ERR(sram->clk)) > + clk_disable_unprepare(sram->clk); > + > return 0; > } > > -- > 1.7.10.4 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ _______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
