Re: [PATCH v6 3/4] ALSA: cs35l41: Add shared boost feature

On 2/22/23 03:32, Lucas Tanure wrote:

+static const struct reg_sequence cs35l41_actv_seq[] = {

• /* SYNC_BST_CTL_RX_EN = 1; SYNC_BST_CTL_TX_EN = 1 */
• {CS35L41_MDSYNC_EN, 0x00003000},
• /* BST_CTL_SEL = MDSYNC */
• {CS35L41_BSTCVRT_VCTRL2, 0x00000002},

+};

+static const struct reg_sequence cs35l41_pass_seq[] = {

• /* SYNC_BST_CTL_RX_EN = 1; SYNC_BST_CTL_TX_EN = 0 */
• {CS35L41_MDSYNC_EN, 0x00002000},
• /* BST_EN = 0 */
• {CS35L41_PWR_CTRL2, 0x00003300},
• /* BST_CTL_SEL = MDSYNC */
• {CS35L41_BSTCVRT_VCTRL2, 0x00000002},

+};

The passive device only needs to use TX, not RX. It is transmitting its boost target voltage to the bus so that the active amp can boost according to the combined target.

Should be: static const struct reg_sequence cs35l41_pass_seq[] = { /* SYNC_BST_CTL_RX_EN = 0; SYNC_BST_CTL_TX_EN = 1 */ {CS35L41_MDSYNC_EN, 0x00001000}, ...

Thanks, David