Re: [PATCH 2/2] [media] pxa_camera: remove an unused structure pointer

2016-09-18 Thread Robert Jarzmik 
Mauro Carvalho Chehab  writes:

> As reported by smatch:
>
> drivers/media/platform/pxa_camera.c: In function 'pxa_dma_start_channels':
> drivers/media/platform/pxa_camera.c:457:21: warning: variable 'active' set 
> but not used [-Wunused-but-set-variable]
>   struct pxa_buffer *active;
>  ^~
>
> Signed-off-by: Mauro Carvalho Chehab 
Acked-by: Robert Jarzmik 

Cheers.

-- 
Robert
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 2/2] [media] pxa_camera: remove an unused structure pointer

2016-09-09 Thread Mauro Carvalho Chehab 
As reported by smatch:

drivers/media/platform/pxa_camera.c: In function 'pxa_dma_start_channels':
drivers/media/platform/pxa_camera.c:457:21: warning: variable 'active' set but 
not used [-Wunused-but-set-variable]
  struct pxa_buffer *active;
 ^~

Signed-off-by: Mauro Carvalho Chehab 
---
 drivers/media/platform/pxa_camera.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/media/platform/pxa_camera.c 
b/drivers/media/platform/pxa_camera.c
index 733677f06cb4..11478364c6d6 100644
--- a/drivers/media/platform/pxa_camera.c
+++ b/drivers/media/platform/pxa_camera.c
@@ -455,9 +455,6 @@ static void pxa_videobuf_set_actdma(struct pxa_camera_dev 
*pcdev,
 static void pxa_dma_start_channels(struct pxa_camera_dev *pcdev)
 {
int i;
-   struct pxa_buffer *active;
-
-   active = pcdev->active;
 
for (i = 0; i < pcdev->channels; i++) {
dev_dbg(pcdev_to_dev(pcdev),
-- 
2.7.4

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html