On Mon, May 24, 2010 at 12:57:14PM +0200, Daniel Mack wrote: > Propagte correct error values instead of returning -1 which just means > -EPERM ("Permission denied") > > Signed-off-by: Daniel Mack <daniel@xxxxxxxx> > Cc: Wolfram Sang <w.sang@xxxxxxxxxxxxxx> > Cc: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> > Cc: Jiri Slaby <jslaby@xxxxxxx> > Cc: Dmitry Torokhov <dtor@xxxxxxx> > Cc: Devin Heitmueller <dheitmueller@xxxxxxxxxxxxxx> > Cc: linux-media@xxxxxxxxxxxxxxx I am no specialist for this driver, but as I am on cc and it looks good to me (no big changes, after all): Acked-by: Wolfram Sang <w.sang@xxxxxxxxxxxxxx> -- Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ |
Attachment:
signature.asc
Description: Digital signature