Rockbox

Tasklist

FS#10091 - Boots instead of charging screen ('Car Mode' is 'off')

Attached to Project: Rockbox
Opened by Alun Thomas (tspoon) - Thursday, 02 April 2009, 20:02 GMT
Last edited by Boris Gjenero (dreamlayers) - Monday, 06 April 2009, 01:49 GMT
Task Type Bugs
Category Battery/Charging
Status Closed
Assigned To No-one
Operating System Archos Recorder
Severity Low
Priority Normal
Reported Version Version 3.2
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

With 3.2 loaded, Rockbox starts automatically when the DC jack is inserted, even though the 'Car Mode' Setting is set to 'off'. It normally would show the charging screen and not boot into rockbox.
This task depends upon

Closed by  Boris Gjenero (dreamlayers)
Monday, 06 April 2009, 01:49 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in r20634
Comment by Boris Gjenero (dreamlayers) - Thursday, 02 April 2009, 23:25 GMT
Are you starting Rockbox from flash? Does charger detection work?
Comment by Alun Thomas (tspoon) - Friday, 03 April 2009, 01:11 GMT
Yes I have set the 'P' and 'C' (plugged and charging) flags to show on my wps, and they are functioning correctly. I am starting from flash, with bootloader 3.0, although it also does for bootloader 2.0, but not if I return to an earlier release of rockbox in flash.
Comment by Boris Gjenero (dreamlayers) - Friday, 03 April 2009, 03:58 GMT
The return value of charger_inserted() is based on periodic polling of hardware by power_thread(). When init() checks whether to go to the charging screen, power_thread() has not yet checked charger status for the first time, and so charger_inserted() returns false. This patch checks the status of the hardware directly via power_input_status().
Comment by Alun Thomas (tspoon) - Friday, 03 April 2009, 04:00 GMT
Thanks boris, I'm not set up to compile right now, but hopefully your work will result in a quick fix in the daily builds.
Comment by Boris Gjenero (dreamlayers) - Friday, 03 April 2009, 04:12 GMT

Loading...