Re: [PATCH v4 09/12] [media] vivid: Local optimization

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Philipp:

On Mon, Jul 18, 2016 at 4:16 PM, Philipp Zabel <p.zabel@xxxxxxxxxxxxxx> wrote:

> I had the best looking result with this variant, btw:
>
>         y >>= 4;
>         cb >>= 4;
>         cr >>= 4;
>         if (tpg->real_quantization == V4L2_QUANTIZATION_LIM_RANGE) {
>                 y = clamp(y, 16, 235);
>                 cb = clamp(cb, 16, 240);
>                 cr = clamp(cr, 16, 240);
>         } else {
>                 y = clamp(y, 1, 254);
>                 cb = clamp(cb, 1, 254);
>                 cr = clamp(cr, 1, 254);
>         }

I like this variant much better than mine. I have applied to my local
tree. So it will be what I post on v5

Thanks for you comments :)


btw: it is scary what the compiler optimizations are capable of doing these days


-- 
Ricardo Ribalda
--
To unsubscribe from this list: send the line "unsubscribe linux-doc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux