Hi Marc, Thank you for prompt feedback.
> -----Original Message----- > From: Marc Kleine-Budde [mailto:m...@pengutronix.de] > Sent: 2015年11月18日 18:26 > To: Yang, Wenyou; Wolfgang Grandegger > Cc: linux-...@vger.kernel.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; Ferre, Nicolas > Subject: Re: [PATCH 1/2] can: m_can: Increase delay to ensure written INIT > accepted > > On 11/18/2015 11:04 AM, Wenyou Yang wrote: > > Increase the delay time until the value written to INIT can be read > > back to ensure that the previous value written to INIT has been > > accepted. I tested it again with different clock frequencies, it works without this patch. Maybe I made a mistake. Sorry about it. Please ignore. > > > > Signed-off-by: Wenyou Yang <wenyou.y...@atmel.com> > > The patch looks ok, can you please add to your commit message which SoCs are > affected by this problem. > > > --- > > > > drivers/net/can/m_can/m_can.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/net/can/m_can/m_can.c > > b/drivers/net/can/m_can/m_can.c index ef65517..fd1caa0 100644 > > --- a/drivers/net/can/m_can/m_can.c > > +++ b/drivers/net/can/m_can/m_can.c > > @@ -320,7 +320,7 @@ static inline void m_can_config_endisable(const struct > m_can_priv *priv, > > bool enable) > > { > > u32 cccr = m_can_read(priv, M_CAN_CCCR); > > - u32 timeout = 10; > > + u32 timeout = 1000; > > u32 val = 0; > > > > if (enable) { > > > > Marc > > -- > Pengutronix e.K. | Marc Kleine-Budde | > Industrial Linux Solutions | Phone: +49-231-2826-924 | > Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | > Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | Best Regards, Wenyou Yang
