Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:7 as permitted sender) client-ip=2620:137:e000::3:7;
Message-ID: <1f2943e2-f1f0-449d-a1f2-937bae6a0af9@amd.com>
Date:   Fri, 27 Oct 2023 23:02:39 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 09/13] ASoC: amd: acp: add machine driver support for pdm
 use case
Content-Language: en-US
To:     Mark Brown <broonie@kernel.org>,
        Mario Limonciello <mario.limonciello@amd.com>
Cc:     Krzysztof Kozlowski <krzk@kernel.org>,
        Syed Saba Kareem <Syed.SabaKareem@amd.com>,
        alsa-devel@alsa-project.org, Vijendar.Mukunda@amd.com,
        Basavaraj.Hiregoudar@amd.com, Sunil-kumar.Dommati@amd.com,
        venkataprasad.potturu@amd.com, arungopal.kondaveeti@amd.com,
        mastan.katragadda@amd.com, juan.martinez@amd.com,
        Liam Girdwood <lgirdwood@gmail.com>,
        Jaroslav Kysela <perex@perex.cz>,
        Takashi Iwai <tiwai@suse.com>,
        Marian Postevca <posteuca@mutex.one>,
        Alper Nebi Yasak <alpernebiyasak@gmail.com>,
        Claudiu Beznea <claudiu.beznea@microchip.com>,
        Gaosheng Cui <cuigaosheng1@huawei.com>,
        Jarkko Nikula <jarkko.nikula@bitmer.com>,
        open list <linux-kernel@vger.kernel.org>
References: <20231021145110.478744-1-Syed.SabaKareem@amd.com>
 <20231021145110.478744-9-Syed.SabaKareem@amd.com>
 <4d5a4c67-2f4b-4111-b98b-ef575543fa6e@kernel.org>
 <3ec97548-1f91-49d0-adfb-4f8051ca9a97@amd.com>
 <f8f8017c-4e76-4d70-918f-d7cb45186184@kernel.org>
 <c0ea139c-9861-4ea1-b547-6e3c380301b3@amd.com>
 <ZTvkCAYsrS62/82u@finisterre.sirena.org.uk>
