RE: [PATCH 1/2] can: m_can: Increase delay to ensure written INIT accepted
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 > > 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- | > Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | Best Regards, Wenyou Yang
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. > > Signed-off-by: Wenyou Yang 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- | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | signature.asc Description: OpenPGP digital signature
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. > > Signed-off-by: Wenyou YangThe 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- | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | signature.asc Description: OpenPGP digital signature
RE: [PATCH 1/2] can: m_can: Increase delay to ensure written INIT accepted
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- | > Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de | Best Regards, Wenyou Yang