Gianluca,

One quick comment about your patch; I've noticed that you've declared two new "GPL only" symbols:

EXPORT_SYMBOL_GPL(em28xx_capture_start);
EXPORT_SYMBOL_GPL(em28xx_alloc_isoc);

I'm not sure what the exact policy is with GPL symbols, but I do know what Al Viro posted recently on the subject:

http://thread.gmane.org/gmane.linux.file-systems/61372

Do we really need EXPORT_SYMBOL_GPL() here?

Cheers,
Chris
--
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

Reply via email to