[POWERPC] Simplify dma_ops bug conditions

Use BUG_ON rather than BUG to simplify the dma_ops handing,
and remove the now-unnecessary return cases.

Booted on pseries.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
This commit is contained in:
Jeremy Kerr 2006-07-13 16:32:52 +10:00 committed by Paul Mackerras
parent 931b261f44
commit 5d33eebee8

View file

@ -35,10 +35,9 @@ int dma_supported(struct device *dev, u64 mask)
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
return dma_ops->dma_supported(dev, mask); return dma_ops->dma_supported(dev, mask);
BUG();
return 0;
} }
EXPORT_SYMBOL(dma_supported); EXPORT_SYMBOL(dma_supported);
@ -66,10 +65,9 @@ void *dma_alloc_coherent(struct device *dev, size_t size,
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
return dma_ops->alloc_coherent(dev, size, dma_handle, flag); return dma_ops->alloc_coherent(dev, size, dma_handle, flag);
BUG();
return NULL;
} }
EXPORT_SYMBOL(dma_alloc_coherent); EXPORT_SYMBOL(dma_alloc_coherent);
@ -78,10 +76,9 @@ void dma_free_coherent(struct device *dev, size_t size, void *cpu_addr,
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
dma_ops->free_coherent(dev, size, cpu_addr, dma_handle); dma_ops->free_coherent(dev, size, cpu_addr, dma_handle);
else
BUG();
} }
EXPORT_SYMBOL(dma_free_coherent); EXPORT_SYMBOL(dma_free_coherent);
@ -90,10 +87,9 @@ dma_addr_t dma_map_single(struct device *dev, void *cpu_addr, size_t size,
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
return dma_ops->map_single(dev, cpu_addr, size, direction); return dma_ops->map_single(dev, cpu_addr, size, direction);
BUG();
return (dma_addr_t)0;
} }
EXPORT_SYMBOL(dma_map_single); EXPORT_SYMBOL(dma_map_single);
@ -102,10 +98,9 @@ void dma_unmap_single(struct device *dev, dma_addr_t dma_addr, size_t size,
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
dma_ops->unmap_single(dev, dma_addr, size, direction); dma_ops->unmap_single(dev, dma_addr, size, direction);
else
BUG();
} }
EXPORT_SYMBOL(dma_unmap_single); EXPORT_SYMBOL(dma_unmap_single);
@ -115,11 +110,10 @@ dma_addr_t dma_map_page(struct device *dev, struct page *page,
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
return dma_ops->map_single(dev,
(page_address(page) + offset), size, direction); return dma_ops->map_single(dev, page_address(page) + offset, size,
BUG(); direction);
return (dma_addr_t)0;
} }
EXPORT_SYMBOL(dma_map_page); EXPORT_SYMBOL(dma_map_page);
@ -128,10 +122,9 @@ void dma_unmap_page(struct device *dev, dma_addr_t dma_address, size_t size,
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
dma_ops->unmap_single(dev, dma_address, size, direction); dma_ops->unmap_single(dev, dma_address, size, direction);
else
BUG();
} }
EXPORT_SYMBOL(dma_unmap_page); EXPORT_SYMBOL(dma_unmap_page);
@ -140,10 +133,9 @@ int dma_map_sg(struct device *dev, struct scatterlist *sg, int nents,
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
return dma_ops->map_sg(dev, sg, nents, direction); return dma_ops->map_sg(dev, sg, nents, direction);
BUG();
return 0;
} }
EXPORT_SYMBOL(dma_map_sg); EXPORT_SYMBOL(dma_map_sg);
@ -152,9 +144,8 @@ void dma_unmap_sg(struct device *dev, struct scatterlist *sg, int nhwentries,
{ {
struct dma_mapping_ops *dma_ops = get_dma_ops(dev); struct dma_mapping_ops *dma_ops = get_dma_ops(dev);
if (dma_ops) BUG_ON(!dma_ops);
dma_ops->unmap_sg(dev, sg, nhwentries, direction); dma_ops->unmap_sg(dev, sg, nhwentries, direction);
else
BUG();
} }
EXPORT_SYMBOL(dma_unmap_sg); EXPORT_SYMBOL(dma_unmap_sg);