[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20150806.185406.1871113961208590119.davem@davemloft.net>
Date: Thu, 06 Aug 2015 18:54:06 -0700 (PDT)
From: David Miller <davem@...emloft.net>
To: l.stach@...gutronix.de
Cc: andrew@...n.ch, u.kleine-koenig@...gutronix.de,
netdev@...r.kernel.org, kernel@...gutronix.de,
patchwork-lst@...gutronix.de
Subject: Re: [PATCH] net: fec: fix initial runtime PM refcount
From: Lucas Stach <l.stach@...gutronix.de>
Date: Mon, 3 Aug 2015 17:50:11 +0200
> The clocks are initially active and thus the device is marked active.
> This still keeps the PM refcount at 0, the pm_runtime_put_autosuspend()
> call at the end of probe then leaves us with an invalid refcount of -1,
> which in turn leads to the device staying in suspended state even though
> netdev open had been called.
>
> Fix this by initializing the refcount to be coherent with the initial
> device status.
>
> Fixes:
> 8fff755e9f8 (net: fec: Ensure clocks are enabled while using mdio bus)
>
> Signed-off-by: Lucas Stach <l.stach@...gutronix.de>
Applied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists