From: Stefan Wahren Date: Thu, 25 May 2017 22:26:12 +0000 (+0200) Subject: staging: vchiq_2835_arm: Reduce scope of i in free_pagelist X-Git-Tag: v4.14-rc1~623^2~234 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=158ef80a8741e55c0214c174509b42bc5de6c7e1;p=platform%2Fkernel%2Flinux-rpi.git staging: vchiq_2835_arm: Reduce scope of i in free_pagelist We can reduce the scope of the counting variable i. This has been found by CppCheck. Signed-off-by: Stefan Wahren Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c index d04db3f..9105545 100644 --- a/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c +++ b/drivers/staging/vc04_services/interface/vchiq_arm/vchiq_2835_arm.c @@ -577,7 +577,6 @@ static void free_pagelist(struct vchiq_pagelist_info *pagelistinfo, int actual) { - unsigned int i; PAGELIST_T *pagelist = pagelistinfo->pagelist; struct page **pages = pagelistinfo->pages; unsigned int num_pages = pagelistinfo->num_pages; @@ -633,6 +632,8 @@ free_pagelist(struct vchiq_pagelist_info *pagelistinfo, /* Need to mark all the pages dirty. */ if (pagelist->type != PAGELIST_WRITE && pagelistinfo->pages_need_release) { + unsigned int i; + for (i = 0; i < num_pages; i++) set_page_dirty(pages[i]); }