diff mbox

[3/15] - tm6000 bugfix hunk in init_dev

Message ID 4B69D950.2090504@arcor.de (mailing list archive)
State Superseded, archived
Headers

Commit Message

Stefan Ringel Feb. 3, 2010, 8:15 p.m. UTC 
  signed-off-by: Stefan Ringel <stefan.ringel@arcor.de>

Comments

Mauro Carvalho Chehab Feb. 3, 2010, 8:22 p.m. UTC | #1 
This is too generic:

 tm6000 bugfix hunk in init_dev

A better subject would be:

[PATCH 3/15] tm6000: avoid unregister the driver after success at tm6000_init_dev

PS.: don't mind to re-submit those patches where just the subject has troubles.
I'll fix it locally. I'm pointing the issues to save me some time on your next
patcheset submission ;)

For the patchsets that you forgot to sign (1/15 for example), just reply your own
email with Signed-off-by:.

Cheers,
Mauro 


Stefan Ringel wrote:
> signed-off-by: Stefan Ringel <stefan.ringel@arcor.de>
> 
> --- a/drivers/staging/tm6000/tm6000-cards.c
> +++ b/drivers/staging/tm6000/tm6000-cards.c
> @@ -402,6 +448,7 @@ static int tm6000_init_dev(struct tm6000_core *dev)
>          }
>  #endif
>      }
> +    return 0;
>  
>  err2:
>      v4l2_device_unregister(&dev->v4l2_dev);
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

--- a/drivers/staging/tm6000/tm6000-cards.c
+++ b/drivers/staging/tm6000/tm6000-cards.c
@@ -402,6 +448,7 @@  static int tm6000_init_dev(struct tm6000_core *dev)
         }
 #endif
     }
+    return 0;
 
 err2:
     v4l2_device_unregister(&dev->v4l2_dev);