Bug 12612 – In Store Demo does not survive a power cycle

Bug 12612 - In Store Demo does not survive a power cycle

Summary:

In Store Demo does not survive a power cycle

Status:	RESOLVED FIXED

Product:	SB Radio
Classification:	Unclassified
Component:	Settings
Version:	Include FW version in comment
Platform:	PC All

Importance:	P2 major (vote)
Target Milestone:	MP
Assigned To:	Richard Titmuss

URL:
Keywords:

Depends on:
Blocks:	12703
	Show dependency tree / graph

Reported:	2009-06-30 10:42 UTC by James Richardson
Modified:	2009-09-08 09:15 UTC (History)
CC List:	0 users

See Also:
Category:	---

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description James Richardson 2009-06-30 10:42:19 UTC

Baby r6297

Start In Store Demo
Pull Power out of Baby Boom
Plug Power into Baby Boom

Notice that the in store demo is not running, and normal menu UI is displayed.

Comment 1 Ben Klaas 2009-06-30 11:51:23 UTC

That's baffling to me, I've tested that at least 20 times without fail.

Comment 2 Ben Klaas 2009-06-30 11:53:35 UTC

bah, you're right. with 6297 this is broken. Looking into it

Comment 3 Ben Klaas 2009-07-02 10:35:50 UTC

after enabling the Demo, the settings file for Demo.lua:

# cat /etc/squeezeplay/userpath/settings/Demo.lua 
settings = {startDemo=true,}
#

after reboot:
# cat /etc/squeezeplay/userpath/settings/Demo.lua 
# 

do we have a (new) file sync issue with Baby? This previously worked, but clearly the settings file is not surviving reboot.

Comment 4 Richard Titmuss 2009-07-03 16:15:15 UTC

This should not be fixed in r6365, but I'm leaving the bug open as I need to add some error handling in system_atomic_write() and also test it a bit more.

Comment 5 James Richardson 2009-07-06 15:33:31 UTC

r6372 demo now starts after a power up, but, the audio cuts out.

Comment 6 Chris Owens 2009-07-07 10:15:07 UTC

This bug has been identified as one that may not be required for MPQ since it now seems clear that MPQ and MP firmwares will be different.  Please comment if you disagree with this assessment.  Thanks!

Comment 7 Richard Titmuss 2009-07-15 06:43:04 UTC

Error checked added in fab4 r6579.