diff options
-rw-r--r-- | mm/dmapool.c | 24 |
1 files changed, 8 insertions, 16 deletions
diff --git a/mm/dmapool.c b/mm/dmapool.c index 8058fcd7ae91..a3a1bfe91110 100644 --- a/mm/dmapool.c +++ b/mm/dmapool.c | |||
@@ -170,24 +170,16 @@ struct dma_pool *dma_pool_create(const char *name, struct device *dev, | |||
170 | retval->boundary = boundary; | 170 | retval->boundary = boundary; |
171 | retval->allocation = allocation; | 171 | retval->allocation = allocation; |
172 | 172 | ||
173 | if (dev) { | 173 | INIT_LIST_HEAD(&retval->pools); |
174 | int ret; | ||
175 | 174 | ||
176 | mutex_lock(&pools_lock); | 175 | mutex_lock(&pools_lock); |
177 | if (list_empty(&dev->dma_pools)) | 176 | if (list_empty(&dev->dma_pools) && |
178 | ret = device_create_file(dev, &dev_attr_pools); | 177 | device_create_file(dev, &dev_attr_pools)) { |
179 | else | 178 | kfree(retval); |
180 | ret = 0; | 179 | return NULL; |
181 | /* note: not currently insisting "name" be unique */ | ||
182 | if (!ret) | ||
183 | list_add(&retval->pools, &dev->dma_pools); | ||
184 | else { | ||
185 | kfree(retval); | ||
186 | retval = NULL; | ||
187 | } | ||
188 | mutex_unlock(&pools_lock); | ||
189 | } else | 180 | } else |
190 | INIT_LIST_HEAD(&retval->pools); | 181 | list_add(&retval->pools, &dev->dma_pools); |
182 | mutex_unlock(&pools_lock); | ||
191 | 183 | ||
192 | return retval; | 184 | return retval; |
193 | } | 185 | } |