Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v2 1/3] ASoC: soc-acpi: add comp_ids field for machine
 driver matching
To:     Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        Brent Lu <brent.lu@intel.com>, <alsa-devel@alsa-project.org>
CC:     Liam Girdwood <lgirdwood@gmail.com>,
        Mark Brown <broonie@kernel.org>,
        Jaroslav Kysela <perex@perex.cz>,
        Takashi Iwai <tiwai@suse.com>,
        Jie Yang <yang.jie@linux.intel.com>,
        Kai Vehmanen <kai.vehmanen@linux.intel.com>,
        Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>,
        Yong Zhi <yong.zhi@intel.com>,
        Vamshi Krishna Gopal <vamshi.krishna.gopal@intel.com>,
        <linux-kernel@vger.kernel.org>,
        Rander Wang <rander.wang@intel.com>,
        "Bard Liao" <bard.liao@intel.com>,
        Malik_Hsu <malik_hsu@wistron.corp-partner.google.com>,
        Libin Yang <libin.yang@intel.com>,
        Hans de Goede <hdegoede@redhat.com>,
        Charles Keepax <ckeepax@opensource.cirrus.com>,
        Paul Olaru <paul.olaru@oss.nxp.com>,
        "Curtis Malainey" <cujomalainey@chromium.org>,
        Mac Chiang <mac.chiang@intel.com>,
        Gongjun Song <gongjun.song@intel.com>
References: <20211007133516.1464655-1-brent.lu@intel.com>
 <20211007133516.1464655-2-brent.lu@intel.com>
 <7e6cf92f-b889-aed9-b13a-615d4d5f9b93@intel.com>
 <2ae5e8ef-76cb-8335-e250-64b0246175f3@linux.intel.com>
From:   Cezary Rojewski <cezary.rojewski@intel.com>
Message-ID: <6d2eb853-60f4-73ee-a465-aaa4528c100f@intel.com>
Date:   Thu, 7 Oct 2021 20:46:08 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Firefox/78.0 Thunderbird/78.13.0
In-Reply-To: <2ae5e8ef-76cb-8335-e250-64b0246175f3@linux.intel.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?aUFlVnlzbStmbmVFakpBdU9TdGorSURvYkJMSVhOL3AzRCs2eEhScCtZMDBz?=
 =?utf-8?B?MWNNRTQ5TjB0TUNpdWJPRHFxSHNZd2kycmlkZXpRVXpocGlXMVhLcXcrUzFC?=
 =?utf-8?B?bGNjM0hkZXFyb0w2cGloRjdtbjZIcjl5MWE5ek4rQkQ4cGNzeTFoVzYvQ1VQ?=
 =?utf-8?B?SENLekp0MW4rWGVaNHlYQWlnQitOY0duc1JEWEI4b1dXdjUrdlNSdDNRVjdl?=
 =?utf-8?B?Sis2eXUwcWJaUDFvTzhKWERySUNNNklmT1o5eVBOTU1qVVJwMGZoVzY2V081?=
 =?utf-8?B?OW5NaFMzazMweVpyV2wza09UbmhrY1RIQjRUU3BzZGRKdXVDSEZQSFowOG1S?=
 =?utf-8?B?RjZoWE5JRlgwQVErZFlYNkZjQ0lqR1gzeHo4bGlXcVpEQ1NkMElrUll4anJ2?=
 =?utf-8?B?QVJ2ZThNNXlFVlJac2JmR1V0RFE5dm1uYXEzN3Q5MDdNUjJtL1lwbFd0RlZU?=
 =?utf-8?B?QkRBallxTnFiUmJLaStBK1VGeUdNU05aZEtmN1hFWjBzZGtZWjZjRXBrTjho?=
 =?utf-8?B?S1Z4ODVlWnpZZ3lBdzIxK093cTNBcjBLYlN1ZlZyVUl5djBkSmhyblVyWkcy?=
 =?utf-8?B?YSsxOHBDejhQZkZQUEFzYStkOFBUUXB3SUdoYUVlZFl0ci8zTng0a2c2UzJi?=
 =?utf-8?B?TEcvMkpOWUQ0cHVKQTI3eHZoZ3ZlRGljYml1cXZubVRCNm1IR3Yra1RDazBy?=
 =?utf-8?B?ek9HMHVRTzVSY2FsTFNoUXFtRkRYaXBwbzRyU0VZT0ZIZVFjRkFod1kvMzI1?=
 =?utf-8?B?eTR5RjhKSkZGNUxKV3hCdjBhNWhaSlNIQTNGOS9wc3FzdTE2SmxrMk5yMjBO?=
 =?utf-8?B?YUE5SjdTVlFtVTk3Rm5Dc3B4dGExV0FKZWtWbDhCdHg3c1kvNEc2Rm15WjQv?=
 =?utf-8?B?amt4T3E4cmlzeTA1OG9EWjdjQW1BR2lpSTdQbHg5TDRhdzd5NHNQK04rS0NX?=
 =?utf-8?B?VEtHMytzMXJ2clFVOUNNM1Q5THdOMlV4N0tMWXR6SHdQZ1h4Q0grSW9QMjhF?=
 =?utf-8?B?aU50dUVIMlpiMUtRNVBNRlBPNGZ1Zkk4Q01VZU10MVd5RjNkeEY3dThObS9X?=
 =?utf-8?B?aHpLWmFnSXgzRHptMkhiYUFxOXhKOEg1OXpWdGl1MjR6UGR4YkZsVmhQa3I4?=
 =?utf-8?B?YnNicWt2c0k0N3BTbnFpR2RUOFczcDJxQkxNZURQZlE3OVk5NTNTYUJsZ01Q?=
 =?utf-8?B?S2RWWWVvUnVrVHY0U2pIdDM0ZTlSaGZ2cWkyRnRZTUJYdlJaYjNZQmlBTXhy?=
 =?utf-8?B?U2U1Y1BCMkxTWEx6cWUwNm1vM3VuUGdxejhDdnJZV1pxQzhtNHR1MGpJbjR5?=
 =?utf-8?B?RkV4SlpiREJ3WkljbHFEL0MxTnk3SDVRblBnbU5iTnBCenFqdEJPcVFVWkcz?=
 =?utf-8?B?QjUyMGlKS2VHMG04ZW9FaGN2dm10WmFRQ0x4amhic3Zkd0E4VDl1N3kxcjVQ?=
 =?utf-8?B?Y2pMb1dkamNFdHI2aThQckVBRmtBRm1ENC9jRXo2QzRlUXpxZkZBR0Iva2Nk?=
 =?utf-8?B?RmhYN3d2Mjh5S3dpTWQrWUtJTnZQK1Zvc0ludGxsUjF3OFJzOWtoL1ZqLyt0?=
 =?utf-8?B?cWVFckJRc0tDeld1ZHNsYW44SVIxSURmalRpcVZVRFF4QjlvSUVaQktqM01s?=
 =?utf-8?B?YXN1Nk1YeWFmS0dlL09Cc0JtVUJVa2RJUFpSaGg3MHB6c3ZFTkZxOFhrZW5Q?=
 =?utf-8?B?Vk1PYUM2MTltT0RHNDU4dHlxM0FKTGFhYnB1anpDNkFlODVMV0VSWEljWjNv?=
 =?utf-8?Q?5jV64fBsWKp7leHKpSa8IMZGy/XZ6iysdlFeopV?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 7686f240-ba93-4ad9-3e06-08d989c2befd
