Re: [PATCH v1 1/2] ASoC: tac5xx2-sdw: update default regmap registers

From: Mark Brown

Date: Wed Aug 19 2026 - 09:46:42 EST


On Wed, Aug 19, 2026 at 11:04:26AM +0530, Niranjan H Y wrote:

> + /* SA */
> + {TAC_VOLUME_REG(SA, FU21, LEFT), 0x9c00},
> + {TAC_VOLUME_REG(SA, FU21, RIGHT), 0x9c00},
> + {TAC_GAIN_REG(SA, FU23, LEFT), 0x0},
> + {TAC_GAIN_REG(SA, FU23, RIGHT), 0x0},
> + {TAC_MUTE_REG(SA, FU21, LEFT), 0x1},
> + {TAC_MUTE_REG(SA, FU21, RIGHT), 0x1},
> + {TAC_MUTE_REG(SA, FU23, LEFT), 0x1},
> + {TAC_MUTE_REG(SA, FU23, RIGHT), 0x1},

Did you check the sorting of the table here?

> - REG_SEQ0(SDW_SDCA_CTL(TAC_FUNCTION_ID_SA, TAC_SDCA_ENT_FU23,
> - TAC_SDCA_CHANNEL_GAIN, TAC_CHANNEL_LEFT), 0),
> - REG_SEQ0(SDW_SDCA_CTL(TAC_FUNCTION_ID_SA, TAC_SDCA_ENT_FU23,
> - TAC_SDCA_CHANNEL_GAIN, TAC_CHANNEL_RIGHT), 0),

The new macros use _CHANEL_VOLUME and _MASTER_GAIN, there's none using
_CHANNEL_GAIN?

Attachment: signature.asc
Description: PGP signature