Bug 13038 - Baby display has setup-time issues and causes horizontal stripe test pattern
: Baby display has setup-time issues and causes horizontal stripe test pattern
Status: CLOSED FIXED
Product: SB Radio
Classification: Unclassified
Component: Product Quality
: Include FW version in comment
: PC Other
: P3 normal (vote)
: 7.4.0
Assigned To: Richard Titmuss
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-07-24 12:55 UTC by Caleb Crome
Modified: 2009-10-05 14:26 UTC (History)
2 users (show)

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Caleb Crome 2009-07-24 12:55:48 UTC
Okay, we have some more information on this.  

The cause seems to be due to the change that Ricahrd and I implemented to comply with the LCD driver chip's errata (https://svn.slimdevices.com/repos/player/trunk/hardware/babyboom/docs/datasheets/display/Novatech%20Errata%20NT39116%20Abnormal%20Display%20Issue%20RGB%20Interface_090117.pdf).

The problem is that the Novatech proposed fix does not work, because it forces the data to change on the falling clock edge, which reduces setup time to almost nothing. I measured 3-5 ns setup time, whereas the spec is 15ns.

I was able to "fix" the issue by setting the LCDC data slew rate to fast, and the LCDC clock to slow.  This gives enough skew to allow the chip to function properly.  

We'll check with Wintek to see if there are any other fixes possible.  In the meantime, we can resolve this in a couple of ways:

Set the drive settings as in my 'fix'.  This is not likely to be a robust solution, as the setup time is too short.
Use SLCDC mode.  This reuquires that we actually get the driver from Freescale, which requires I (or maurice) spend a little time with Galib.  And that the hardware works for this mode, which has not been tested.
Perhaps we can give up the first and last lines of the display, and just use it as a 320x238 display.  This would allow us (I think) to set the data and clock such that the setup time is guaranteed.  But the whole display may move up or down a pixel.  This option needs to be investigated.
For the moment, I'll send Richard a patch that does the quick-and-dirty workaround that seems to make the problem go away.
Comment 1 Caleb Crome 2009-07-24 12:57:22 UTC
Commited in baby-mp branch 6776.
Comment 2 Ben Klaas 2009-08-26 07:51:52 UTC
this is an administrative shuffle on priority fields to help make better judgment on the top end of the priority list. P4->P5, P3->P4, and P2->P3.
Comment 3 James Richardson 2009-10-05 14:26:07 UTC
This bug has been marked as fixed in the 7.4.0 release version of SqueezeBox Server!
    * SqueezeCenter: 28672
    * Squeezebox 2 and 3: 130
    * Transporter: 80
    * Receiver: 65
    * Boom: 50
    * Controller: 7790
    * Radio: 7790  

Please see the Release Notes for all the details: http://wiki.slimdevices.com/index.php/Release_Notes

If you haven't already, please download and install the new version from http://www.logitechsqueezebox.com/support/download-squeezebox-server.html

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.