Received-SPF: pass (google.com: domain of linux-kernel+bounces-72404-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID: <343f7b1d-1b4e-402e-bd54-4473b91b6f79@amd.com>
Date: Tue, 20 Feb 2024 11:53:44 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] ASoC: SOF: amd: fix soundwire dependencies
Content-Language: en-US
To: Arnd Bergmann <arnd@arndb.de>, Arnd Bergmann <arnd@kernel.org>,
 Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
 Liam Girdwood <lgirdwood@gmail.com>,
 Peter Ujfalusi <peter.ujfalusi@linux.intel.com>,
 Bard Liao <yung-chuan.liao@linux.intel.com>,
 Ranjani Sridharan <ranjani.sridharan@linux.intel.com>,
 Daniel Baluta <daniel.baluta@nxp.com>, Mark Brown <broonie@kernel.org>
Cc: Kai Vehmanen <kai.vehmanen@linux.intel.com>,
 Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
 V sujith kumar Reddy <Vsujithkumar.Reddy@amd.com>,
 Venkata Prasad Potturu <venkataprasad.potturu@amd.com>,
 sound-open-firmware@alsa-project.org, linux-sound@vger.kernel.org,
 linux-kernel@vger.kernel.org
References: <20240219093900.644574-1-arnd@kernel.org>
 <465d1076-163c-4933-a9b5-e4e8736f5748@amd.com>
 <412c0a42-4c19-48dd-aa4e-de39b6ec3099@app.fastmail.com>
