diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-03-22 23:31:15 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-03-22 23:31:15 -0400 |
commit | 34699403e9916060af8ae23f5e4705a6c078e79d (patch) | |
tree | e149ca6354171caf61132d80508ad878b00878c9 /sound/firewire | |
parent | 7fc86a7908a4e9eb2da4b6498f86193d113842d3 (diff) | |
parent | d1bbd20972936b9b178fda3eb1ec417cb27fdc01 (diff) |
Merge tag 'firewire-updates' of git://git.kernel.org/pub/scm/linux/kernel/git/ieee1394/linux1394
Pull IEEE 1394 (FireWire) subsystem updates post v3.3 from Stefan Richter:
- Some SBP-2 initiator fixes, side product from ongoing work on a target.
- Reintroduction of an isochronous I/O feature of the older ieee1394 driver
stack (flush buffer completions); it was evidently rarely used but not
actually unused. Matching libraw1394 code is already available.
- Be sure to prefix all kernel log messages with device name or card name,
and other logging related cleanups.
- Misc other small cleanups, among them a small API change that affects
sound/firewire/ too. Clemens Ladisch is aware of it.
* tag 'firewire-updates' of git://git.kernel.org/pub/scm/linux/kernel/git/ieee1394/linux1394: (26 commits)
firewire: allow explicit flushing of iso packet completions
firewire: prevent dropping of completed iso packet header data
firewire: ohci: factor out iso completion flushing code
firewire: ohci: simplify iso header pointer arithmetic
firewire: ohci: optimize control bit checks
firewire: ohci: remove unused excess_bytes field
firewire: ohci: copy_iso_headers(): make comment match the code
firewire: cdev: fix IR multichannel event documentation
firewire: ohci: fix too-early completion of IR multichannel buffers
firewire: ohci: move runtime debug facility out of #ifdef
firewire: tone down some diagnostic log messages
firewire: sbp2: replace a GFP_ATOMIC allocation
firewire: sbp2: Fix SCSI sense data mangling
firewire: sbp2: Ignore SBP-2 targets on the local node
firewire: sbp2: Take into account Unit_Unique_ID
firewire: nosy: Use the macro DMA_BIT_MASK().
firewire: core: convert AR-req handler lock from _irqsave to _bh
firewire: core: fix race at address_handler unregistration
firewire: core: remove obsolete comment
firewire: core: prefix log messages with card name
...
Diffstat (limited to 'sound/firewire')
-rw-r--r-- | sound/firewire/isight.c | 4 | ||||
-rw-r--r-- | sound/firewire/speakers.c | 4 |
2 files changed, 1 insertions, 7 deletions
diff --git a/sound/firewire/isight.c b/sound/firewire/isight.c index cd094ecaca3b..d428ffede4f3 100644 --- a/sound/firewire/isight.c +++ b/sound/firewire/isight.c | |||
@@ -611,7 +611,6 @@ static void isight_card_free(struct snd_card *card) | |||
611 | 611 | ||
612 | fw_iso_resources_destroy(&isight->resources); | 612 | fw_iso_resources_destroy(&isight->resources); |
613 | fw_unit_put(isight->unit); | 613 | fw_unit_put(isight->unit); |
614 | fw_device_put(isight->device); | ||
615 | mutex_destroy(&isight->mutex); | 614 | mutex_destroy(&isight->mutex); |
616 | } | 615 | } |
617 | 616 | ||
@@ -644,7 +643,7 @@ static int isight_probe(struct device *unit_dev) | |||
644 | isight->card = card; | 643 | isight->card = card; |
645 | mutex_init(&isight->mutex); | 644 | mutex_init(&isight->mutex); |
646 | isight->unit = fw_unit_get(unit); | 645 | isight->unit = fw_unit_get(unit); |
647 | isight->device = fw_device_get(fw_dev); | 646 | isight->device = fw_dev; |
648 | isight->audio_base = get_unit_base(unit); | 647 | isight->audio_base = get_unit_base(unit); |
649 | if (!isight->audio_base) { | 648 | if (!isight->audio_base) { |
650 | dev_err(&unit->device, "audio unit base not found\n"); | 649 | dev_err(&unit->device, "audio unit base not found\n"); |
@@ -681,7 +680,6 @@ static int isight_probe(struct device *unit_dev) | |||
681 | 680 | ||
682 | err_unit: | 681 | err_unit: |
683 | fw_unit_put(isight->unit); | 682 | fw_unit_put(isight->unit); |
684 | fw_device_put(isight->device); | ||
685 | mutex_destroy(&isight->mutex); | 683 | mutex_destroy(&isight->mutex); |
686 | error: | 684 | error: |
687 | snd_card_free(card); | 685 | snd_card_free(card); |
diff --git a/sound/firewire/speakers.c b/sound/firewire/speakers.c index cbe6bb9e53b6..297244e658d9 100644 --- a/sound/firewire/speakers.c +++ b/sound/firewire/speakers.c | |||
@@ -656,12 +656,10 @@ static u32 fwspk_read_firmware_version(struct fw_unit *unit) | |||
656 | static void fwspk_card_free(struct snd_card *card) | 656 | static void fwspk_card_free(struct snd_card *card) |
657 | { | 657 | { |
658 | struct fwspk *fwspk = card->private_data; | 658 | struct fwspk *fwspk = card->private_data; |
659 | struct fw_device *dev = fw_parent_device(fwspk->unit); | ||
660 | 659 | ||
661 | amdtp_out_stream_destroy(&fwspk->stream); | 660 | amdtp_out_stream_destroy(&fwspk->stream); |
662 | cmp_connection_destroy(&fwspk->connection); | 661 | cmp_connection_destroy(&fwspk->connection); |
663 | fw_unit_put(fwspk->unit); | 662 | fw_unit_put(fwspk->unit); |
664 | fw_device_put(dev); | ||
665 | mutex_destroy(&fwspk->mutex); | 663 | mutex_destroy(&fwspk->mutex); |
666 | } | 664 | } |
667 | 665 | ||
@@ -718,7 +716,6 @@ static int __devinit fwspk_probe(struct device *unit_dev) | |||
718 | fwspk = card->private_data; | 716 | fwspk = card->private_data; |
719 | fwspk->card = card; | 717 | fwspk->card = card; |
720 | mutex_init(&fwspk->mutex); | 718 | mutex_init(&fwspk->mutex); |
721 | fw_device_get(fw_dev); | ||
722 | fwspk->unit = fw_unit_get(unit); | 719 | fwspk->unit = fw_unit_get(unit); |
723 | fwspk->device_info = fwspk_detect(fw_dev); | 720 | fwspk->device_info = fwspk_detect(fw_dev); |
724 | if (!fwspk->device_info) { | 721 | if (!fwspk->device_info) { |
@@ -767,7 +764,6 @@ err_connection: | |||
767 | cmp_connection_destroy(&fwspk->connection); | 764 | cmp_connection_destroy(&fwspk->connection); |
768 | err_unit: | 765 | err_unit: |
769 | fw_unit_put(fwspk->unit); | 766 | fw_unit_put(fwspk->unit); |
770 | fw_device_put(fw_dev); | ||
771 | mutex_destroy(&fwspk->mutex); | 767 | mutex_destroy(&fwspk->mutex); |
772 | error: | 768 | error: |
773 | snd_card_free(card); | 769 | snd_card_free(card); |