On 03/12/14 11:42, Mauro Carvalho Chehab wrote: > Em Mon, 17 Feb 2014 10:57:29 +0100 > Hans Verkuil <hverkuil@xxxxxxxxx> escreveu: > >> From: Hans Verkuil <hans.verkuil@xxxxxxxxx> >> >> Add core support for matrices. > > Again, this patch has negative values for array index. > > I'll stop analyzing here, as it is hard to keep the mind in a > sane state seeing those crazy things ;) Rather than getting bogged down in these details can you please give your opinion on the public API aspects. There is no point for me to spend time on this and then get it NACKed because you don't like the API itself and want something completely different. Internal things I can change, but I'm not going to spend a second on that unless I know the concept stands. Otherwise it is wasted time. This is something we need to improve on with regards to our processes: when it comes to API enhancements you really need to be involved earlier or it's going to be a huge waste of everyones time it is gets NACked. Not to mention demotivating and frustrating for all concerned. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
