Hi Slawinski, Thanks for your feedback.Here is the target we want to do when submitting all the patchs: 1. rename tas2764.c to tas27xx.c 2.rename tas2764.h to tas27xx.h 3.update Makefile 4.update Kconfig 5.rename tas2764.yaml to tas27xx.yaml 6.update tas27xx.c to support either TAS2764 or TAS2780 7. update tas27xx.h to support either TAS2764 or TAS2780 As just to make everything clear,we plan to do item1 to 5 firstly.Could you let us know what's your suggestion?Should we stop this patch routing and start a new patch submit process? Regarding item 6 to 7,we can submit separate patch process after we finish item 1 to 5. Regards Raphael -----Original Message----- From: Amadeusz Sławiński <amadeuszx.slawinski@xxxxxxxxxxxxxxx> Sent: Wednesday, March 23, 2022 4:29 PM To: Raphael-Xu <13691752556@xxxxxxx>; broonie@xxxxxxxxxx Cc: Navada Kanyana, Mukund <navada@xxxxxx>; alsa-devel@xxxxxxxxxxxxxxxx; Ding, Shenghao <shenghao-ding@xxxxxx>; Xu, Yang <raphael-xu@xxxxxx> Subject: [EXTERNAL] Re: [PATCH v4 3/3] update tas27xx.h to support either TAS2764 or TAS2780 On 3/23/2022 5:26 AM, Raphael-Xu wrote: > Signed-off-by: Raphael-Xu <13691752556@xxxxxxx> > --- > sound/soc/codecs/tas27xx.h | 27 +++++++++++++++++++-------- > 1 file changed, 19 insertions(+), 8 deletions(-) > > diff --git a/sound/soc/codecs/tas27xx.h b/sound/soc/codecs/tas27xx.h > index 67d6fd903c42..02b29c030d37 100644 > --- a/sound/soc/codecs/tas27xx.h > +++ b/sound/soc/codecs/tas27xx.h > @@ -1,18 +1,20 @@ > /* SPDX-License-Identifier: GPL-2.0-only */ > /* > - * tas2764.h - ALSA SoC Texas Instruments TAS2764 Mono Audio > Amplifier > + * tas27xx.h - ALSA SoC Texas Instruments TAS2764/TAS2780 > + * Mono Audio Amplifier > * > - * Copyright (C) 2020 Texas Instruments Incorporated - > https://www.ti.com > + * Copyright (C) 2022 Texas Instruments Incorporated - > + * https://www.ti.com > * > - * Author: Dan Murphy <dmurphy@xxxxxx> > + * Author: > */ > > -#ifndef __TAS2764__ > -#define __TAS2764__ > +#ifndef __TAS27XX__H_ > +#define __TAS27XX__H_ > > /* Book Control Register */ > -#define TAS2764_BOOKCTL_PAGE 0 > -#define TAS2764_BOOKCTL_REG 127 > +#define TAS27XX_BOOKCTL_PAGE 0 > +#define TAS27XX_BOOKCTL_REG 127 > #define TAS2764_REG(page, reg) ((page * 128) + reg) > > /* Page */ > @@ -77,6 +79,10 @@ > #define TAS2764_TDM_CFG3_RXS_SHIFT 0x4 > #define TAS2764_TDM_CFG3_MASK GENMASK(3, 0) > > +/* TDM Configuration Reg4 */ > +#define TAS2764_TDM_CFG4 TAS2764_REG(0X0, 0x0d) > +#define TAS2764_TDM_CFG4_TX_OFFSET_MASK GENMASK(3, 1) > + > /* TDM Configuration Reg5 */ > #define TAS2764_TDM_CFG5 TAS2764_REG(0X0, 0x0e) > #define TAS2764_TDM_CFG5_VSNS_MASK BIT(6) > @@ -89,4 +95,9 @@ > #define TAS2764_TDM_CFG6_ISNS_ENABLE BIT(6) > #define TAS2764_TDM_CFG6_50_MASK GENMASK(5, 0) > > -#endif /* __TAS2764__ */ > +/* INT&CLK CFG */ > +#define TAS27XX_CLK_CFG TAS2764_REG(0X0, 0x5c) > +#define TAS27XX_CLK_CFG_MASK GENMASK(7, 6) > +#define TAS27XX_CLK_CFG_ENABLE (BIT(7) | BIT(6)) > + > +#endif /* __TAS27XX__H_ */ > \ No newline at end of file And this patch should probably go before patch 2, otherwise there will be build failure on patch 2?