diff options
author | Matthew Wilcox <matthew@wil.cx> | 2007-09-09 08:56:38 -0600 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.localdomain> | 2007-10-12 14:48:11 -0400 |
commit | b009bef6cd2c6c2b870088d9ae67dbc4ec2ca317 (patch) | |
tree | 613913c0dc95d2ade63ced852b24d23b9c4b09c4 /lib/irq_regs.c | |
parent | faac48ecf475c8e214190dabe600585e0bd4f455 (diff) | |
download | blackbird-op-linux-b009bef6cd2c6c2b870088d9ae67dbc4ec2ca317.tar.gz blackbird-op-linux-b009bef6cd2c6c2b870088d9ae67dbc4ec2ca317.zip |
[SCSI] advansys: Remove some custom wrappers
- Replace ASC_ASSERT() with BUG_ON().
In a few places, get rid of the assertion altogether -- the ensuing
crash will tell us all we need to know. Use BUG() where it fits better
than BUG_ON(). Also fix a fencepost error in advansys_proc_info().
- Replace DvcSleepMilliSecond with mdelay.
Despite its name using 'sleep', the implementation was a delay.
I've marked some places with XXX where we should probably be using
msleep instead. They need to be audited to be sure we can sleep in
that context.
- Replace DvcDelayMicroSecond with udelay.
- Replace DvcDelayNanoSecond with udelay too.
All callers were multiples of 1000.
- Remove DvcEnterCritical and DvcLeaveCritical.
These functions are no-ops, and as the comments said, the spinlock
protects the critical sections.
Signed-off-by: Matthew Wilcox <matthew@wil.cx>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'lib/irq_regs.c')
0 files changed, 0 insertions, 0 deletions