Modify

Ticket #9971 (new Patch - Bug Fix)

Opened 22 months ago

Last modified 15 months ago

Improve CrystalHD decoding for BCM 70012

Reported by: czhang1974@… Owned by:
Priority: minor Milestone: unknown
Component: MythTV - Video Decoding Version: 0.24-fixes
Severity: medium Keywords: crystalhd
Cc: Ticket locked: no

Description

This patch fixes the green overlay problem reported in the following thread on the bcm70012.

http://www.gossamer-threads.com/lists/mythtv/dev/453116

The default output format from CrystalHD on the bcm70012 is NV12, not YV12. It can be changed to use YV12 by setting some flags if we use the DtsProcOutput? instead of DtsProcOutputNoCopy?, but further reading of the library source code shows that it was converted to YV12 in software anyways. The hardware output is always in NV12, so there is no gain in using YV12.

I also had to make sure that the buffer is always released after decoding. I am still not sure why the code checked for mpa_pic.data[3] before calling the release_buffer, which effectively blocked the call since that would always be NULL for the crystalhd decoder. It most likely is breaking something else that I don't know about, but I will leave that to someone more familiar with the other private decoders.

This now works reasonably well for 720P on my AppleTV. 1080i is still having serious stuttering problem that I can't find the root cause of yet.

Attachments

bcm70012.patch (3.5 KB) - added by czhang1974@… 22 months ago.
crystalhd bcm70012 patch

Change History

Changed 22 months ago by czhang1974@…

crystalhd bcm70012 patch

comment:1 Changed 22 months ago by wagnerrp

  • Owner set to markk
  • Status changed from new to assigned

comment:2 Changed 20 months ago by markk

  • Status changed from assigned to accepted
  • Milestone changed from unknown to 0.25

comment:3 Changed 20 months ago by markk

Finally managed to test this in master. Unfortunately, while some streams play others are pretty broken. From memory, the BCM70012 needs an alignment fix for streams that are not 'normal' widths of 720 or 1920. Additionally, some streams that are straightforward 1080p still look slightly broken.

I'll see what I can do.

(N.B. not regression tested with the BCM70015 either)

comment:4 Changed 16 months ago by markk

  • Owner markk deleted
  • Status changed from accepted to assigned

comment:5 Changed 15 months ago by beirdo

  • Status changed from assigned to new
  • Milestone changed from 0.25 to unknown
View

Add a comment

Modify Ticket

Action
as new
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.