On Thu, Aug 02, 2012 at 18:43:11, AnilKumar, Chimata wrote: > Add Runtime PM support to C_CAN/D_CAN controller. The runtime PM > APIs control clocks for C_CAN/D_CAN IP and prevent access to the > register of C_CAN/D_CAN IP when clock is turned off. > > Signed-off-by: AnilKumar Ch <[email protected]> > --- > drivers/net/can/c_can/c_can_platform.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/net/can/c_can/c_can_platform.c > b/drivers/net/can/c_can/c_can_platform.c > index d0a66cf..83a1e17 100644 > --- a/drivers/net/can/c_can/c_can_platform.c > +++ b/drivers/net/can/c_can/c_can_platform.c > @@ -32,6 +32,7 @@ > #include <linux/clk.h> > #include <linux/of.h> > #include <linux/of_device.h> > +#include <linux/pm_runtime.h> > > #include <linux/can/dev.h> > > @@ -177,6 +178,9 @@ static int __devinit c_can_plat_probe(struct > platform_device *pdev) > goto exit_free_device; > } > > + pm_runtime_enable(&pdev->dev); > + pm_runtime_get_sync(&pdev->dev); > +
If module is inserted or built into the kernel, module stays in enabled state always, isn't that wrong? Ideally, you should enable the module when it is required or being used. Thanks, Vaibhav _______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
