Opened 13 years ago
Closed 11 years ago
Last modified 11 years ago
#9971 closed Patch - Feature (Won't Fix)
Improve CrystalHD decoding for BCM 70012
Reported by: | 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 (1)
Change History (8)
Changed 13 years ago by
Attachment: | bcm70012.patch added |
---|
comment:1 Changed 13 years ago by
Owner: | set to markk |
---|---|
Status: | new → assigned |
comment:2 Changed 12 years ago by
Milestone: | unknown → 0.25 |
---|---|
Status: | assigned → accepted |
comment:3 Changed 12 years ago by
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 12 years ago by
Owner: | markk deleted |
---|---|
Status: | accepted → assigned |
comment:5 Changed 12 years ago by
Milestone: | 0.25 → unknown |
---|---|
Status: | assigned → new |
comment:6 Changed 11 years ago by
Resolution: | → Won't Fix |
---|---|
Status: | new → closed |
I don't think any of the current developers own the hardware to work on this issue. Mark identified issues with the attached patch so this ticket is going nowhere. We don't claim to support the BCM 70012 making this a new feature, not a bug fix.
Instead of leaving the ticket open indefinitely I'm going to close it. If someone with the appropriate hardware takes an interest in adding working support for the 70012 they can open a new ticket.
comment:7 Changed 11 years ago by
Type: | Patch - Bug Fix → Patch - Feature |
---|
crystalhd bcm70012 patch