RE: [PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-11 Thread Christian Magnusson
leine-Budde Cc: w...@grandegger.com; andri.yngva...@marel.com; linux-...@vger.kernel.org; netdev@vger.kernel.org; linux-kernel; Christian Magnusson Subject: Re: [PATCH 1/1] can: sja1000: clear interrupts on start 2015-11-11 13:37 GMT+01:00 Marc Kleine-Budde : >> Some cred should go to Chris

Re: [PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-11 Thread Marc Kleine-Budde
On 11/11/2015 01:46 PM, Mirza Krak wrote: > 2015-11-11 13:37 GMT+01:00 Marc Kleine-Budde : >>> Some cred should go to Christian (added him to CC) for finding this >>> problem and reporting it to me. >> >> I'll add a >> >> Reported-by: Christian Magnusson >> >> to the patch. > > Can we just get an

Re: [PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-11 Thread Mirza Krak
2015-11-11 13:37 GMT+01:00 Marc Kleine-Budde : >> Some cred should go to Christian (added him to CC) for finding this >> problem and reporting it to me. > > I'll add a > > Reported-by: Christian Magnusson > > to the patch. > Can we just get an ACK that it is OK from Christian before we do it.

Re: [PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-11 Thread Marc Kleine-Budde
On 11/11/2015 01:29 PM, Mirza Krak wrote: >> Can you put a scope on the interrupt pin and see if it's a problem of >> the SoC and/or Linux or the SJA1000 not pulling the IRQ line. > > A little demonstration on what it looks like when the system is > resumed (added a printk in sja1000_start without

Re: [PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-11 Thread Mirza Krak
2015-11-11 11:06 GMT+01:00 Marc Kleine-Budde : > On 11/11/2015 10:54 AM, Mirza Krak wrote: >> 2015-11-11 9:04 GMT+01:00 Marc Kleine-Budde : >>> This problem occurs only an hardware, where the SJA1000 is powered >>> during system suspend? >> >> Correct, and the controller is in UP state. >> >>> Does

Re: [PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-11 Thread Marc Kleine-Budde
On 11/11/2015 10:54 AM, Mirza Krak wrote: > 2015-11-11 9:04 GMT+01:00 Marc Kleine-Budde : >> This problem occurs only an hardware, where the SJA1000 is powered >> during system suspend? > > Correct, and the controller is in UP state. > >> Doesn't the SJA1000 trigger an interrupt that is detected

Re: [PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-11 Thread Mirza Krak
2015-11-11 9:04 GMT+01:00 Marc Kleine-Budde : > This problem occurs only an hardware, where the SJA1000 is powered > during system suspend? Correct, and the controller is in UP state. > Doesn't the SJA1000 trigger an interrupt that is detected after > resuming? You add the fix to the open() func

Re: [PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-11 Thread Marc Kleine-Budde
On 11/10/2015 02:59 PM, Mirza Krak wrote: > From: Mirza Krak > > According to SJA1000 data sheet error-warning (EI) interrupt is not > cleared by setting the controller in to reset-mode. > > Then if we have the following case: > - system is suspended (echo mem > /sys/power/state) and SJA1000 is

[PATCH 1/1] can: sja1000: clear interrupts on start

2015-11-10 Thread Mirza Krak
From: Mirza Krak According to SJA1000 data sheet error-warning (EI) interrupt is not cleared by setting the controller in to reset-mode. Then if we have the following case: - system is suspended (echo mem > /sys/power/state) and SJA1000 is left in operating state - A bus error condition occurs w