From: Vladimir Oltean <olteanv@gmail.com>
To: Joakim Zhang <qiangqing.zhang@nxp.com>
Cc: "peppe.cavallaro@st.com" <peppe.cavallaro@st.com>,
"alexandre.torgue@foss.st.com" <alexandre.torgue@foss.st.com>,
"joabreu@synopsys.com" <joabreu@synopsys.com>,
"davem@davemloft.net" <davem@davemloft.net>,
"kuba@kernel.org" <kuba@kernel.org>,
dl-linux-imx <linux-imx@nxp.com>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH net] net: stmmac: fix system hang caused by eee_ctrl_timer during suspend/resume
Date: Fri, 12 Nov 2021 15:43:31 +0200 [thread overview]
Message-ID: <20211112134331.initwvks6ytlfi4x@skbuf> (raw)
In-Reply-To: <DB8PR04MB6795FCF1B1E5E0B03CA49B90E6959@DB8PR04MB6795.eurprd04.prod.outlook.com>
On Fri, Nov 12, 2021 at 01:21:57AM +0000, Joakim Zhang wrote:
> Hi Vladimir,
>
> > Hi Joakim,
> >
> > This patch looks strange to me. You are basically saying that the LPI timer for
> > MAC-level EEE is clocked from the clk_ptp_ref clock?! Are you sure this is
> > correct? I thought this clock is only used for the PTP timestamping counter.
> > Maybe the clock definitions in imx8mp.dtsi are not correct?
>
> No, MAC-level EEE is not clocked from the clk_ptp_ref clock, this clock is for PTP. To fix this issue,
> I can only move pm_runtime_force_suspend() into noirq suspend stage. As commit message said,
> "postpone clocks management from suspend stage to noirq suspend stage", that means I move all
> the clocks management into noirq suspend stage, including PTP clock, it's should be harmless for PTP
> function. Do you find any regression with this patch?
No, I didn't find any regression, I was just frustrated yesterday that a
bugfix patch I wanted to send to "stable" on the handling of clk_ptp_ref
is conflicting with your (apparently unnecessary) movement of this clock
to the noirq suspend stage. I'm better now.
prev parent reply other threads:[~2021-11-12 13:43 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-08 7:43 [PATCH net] net: stmmac: fix system hang caused by eee_ctrl_timer during suspend/resume Joakim Zhang
2021-09-08 11:40 ` patchwork-bot+netdevbpf
2021-09-09 8:46 ` kernel test robot
2021-11-11 15:16 ` Vladimir Oltean
2021-11-12 1:21 ` Joakim Zhang
2021-11-12 13:43 ` Vladimir Oltean [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211112134331.initwvks6ytlfi4x@skbuf \
--to=olteanv@gmail.com \
--cc=alexandre.torgue@foss.st.com \
--cc=davem@davemloft.net \
--cc=joabreu@synopsys.com \
--cc=kuba@kernel.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
--cc=peppe.cavallaro@st.com \
--cc=qiangqing.zhang@nxp.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.