Hans de Goede wrote:
> Hi,

Hi Hans, Yordan,

> First of all my apologies for taking so long to get around to
> reviewing this.
> 
> Over all it looks good, I've put some small remarks inline, if
> you fix these I can merge this. I wonder though, given the recent
> limbo around Nokia's change of focus, if there are any plans to
> actually move forward with a plugin using this... ?

Yes, there are. That hasn't changed a bit.

> The reason I'm asking is that adding the plugin framework if nothing
> is going to use it seems a bit senseless.

I agree with that. I also hope others would find the plugin framework
useful as well. :-)

A minor comment on the patch itself: there are a few checkpatch.pl
warnings from it, mostly lines over 80 characters, but also others.

Cheers,

-- 
Sakari Ailus
sakari.ai...@maxwell.research.nokia.com
--
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