On Thu, 12 Apr 2012 18:15:30 -0500 mjr@xxxxxxxxxxx wrote: > From: Matt Renzelmann <mjr@xxxxxxxxxxx> > > All calls to ks8851_rdreg* and ks8851_wrreg* should be protected with > the driver's lock mutex. A spurious interrupt may otherwise cause a > crash. Thanks to Stephen Boyd, Flavio Leitner, and Ben Hutchings for > feedback. > > Signed-off-by: Matt Renzelmann <mjr@xxxxxxxxxxx> > --- > > This modified version incorporates Ben Hutchings' bugfix by removing > the incorrect call to CIDER_REV_GET. > > drivers/net/ethernet/micrel/ks8851.c | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/net/ethernet/micrel/ks8851.c b/drivers/net/ethernet/micrel/ks8851.c > index c722aa6..e5dc075 100644 > --- a/drivers/net/ethernet/micrel/ks8851.c > +++ b/drivers/net/ethernet/micrel/ks8851.c > @@ -1418,6 +1418,7 @@ static int __devinit ks8851_probe(struct spi_device *spi) > struct net_device *ndev; > struct ks8851_net *ks; > int ret; > + unsigned cider; Basically what I have proposed, but I used 'unsigned int cider'. Anyway, the driver uses just 'unsigned' everywhere. Signed-off-by: Flavio Leitner <fbl@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html