Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Liao, Bard" <bard.liao@intel.com>
To:     Bard Liao <yung-chuan.liao@linux.intel.com>,
        "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
        "vkoul@kernel.org" <vkoul@kernel.org>
CC:     "vinod.koul@linaro.org" <vinod.koul@linaro.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "pierre-louis.bossart@linux.intel.com" 
        <pierre-louis.bossart@linux.intel.com>
Subject: RE: [PATCH 7/7] soundwire: intel: split auxdevice to different file
Thread-Topic: [PATCH 7/7] soundwire: intel: split auxdevice to different file
Thread-Index: AQHY9W0kVhKMfSQpDE6oqDEeqqAfF6448o+Q
Date:   Fri, 11 Nov 2022 01:47:28 +0000
Message-ID: <DM6PR11MB407474B2B8CA4C10AE084AC3FF009@DM6PR11MB4074.namprd11.prod.outlook.com>
References: <20221111013135.38289-1-yung-chuan.liao@linux.intel.com>
 <20221111013135.38289-8-yung-chuan.liao@linux.intel.com>
In-Reply-To: <20221111013135.38289-8-yung-chuan.liao@linux.intel.com>
Accept-Language: en-US
Content-Language: en-US
dlp-version: 11.6.500.17
dlp-reaction: no-action
dlp-product: dlpe-windows
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?zHwnad0UJUPJ+NrPtJx02nw2Th0TY8siTB5pAIkXhCL1HlZnuQWY+MOLg2vw?=
 =?us-ascii?Q?IM7GOP2pXZt9z7Mk34e3jyV/8EEbklp2D5twDw/C6iSAY4G8RnVPdaXb+Ek7?=
 =?us-ascii?Q?acQFiWzMaB+pKlK5VynRLfFylZ7/G8ghfBwSkPO9YwNd1LWQRLTCRgm8g2ND?=
 =?us-ascii?Q?aUvR02jpHAd6tW95rkMOwqvXm9XU76BozTECZZorRZApC7WI09+jMBgjFxbZ?=
 =?us-ascii?Q?S9cL/QXkSQeciWI1Dii1uBtOB9bMYWVvtKT9rsn1csYbr3+k/PAuGITlq64T?=
 =?us-ascii?Q?oRVQEFetg0w5oFTFA+P1U1udFdCcfY1Ws+F7NsibXWEKb0OYRvE6anjs5GCm?=
 =?us-ascii?Q?skV2jn74BNKYctlcBOMjvEsDvIzs1WOqpFELglf2JxLzUENh1UkDgLNktbdG?=
 =?us-ascii?Q?RVwfpUNKx2j2CbaYTm7b1QQi6n5vDtLMqluiLlVFWPtXhV5FL2C4vD0zwI2X?=
 =?us-ascii?Q?Qei9J+0U8UvN+U9+DxjJe/9iEkDXevmK84UhrVq9sJAmAvUPpPDE8atJbk+0?=
 =?us-ascii?Q?GV7z72l7TKVR5v9vf9XoESaQ9XTA2uZTgtCN1dO9rsmLX4PCakTgO/sk+6O6?=
 =?us-ascii?Q?/EguW2w0Cs4LTNtsqG1Rals1McenDSm1Dw739iwJfWcWn7zmk5/GbzlAghPF?=
 =?us-ascii?Q?hdzVsfymSoI4/neSAuNOPP31MhaA+/ylQ6jNB5HhRWJLrfLe+wfMMiYCOjoR?=
 =?us-ascii?Q?nAY5WtpDqWqqjkpB0mZYjG+lr5bhPdTA3JAIRb3i3vtEPEZczVKuQsw/ugqB?=
 =?us-ascii?Q?tkGGzE0Aela9gh30OygOpIW2qaIGMwXcgclAxzhOxOmRKxw6aUXQ53p/myvg?=
 =?us-ascii?Q?1b0GeukjS2ozv+vt5XaTfz+wDvyv5XiLDZPTqR2sAdYnnBdwY39YCH61tNHS?=
 =?us-ascii?Q?TFxjoCakAAMkU3zTWsCKCLBQlMQgLvyDlT8GbD6JvRffkoNhFRcymXY+2bF+?=
 =?us-ascii?Q?xwEyeHZac8LhVVaTBDIscuyc2CpoG/Y/YLW/nZXhj0UAfl101DmjAUbC/ZR7?=
 =?us-ascii?Q?Ynq9vQkz56ul+YmK8GMo6QbL/uezLUgUBjwT+IE0+x/hgd/XZQYivcaUB1vV?=
 =?us-ascii?Q?ynsWjERW9VFNfGDyHHPET3B6PBncObxFwMNvyau8NT+LNUvVJhM90hwocsft?=
 =?us-ascii?Q?AcltdpUZJMW4BmCVKGYSSGJtnnxpc+cDU7oCnZarGz5mR5tf1bft9UEkNFxR?=
 =?us-ascii?Q?S4j28LWihFQ8onjqTqykKU4iLfPjwZ3ObpAPmpPYUFjfTSrePk1i/9WkJuBk?=
 =?us-ascii?Q?2imWHvdTp0O944qGUQDYzrTqmO8bBis9XC/VRTmJwSsoJYm0IEgXgBy1sfO9?=
 =?us-ascii?Q?kckEF+qK9rI/SulU9d90a7LTzUtRzCt9xNGYqK/lQWcNScCwx2IGc067bSp4?=
 =?us-ascii?Q?8GIz1hfDYuOokJQxzQj622RSp89vUmBxBZmJyEB+4Mvea1A+Rk8SVcsjDnlD?=
 =?us-ascii?Q?jpruQ+RZCOnvtds9k61DMlLLyWQ8FFcdLjNPRFtSXa479MLu6syc5lH92AaT?=
 =?us-ascii?Q?HwHoKTWq7q7uZRrrzqpUMXVfMlOQJsjiPs7PBHYfD98bubYKKGrRTN+58QNw?=
 =?us-ascii?Q?r73LJ7ibpL3ABCg+pHYaVyFh3Ia1FVHPjCvueIu4?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB4074.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bfec97c9-0afe-49dd-7c9e-08dac386b0c6
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Nov 2022 01:47:28.4866
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 7N4AlSw+LkmzrGvM61KDf/XLFO6Lc9yPxREuT0buMKG7px2vIZqHZ62mukKjyT0itOkFgPjsjQq2sGEZg1GzJQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR11MB6126
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,
        SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> -----Original Message-----
> From: Bard Liao <yung-chuan.liao@linux.intel.com>
> Sent: Friday, November 11, 2022 9:32 AM
> To: alsa-devel@alsa-project.org; vkoul@kernel.org
> Cc: vinod.koul@linaro.org; linux-kernel@vger.kernel.org; pierre-
> louis.bossart@linux.intel.com; Liao, Bard <bard.liao@intel.com>
> Subject: [PATCH 7/7] soundwire: intel: split auxdevice to different file
>=20
> From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
>=20
> The auxdevice layer is completely generic, it should be split from
> intel.c which is only geared to the 'cnl' hw_ops now.
>=20
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com=
>
> Reviewed-by: Rander Wang <rander.wang@intel.com>
> Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

Hi Vinod,

Note that "soundwire: intel: Initialize clock stop timeout" which is in
your fixes branch, but not in your next branch is needed before applying
this patch.