diff options
author | Oliver Hartkopp <socketcan@hartkopp.net> | 2010-05-19 08:46:38 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-05-21 08:21:45 +0200 |
commit | 1f01bfd202bc539bccd282befa2bbdb8d6ad80ee (patch) | |
tree | 77930cd6a9c7ffb724cd14e99ddc16f75787afc5 /drivers/net | |
parent | sh_eth: Fix memleak in sh_mdio_release (diff) | |
download | linux-1f01bfd202bc539bccd282befa2bbdb8d6ad80ee.tar.xz linux-1f01bfd202bc539bccd282befa2bbdb8d6ad80ee.zip |
can: SJA1000 add missing spin_lock_init()
As remarked by Sam Ravnborg the spin_lock variable, that has been introduced
in commit 57c8a456640fa3ca777652f11f2db4179a3e66b6 ("can: Fix SJA1000 command
register writes on SMP systems") has not been initialized properly.
This patch adds the initialization to allow spinlock debugging.
Signed-off-by: Oliver Hartkopp <socketcan@hartkopp.net>
CC: Sam Ravnborg <sam@ravnborg.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r-- | drivers/net/can/sja1000/sja1000.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/can/sja1000/sja1000.c b/drivers/net/can/sja1000/sja1000.c index 85f7cbfe8e5f..0a8de01d52f7 100644 --- a/drivers/net/can/sja1000/sja1000.c +++ b/drivers/net/can/sja1000/sja1000.c @@ -599,6 +599,8 @@ struct net_device *alloc_sja1000dev(int sizeof_priv) priv->can.ctrlmode_supported = CAN_CTRLMODE_3_SAMPLES | CAN_CTRLMODE_BERR_REPORTING; + spin_lock_init(&priv->cmdreg_lock); + if (sizeof_priv) priv->priv = (void *)priv + sizeof(struct sja1000_priv); |