From:   syed saba kareem <ssabakar@amd.com>
In-Reply-To: <ZTvkCAYsrS62/82u@finisterre.sirena.org.uk>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Y25FdW9DQnBSYk5FRHhCc1Rudy96dmZHeGFuYnl0QWxsbGFrcm1hVTlGaXFN?=
 =?utf-8?B?KzloaFFqcExhSXU3OXpmbkdCSWtnYy9ScGRMYTdIUWFrQnArSnhOUUttemdn?=
 =?utf-8?B?VnNlZmxJMTlZYnF4aHdmYnVMR3pLRFd5T1BhdlpmMzY2S28wTWZKNlhJUDBl?=
 =?utf-8?B?U3hnRFFRbGdDZnl5dkROaktzcmFqZkp2SkVFSTc0b0NtaDJSRnVlbWJidzhr?=
 =?utf-8?B?UHZwdUdkNlJCZVlOZ1prZitVRG9KNWhZUThnSjNnc1F3S2hsNTRVbno0UTlS?=
 =?utf-8?B?S3FQQTJOVm5wTFg0ZVJpMzdkS0VRTzBsK1dVZEJMU2plN0VoMlRSZXYwMGpr?=
 =?utf-8?B?TkxJeTNHbXB6emxiaWFBNTd3K3pkcG9yMG1ibFNHdjk4SUlZWHoydHUwcjRK?=
 =?utf-8?B?czhIQndrZDVveW9vbkdvWUFNMk1UektXL0p4RVhkbVdrRTJ4SmVZUDd5Zjd5?=
 =?utf-8?B?a2NybjhyZ0tBcUM0bEhqWm15NisrN2VsZno5WnV6K2ZmbUlQd3VlU3NqQ0VD?=
 =?utf-8?B?MWVqQ0ZXSkhZOWZCQ0dOZXUwbGFLalNHR244YUFpNXhVY3ZJcmtORm5Kc1Jv?=
 =?utf-8?B?NmtzaGliUkJBVzBHSlkxM25ZZk5SWHZEZWJQSFNHWnJRUGNLRHJRcVJiczV6?=
 =?utf-8?B?aWxZRlE2bisrOHN3alZqY2dGS1ZjVThHd2xFVGQ0VHBvR2cxcGFJV0o4ZE8y?=
 =?utf-8?B?SWRLQ3JVMTRSdndSajVZcCtVSFBmOENhRStaTlIybEVQTlJyYTZSVWM3THpo?=
 =?utf-8?B?dWxzaHl5cWo3T0ZLcGYxM3VodmtBOG83dFhVZHd5RkVDcWNGMU1TbXo0SVZv?=
 =?utf-8?B?V3lyNzZEbm43VkxOM05rcXlGRktKLzJ4d2lwcHF4aHFtV09nN0NpVmMrL2VI?=
 =?utf-8?B?bU43VmNkQ2h6Vm00cjI0NjdJclFhb0RjUzR1dVhxd1M5Z0FFNE9RaGtOUWVN?=
 =?utf-8?B?R2JXMmVrSC92TmhSdlB1WkVWVXpibEtCdzdTWkM1b1lhU0dvN2Z2ZkRvRS90?=
 =?utf-8?B?eE83aCtDMjcyWkdBQ0R4UVA2Vm5VbDdjUk93SHJCRmhyaDcyRU5TNkhmb2F1?=
 =?utf-8?B?VkM4M2FIa2Z3WDNlYUNaWkV1WTc2eE9vZ0UwY2Vxa2NXeGw0aEFHT2VuSmtR?=
 =?utf-8?B?RVFOUU9idXc4REwrM3lRckdHaWROZXpjOWVkcDBqQ0t3VnBGZkNUbDNCQ0Vt?=
 =?utf-8?B?K0ZjQndFM3JSa213S1JTKzBrWnc5Q0RDZmV0OEdkNVVXMWY4R1JJYUROTWUw?=
 =?utf-8?B?QmJHeDZzVWpuZjJwdDhhTXlOQTRYcWVZenQ1c0ZnL2s3enBqdG00NTNZeWNa?=
 =?utf-8?B?U05QYTcyK24rVWsyay9NbVJuY2RLcXBCcHV4eTFwR2Q1djdXeGNWQ2x4eHdt?=
 =?utf-8?B?bWthMFg0YlN0cE9tMWM0bjA3Y1l1YmJXNXB4Y3lhVFVGMUNFU2NnUTlYQ2Z5?=
 =?utf-8?B?Uld2Z01ybHcycXJ5b3lxcGF6QzcvRE1FVHp6cUhMU2VSTkJEZ3E0a1V0NlBQ?=
 =?utf-8?B?bzdmS1llUGlaWGZPeGpuNU9IVVVGYVF4akNkWURETHhLK2JUeGdUVWE4Uy8z?=
 =?utf-8?B?eU5sY0ZQOXkzbllJSy92UEFnbGZ1d3lMNEMrbUFUZG04VUpBRkVaUHM1TXQ2?=
 =?utf-8?B?K2lGS21DYUdyQ2ZVaGcvcVpJQS9kZjF6V3FGTFNDSC9zOWlGc0VOMFlYL3Fl?=
 =?utf-8?B?dXFnSS9CWTdmelFGZFFydzh3VVdHVXRNVlJYcEtIelZ6ZGpnM3ZvNEVaN0Yz?=
 =?utf-8?B?OG9qenlDUnNYbitVTVVNeGN3ZExXblVsZ0FCQXN6UGhncW1HaDVYUlByRlo5?=
 =?utf-8?B?RnIzaE1peDRuTzVHa1V1blZDQVZESnBXUVcwckZiaUhJbnVVWktVeTlIcHQy?=
 =?utf-8?B?WC9FenM4YVVSZG44N0QxTVNOS0tnT3RYVVBDQkJMM1B5a2dhbnJVMnI5YTJT?=
 =?utf-8?B?djUrWUpFOFAyZDhnZ0hkcXY0eHhVWmo5SmxKdGtCc2tDc0Z0RWV0TVJzKzdW?=
 =?utf-8?B?amJ5Q0drb1JpY2o1dHN3bVZSQjduZ2xKeWNEaGtnWXRwNTFMdU5XN0lhZC9C?=
 =?utf-8?B?bk1pQmkwVU5OTGZ3SENRNmpJc093UVZ6U0ZlTWw4Mk4yV0FOdis1YlRja1hB?=
 =?utf-8?Q?wjNx2O9a5OhQW1RkJg6XF8SH+?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b74d4aac-f319-4c59-56b0-08dbd712bedc
X-MS-Exchange-CrossTenant-AuthSource: MN2PR12MB3661.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Oct 2023 17:32:51.8130
 (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: 8gKeCsq00jKm2zCeZNZ5qDvc58TD1GuPi5/f6Rr133csAxDKYTAExJp5kvWM5GWAFyK4LEqvneeXZpWLKa/XZA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW4PR12MB6683
X-Spam-Status: No, score=-1.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE,
        URIBL_BLOCKED autolearn=no 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
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (snail.vger.email [0.0.0.0]); Fri, 27 Oct 2023 10:33:00 -0700 (PDT)


On 10/27/23 21:53, Mark Brown wrote:
> On Fri, Oct 27, 2023 at 10:54:47AM -0500, Mario Limonciello wrote:
>
>> What would actually go into MODULE_DEVICE_TABLE?
>> The platform devices created are contingent upon what was found during the
>> top level ACP driver probe.  You don't want all the "child" platform drivers
>> to load unless they're needed.
> You want
>
> 	MODULE_DEVICE_TABLE(platform, board_ids);
>
> which is effectively the same as all the MODULE_ALIAS items you have
> there (which can be removed).

@krzk:as Mark Brown explained we can use platform device id table

instead of MODULE_ALIAS. As effectively there is no difference between

using platform device id table and MODULE_ALIAS.

If you are still expecting us to use platform id table instead of 
MODULE_ALIAS

we will provide the changes as an incremental patch.