Jiri Slaby
7fa57a0cd9
Char: cyclades, fix potential NULL dereference
...
ztxdone is jumped to even if tty is NULL and tty_wakeup placed after
this label doesn't expect NULLed parameter, so this will cause an oops
in some situations (why they scheduled a wakeup there before remove
bottom half processing patch?).
wakeup only in the case when we have non-null tty struct.
Spotted by Adrian Bunk.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Cc: Adrian Bunk <bunk@kernel.org >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-10-23 08:32:06 -07:00
Robert P. J. Day
23d028f69a
Documentation: Remove references to dead "cyclades=" boot option
...
Signed-off-by: Robert P. J. Day <rpjday@mindspring.com >
Signed-off-by: Adrian Bunk <bunk@kernel.org >
2007-10-20 00:01:12 +02:00
Jan Engelhardt
96de0e252c
Convert files to UTF-8 and some cleanups
...
* Convert files to UTF-8.
* Also correct some people's names
(one example is Eißfeldt, which was found in a source file.
Given that the author used an ß at all in a source file
indicates that the real name has in fact a 'ß' and not an 'ss',
which is commonly used as a substitute for 'ß' when limited to
7bit.)
* Correct town names (Goettingen -> Göttingen)
* Update Eberhard Mönkeberg's address (http://lkml.org/lkml/2007/1/8/313 )
Signed-off-by: Jan Engelhardt <jengelh@gmx.de >
Signed-off-by: Adrian Bunk <bunk@kernel.org >
2007-10-19 23:21:04 +02:00
Jesper Juhl
d941ea7d49
Cyclades: Avoid label defined but not used warning
...
CC drivers/char/cyclades.o
drivers/char/cyclades.c: In function 'cy_init':
drivers/char/cyclades.c:5488: warning: label 'err_unr' defined but not used
Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com >
Acked-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-10-18 14:37:26 -07:00
Jiri Slaby
65f76a82ec
Char: cyclades, fix some -W warnings
...
Most of them are signedness, the rest unused function parameters.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-10-18 14:37:26 -07:00
Jiri Slaby
1c0a387c1f
Char: cyclades, move spin_lock to one place
...
Lock whole processing in isr, avoid error-prone locking/unlocking in rx/tx
esp. On fail paths (there was a bug in the past yet).
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-10-18 14:37:26 -07:00
Jiri Slaby
ce97a09767
Char: cyclades, make the isr code readable
...
Due to large indent the code was wrapped and unreadable. Create 3 function
instead of one and reorder the code, so it is readable now.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-10-18 14:37:26 -07:00
Jiri Slaby
ebafeeff0f
Char: cyclades, remove bottom half processing
...
The work done in bottom half doesn't cost much cpu time (e.g. tty_hangup
itself schedules its own bottom half), it's possible to do the work in isr
directly and save hence some .text.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk >
Cc: Paul Fulghum <paulkf@microgate.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-10-18 14:37:26 -07:00
Jeff Garzik
a6343afb6e
drivers/*: mark variables with uninitialized_var()
...
Mark variables in drivers/* with uninitialized_var() if such a warning
appears, and analysis proves that the var is initialized properly on all
paths it is used.
Signed-off-by: Jeff Garzik <jeff@garzik.org >
2007-07-17 16:23:19 -04:00
Jiri Slaby
c4923b4f13
Char: cyclades, fix sparse warning
...
cyclades, fix sparse warning
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-07-17 10:23:10 -07:00
Jiri Slaby
054f5b0aaa
Char: cyclades, add firmware loading
...
cyclades, add firmware loading
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-07-17 10:23:10 -07:00
Alan Cox
7e7d136e9e
serial: remove termios checks from various old char serial drivers
...
Signed-off-by: Alan Cox <alan@redhat.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-07-16 09:05:52 -07:00
Jiri Slaby
3fcbc72965
Char: cyclades, fix deadlock
...
An omitted unlock.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Cc: <stable@kernel.org >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-23 20:14:13 -07:00
Jiri Slaby
c8e1693a4f
Char: cyclades, copyright and version changes
...
- add copyright
- move version one number upper and use MODULE_VERSION
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
3137553d3f
Char: cyclades, probe cleanup
...
- add fail paths
- merge 3 similar initializations into one (Z, Ze, Y)
[akpm@linux-foundation.org: build fix]
Signed-off-by: Jiri Slaby <jirislaby@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2007-05-08 11:15:25 -07:00
Jiri Slaby
dd025c0c7a
Char: cyclades, dynamic ports
...
and save thus approx. 160k of .bss
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
f2462bfe55
Char: cyclades, fix tty device unregister
...
put_tty_driver after unregistering devices
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
f742903424
Char: cyclades, conditions cleanup
...
- 0 is not NULL
- use unlikely to hit the icache in isr more likely
- remove or comment empty if/else paths
[akpm@linux-foundation.org: build fix]
Signed-off-by: Jiri Slaby <jirislaby@gmail.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
2007-05-08 11:15:25 -07:00
Jiri Slaby
9fa1b3b185
Char: cyclades, remove locking macros
...
and use locally stored card structure if possible
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
6a0aa67b17
Char: cyclades, remove unused timestamps
...
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
3991428d9e
Char: cyclades, timers cleanup
...
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
ce71b0ffd0
Char: cyclades, fix blockmove
...
tty has no longer flip buffers accessible externally. Fix it by moving the
code to the tty_*flip* helpers.
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
2c7fea9921
Char: cyclades, remove sleep_on
...
convert to wait_* and completion
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
875b206b5f
Char: cyclades, make info->card a pointer
...
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00
Jiri Slaby
6d8248e850
Char: cyclades, get rid of phys addresses
...
Signed-off-by: Jiri Slaby <jirislaby@gmail.com >
Signed-off-by: Andrew Morton <akpm@linux-foundation.org >
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org >
2007-05-08 11:15:25 -07:00