Rockbox

This doesn't seem to cause any problem on old model E200s - can anyone else confirm?

The fact that you found this solution should lead to a genuine one. Try changing the __attribute__((aligned(16))) to __attribute__((aligned(176*2))) for lcd_driver_framebuffer. Study what that does and find the smallest alignment that works. I really have suspected alignment but cannot test so let us just take the next logical step.

Any chance anyone will ever check it for framebuffer alignment issues? c'mon. so easy to do. :)

I can confirm that this does not cause problems for my old Sansa, and several builds on Anything But iPod have this patch added, with no reports of problems.

I have something like this bug. I get the two lines in rockbox, but only until it shows the logo, then the display returns to normal. I have tried playing around with various alignments all the way up to 0x10000 to no avail. I also experienced this problem with the bootloader in the past but have not been able to reproduce it recently.

I do own a newer Sansa e280 (no FM) that does show the screen glitch. Running the patched Rockbox version from cpchan the scrambled line has disappeared. So I would appreciate to see this patch in the original Rockbox trunk.

Barry,
JdGordon tried some alignment stuff for me. I come to find out alignments must be an integer power of two. Didn't seem to have any effect. :( I can do it by holding any button other than "|<<" with the bootloader I installed right when I got the player. You never poked at the register settings? Perhaps messing around with the lcd driver would work. Maybe sync pulse polarity. Some things could be set differently depending on model if that can be detected.

Arno,
It's not really a genuine fix so I personally don't feel this patch is appropriate for the trunk esp. since it puts a burdon on e200s that don't have the problem. I just find it weird that the first two screen line are taken to be at the highest address by the controller and then it wraps.

"since it puts a burdon on e200s that don't have the problem."

The three builds on anythingbutipod.com have the patch added (mine, cpchan's, and digerati1338's) and I haven't noticed any problems with having it on my Sansa. I'm not sure why Ghost had warned not to put it on the e200s without the problem, but I haven't heard of any complaints about it on anythingbutipod. However, since it isn't a genuine fix, I don't think the patch should be added to SVN.

Here's a cleaner patch. Unfortunately, I haven't been able to build sansapatcher on my machine to install the new bootloader that results. YMMV. If I can get the new bootloader, the patch should be smaller.

Here's the final patch. In order to work, you must build a bootloader as well (install with sansapatcher -a new_bootloader). It has the added advantage of getting rid of the garbage when you load the original firmware. It should work with original e200s. It would be great to have some testing. From my end, it looks ready for SVN. Let me know.

Added new task to include the patch " FS#7654  - LCD garbled top two lines".

Discussion of it can go there.