Hey Wolfgang, Wolfgang Grandegger wrote: >> This core is found on some Freescale SoCs and also some Coldfire >> SoCs. Support for Coldfire is missing though at the moment as >> they have an older revision of the core which does not have RX FIFO >> support. >> >> Signed-off-by: Sascha Hauer <[email protected]> >> Signed-off-by: Marc Kleine-Budde <[email protected]> >> --- >> >> Changes to prev version: >> * The is now GPLv2 (only) as no one complained. >> >> The patch applies to current net-next-2.6/master. >> If there aren't any objections please consider applying this patch. >> Wolfgang, can I an Acked-by? > > I already had a look to the previous version and I realized that > accessing reg_esr is racy. I will dig out my notes and come up with a > full review later today or tomorrow.
Let me have a look.... I think I should remove the read reg_esr in "flexcan_irq_err()" and use the value read in "flexcan_irq()" Without the fix, there's a race window that we loose some error bits from the interrupt handler to the napi function, because the error bits are cleared on read. Regarding the upcoming improvement of the can error frames upon state changes, I'd first like to get this driver merged and then discuss a solution for the error frames. cheers, 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 |
signature.asc
Description: OpenPGP digital signature
_______________________________________________ Socketcan-core mailing list [email protected] https://lists.berlios.de/mailman/listinfo/socketcan-core
