Message-ID: <d80db13a-bb87-039c-da20-7ebf5e68d841@amd.com>
Date:   Thu, 16 Mar 2023 08:59:42 +0530
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.8.0
Subject: Re: [PATCH V6 4/8] soundwire: amd: enable build for AMD SoundWire
 manager driver
To:     Vinod Koul <vkoul@kernel.org>
Cc:     alsa-devel@alsa-project.org, pierre-louis.bossart@linux.intel.com,
        Basavaraj.Hiregoudar@amd.com, Sunil-kumar.Dommati@amd.com,
        Mario.Limonciello@amd.com, amadeuszx.slawinski@linux.intel.com,
        Mastan.Katragadda@amd.com, Arungopal.kondaveeti@amd.com,
        claudiu.beznea@microchip.com,
        Bard Liao <yung-chuan.liao@linux.intel.com>,
        Sanyog Kale <sanyog.r.kale@intel.com>,
        open list <linux-kernel@vger.kernel.org>
References: <20230307133135.545952-1-Vijendar.Mukunda@amd.com>
 <20230307133135.545952-5-Vijendar.Mukunda@amd.com> <ZBGW9ThXgcT0tIui@matsya>
Content-Language: en-US
From:   "Mukunda,Vijendar" <vijendar.mukunda@amd.com>
In-Reply-To: <ZBGW9ThXgcT0tIui@matsya>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?aEVueU5iSmZKQ1ZVN21vUERQV3VvbUV2RUhwNGlRZEFKT0dtazJGZ2lVbUR3?=
 =?utf-8?B?ejRVVUV0RGdZM3V5d3pNQ2hWd1dMd1dyRUJ2bmhhOG5MQjVKNjFIMDV4aEFK?=
 =?utf-8?B?SU9vTkp4QnMxWVZsR1NLM2N1QVV6WkR0bHpzWUwxUDZETDZXUTFCNjNoTzEv?=
 =?utf-8?B?SlhseWtzLzVabjU5QXdoNlZQVE1NZWNaMCt5YUdmVFpyNnRSN3FRWGFLTnlK?=
 =?utf-8?B?QWlzM2VzS1htMnBWNTlJN0pNNmJCZEdCVnFNQzNEd1BjRUV5OWlTcXREazdE?=
 =?utf-8?B?TDZmdG9TWDlTK3hoS1poSmxKN3hFOEttWXAxWm1MSUxDRTV5aWQvVlN0Zksx?=
 =?utf-8?B?WXhiY0JwK0RLR2tVYk9TOWZNZTk0NGdTamM3TVVjOVdiOFJ2SDZKaHJBT1N2?=
 =?utf-8?B?OXEyVjdTODRkSkp6U3lDN3JvVDdadFlGVEY1MVR0SkZkVjZ3UXVOTmlaY3Ns?=
 =?utf-8?B?NTBhUDliRXJCS29xa29lWExRYU1zbkdWSVk4S1RZNTdycVFRNlNTR3E0TENa?=
 =?utf-8?B?c2FvSkRmTnVmYnVBcTBLZUNQUEUybjFYVDRMYVBnam5ScjdUd3BMcWdVam52?=
 =?utf-8?B?NUYyeHZJdi8rdHJKSDRSYTd1V0U4a0owQVUyNEFtZFpEN2FDaDhjZ1Y2SDda?=
 =?utf-8?B?S09YN0p4REN5R1RsT1Bjbk01b0JIREVCTmxtOTBQZVRNRTRuWlM2bFY5MVZY?=
 =?utf-8?B?cU1oOG83TzZGRzNtQ2V3WUtYSUNUSURIUnJ1ZTJDUTkxZEJ5S1c1L0YxOTZM?=
 =?utf-8?B?TmJXRGQ0dnBodDM3dXZmV3RHdU1sWWhGa25meXpwbVU4ZUNNRDBsNmdBbDIv?=
 =?utf-8?B?Q3FsMzRqUFQ3S3ByYm80UFl6Y1FFYVNqcW5OWEFxRUtCMXc1dUozNUxDOXYy?=
 =?utf-8?B?UVIwOEJLWDM3SldHc1Y0ZlRSMy9CT0hqSGVGWDUrK3lwVjlJSCsvdjlXcHNo?=
 =?utf-8?B?a1B2cFpId2ZzYUxmamZ3KzQ1WFdmV1N5bXZFTDFxWXdyY2JoS2lOamkvSGpJ?=
 =?utf-8?B?K0JSVVZ4ZjdTc1JIdXVkVkZnQ05ocDlrdWZuSWluS21oOXRpdTVrRjExOG5Y?=
 =?utf-8?B?d1ppK2dyQ1MwOWQ4NWFESlBTdWplUXBqNmRJVnA5RW80ei91VHE0ZzJTTzhU?=
 =?utf-8?B?bkVQNUUydFVMeTlBMVpaU1cyV1AxYzd2S2dVL29hemxoSjZIZ0g5QlMrM2U5?=
 =?utf-8?B?YmJST0FMRzZ2azBlc3c5dFpVWG41eVExRkpaNUprcHBSWFl3bTFCVkdnbmtl?=
 =?utf-8?B?ekJ4SXFRRkRMRkFJclo3MWNZbVRTM1FCS2tjeXE1M2QvcEkyay9RdGdMbFJj?=
 =?utf-8?B?T2xSWjgwSUZOS0ZNZTRuYVVUZ0cvQTNwVlUwTk5lY0MxNkQ5V3R4cm0rRXBq?=
 =?utf-8?B?bWVxVGl5YlRhZkt3YU5xbEJtZ0gzTnNiN0NrbmVLOVFLaDJTdk9KQkVXWFd2?=
 =?utf-8?B?R1IvMWxPZElQSE1pMU10T1FjNGk0Y2xRT29lblNSTER2TmxydmF2MjA3THl5?=
 =?utf-8?B?S3kydXlEQm5ZSUtDRVZpUTR4Rms2aGJSQUNoeXVpc0RUTGVodGg1S3ZhTDRx?=
 =?utf-8?B?UERDajNrOGJ1V25ES1lRSGdwN0xhKy8vRXM4NExiMjA1SXdqcndiQmNCdy8w?=
 =?utf-8?B?bTVCWTZHd0dFb2ZiMm9zeUU4VkNQcVJyZ0YvS1ZlZ0lCZ2JLWktkb0VnUFUy?=
 =?utf-8?B?L2hlcjFrbG5oUm1BUFFSazJ2MFVUZEwzMFVpSXdDcVBYSHV5ZDRRRWlWVHRn?=
 =?utf-8?B?OXRhN2JpVzB5QjFxWEZmQTFIeG1SczlpU3BYdTY3WGRrRUxCRTNOWG1PNjhM?=
 =?utf-8?B?SjRnRmNSZ2FtQ3ZWK3p3VzlZWDhyZ29sSWtGalZ2Z29QL2x4ckJORDNnSkZC?=
 =?utf-8?B?ci9IRkxBN0o2UDdXZDNWNk5uOU9hdzd3cnQzTzkvTEdQbHhxSmdtREpueGdI?=
 =?utf-8?B?eHBXekpqVVpNajcvRlBjSXJESFlqM3ppdTVXS1BFcnJ0YnVSZVpwR2ZTa2Ri?=
 =?utf-8?B?SlVFUE9KU05OZEdhQXhXeUZGZ2JSb2N4dm9pY0gxUm9Zd1ArM29UVHg2ZFF2?=
 =?utf-8?B?ajVoZHZqYnlzWDB3UE10L0tEU2JSdmp6V0VHM2lJSGplQ1M3T3dQczVOYWRO?=
 =?utf-8?Q?h5jsqav8uyOfczTMgGcVma2BL?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 73874f14-9c32-4732-c8e2-08db25ce2803
