diff options
author | Shannon Nelson <shannon.nelson@intel.com> | 2007-10-18 06:07:13 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-18 17:37:32 -0400 |
commit | 5149fd010f404889b7d8f79159057791fbb817b1 (patch) | |
tree | 0ec09c009f35d59be64e3b952b6deba39b61f6e1 /drivers/dma/ioat.c | |
parent | dfe2299e7b35a0adfc87f04d3e725ccc508d7626 (diff) |
I/OAT: clean up error handling and some print messages
Make better use of dev_err(), and catch an error where the transaction
creation might fail.
Signed-off-by: Shannon Nelson <shannon.nelson@intel.com>
Cc: Dan Williams <dan.j.williams@intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/dma/ioat.c')
-rw-r--r-- | drivers/dma/ioat.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/dma/ioat.c b/drivers/dma/ioat.c index a45872f32d50..f204c39fb412 100644 --- a/drivers/dma/ioat.c +++ b/drivers/dma/ioat.c | |||
@@ -34,7 +34,7 @@ | |||
34 | #include "ioatdma_registers.h" | 34 | #include "ioatdma_registers.h" |
35 | #include "ioatdma_hw.h" | 35 | #include "ioatdma_hw.h" |
36 | 36 | ||
37 | MODULE_VERSION("1.24"); | 37 | MODULE_VERSION(IOAT_DMA_VERSION); |
38 | MODULE_LICENSE("GPL"); | 38 | MODULE_LICENSE("GPL"); |
39 | MODULE_AUTHOR("Intel Corporation"); | 39 | MODULE_AUTHOR("Intel Corporation"); |
40 | 40 | ||
@@ -85,6 +85,7 @@ static void ioat_shutdown_functionality(struct pci_dev *pdev) | |||
85 | { | 85 | { |
86 | struct ioat_device *device = pci_get_drvdata(pdev); | 86 | struct ioat_device *device = pci_get_drvdata(pdev); |
87 | 87 | ||
88 | dev_err(&pdev->dev, "Removing dma and dca services\n"); | ||
88 | if (device->dca) { | 89 | if (device->dca) { |
89 | unregister_dca_provider(device->dca); | 90 | unregister_dca_provider(device->dca); |
90 | free_dca_provider(device->dca); | 91 | free_dca_provider(device->dca); |