On Tue, Jan 13, 2015 at 12:54:40PM -0800, Tony Lindgren wrote: > * Felipe Balbi <[email protected]> [150113 11:55]: > > On Tue, Jan 13, 2015 at 11:29:24AM -0800, Tony Lindgren wrote: > > > --- a/drivers/net/ethernet/ti/davinci_emac.c > > > +++ b/drivers/net/ethernet/ti/davinci_emac.c > > > @@ -1538,7 +1538,7 @@ static int emac_dev_open(struct net_device *ndev) > > > int i = 0; > > > struct emac_priv *priv = netdev_priv(ndev); > > > > > > - pm_runtime_get(&priv->pdev->dev); > > > + pm_runtime_get_sync(&priv->pdev->dev); > > > > gotta check return value on all pm_runtime_get_sync() calls. IIRC, > > there's a coccinelle script for checking and patching this. > > Sure, here's an updated patch with error checking added. > > Regards, > > Tony > > 8< --------------------- > From: Tony Lindgren <[email protected]> > Date: Mon, 22 Dec 2014 08:19:06 -0800 > Subject: [PATCH] net: davinci_emac: Fix runtime pm calls for davinci_emac > > Commit 3ba97381343b ("net: ethernet: davinci_emac: add pm_runtime support") > added support for runtime PM, but it causes issues on omap3 related devices > that actually gate the clocks: > > Unhandled fault: external abort on non-linefetch (0x1008) > ... > [<c04160f0>] (emac_dev_getnetstats) from [<c04d6a3c>] > (dev_get_stats+0x78/0xc8) > [<c04d6a3c>] (dev_get_stats) from [<c04e9ccc>] (rtnl_fill_ifinfo+0x3b8/0x938) > [<c04e9ccc>] (rtnl_fill_ifinfo) from [<c04eade4>] (rtmsg_ifinfo+0x68/0xd8) > [<c04eade4>] (rtmsg_ifinfo) from [<c04dd35c>] (register_netdevice+0x3a0/0x4ec) > [<c04dd35c>] (register_netdevice) from [<c04dd4bc>] > (register_netdev+0x14/0x24) > [<c04dd4bc>] (register_netdev) from [<c041755c>] > (davinci_emac_probe+0x408/0x5c8) > [<c041755c>] (davinci_emac_probe) from [<c0396d78>] > (platform_drv_probe+0x48/0xa4) > > Let's fix it by moving the pm_runtime_get() call earlier, and also add it to > the emac_dev_getnetstats(). Also note that we want to use > pm_runtime_get_sync() > as we don't want to have deferred_resume happen. And let's also check the > return value for pm_runtime_get_sync() as noted by Felipe Balbi > <[email protected]>. > > Cc: Brian Hutchinson <[email protected]> > Cc: Felipe Balbi <[email protected]> > Cc: Mark A. Greer <[email protected]> > Signed-off-by: Tony Lindgren <[email protected]>
Tony, Sorry for the sloppy pm_runtime patch and thanks for fixing it. Acked-by: Mark A. Greer <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
