Bug 8260 - Headphone & Speakers active at the same time
: Headphone & Speakers active at the same time
Status: CLOSED FIXED
Product: SB Boom
Classification: Unclassified
Component: Hardware
: 17
: PC Windows XP
: -- normal (vote)
: 7.2
Assigned To: Felix Mueller
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2008-05-28 10:42 UTC by James Richardson
Modified: 2008-12-15 12:39 UTC (History)
0 users

See Also:
Category: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description James Richardson 2008-05-28 10:42:47 UTC
1) set headphone out to on
2) play something to verify speakers are off
3) cut power to the boom (pull ac out)
4) plug boom back in, play music, both speakers and headphones active
Comment 1 Felix Mueller 2008-05-28 11:18:25 UTC
The issue is the DAC update that happens when Boom reconnects to SC. When Boom boots it senses whether something is plugged into headphone out and sets the speakers accordingly. But as soon as Boom connects the DAC update happens and that seems to kill this settings.

If you move the plugin ABTester out of the Plugin directory and restart SC everything is fine. At least that is what I am seeing.
Comment 2 Blackketter Dean 2008-06-04 16:46:14 UTC
James: Can you confirm that this issue is fixed?
Comment 3 James Richardson 2008-06-05 07:10:42 UTC
This issues is still present in FW 11, with SC 7.2 - 20432 (default install)

To Repro do the following:
Set line out to HeadPhone, verify audio path
Switch to SN, verify audio path
Switch back to SC - Audio will be on both Speakers and Headphone.

If I manually remove the ABTester plug in, then this issue is resolved.

Will the ABTester plug in be removed from SC before the product goes to market? if so, then we can close this bug as fixed, if not, then we need to address the issue.
Comment 4 Erland Isaksson 2008-06-05 10:26:58 UTC
Is the headphone out setting something that is stored in the DSP ?

The only thing ABTester does is reloading the DSP image at player reconnect, it needs to do this since there is no way to check if the correct DSP image is already loaded. I think Caleb has a point on his todo list to make it possible to check which DSP image that is current loaded.

The ABTester issues a number of SlimProto bdac command when a player reconnects to reload the default.i2c DSP image file in the Slim/Plugin/ABTester/StandardImages/ directory. Basically it calls Slim::Player::Boom::upgradeDAC with the default.i2c file as parameter.

There is also an option in SqueezeCenter Settings/ABTester to disable the automatic load of DSP images at player reconnect.
Comment 5 Chris Owens 2008-06-05 15:35:14 UTC
The firmware detects if something is plugged in, and changes settings in the DAC to make the audio paths appropriate.  However, whenever the DAC gets updated which happens whenever you connect to SN or SC (or some other times that aren't currently clear to me) the settings are lost.

Felix proposes that after a DAC update, the firmware should always re-detect the headphone connector state, which might fix the problem?
Comment 6 Erland Isaksson 2008-06-06 00:02:39 UTC
(In reply to comment #5)
> 
> Felix proposes that after a DAC update, the firmware should always re-detect
> the headphone connector state, which might fix the problem?
> 
Yes, I think that should work.

I suspect it at least need to happen after: 
bdac (+byte4=2) (End of loading custom DSP image)

But probably also after:
bdac (+byte4=3) (End of restoring to default DSP image in firmware)

And possibly also after:
bdac (+byte4=4) (After a write to a specific DSP image address)
(byte4=4 will never be issued at reconnect by ABTester, but it will happen when the woofer or stereoXL settings are changed)
Comment 7 James Richardson 2008-06-12 09:15:42 UTC
Still a problem with FW 12

Reported by CAT testers as well, PR-0023, 24, 49
Comment 8 James Richardson 2008-06-26 11:45:39 UTC
Verified still an issue in FW r17.  Can replicate by power cycling the boom after setting Audio > Out to Headphone
Comment 9 James Richardson 2008-06-26 11:47:00 UTC
More CAT reports PR-0057, 67
Comment 10 Felix Mueller 2008-07-03 13:32:45 UTC
Fixed in rev4464
Comment 11 James Richardson 2008-07-08 11:16:10 UTC
Verified fixed in 7.2-21573 r18
Comment 12 James Richardson 2008-12-15 12:39:02 UTC
This bug has been fixed in the 7.3.0 release version of SqueezeCenter!

Please download the new version from http://www.slimdevices.com/su_downloads.html if you haven't already.  

If you are still experiencing this problem, feel free to reopen the bug with your new comments and we'll have another look.