Re: [patch, libv4l]: Introduce define for lookup table size

2017-05-16 Thread Pavel Machek
Hi! > >>> Make lookup table size configurable at compile-time. > >> > >> I don't think I'll take this patch. The problem is that if we really add > >> support for 10 or 12 bit lookup tables in the future, then just changing > >> LSIZE isn't enough. > >> > >> This patch doesn't really add anything

Re: [patch, libv4l]: Introduce define for lookup table size

2017-05-16 Thread Hans Verkuil
On 16/05/17 14:45, Pavel Machek wrote: > Hi! > >>> Make lookup table size configurable at compile-time. >> >> I don't think I'll take this patch. The problem is that if we really add >> support for 10 or 12 bit lookup tables in the future, then just changing >> LSIZE isn't enough. >> >> This patch

Re: [patch, libv4l]: Introduce define for lookup table size

2017-05-16 Thread Pavel Machek
Hi! > > Make lookup table size configurable at compile-time. > > I don't think I'll take this patch. The problem is that if we really add > support for 10 or 12 bit lookup tables in the future, then just changing > LSIZE isn't enough. > > This patch doesn't really add anything as it stands. Wel

Re: [patch, libv4l]: Introduce define for lookup table size

2017-05-16 Thread Hans Verkuil
On 09/05/17 13:04, Pavel Machek wrote: > > Make lookup table size configurable at compile-time. I don't think I'll take this patch. The problem is that if we really add support for 10 or 12 bit lookup tables in the future, then just changing LSIZE isn't enough. This patch doesn't really add anyt

[patch, libv4l]: Introduce define for lookup table size

2017-05-09 Thread Pavel Machek
Make lookup table size configurable at compile-time. Signed-off-by: Pavel Machek diff --git a/lib/libv4lconvert/processing/libv4lprocessing-priv.h b/lib/libv4lconvert/processing/libv4lprocessing-priv.h index e4a29dd..55e1687 100644 --- a/lib/libv4lconvert/processing/libv4lprocessing-priv.h