Re: [PATCH] USB: add Sensoray 2255 v4l driver

From: dean
Date: Fri May 16 2008 - 11:57:34 EST


Hi Oliver,

Just to be clear. You mean fixing the loading issues(ugly msleep, etc...) that Mauro mentioned, not the YUV formats(which seem unrelated)?

Would it be ok to have just 422P and greyscale in the driver for now if we can't get a firmware update in a reasonable timeframe? It's certainly not ideal, but we could direct our customers to the libraries, at least until we get a proper HW fix.

Best regards,

Dean


Oliver Neukum wrote:
Am Freitag 16 Mai 2008 04:51:02 schrieb Mauro Carvalho Chehab:
+static DEFINE_MUTEX(usb_s2255_open_mutex);
Hmmm... what happens if the user plugs two or more of such devices?

Concurrent calls to open() will be delayed. Not a problem, if the
firmware issue is fixed as it needs to be.

Regards
Oliver



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/