From: "Mukunda,Vijendar" <vijendar.mukunda@amd.com>
In-Reply-To: <412c0a42-4c19-48dd-aa4e-de39b6ec3099@app.fastmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?TEdzaXRwMmNuRzJVd0ZTRkZCUWl6N1pQV1ZZcE5PUzcwWExyS2JSMnBPOGdl?=
 =?utf-8?B?RUNrcmJtZGIyWXRhcVdTZVh6c0dKY043SUY4TjEyVnJ0eTJWWFR2N0N3bjNz?=
 =?utf-8?B?SUM0b0NVSSs4T0NzVm5qSUpEYUZOU1pUZVU2NnJIQU1Mb1E2RGVmUzJrcWpT?=
 =?utf-8?B?L3VKcXZhRTRtd3FidTQ4ZnV5dXorMU9zV1BKR2NFQ2lEU0ludjZuV3IrRUhh?=
 =?utf-8?B?WWhTRkxXNnJJcnZYY1pZbm5zc3Q5VjNsZWNNWEd4c2FURm5KZVRsNmViZzVG?=
 =?utf-8?B?Tlg2Mk1FY3JndHJINW1xNm5kSk5oeWV5KzVmSnUzOXlFOTcvVlZWTjd5OW5r?=
 =?utf-8?B?V1hVcFAzYmNuZW9EcDJnWGk4TEVIY0RYYVpCTWptWG9DZ3gvbHROL0ZOc0Ir?=
 =?utf-8?B?cFBtSVNXdDd3WW13ajdsYi9zRUFUSUhBOTlIdUZla0hGZ1FubllUeVM1VU5U?=
 =?utf-8?B?Vko1azZ3ZU5od1N2T1IxT21hK0hQc2VpNmVFYXlCeWZ4TEkvR0l5SE9XZEFY?=
 =?utf-8?B?ZVBBeFk5eWhoamo4cUMvZndaQTZiL0RJTHIrUGtUUWk4ckZ5OTUwYTF1cElG?=
 =?utf-8?B?RDRVN3cvTlhCY2Q0MXNsVzRUSDQrZWZiTVp5YTAxWVp5b08wMkRHVksxdnYw?=
 =?utf-8?B?Nytla0Izc0lWRlRoK2NUWGZPOHBJMDhJbHNqYlVZTDJZMU9xWk1WZkJZcTNN?=
 =?utf-8?B?WXZFTHNJdTNDdjE0L0FHT1M1cHJDRGRYVU00VFRxaHlnSmhpQXRySEQ5bW1y?=
 =?utf-8?B?WTVLVXc5Y0RwN1BMUVJHa3JQK1Y4WEpHUE5uS1NoN1pNRkxRNlZQQ3hvbGpT?=
 =?utf-8?B?c1dmdHBnQ3ZoYXpqVXVHVjBLNGhpbG9xRU80ODl2bHYzVnl0eEJJQUkxOUY3?=
 =?utf-8?B?UFRvS2FCakNoQ01hSG1MUmJ3Z0F1bDMrV1hhOVprd1FOTHlCUmdaNlZraWNS?=
 =?utf-8?B?cUl4RTNpNXdKZGVDK1p0eFdjMjM0OWFySmlWSVFqRnZqYUhydFhDekJXOGFU?=
 =?utf-8?B?TDZIOUN0S09FZkxzcDFtdnFlR2QxZ2FnNjBqTHFMY21XLzJodUZVUEpPWHlT?=
 =?utf-8?B?OHY1T2xoeUlEa0REbldzSnJGblBWQjVMVUZQVkhTZE1STlVCUmZ6VTg1NC9C?=
 =?utf-8?B?UmxCQTlhQy8vNWwzc0RXck1qTmwvWFprdnJmci9iMnNxc2FCQ2NITmdNOVRt?=
 =?utf-8?B?d0hnampZcHNTVkZhSDJWL0lJSy8wVmRNQVo2NDVrS3JMTUNNVXl6MmFBVWl6?=
 =?utf-8?B?cW5Ed0Z3MEk2UWtmUGJOK055bE1aQzFON0NUdldGYThCbDVjYndhT2tsUXh3?=
 =?utf-8?B?N3NmWlNTT2ZuREFYcXZXLzE5NlBZTGxsdE1yNTlya2VqSnIwT2Z2T3Qrc2ZE?=
 =?utf-8?B?TFRqYUJLVFhDTWpaLzdPbzl5Q2pVcUtOWHBEL1lUNVFDeFlwdFFPTWtrZEF6?=
 =?utf-8?B?MHR2a3RHT1RSQmo2ZndtTEptTGhmZ3JEN1VJVnFKVVNUMnk3Z1lCQjl5MWc2?=
 =?utf-8?B?OWo5MDBEbXhtTThBdDcvOU9rTzgrU0xIVlpzVGdna2NUZ1MwalJBN29vY1ZT?=
 =?utf-8?B?WDc2V24vME93NFRyT2l4QXplbDEwVlYyUks1eCtxRTdlVVF4eHhScURwYkpo?=
 =?utf-8?B?OU9OTlZGYXcvMFg3NG9Ha09Xcyt4YzdCUmQzbjZrbHlQKzNSZzU0Y2pPa2pt?=
 =?utf-8?B?ZnA4K2w2WkJFMkNVZjNMVlE1aFVhVzYvekZJbWswQytVQVMza0hYeGlpTE5p?=
 =?utf-8?B?MUJBWjdRMmxVbmZaTDlWWU5sTnN2QmowYWFwRlVJaGo2L3g3b2tIckUyNzJE?=
 =?utf-8?B?blJmdVV2cFdCdnFTRi9jQ2FZU0Q4Z3hiZlp2UFNDblpsbk5wNVdwN1E0TE93?=
 =?utf-8?B?Q1pGaU54em81TngrWmlnQlcyL2pEei9iVU9meG94ajA1RE9HekJFb1dqZW5K?=
 =?utf-8?B?ekhwMGxaZFJaQnVIUWhXRHZiQzFLZkxDaXByL1UxN1FNREFNYnNwOU9TRUFQ?=
 =?utf-8?B?Q2tJNEtsaHF2TVI2VzFXekFoNWxMNnJxWWVHOG13RWxlWm15bFp5VWI0L0Jw?=
 =?utf-8?B?REtyQUlaWkE1REFpa1FJWE5DNEJSWjkvbkJVRnA5cytJMVM0dFp6UXFNTldK?=
 =?utf-8?Q?ma6KebkPKZg2BJ8YU0XejYrTy?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 06d1b5a5-a446-4501-3ea1-08dc31dc8383
X-MS-Exchange-CrossTenant-AuthSource: DM6PR12MB4123.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Feb 2024 06:23:55.2027
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: T9K4QX1T1lmFiD808zY/ky6BpI4izcXT3IkjL9MLXC47uXsR7hIbII5IdC6Ewgxsdr2T18GtuDnXLwjlYU3Z8g==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5093

On 20/02/24 11:43, Arnd Bergmann wrote:
> On Tue, Feb 20, 2024, at 06:57, Mukunda,Vijendar wrote:
>> On 19/02/24 15:08, Arnd Bergmann wrote:
>>> From: Arnd Bergmann <arnd@arndb.de>
>>> In normal configs, they should all either be built-in or all loadable
>>> modules anyway, so this simplification does not limit any real usecases.
>> Tested this patch. SOUNWIRE_AMD flag is not selected by default causing
>> AMD SOF driver for ACP 6.3 platform is build without enabling SoundWire.
> Yes, that is what I described. But as SOUNWIRE_AMD is a user visible
> symbol, there is no problem in expecting users to enable it when they
> have this hardware, and distros just enable all the drivers anyway.

Want to set SOUNDWIRE_AMD flag by default, similar to Intel & Qcom
platforms instead of explicitly enabling the Kconfig option.
>
>     Arnd