Rockbox

Tasklist

FS#10926 - Backlight off causes track info to freeze

Attached to Project: Rockbox
Opened by Mike Holden (mikeholden) - Thursday, 21 January 2010, 11:01 GMT
Last edited by Jonathan Gordon (jdgordon) - Tuesday, 03 August 2010, 07:28 GMT
Task Type Bugs
Category Themes
Status Closed
Assigned To No-one
Operating System Iriver H300 series
Severity Low
Priority Normal
Reported Version Release 3.4
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

On my H340, once the backlight fades to off, the now/next track names in the WPS freeze. They remain frozen once the backlight comes on again (due to button press), even though playback has now progressed onto subsequent tracks the track 1 & 2 info remain on screen. Other info (track progress, time remaining etc are still correct, just the track title, track number etc are stuck.

This is tested with current CVS r24271, and a "clean" set of settings (i.e a fresh .rockbox tree from the build, all default settings).

To reproduce:
1. Play a playlist or directory with multiple tracks (at least 2).
2. Let backlight fade to off during 1st track.
3. Wait until track transition occurs to track 2.
4. Press Vol Up to activate backlight
5. Note the track names displayed are still track 1 & 2, not track 2 & 3.
This task depends upon

Closed by  Jonathan Gordon (jdgordon)
Tuesday, 03 August 2010, 07:28 GMT
Reason for closing:  Fixed
Comment by Reiner Zufall (junkie) - Thursday, 18 February 2010, 15:30 GMT
Actually, the problem only occurs with versions greater than Release 3.4.
Comment by Mike Holden (mikeholden) - Thursday, 18 February 2010, 15:38 GMT
Release 3.5 wasn't an option at the time I filed the bug! "Release 3.4" in this context means 3.4 and/or later. I put a specific SVN revision number in to state explicitly the version with a problem.
Comment by Timothy Pelayo (Crooked Rain) - Tuesday, 23 February 2010, 20:49 GMT
I've been having this problem on my H320 lately and it's getting really annoying, especially when I'm driving. I hope someone can figure out what's wrong.
Comment by Jonathan Gordon (jdgordon) - Wednesday, 03 March 2010, 00:03 GMT
this is almost certainly because the backlight off is stopping the full refresh that happens on track change. what needs to happen is either a refresh happens while the backlight is off, or (probably better) do the refresh when the backlight comes back on
Comment by Jonathan Gordon (jdgordon) - Wednesday, 03 March 2010, 05:53 GMT
let me know if this is fixed... just looking at the code again, I dont think that fix fixed it :p
Comment by Timothy Pelayo (Crooked Rain) - Thursday, 04 March 2010, 20:26 GMT
During one car ride, I thought I noticed it get out of sync again until I went into the file browser and back to the WPS. On the next car ride, it seemed to update correctly and I couldn't seem to reproduce it. Now I'm wondering whether I mistakenly thought it was out of sync on the first trip. At the very least, it seems to work better now. I'll post again if I notice it getting out of sync again.
Comment by Ronald Malbosc (Chiyo-chan) - Friday, 05 March 2010, 08:09 GMT
I've tested it this morning and the infos are still out of sync.
Comment by Steve Bavin (pondlife) - Friday, 05 March 2010, 10:00 GMT
Is it out of sync while the backlight is off, but back in sync when the backlight is turned back on? Or always out of sync?

Could it be that the WPS isn't updatedwhile the backlight is off - even on targets where the display can be read without backlight.
Comment by Ronald Malbosc (Chiyo-chan) - Friday, 05 March 2010, 10:17 GMT
When the track change is done while the backlight is on, the info are updated correctly.
But they aren't refreshed when the backlight goes on after a track change and don't change unless you launch & exit the file browser.
If the refresh is time driven, then it's longer than the backlight fade off timer (30s maybe?).
Comment by Steve Bavin (pondlife) - Friday, 05 March 2010, 10:19 GMT
I don't see why the backlight status should be relevant to the update - at least not when the screen can be read without a backlight.
Comment by Ronald Malbosc (Chiyo-chan) - Friday, 05 March 2010, 10:34 GMT
From what I remember of the svn history, that's exactly what it's trying to do.
Main LCD: not readable when backlight off => no update when backlight off, to power consumption.
Remote LCD: readable when backlight off => update every time.

The problem seems to be that when you turn backlight on, the screen isn't automatically refreshed (maybe it's trying to fast and the screen still says that it's off?).
Comment by Ronald Malbosc (Chiyo-chan) - Monday, 08 March 2010, 10:24 GMT
I've checked a few more times and this time the screen got refreshed every time 1-2s after the backlight came on.
It's the same build that the last time I checked and from wich I wrote that it dind'nt work. Quite strange.
I don't know what's happening, maybe to much work ^^

I'll continue to check to see if it's really ok everytime.
Comment by Ronald Malbosc (Chiyo-chan) - Tuesday, 09 March 2010, 09:46 GMT
There's definitely a problem remaining.
Sometime the infos are refreshed, sometime they aren't.
Can't find a way to easily reproduce though...
Comment by Timothy Pelayo (Crooked Rain) - Wednesday, 24 March 2010, 01:56 GMT
Still having a problem with this. Like Ronald said, sometimes it works right, sometimes it doesn't. I haven't been able to find a pattern.
Comment by Timothy Pelayo (Crooked Rain) - Wednesday, 28 July 2010, 06:53 GMT
Any chance of this getting fixed? Pretty annoying having out of sync track info for the last year.
Comment by Christian Eitner (cE10) - Monday, 02 August 2010, 12:31 GMT
I can confirm this erroneous behavior for 3.6 on my H340.
Comment by Jonathan Gordon (jdgordon) - Monday, 02 August 2010, 14:13 GMT
I think it is now fixed (again :D )

in 27666. let me know how it goes
Comment by Christian Eitner (cE10) - Monday, 02 August 2010, 15:08 GMT
Well, build 27666 simply ignores my WPS theme (as described in  FS#11514 ) and uses the default.
Comment by Jonathan Gordon (jdgordon) - Monday, 02 August 2010, 23:01 GMT
because that wps is broken... ( and ) need to be escaped (i.e %( and %) )
Comment by Michael Chicoine (mc2739) - Monday, 02 August 2010, 23:06 GMT
Christian:

Lines 5,7 and 8 need updating due to skin syntax changes. Here are the updated lines:

%s%ac%?ia<%ia|%?d(2)<%d(2)|%(root%)>>
%s%ac%?id<%id|%?d(1)<%d(1)|%(root%)>>
%ac%?iy<%(%iy%)|>


Comment by Ronald Malbosc (Chiyo-chan) - Tuesday, 03 August 2010, 05:55 GMT
With build 27666, the informations are now on sync. Do they get refreshed every every time when the track change or are they just refreshed when the backlight goes on?
I did notice that the progress bar got broken? Did progress bar wps syntax changed recently?
Comment by Christian Eitner (cE10) - Tuesday, 03 August 2010, 07:21 GMT
Well, build 27666 simply ignores my WPS theme (as described in  FS#11514 ) and uses the default.
Comment by Christian Eitner (cE10) - Tuesday, 03 August 2010, 07:22 GMT
Please ignore my last double post. Firefox resent some old information.
Comment by Christian Eitner (cE10) - Tuesday, 03 August 2010, 07:27 GMT
With Michael's help on the new syntax, all works fine now. I do have a progress bar in my WPS theme, and it seems to work allright.
Comment by Jonathan Gordon (jdgordon) - Tuesday, 03 August 2010, 07:28 GMT
Ronald, it is doing what it was always supposed to and only updating when the backlight is on. progressbars may have chaged since you last updated...

Loading...