diff options
author | Yisheng Xie <xieyisheng1@huawei.com> | 2018-02-12 18:43:13 +0800 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2018-02-16 17:29:07 +0100 |
commit | acb42f8e89efcf116aa56f4be31776d809e8aa79 (patch) | |
tree | 668dccf2ba2f6d01f8c22518d51e26f673dc63c6 /drivers/staging/android/ion | |
parent | staging: android: ion: Return void instead of int (diff) | |
download | linux-dev-acb42f8e89efcf116aa56f4be31776d809e8aa79.tar.xz linux-dev-acb42f8e89efcf116aa56f4be31776d809e8aa79.zip |
staging: android: ion: Cleanup ion_page_pool_alloc_pages
ion_page_pool_alloc_pages calls alloc_pages to allocate pages for page
pools. If alloc_pages return NULL, it will return NULL, or it will
return the pages allocate from alloc_pages. So we can just return
alloc_pages without any judgement.
Acked-by: Sumit Semwal <sumit.semwal@linaro.org>
Signed-off-by: Yisheng Xie <xieyisheng1@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/android/ion')
-rw-r--r-- | drivers/staging/android/ion/ion_page_pool.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/staging/android/ion/ion_page_pool.c b/drivers/staging/android/ion/ion_page_pool.c index e3a6e327ec0a..6d2caf04943c 100644 --- a/drivers/staging/android/ion/ion_page_pool.c +++ b/drivers/staging/android/ion/ion_page_pool.c @@ -11,13 +11,9 @@ #include "ion.h" -static void *ion_page_pool_alloc_pages(struct ion_page_pool *pool) +static inline struct page *ion_page_pool_alloc_pages(struct ion_page_pool *pool) { - struct page *page = alloc_pages(pool->gfp_mask, pool->order); - - if (!page) - return NULL; - return page; + return alloc_pages(pool->gfp_mask, pool->order); } static void ion_page_pool_free_pages(struct ion_page_pool *pool, |