Bug 10708 - Boom cuts off audio when put into power off state.
: Boom cuts off audio when put into power off state.
Status: NEW
Product: Logitech Media Server
Classification: Unclassified
Component: Audio
: 7.3.2
: All All
: P1 normal (vote)
: 8.0.0
Assigned To: Andy Grundman
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2009-01-15 09:24 UTC by Spies Steven
Modified: 2009-07-29 14:58 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 Spies Steven 2009-01-15 09:24:01 UTC
This bug is a result of bug 7681 being fixed in 7.3.2 which turns off the amps in Boom when in a off state.  Before this change while something is paying on Boom and one puts boom into an off state there is a short fade of the audio.  After this change the boom shuts off the amps before the fade is complete.  This results in a unpleasant  cut off of the audio.

I believe this can be solved in SC by delaying the request of the off state until after the audio has faded.
Comment 1 KDF 2009-03-15 01:29:00 UTC
I believe the fade is part of moving to the off state.  However, if the audio_outputs_enable() function is the one controlling the amp, then this could be done via a timer callback a few seconds after power down or it could be relocated so that it is only called after the fade routine has completed?

pointers/suggestions?
Comment 2 Chris Owens 2009-03-16 09:48:37 UTC
We are now planning to make a 7.3.3 release.  Please review your bugs (all marked open against 7.3.3) to see if they can be fixed in the next few weeks, or if they should be retargeted for 7.4 or future.

Thanks!
Comment 3 Felix Mueller 2009-03-17 04:21:04 UTC
audio_outputs_enable() is controlling the amp. So, yes, a timer to delay that until the fading is done should solve the issue. Thanks.
Comment 4 Chris Owens 2009-03-30 17:32:03 UTC
Since there's now a planned 7.3.3 release, bugs which won't make the cut-off are being moved to the next target out.  If you feel that this bug needs to be addressed more (or less) urgently than the 7.4 release, please cc chris@slimdevices.com and leave a comment in the bug to that effect so we can review it.

Thanks.
Comment 5 Chris Owens 2009-03-31 08:54:29 UTC
For some reason Bugzilla did not change the target when I did this yesterday.  Or maybe it was me.  In either case, I'm trying it again.
Comment 6 Andy Grundman 2009-07-29 14:58:56 UTC
Moving 7.4 bugs to 8.0.