Re: [PATCH] tinyjpeg: Better luminance quantization table for Pixart JPEG

[Hans de Goede]

Hi,

Thanks for the patch, I've pushed it to v4l-utils master and I will
cherry-pick it into the stable-0.8 branch after this mail.

I noticed while testing with a pac7302 camera, that under certain
circumstances the jpeg decompression still goes wrong. When you
point the camera at a high contrast picture in proper daylight,
it switches to a higher compression for certain areas of the picture,
and these areas become quite "blockey" when this happens the marker
changes to a different value. I think we should use a different
quant. table when this happens. It would be great if you can reproduce
this and find out a way to make the image less blocky in these
cases.

Thanks & Regards,

Hans


On 03/23/2012 08:19 PM, Jean-Francois Moine wrote:

An other luminance quantization table gives a better quality to the
Pixart images created by the webcams handled by the gspca drivers
pac7302 and pac7311 (pixel format 'PJPG').

Tests have been done with 5 different pac7302 webcams. The marker was
always 0x44.

Signed-off-by: Jean-François Moine<moin...@free.fr>

diff --git a/lib/libv4lconvert/tinyjpeg.c b/lib/libv4lconvert/tinyjpeg.c
index e308f63..687e69c 100644
--- a/lib/libv4lconvert/tinyjpeg.c
+++ b/lib/libv4lconvert/tinyjpeg.c
@@ -206,14 +206,14 @@ static const unsigned char val_ac_chrominance[] = {
  };

  const unsigned char pixart_quantization[][64] = { {
-               0x07, 0x07, 0x08, 0x0a, 0x09, 0x07, 0x0d, 0x0b,
-               0x0c, 0x0d, 0x11, 0x10, 0x0f, 0x12, 0x17, 0x27,
-               0x1a, 0x18, 0x16, 0x16, 0x18, 0x31, 0x23, 0x25,
-               0x1d, 0x28, 0x3a, 0x33, 0x3d, 0x3c, 0x39, 0x33,
-               0x38, 0x37, 0x40, 0x48, 0x5c, 0x4e, 0x40, 0x44,
-               0x57, 0x45, 0x37, 0x38, 0x50, 0x6d, 0x51, 0x57,
-               0x5f, 0x62, 0x67, 0x68, 0x67, 0x3e, 0x4d, 0x71,
-               0x79, 0x70, 0x64, 0x78, 0x5c, 0x65, 0x67, 0x63,
+               0x08, 0x08, 0x08, 0x08, 0x08, 0x08, 0x10, 0x10,
+               0x10, 0x10, 0x10, 0x10, 0x10, 0x10, 0x10, 0x10,
+               0x10, 0x10, 0x10, 0x10, 0x10, 0x10, 0x10, 0x10,
+               0x10, 0x10, 0x10, 0x10, 0x20, 0x20, 0x20, 0x20,
+               0x20, 0x20, 0x20, 0x20, 0x20, 0x20, 0x20, 0x20,
+               0x20, 0x20, 0x20, 0x40, 0x40, 0x40, 0x40, 0x40,
+               0x40, 0x40, 0x40, 0x40, 0x40, 0x40, 0x40, 0x40,
+               0x40, 0x40, 0x40, 0x40, 0x40, 0x40, 0x40, 0x40,
        },
        {
                0x11, 0x12, 0x12, 0x18, 0x15, 0x18, 0x2f, 0x1a,

--
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