summaryrefslogtreecommitdiffstats
path: root/drivers
Commit message (Collapse)AuthorAgeFilesLines
* tlckl: BKL pushdownJonathan Corbet2008-06-201-7/+12
| | | | | | Put explicit lock_kernel calls into tlclk_open() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* xilinx icap: BKL pushdownJonathan Corbet2008-06-201-1/+5
| | | | | | | Add explicit lock_kernel() calls to hwicap_open() even though the existing locking looks adequate. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* vcs: BKL pushdownJonathan Corbet2008-06-201-2/+7
| | | | | | Add explicit BKL to vcs_open(). Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* spidev: BKL pushdownJonathan Corbet2008-06-201-0/+3
| | | | | | | Add the BKL to spidev_open(), even though the existing locking looks adequate. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* Add "no BKL needed" comments to several driversJonathan Corbet2008-06-2014-2/+21
| | | | | | | | This documents the fact that somebody looked at the relevant open() functions and concluded that, due to their trivial nature, no locking was needed. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* tty: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-2/+25
| | | | Parts of the serial code actually BUG() if we don't do this.
* snsc: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-1/+4
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* fbmem: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-4/+11
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* dvb: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-0/+4
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* usbcore: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-0/+3
| | | | | | | usb_open() is protected by a down_read(&minor_rwsem), but I'm not sure I trust it to protect everything including subsidiary open() functions. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* isdn: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-1/+2
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* gdth: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-0/+3
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* st: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-2/+9
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* aacraid: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-0/+3
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* osst: cdev lock_kernel() pushdown.Jonathan Corbet2008-06-201-1/+14
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* sg: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-3/+13
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* ide-tape: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-1/+6
| | | | | | ->release() already has explicit lock_kernel() calls... Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* phonedev: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-0/+3
| | | | | | | phone_open() looks OK, but I don't trust the subsidiary drivers (and ixj in particular). Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* drm: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-2/+7
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* lp: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-12/+26
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* mbcs: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-0/+5
| | | | | | This driver would appear to have no internal locking at all. Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* viotape: cdev lock_kernel pushdown ()Jonathan Corbet2008-06-201-0/+3
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* misc: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-0/+3
| | | | | | | misc_open() looks fine, but who knows what all of the misc drivers are doing in their open() functions? Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* mem: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-2/+8
| | | | | | | It's really hard to tell if this is necessary - lots of weird magic happens by way of map_devmem() Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* ipmi: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-2/+6
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* UIO: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-5/+12
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* Input: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-4/+12
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* HID: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-0/+3
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* UBI: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-1/+6
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* mtdchar: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-6/+16
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* paride: cdev lock_kernel() pushdownJonathan Corbet2008-06-202-7/+23
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* AoE: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-1/+6
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* drivers/s390: cdev lock_kernel() pushdownJonathan Corbet2008-06-204-14/+41
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* rtc: cdev lock_kernel() pushdownJonathan Corbet2008-06-201-3/+9
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* pcmcia: cdev lock_kernel() pushdownJonathan Corbet2008-05-181-7/+18
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* cosa: cdev lock_kernel() pushdownJonathan Corbet2008-05-181-6/+16
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* i2c: cdev lock_kernel() pushdownJonathan Corbet2008-05-181-6/+16
| | | | Signed-off-by: Jonathan Corbet <corbet@lwn.net>
* Merge branch 'i2c-for-linus' of git://jdelvare.pck.nerim.net/jdelvare-2.6Linus Torvalds2008-05-1820-64/+173
|\ | | | | | | | | | | | | | | | | | | | | | | * 'i2c-for-linus' of git://jdelvare.pck.nerim.net/jdelvare-2.6: i2c/max6875: Really prevent 24RF08 corruption i2c-amd756: Fix functionality flags i2c: Kill the old driver matching scheme i2c: Convert remaining new-style drivers to use module aliasing i2c: Switch pasemi to the new device/driver matching scheme i2c: Clean up Blackfin BF527 I2C device declarations i2c-nforce2: Disable the second SMBus channel on the DFI Lanparty NF4 Expert i2c: New co-maintainer
| * i2c/max6875: Really prevent 24RF08 corruptionJean Delvare2008-05-181-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | i2c-core takes care of the possible corruption of 24RF08 chips for quite some times, so device devices no longer need to do it. And they really should not, as applying the prevention twice voids it. I thought that I had fixed all drivers long ago but apparently I had missed that one. Signed-off-by: Jean Delvare <khali@linux-fr.org> Cc: Ben Gardner <bgardner@wabtec.com>
| * i2c-amd756: Fix functionality flagsJean Delvare2008-05-181-1/+1
| | | | | | | | | | | | | | The i2c-amd756 driver pretends to support SMBus process call transactions but actually does not. Fix it. Signed-off-by: Jean Delvare <khali@linux-fr.org>
| * i2c: Kill the old driver matching schemeJean Delvare2008-05-181-17/+5
| | | | | | | | | | | | | | Remove the old driver_name/type scheme for i2c driver matching. Only the standard aliasing model will be used from now on. Signed-off-by: Jean Delvare <khali@linux-fr.org>
| * i2c: Convert remaining new-style drivers to use module aliasingJean Delvare2008-05-1816-39/+143
| | | | | | | | | | | | | | | | | | | | | | | | Update all the remaining new-style i2c drivers to use standard module aliasing instead of the old driver_name/type driver matching scheme. Note that the tuner driver is a bit quirky at the moment, as it overwrites i2c_client.name with arbitrary strings. We write "tuner" back on remove, to make sure that driver cycling will work properly, but there may still be troublesome corner cases. Signed-off-by: Jean Delvare <khali@linux-fr.org>
| * i2c-nforce2: Disable the second SMBus channel on the DFI Lanparty NF4 ExpertJean Delvare2008-05-181-4/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | There is a strange chip at 0x2e on the second SMBus channel of the DFI Lanparty NF4 Expert motherboard. Accessing the chip reboots the system. As there's nothing interesting on this SMBus channel, the easiest and safest thing to do is to disable it on that board. This is a better fix to bug #5889 than the it87 driver update that was done originally: http://bugzilla.kernel.org/show_bug.cgi?id=5889 Signed-off-by: Jean Delvare <khali@linux-fr.org>
* | m68k: Return -ENODEV if no device is foundGeert Uytterhoeven2008-05-185-8/+8
| | | | | | | | | | | | | | | | According to the tests in do_initcalls(), the proper error code in case no device is found is -ENODEV, not -ENXIO or -EIO. Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
* | m68k: Some input drivers do not check the platformGeert Uytterhoeven2008-05-183-0/+14
| | | | | | | | | | | | | | | | Some input drivers do not check whether they're actually running on the correct platform, causing multi-platform kernels to crash if they are not. Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
* | m68k: Some network drivers do not check the platformGeert Uytterhoeven2008-05-185-0/+19
| | | | | | | | | | | | | | | | Some network drivers do not check whether they're actually running on the correct platform, causing multi-platform kernels to crash if they are not. Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
* | m68k: dnfb doesnt check for ApolloGeert Uytterhoeven2008-05-181-0/+3
| | | | | | | | | | | | | | | | The Apollo frame buffer device driver (dnfb) doesn't check whether it's actually running on Apollo hardware, causing a crash if it isn't. Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
* | m68k: macide doesnt check for MacGeert Uytterhoeven2008-05-181-0/+3
| | | | | | | | | | | | | | | | The Macintosh IDE driver (macide) doesn't check whether it's actually running on Mac hardware, causing a crash if it isn't. Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
* | m68k vme_scc: avoid global namespace pollutionGeert Uytterhoeven2008-05-181-3/+1
| | | | | | | | | | | | | | | | | | m68k vme_scc: - make scc_ports[] static - kill unused global scc_initialized Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
* | m68k: FB_HP300 depends on DIO and doesnt need FB_CFB_FILLRECTGeert Uytterhoeven2008-05-181-2/+1
| | | | | | | | | | | | | | | | | | | | Correct FB_HP300 dependencies: - FB_HP300 doesn't depend only on HP300, but also on DIO (which depends on HP300) - FB_HP300 does not need FB_CFB_FILLRECT Signed-off-by: Geert Uytterhoeven <geert@linux-m68k.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
OpenPOWER on IntegriCloud