Mailinglist Archive: radeonhd (288 mails)

[bugzilla-daemon@xxxxxxxxxxxxxxx]

http://bugs.freedesktop.org/show_bug.cgi?id=22901


Matthias Hopf <mhopf@xxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




--- Comment #2 from Matthias Hopf <mhopf@xxxxxxx>  2009-07-23 04:16:11 PST ---
(In reply to comment #1)
> (In reply to comment #0)
> > Created an attachment (id=27928)
 --> (http://bugs.freedesktop.org/attachment.cgi?id=27928) [details] [details]
> > patch to fix YUV scaling and use Rec709 coefficients
> >
> > radeonhd uses certain constants in the conversion from YUV to RGB in the Xv
> > code.
> > However, there's a minor scaling bug where 256 is used instead of 255 as the
> > maximum value of an 8-bit unsigned.
>
> Good catch. Those off-by-ones always gets ya. ;)
>
> I'll push this change once I get my commit access (soon), if no one else has
> done it already.

Thanks a lot, Javeed. I agree that HDTV color space is more appropriate as a
default.
Also thanks for the detailed description.

Yang, I pushed this pash (slightly modified due to Egbert's cleanup actions)
already. Nevertheless I hope that you get your commit rights soon :-)


-- 
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
_______________________________________________
xorg-team mailing list
xorg-team@xxxxxxxxxxx
http://lists.x.org/mailman/listinfo/xorg-team
-- 
To unsubscribe, e-mail: radeonhd+unsubscribe@xxxxxxxxxxxx
For additional commands, e-mail: radeonhd+help@xxxxxxxxxxxx