diff options
| author | Oliver Hartkopp <socketcan@hartkopp.net> | 2010-05-19 06:46:38 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-05-20 23:21:45 -0700 |
| commit | 1f01bfd202bc539bccd282befa2bbdb8d6ad80ee (patch) | |
| tree | 77930cd6a9c7ffb724cd14e99ddc16f75787afc5 | |
| parent | 0f0b405cd16f7aaff84a935984cae421897d725d (diff) | |
| download | linux-1f01bfd202bc539bccd282befa2bbdb8d6ad80ee.tar.gz linux-1f01bfd202bc539bccd282befa2bbdb8d6ad80ee.tar.bz2 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>
| -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); |