From: Takashi Iwai <tiwai@suse.de>
To: Bill Pemberton <wfp5p@virginia.edu>
Cc: gregkh@linuxfoundation.org, alsa-devel@alsa-project.org
Subject: Re: [PATCH 00/20] ALSA: remove __dev* attributes for retirement of HOTPLUG
Date: Thu, 06 Dec 2012 18:41:27 +0100 [thread overview]
Message-ID: <s5hd2ynqdrs.wl%tiwai@suse.de> (raw)
In-Reply-To: <1354815329-20221-1-git-send-email-wfp5p@virginia.edu>
At Thu, 6 Dec 2012 12:35:09 -0500,
Bill Pemberton wrote:
>
> Remove use of __devinit, __devexit_p, __devinitdata, __devinitconst,
> and __devexit so that HOTPLUG can be retired.
>
> Bill Pemberton (20):
> ALSA: pci: remove __dev* attributes
> ALSA: AACI: remove __dev* attributes
> ALSA: pxa2xx: remove __dev* attributes
> ALSA: atmel: remove __dev* attributes
> ALSA: firewire-speakers: remove __dev* attributes
> ALSA: mips: remove __dev* attributes
> ALSA: parisc/harmony: remove __dev* attributes
> ALSA: oss: remove __dev* attributes
> ALSA: ad1889: remove __dev* attributes
> ALSA: bt87X: remove __dev* attributes
> ALSA: oxygen: remove __dev* attributes
Any reason the three above to be separate patches?
> ALSA: isa: remove __dev* attributes
> ALSA: sound/ps3: remove __dev* attributes
> ALSA: ppc: remove __dev* attributes
> ALSA: sh: remove __dev* attributes
> ALSA: sparc: remove __dev* attributes
> ALSA: at73c213: remove __dev* attributes
> ALSA: drivers: remove __dev* attributes
> ALSA: snd-usb-6fire: remove __dev* attributes
> ALSA: snd-usb-caiaq: remove __dev* attributes
Also, these can be a single clean up for sound/usb/*...
It's no big matter, but I just wonder since I don't see a logic...
Takashi
> sound/arm/aaci.c | 18 +++---
> sound/arm/pxa2xx-ac97-lib.c | 2 +-
> sound/arm/pxa2xx-ac97.c | 6 +-
> sound/atmel/abdac.c | 8 +--
> sound/atmel/ac97c.c | 10 ++--
> sound/drivers/aloop.c | 16 +++---
> sound/drivers/dummy.c | 14 ++---
> sound/drivers/ml403-ac97cr.c | 10 ++--
> sound/drivers/mpu401/mpu401.c | 18 +++---
> sound/drivers/mtpav.c | 14 ++---
> sound/drivers/mts64.c | 40 +++++++-------
> sound/drivers/pcsp/pcsp.c | 14 ++---
> sound/drivers/pcsp/pcsp_input.c | 2 +-
> sound/drivers/pcsp/pcsp_input.h | 2 +-
> sound/drivers/pcsp/pcsp_lib.c | 2 +-
> sound/drivers/pcsp/pcsp_mixer.c | 10 ++--
> sound/drivers/portman2x4.c | 18 +++---
> sound/drivers/serial-u16550.c | 32 +++++------
> sound/drivers/virmidi.c | 6 +-
> sound/firewire/speakers.c | 8 +--
> sound/isa/ad1816a/ad1816a.c | 18 +++---
> sound/isa/ad1816a/ad1816a_lib.c | 19 ++++---
> sound/isa/ad1848/ad1848.c | 8 +--
> sound/isa/adlib.c | 8 +--
> sound/isa/als100.c | 22 ++++----
> sound/isa/azt2320.c | 26 ++++-----
> sound/isa/cmi8328.c | 8 +--
> sound/isa/cmi8330.c | 42 +++++++-------
> sound/isa/cs423x/cs4231.c | 8 +--
> sound/isa/cs423x/cs4236.c | 50 ++++++++---------
> sound/isa/es1688/es1688.c | 28 +++++-----
> sound/isa/es18xx.c | 80 +++++++++++++--------------
> sound/isa/galaxy/galaxy.c | 26 ++++-----
> sound/isa/gus/gusclassic.c | 15 ++---
> sound/isa/gus/gusextreme.c | 24 ++++----
> sound/isa/gus/gusmax.c | 16 +++---
> sound/isa/gus/interwave.c | 54 +++++++++---------
> sound/isa/msnd/msnd.h | 2 +-
> sound/isa/msnd/msnd_pinnacle.c | 44 +++++++--------
> sound/isa/msnd/msnd_pinnacle_mixer.c | 2 +-
> sound/isa/opl3sa2.c | 40 +++++++-------
> sound/isa/opti9xx/miro.c | 68 +++++++++++------------
> sound/isa/opti9xx/opti92x-ad1848.c | 42 +++++++-------
> sound/isa/sb/emu8000.c | 22 ++++----
> sound/isa/sb/jazz16.c | 18 +++---
> sound/isa/sb/sb16.c | 26 ++++-----
> sound/isa/sb/sb8.c | 8 +--
> sound/isa/sc6000.c | 38 ++++++-------
> sound/isa/sscape.c | 32 +++++------
> sound/isa/wavefront/wavefront.c | 53 +++++++++---------
> sound/isa/wavefront/wavefront_fx.c | 2 +-
> sound/isa/wavefront/wavefront_midi.c | 2 +-
> sound/isa/wavefront/wavefront_synth.c | 14 ++---
> sound/mips/au1x00.c | 4 +-
> sound/mips/hal2.c | 14 ++---
> sound/mips/sgio2audio.c | 28 +++++-----
> sound/oss/ad1848.c | 2 +-
> sound/oss/kahlua.c | 10 ++--
> sound/parisc/harmony.c | 12 ++--
> sound/pci/ad1889.c | 18 +++---
> sound/pci/ak4531_codec.c | 10 ++--
> sound/pci/ali5451/ali5451.c | 32 +++++------
> sound/pci/als300.c | 14 ++---
> sound/pci/als4000.c | 12 ++--
> sound/pci/asihpi/asihpi.c | 55 +++++++++----------
> sound/pci/asihpi/hpioctl.c | 6 +-
> sound/pci/asihpi/hpioctl.h | 6 +-
> sound/pci/atiixp.c | 32 +++++------
> sound/pci/atiixp_modem.c | 20 +++----
> sound/pci/au88x0/au88x0.c | 10 ++--
> sound/pci/au88x0/au88x0_a3d.c | 6 +-
> sound/pci/au88x0/au88x0_core.c | 2 +-
> sound/pci/au88x0/au88x0_eq.c | 10 ++--
> sound/pci/au88x0/au88x0_game.c | 2 +-
> sound/pci/au88x0/au88x0_mixer.c | 2 +-
> sound/pci/au88x0/au88x0_mpu401.c | 2 +-
> sound/pci/au88x0/au88x0_pcm.c | 6 +-
> sound/pci/aw2/aw2-alsa.c | 28 +++++-----
> sound/pci/azt3328.c | 22 ++++----
> sound/pci/bt87x.c | 22 ++++----
> sound/pci/ca0106/ca0106_main.c | 12 ++--
> sound/pci/ca0106/ca0106_mixer.c | 26 ++++-----
> sound/pci/ca0106/ca0106_proc.c | 2 +-
> sound/pci/ca0106/ca_midi.c | 2 +-
> sound/pci/cmipci.c | 48 ++++++++--------
> sound/pci/cs4281.c | 30 +++++-----
> sound/pci/cs46xx/cs46xx.c | 8 +--
> sound/pci/cs46xx/cs46xx_lib.c | 35 ++++++------
> sound/pci/cs5530.c | 16 +++---
> sound/pci/cs5535audio/cs5535audio.c | 18 +++---
> sound/pci/cs5535audio/cs5535audio.h | 10 ++--
> sound/pci/cs5535audio/cs5535audio_olpc.c | 10 ++--
> sound/pci/cs5535audio/cs5535audio_pcm.c | 2 +-
> sound/pci/ctxfi/ctatc.c | 20 +++----
> sound/pci/ctxfi/ctatc.h | 8 +--
> sound/pci/ctxfi/cthardware.c | 4 +-
> sound/pci/ctxfi/cthw20k1.c | 4 +-
> sound/pci/ctxfi/cthw20k2.c | 4 +-
> sound/pci/ctxfi/xfi.c | 6 +-
> sound/pci/echoaudio/echoaudio.c | 46 ++++++++--------
> sound/pci/echoaudio/echoaudio.h | 4 +-
> sound/pci/echoaudio/midi.c | 4 +-
> sound/pci/emu10k1/emu10k1.c | 8 +--
> sound/pci/emu10k1/emu10k1_main.c | 4 +-
> sound/pci/emu10k1/emu10k1x.c | 29 +++++-----
> sound/pci/emu10k1/emufx.c | 25 +++++----
> sound/pci/emu10k1/emumixer.c | 22 ++++----
> sound/pci/emu10k1/emumpu401.c | 6 +-
> sound/pci/emu10k1/emupcm.c | 11 ++--
> sound/pci/emu10k1/emuproc.c | 2 +-
> sound/pci/emu10k1/p16v.c | 8 +--
> sound/pci/emu10k1/timer.c | 2 +-
> sound/pci/ens1370.c | 52 +++++++++---------
> sound/pci/es1938.c | 20 +++----
> sound/pci/es1968.c | 42 +++++++-------
> sound/pci/fm801.c | 26 ++++-----
> sound/pci/hda/hda_intel.c | 48 ++++++++--------
> sound/pci/ice1712/amp.c | 6 +-
> sound/pci/ice1712/aureon.c | 24 ++++----
> sound/pci/ice1712/delta.c | 42 +++++++-------
> sound/pci/ice1712/ews.c | 28 +++++-----
> sound/pci/ice1712/hoontech.c | 26 ++++-----
> sound/pci/ice1712/ice1712.c | 94 ++++++++++++++++----------------
> sound/pci/ice1712/ice1724.c | 64 +++++++++++-----------
> sound/pci/ice1712/juli.c | 26 ++++-----
> sound/pci/ice1712/maya44.c | 16 +++---
> sound/pci/ice1712/phase.c | 22 ++++----
> sound/pci/ice1712/pontis.c | 10 ++--
> sound/pci/ice1712/prodigy192.c | 12 ++--
> sound/pci/ice1712/prodigy_hifi.c | 20 +++----
> sound/pci/ice1712/psc724.c | 8 +--
> sound/pci/ice1712/quartet.c | 24 ++++----
> sound/pci/ice1712/revo.c | 28 +++++-----
> sound/pci/ice1712/se.c | 20 +++----
> sound/pci/ice1712/vt1720_mobo.c | 10 ++--
> sound/pci/ice1712/wtm.c | 10 ++--
> sound/pci/intel8x0.c | 56 +++++++++----------
> sound/pci/intel8x0m.c | 30 +++++-----
> sound/pci/korg1212/korg1212.c | 12 ++--
> sound/pci/lola/lola.c | 14 ++---
> sound/pci/lola/lola_clock.c | 2 +-
> sound/pci/lola/lola_mixer.c | 32 +++++------
> sound/pci/lola/lola_pcm.c | 4 +-
> sound/pci/lola/lola_proc.c | 2 +-
> sound/pci/lx6464es/lx6464es.c | 30 +++++-----
> sound/pci/lx6464es/lx_core.c | 2 +-
> sound/pci/lx6464es/lx_core.h | 2 +-
> sound/pci/maestro3.c | 24 ++++----
> sound/pci/mixart/mixart.c | 12 ++--
> sound/pci/nm256/nm256.c | 18 +++---
> sound/pci/oxygen/oxygen.c | 10 ++--
> sound/pci/oxygen/virtuoso.c | 10 ++--
> sound/pci/oxygen/xonar_cs43xx.c | 4 +-
> sound/pci/oxygen/xonar_pcm179x.c | 4 +-
> sound/pci/oxygen/xonar_wm87x6.c | 4 +-
> sound/pci/pcxhr/pcxhr.c | 14 ++---
> sound/pci/riptide/riptide.c | 20 +++----
> sound/pci/rme32.c | 10 ++--
> sound/pci/rme96.c | 14 ++---
> sound/pci/rme9652/hdsp.c | 14 ++---
> sound/pci/rme9652/hdspm.c | 41 +++++++-------
> sound/pci/rme9652/rme9652.c | 22 ++++----
> sound/pci/sis7019.c | 16 +++---
> sound/pci/sonicvibes.c | 37 +++++++------
> sound/pci/trident/trident.c | 8 +--
> sound/pci/trident/trident_main.c | 46 ++++++++--------
> sound/pci/via82xx.c | 50 ++++++++---------
> sound/pci/via82xx_modem.c | 26 ++++-----
> sound/pci/vx222/vx222.c | 14 ++---
> sound/pci/ymfpci/ymfpci.c | 12 ++--
> sound/pci/ymfpci/ymfpci_main.c | 44 ++++++++-------
> sound/ppc/awacs.c | 54 +++++++++---------
> sound/ppc/beep.c | 2 +-
> sound/ppc/burgundy.c | 22 ++++----
> sound/ppc/daca.c | 2 +-
> sound/ppc/keywest.c | 4 +-
> sound/ppc/pmac.c | 12 ++--
> sound/ppc/powermac.c | 6 +-
> sound/ppc/snd_ps3.c | 12 ++--
> sound/ppc/tumbler.c | 16 +++---
> sound/sh/aica.c | 13 ++---
> sound/sh/sh_dac_audio.c | 10 ++--
> sound/sparc/amd7930.c | 16 +++---
> sound/sparc/cs4231.c | 38 ++++++-------
> sound/sparc/dbri.c | 28 +++++-----
> sound/spi/at73c213.c | 20 +++----
> sound/usb/6fire/chip.c | 4 +-
> sound/usb/6fire/comm.c | 2 +-
> sound/usb/6fire/comm.h | 2 +-
> sound/usb/6fire/control.c | 8 +--
> sound/usb/6fire/control.h | 2 +-
> sound/usb/6fire/firmware.h | 2 +-
> sound/usb/6fire/midi.c | 2 +-
> sound/usb/6fire/midi.h | 2 +-
> sound/usb/6fire/pcm.c | 8 +--
> sound/usb/6fire/pcm.h | 2 +-
> sound/usb/caiaq/control.c | 8 +--
> sound/usb/caiaq/device.c | 6 +-
> 198 files changed, 1794 insertions(+), 1785 deletions(-)
>
> --
> 1.8.0.1
>
next prev parent reply other threads:[~2012-12-06 17:41 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1354815329-20221-1-git-send-email-wfp5p@virginia.edu>
2012-12-06 17:35 ` [PATCH 03/20] ALSA: pxa2xx: remove __dev* attributes Bill Pemberton
2012-12-06 17:35 ` Bill Pemberton
2012-12-08 16:24 ` Haojian Zhuang
2012-12-08 16:24 ` Haojian Zhuang
2012-12-06 17:35 ` [PATCH 07/20] ALSA: parisc/harmony: " Bill Pemberton
2012-12-06 17:35 ` [PATCH 09/20] ALSA: ad1889: " Bill Pemberton
2012-12-06 17:35 ` [PATCH 13/20] ALSA: sound/ps3: " Bill Pemberton
2012-12-06 22:24 ` Geoff Levand
[not found] ` <1354815329-20221-21-git-send-email-wfp5p@virginia.edu>
2012-12-06 17:36 ` [PATCH 20/20] ALSA: snd-usb-caiaq: " Daniel Mack
2012-12-06 17:41 ` Takashi Iwai [this message]
[not found] ` <20121206174556.D585580213@viridian.itc.virginia.edu>
2012-12-07 8:39 ` [PATCH 00/20] ALSA: remove __dev* attributes for retirement of HOTPLUG Takashi Iwai
[not found] ` <1354815329-20221-6-git-send-email-wfp5p@virginia.edu>
2012-12-06 18:13 ` [PATCH 05/20] ALSA: firewire-speakers: remove __dev* attributes Clemens Ladisch
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=s5hd2ynqdrs.wl%tiwai@suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@alsa-project.org \
--cc=gregkh@linuxfoundation.org \
--cc=wfp5p@virginia.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.