Remove unused parameters, shorten the function name.

Signed-off-by: Christian König <christian.koe...@amd.com>
---
 drivers/gpu/drm/ttm/ttm_page_alloc.c | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/drivers/gpu/drm/ttm/ttm_page_alloc.c 
b/drivers/gpu/drm/ttm/ttm_page_alloc.c
index 912c30dcc9db..111031cbb6df 100644
--- a/drivers/gpu/drm/ttm/ttm_page_alloc.c
+++ b/drivers/gpu/drm/ttm/ttm_page_alloc.c
@@ -467,11 +467,11 @@ static int ttm_set_pages_caching(struct page **pages,
  * any pages that have changed their caching state already put them to the
  * pool.
  */
-static void ttm_handle_caching_state_failure(struct list_head *pages,
-               int ttm_flags, enum ttm_caching_state cstate,
-               struct page **failed_pages, unsigned cpages)
+static void ttm_handle_caching_failure(struct page **failed_pages,
+                                      unsigned cpages)
 {
        unsigned i;
+
        /* Failed pages have to be freed */
        for (i = 0; i < cpages; ++i) {
                list_del(&failed_pages[i]->lru);
@@ -517,9 +517,8 @@ static int ttm_alloc_new_pages(struct list_head *pages, 
gfp_t gfp_flags,
                                r = ttm_set_pages_caching(caching_array,
                                                          cstate, cpages);
                                if (r)
-                                       ttm_handle_caching_state_failure(pages,
-                                               ttm_flags, cstate,
-                                               caching_array, cpages);
+                                       
ttm_handle_caching_failure(caching_array,
+                                                                  cpages);
                        }
                        r = -ENOMEM;
                        goto out;
@@ -542,9 +541,8 @@ static int ttm_alloc_new_pages(struct list_head *pages, 
gfp_t gfp_flags,
                                r = ttm_set_pages_caching(caching_array,
                                                cstate, cpages);
                                if (r) {
-                                       ttm_handle_caching_state_failure(pages,
-                                               ttm_flags, cstate,
-                                               caching_array, cpages);
+                                       
ttm_handle_caching_failure(caching_array,
+                                                                  cpages);
                                        goto out;
                                }
                                cpages = 0;
@@ -555,9 +553,7 @@ static int ttm_alloc_new_pages(struct list_head *pages, 
gfp_t gfp_flags,
        if (cpages) {
                r = ttm_set_pages_caching(caching_array, cstate, cpages);
                if (r)
-                       ttm_handle_caching_state_failure(pages,
-                                       ttm_flags, cstate,
-                                       caching_array, cpages);
+                       ttm_handle_caching_failure(caching_array, cpages);
        }
 out:
        kfree(caching_array);
-- 
2.17.1

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to