Structured comments updated.

[lincan.git] / lincan / src / ssv.c

diff --git a/lincan/src/ssv.c b/lincan/src/ssv.c
index 7cb10b94f865374882b1441ac59ae4f0590e810f..802aee76450ef7c6dd4dfa2fa355338103237ffd 100644 (file)
--- a/lincan/src/ssv.c
+++ b/lincan/src/ssv.c
@@ -5,14 +5,8 @@
  * Version 0.6  18 Sept 2000
  */ 
 
-#include <linux/autoconf.h>
-
-#include <linux/ioport.h>
-#include <linux/delay.h>
-#include <asm/errno.h>
-#include <asm/io.h>
-#include <asm/irq.h>
-
+#include "../include/can.h"
+#include "../include/can_sysdep.h"
 #include "../include/main.h"
 #include "../include/ssv.h"
 #include "../include/i82527.h"
@@ -20,6 +14,8 @@
 int ssvcan_irq[2]={-1,-1};
 unsigned long ssvcan_base=0x0;
 
+static can_spinlock_t ssv_port_lock=SPIN_LOCK_UNLOCKED;
+
 /* IO_RANGE is the io-memory range that gets reserved, please adjust according
  * your hardware. Example: #define IO_RANGE 0x100 for i82527 chips or
  * #define IO_RANGE 0x20 for sja1000 chips.
@@ -110,7 +106,7 @@ int ssv_reset(struct candevice_t *candev)
  * RESET_ADDR represents the io-address of the hardware reset register.
  * NR_82527 represents the number of intel 82527 chips on the board.
  * NR_SJA1000 represents the number of philips sja1000 chips on the board.
- * The flags entry can currently only be PROGRAMMABLE_IRQ to indicate that
+ * The flags entry can currently only be CANDEV_PROGRAMMABLE_IRQ to indicate that
  * the hardware uses programmable interrupts.
  */
 #define RESET_ADDR 0x02
@@ -123,7 +119,7 @@ int ssv_init_hw_data(struct candevice_t *candev)
     candev->nr_82527_chips=NR_82527;
     candev->nr_sja1000_chips=0;
     candev->nr_all_chips=NR_82527;
-    candev->flags |= PROGRAMMABLE_IRQ;
+    candev->flags |= CANDEV_PROGRAMMABLE_IRQ;
 
     return 0;
 }
@@ -170,14 +166,13 @@ int ssv_init_obj_data(struct chip_t *chip, int objnr)
 
     chip->msgobj[objnr]->obj_base_addr=
         chip->chip_base_addr+(objnr+1)*0x10;
-    chip->msgobj[objnr]->flags=0;
        
     return 0;
 }
 
 /* The function template_program_irq is used for hardware that uses programmable
  * interrupts. If your hardware doesn't use programmable interrupts you should
- * not set the candevices_t->flags entry to PROGRAMMABLE_IRQ and leave this
+ * not set the candevices_t->flags entry to CANDEV_PROGRAMMABLE_IRQ and leave this
  * function unedited. Again this function is hardware specific so there's no
  * example code.
  */
@@ -216,21 +211,22 @@ unsigned ssv_read_register(unsigned long address)
        want to read in a first time. In a second time we read the
        data */
     unsigned char ret;
+    can_spin_irqflags_t flags;
     
 
     if((address-ssvcan_base)<0x100)
     {
-       disable_irq(ssvcan_irq[0]);
+       can_spin_lock_irqsave(&ssv_port_lock,flags);
        outb(address-ssvcan_base, ssvcan_base);
        ret=inb(ssvcan_base+1);
-       enable_irq(ssvcan_irq[0]);
+       can_spin_unlock_irqrestore(&ssv_port_lock,flags);
     }
     else
     {
-       disable_irq(ssvcan_irq[1]);
+       can_spin_lock_irqsave(&ssv_port_lock,flags);
        outb(address-ssvcan_base-0x100, ssvcan_base+0x02);
        ret=inb(ssvcan_base+1+0x02);
-       enable_irq(ssvcan_irq[1]);
+       can_spin_unlock_irqrestore(&ssv_port_lock,flags);
     }
 
     return ret;