X-MS-Exchange-CrossTenant-AuthSource: BN6PR11MB4049.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Oct 2021 18:46:17.2992
 (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: CO5Qj0U6dCzsPtSWQFLIiSJ3by92SMnDqF+1bAVav9XPpz1s9UPBZ8Rvee75pWgcovdDS823gpixYcFAhJ7DwBSgfGu18seoFmKCWWXlJZc=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN7PR11MB2625
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 2021-10-07 7:27 PM, Pierre-Louis Bossart wrote:
> 
>>>    struct snd_soc_acpi_mach *
>>>    snd_soc_acpi_find_machine(struct snd_soc_acpi_mach *machines)
>>>    {
>>>        struct snd_soc_acpi_mach *mach;
>>>        struct snd_soc_acpi_mach *mach_alt;
>>>    -    for (mach = machines; mach->id[0]; mach++) {
>>> -        if (acpi_dev_present(mach->id, NULL, -1)) {
>>> +    for (mach = machines; mach->id[0] || mach->comp_ids; mach++) {
>>
>> Such loops are hard to maintain i.e. 'comp_ids' acts here like a flex
>> array that follows 'id'. Removal of 'id' field and streamlining code to
>> only use 'comp_ids' should make this loop more intuitive.
> 
> Changing all the tables adds more noise IMHO. There are 15 files and
> about 100 ids.
> 
> This patch provides an opportunity to reduce duplication, that's good,
> but let's leave all the existing unique table entries alone, shall we?
> 

Well, we could have mentioned files updated in the follow up patches 
i.e. treat this patch as a 'preparation step'. In the long run, having 
two places for id initialization will cost us more than updating all 
those files.

Have no problem with leaving current patch as is if the end goal is 
removal of 'id' field. In some future series I guess..

Czarek