Re: [PATCH 1/6] adv7180: Remove duplicate unregister call

2014-07-17 Thread Hans Verkuil
On 07/06/2014 12:26 AM, Ben Dooks wrote:
 From: Ian Molton ian.mol...@codethink.co.uk
 
 This driver moved over to v4l2_async_unregister_subdev()
 but still retained a call to v4l2_unregister_subdev(). Remove.
 
 Signed-off-by: Ian Molton ian.mol...@codethink.co.uk
 Signed-off-by: Ben Dooks ben.do...@codethink.co.uk

Acked-by: Hans Verkuil hans.verk...@cisco.com

Thanks,

Hans

 ---
  drivers/media/i2c/adv7180.c | 1 -
  1 file changed, 1 deletion(-)
 
 diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c
 index ac1cdbe..821178d 100644
 --- a/drivers/media/i2c/adv7180.c
 +++ b/drivers/media/i2c/adv7180.c
 @@ -663,7 +663,6 @@ static int adv7180_remove(struct i2c_client *client)
   if (state-irq  0)
   free_irq(client-irq, state);
  
 - v4l2_device_unregister_subdev(sd);
   adv7180_exit_controls(state);
   mutex_destroy(state-mutex);
   return 0;
 

--
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 1/6] adv7180: Remove duplicate unregister call

2014-07-05 Thread Ben Dooks
From: Ian Molton ian.mol...@codethink.co.uk

This driver moved over to v4l2_async_unregister_subdev()
but still retained a call to v4l2_unregister_subdev(). Remove.

Signed-off-by: Ian Molton ian.mol...@codethink.co.uk
Signed-off-by: Ben Dooks ben.do...@codethink.co.uk
---
 drivers/media/i2c/adv7180.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/media/i2c/adv7180.c b/drivers/media/i2c/adv7180.c
index ac1cdbe..821178d 100644
--- a/drivers/media/i2c/adv7180.c
+++ b/drivers/media/i2c/adv7180.c
@@ -663,7 +663,6 @@ static int adv7180_remove(struct i2c_client *client)
if (state-irq  0)
free_irq(client-irq, state);
 
-   v4l2_device_unregister_subdev(sd);
adv7180_exit_controls(state);
mutex_destroy(state-mutex);
return 0;
-- 
2.0.0

--
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