login
Login
/
Register
Search
Search this site:
Forums
News
Blogs
Features
Site
Home
»
Mailing list archives
»
linux-kernel
»
2007
»
November
»
10
Re: [RFC 6/13] Char: nozomi, reorder and cleanup probe, remove
view
thread
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
[view in full thread]
From: Frank Seidel
Subject:
Re: [RFC 6/13] Char: nozomi, reorder and cleanup probe, remove
Date: Saturday, November 10, 2007 - 8:42 am
On Samstag 10 November 2007 00:46:51, you (Jiri Slaby) wrote:
quoted text
> nozomi, reorder and cleanup probe, remove > - remap space after requesting region, to not map something we cannot use > anyway > - init spin lock before request_irq, because due to shared irq debug, isr > might be called immediately, where the lock is being held > - remove INIT_LIST_HEAD before list_add, it's useless, because list_add > re-sets prev and next > - reorder cleanup to be the same as fail path of probe (or in other words, > the reverse of probe) > - no need to call nozomi_setup_private_data in remove, pointers are set in > probe
All perfect catches IMHO. Applied without changes. Thanks, Frank -
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to
majordomo@vger.kernel.org
More majordomo info at
http://vger.kernel.org/majordomo-info.html
Please read the FAQ at
http://www.tux.org/lkml/
Previous message: [
thread
] [
date
] [
author
]
Next message: [
thread
] [
date
] [
author
]
Messages in current thread:
[RFC 1/13] Char: nozomi, remove unneded stuff
, Jiri Slaby
, (Fri Nov 9, 4:43 pm)
[RFC 2/13] Char: nozomi, expand some functions
, Jiri Slaby
, (Fri Nov 9, 4:44 pm)
[RFC 3/13] Char: nozomi, fix fail paths
, Jiri Slaby
, (Fri Nov 9, 4:44 pm)
[RFC 4/13] Char: nozomi, tty index cleanup
, Jiri Slaby
, (Fri Nov 9, 4:45 pm)
[RFC 5/13] Char: nozomi, ioctls cleanup
, Jiri Slaby
, (Fri Nov 9, 4:46 pm)
[RFC 6/13] Char: nozomi, reorder and cleanup probe, remove
, Jiri Slaby
, (Fri Nov 9, 4:46 pm)
[RFC 7/13] Char: nozomi, remove struct irq
, Jiri Slaby
, (Fri Nov 9, 4:47 pm)
[RFC 8/13] Char: nozomi, tty cleanup
, Jiri Slaby
, (Fri Nov 9, 4:48 pm)
[RFC 9/13] Char: nozomi, lock cleanup
, Jiri Slaby
, (Fri Nov 9, 4:48 pm)
[RFC! 10/13] Char: nozomi, fix tty_flip_buffer_push
, Jiri Slaby
, (Fri Nov 9, 4:49 pm)
[RFC 11/13] Char: nozomi, remove unused includes
, Jiri Slaby
, (Fri Nov 9, 4:50 pm)
[RFC 12/13] Char: nozomi, remove void acc char2 char3 more ...
, Jiri Slaby
, (Fri Nov 9, 4:50 pm)
[RFC 13/13] Char: nozomi, cleanup read and write
, Jiri Slaby
, (Fri Nov 9, 4:51 pm)
Re: [RFC 1/13] Char: nozomi, remove unneded stuff
, Frank Seidel
, (Sat Nov 10, 8:41 am)
Re: [RFC 2/13] Char: nozomi, expand some functions
, Frank Seidel
, (Sat Nov 10, 8:41 am)
Re: [RFC 3/13] Char: nozomi, fix fail paths
, Frank Seidel
, (Sat Nov 10, 8:41 am)
Re: [RFC 4/13] Char: nozomi, tty index cleanup
, Frank Seidel
, (Sat Nov 10, 8:41 am)
Re: [RFC 5/13] Char: nozomi, ioctls cleanup
, Frank Seidel
, (Sat Nov 10, 8:41 am)
Re: [RFC 6/13] Char: nozomi, reorder and cleanup probe, remove
, Frank Seidel
, (Sat Nov 10, 8:42 am)
Re: [RFC 7/13] Char: nozomi, remove struct irq
, Frank Seidel
, (Sat Nov 10, 8:42 am)
Re: [RFC 8/13] Char: nozomi, tty cleanup
, Frank Seidel
, (Sat Nov 10, 8:42 am)
Re: [RFC 9/13] Char: nozomi, lock cleanup
, Frank Seidel
, (Sat Nov 10, 8:42 am)
Re: [RFC! 10/13] Char: nozomi, fix tty_flip_buffer_push
, Frank Seidel
, (Sat Nov 10, 8:43 am)
Re: [RFC 11/13] Char: nozomi, remove unused includes
, Frank Seidel
, (Sat Nov 10, 8:43 am)
Re: [RFC 12/13] Char: nozomi, remove void acc char2 char3 ...
, Frank Seidel
, (Sat Nov 10, 8:43 am)
Re: [RFC 13/13] Char: nozomi, cleanup read and write
, Frank Seidel
, (Sat Nov 10, 8:43 am)
Re: [RFC 4/13] Char: nozomi, tty index cleanup
, Jiri Slaby
, (Sat Nov 10, 8:50 am)
Re: [RFC 1/13] Char: nozomi, remove unneded stuff
, Jiri Slaby
, (Sat Nov 10, 8:55 am)
Re: [RFC 13/13] Char: nozomi, cleanup read and write
, Adrian Bunk
, (Sat Nov 10, 9:15 am)
Re: [RFC 13/13] Char: nozomi, cleanup read and write
, Jiri Slaby
, (Sat Nov 10, 3:04 pm)
Re: [RFC 13/13] Char: nozomi, cleanup read and write
, Frank Seidel
, (Sat Nov 10, 7:37 pm)
Re: [RFC 13/13] Char: nozomi, cleanup read and write
, Frank Seidel
, (Sun Nov 11, 9:02 am)
Re: [RFC 13/13] Char: nozomi, cleanup read and write
, Adrian Bunk
, (Mon Nov 12, 12:54 am)
Re: [RFC 13/13] Char: nozomi, cleanup read and write
, Frank Seidel
, (Mon Nov 12, 2:43 am)
Re: [RFC 7/13] Char: nozomi, remove struct irq
, Frank Seidel
, (Mon Nov 12, 8:11 am)
Re: [RFC 8/13] Char: nozomi, tty cleanup
, Frank Seidel
, (Mon Nov 12, 11:43 am)
Navigation
Mailing list archives
Recent posts
Popular discussions
linux-kernel
:
Paul Turner
[tg_shares_up rewrite v4 11/11] sched: update tg->shares after cpu.shares write
Matthew Garrett
Re: [PATCH] Enable speedstep for sonoma processors.
Mauro Carvalho Chehab
Re: [PATCH 1/2] media: Add timberdale video-in driver
Peter Zijlstra
[PATCH 23/30] netvm: skb processing
Greg Kroah-Hartman
[PATCH 21/28] cgroupfs: create /sys/fs/cgroup to mount cgroupfs on
git
:
Jan Hudec
Re: GIT push to sftp (feature request)
Steffen Prohaska
[PATCH 0/4] core.ignorecase
Johannes Schindelin
Re: Git checkout preserve timestamp?
Linus Torvalds
[PATCH 1/7] Make unpack_trees_options bit flags actual bitfields
Johan Herland
Re: What's cooking in git.git (Oct 2010, #01; Wed, 13)
linux-netdev
:
David Miller
Re: [PATCH 1/3] f_phonet: dev_kfree_skb instead of dev_kfree_skb_any in TX callback
Richard Cochran
Re: [PATCH v3 3/3] ptp: Added a clock that uses the eTSEC found on the MPC85xx.
Jan Engelhardt
Re: [PATCH] Fix netfilter xt_time's time_mt()'s use of do_div()
Herbert Xu
Re: [RFC PATCH 00/17] virtual-bus
Jeff Kirsher
Re: [net-next-2.6 PATCH] e1000e: don't inadvertently re-set INTX_DISABLE
git-commits-head
:
Linux Kernel Mailing List
ALSA: hda - Enable beep on Realtek codecs with PCI SSID override
Linux Kernel Mailing List
Use path_put() in a few places instead of {mnt,d}put()
Linux Kernel Mailing List
mv643xx_eth: use sw csum for big packets
Linux Kernel Mailing List
arm: fix HAVE_CLK merge goof
Linux Kernel Mailing List
arm: convert pcm037 platform to use smsc911x
freebsd-current
:
David Wolfskill
"interrupt storm..."; seems associated with an0 NIC
Andriy Gapon
Re: letting glabel recognise a media change
Garrett Cooper
Re: Only display ACPI bootmenu key if ACPI is present
Pyun YongHyeon
CFT: msk(4) Rx checksum offloading support
FreeBSD Tinderbox
[head tinderbox] failure on sparc64/sparc64
Colocation donated by:
Syndicate