X-MS-Exchange-CrossTenant-AuthSource: DM6PR12MB4123.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Mar 2023 03:25:57.1803
 (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: IOinaLhOm14UWWB2lbq8uzdsvXquOFxDXnW4C9w4bf6zdA32Ie4gUbJq/rwyygmyK/Hl6w2pavFoSUYMdjoXxw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR12MB5474
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 15/03/23 15:29, Vinod Koul wrote:
> On 07-03-23, 19:01, Vijendar Mukunda wrote:
>> Enable build for SoundWire manager driver for AMD platforms.
>>
>> Signed-off-by: Vijendar Mukunda <Vijendar.Mukunda@amd.com>
>> Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
>> Link: https://lore.kernel.org/lkml/20230220100418.76754-5-Vijendar.Mukunda@amd.com
>> ---
>>  drivers/soundwire/Kconfig  | 10 ++++++++++
>>  drivers/soundwire/Makefile |  4 ++++
>>  2 files changed, 14 insertions(+)
>>
>> diff --git a/drivers/soundwire/Kconfig b/drivers/soundwire/Kconfig
>> index 2b7795233282..983afe3570b2 100644
>> --- a/drivers/soundwire/Kconfig
>> +++ b/drivers/soundwire/Kconfig
>> @@ -46,4 +46,14 @@ config SOUNDWIRE_QCOM
>>  config SOUNDWIRE_GENERIC_ALLOCATION
>>  	tristate
>>  
>> +config SOUNDWIRE_AMD
>> +	tristate "AMD SoundWire Manager driver"
> Alphabetically sorted please
will fix it.
>
>> +	select SOUNDWIRE_GENERIC_ALLOCATION
>> +	depends on ACPI && SND_SOC
>> +	help
>> +	  SoundWire AMD Manager driver.
>> +	  If you have an AMD platform which has a SoundWire Manager then
>> +	  enable this config option to get the SoundWire support for that
>> +	  device.
>> +
>>  endif
>> diff --git a/drivers/soundwire/Makefile b/drivers/soundwire/Makefile
>> index ca97414ada70..5956229d3eb3 100644
>> --- a/drivers/soundwire/Makefile
>> +++ b/drivers/soundwire/Makefile
>> @@ -26,3 +26,7 @@ obj-$(CONFIG_SOUNDWIRE_INTEL) += soundwire-intel.o
>>  #Qualcomm driver
>>  soundwire-qcom-y :=	qcom.o
>>  obj-$(CONFIG_SOUNDWIRE_QCOM) += soundwire-qcom.o
>> +
>> +#AMD driver
>> +soundwire-amd-y :=	amd_manager.o
>> +obj-$(CONFIG_SOUNDWIRE_AMD) += soundwire-amd.o
> here as well
Will fix it.
>
>> -- 
>> 2.34.1