![](https://secure.gravatar.com/avatar/d930951cb00393ecf9c3db3a56d78fa9.jpg?s=120&d=mm&r=g)
27 Jul
2013
27 Jul
'13
7:35 p.m.
On Fri, Jul 26, 2013 at 05:06:12PM +0200, Lars-Peter Clausen wrote:
I guess for the heaphone amplifier the right fix is to set the shift to RT5640_PWR_HA_BIT. For the speaker amplifier I'd assume that the right bit is RT5640_PWR_CLS_D_BIT. But the speaker amplifier widget also as an event callback in which it already seems to set/clear the bit. I think the right fix is to remove the register change from the event callback and set the shift to RT5640_PWR_CLS_D_BIT. But I neither have the hardware nor the datasheet, so this is all just an educated guess.
Or set the register to SND_SOC_NOPM allowing the manual sequencing which is (I presume) needed? If we're going to need an event to do the sequencing we may as well have it do both writes.