From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9EE86EB64DA for ; Wed, 28 Jun 2023 16:31:01 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232227AbjF1Qak (ORCPT ); Wed, 28 Jun 2023 12:30:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38886 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232001AbjF1QaU (ORCPT ); Wed, 28 Jun 2023 12:30:20 -0400 Received: from mail-wr1-x431.google.com (mail-wr1-x431.google.com [IPv6:2a00:1450:4864:20::431]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8E61326AB for ; Wed, 28 Jun 2023 09:30:18 -0700 (PDT) Received: by mail-wr1-x431.google.com with SMTP id ffacd0b85a97d-314090abb67so1155108f8f.2 for ; Wed, 28 Jun 2023 09:30:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1687969817; x=1690561817; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=GMrDQpzoZwdfVDUBjNtjm8VWm39L3gvHFkqK/8Oz7Fg=; b=Ba0XBSFDaUbS4aKjdlM+C4FCmPkBthmSTdfBs6mHnlYMS0NobRCZScMWgsulr6FThL pnVyYRHvQCjtkncraECzLAvQ+7UaaOUOeoLC/FrbqnFzQeidWUji5UyY2HQhg/pYcVVH ZDpPPOqOCvqYv2GZ5sfeTCDWOIDqe8SLoK35Yxpc300QsmkzeAxQVkFdOfKS944KxFT8 SRO7PcD+XNpalbwTg9/mxwiBqFQfoph45g5itQdglK1IfaxhxXXwZznxJF/Ocww5oFpO 9/ExtBr7Oq+j0Ob4taYU+NANwdLBzhD+eJcM7mxFBFjNHNmbpZL6QPC5Q5HQsxo87IWt jI/A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687969817; x=1690561817; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=GMrDQpzoZwdfVDUBjNtjm8VWm39L3gvHFkqK/8Oz7Fg=; b=YlkDFOBIh5ShLmP88gV/Pvn2+nn9x6ezeJYlVaxcp36F5e5QIot6ZagUPp1TC1LYcK k8ZA5X1YhtwkyD6lSFNGHcq23vJaT0GmEICy2z63+otxJjt/Tky2H5cAtt2/RzZgd9CU EwDSeD7ftM2D2OTseSzwymtBXbMdOPgG2kxFCCMXkvjw6fKq06QqVx3Q81haHf+w+qFf 0mmGM1XMfpST5qBhpgJb2RDrDnDP4F+UjWNaBQolaux2jUs/MQN/ZI6MdT4yUJ1OCD0F vKrrf/l5MuJ4WGtWA83Hfce6hBDkE60oFdDDFZkNpwlFR5MTcJwAZgzspP9CvUc8kJEz 36XA== X-Gm-Message-State: AC+VfDw5wewbXgaknJPpe9zhcnl2NoVKiulrbO0sweSn6EkaFeVxSTs/ s3xffYp02EovnzBvLTV+JvUClw== X-Google-Smtp-Source: ACHHUZ5ixMGvBpAM+0EN2kJ24buliqjZdgG6MhG+EaLDx0AocAWUzViTOhuZS9BEglKFygOinlA7EA== X-Received: by 2002:a05:6000:1a4f:b0:314:415:cbf5 with SMTP id t15-20020a0560001a4f00b003140415cbf5mr3635839wry.51.1687969816968; Wed, 28 Jun 2023 09:30:16 -0700 (PDT) Received: from [192.168.1.195] ([5.133.47.210]) by smtp.googlemail.com with ESMTPSA id q14-20020adfdfce000000b003113ccbf388sm3953201wrn.13.2023.06.28.09.30.15 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 28 Jun 2023 09:30:16 -0700 (PDT) Message-ID: Date: Wed, 28 Jun 2023 17:30:15 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH 2/3] ASoC: qcom: q6apm: add support for reading firmware name from DT Content-Language: en-US To: Mark Brown Cc: krzysztof.kozlowski+dt@linaro.org, andersson@kernel.org, robh+dt@kernel.org, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org, dmitry.baryshkov@linaro.org, johan+linaro@kernel.org, perex@perex.cz, tiwai@suse.com, lgirdwood@gmail.com, ckeepax@opensource.cirrus.com, kuninori.morimoto.gx@renesas.com, linux-kernel@vger.kernel.org, pierre-louis.bossart@linux.intel.com, alsa-devel@alsa-project.org References: <20230628102621.15016-1-srinivas.kandagatla@linaro.org> <20230628102621.15016-3-srinivas.kandagatla@linaro.org> From: Srinivas Kandagatla In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 28/06/2023 12:53, Mark Brown wrote: > On Wed, Jun 28, 2023 at 11:26:20AM +0100, Srinivas Kandagatla wrote: >> Currently firmware file name is autogenerated based on card name and model number, >> however this imposed a restriction of finding firmware in a single firmware path. >> Platform specific firmwares are normally located in sub folders of the SoC. >> >> Provide more flexibity by reading firmware-name from DT. > > Why not try a series of firmware names/locations generated using the > identifying information for the card/system? That way we don't have to There is no consistent way with the current state of what is available in linux-firmware and what drivers can generate from DMI, atleast with Qualcomm SoCs. Example for x13s has all the firmwares are under qcom/sc8280xp/LENOVO/21BX for two models 21BX, 21BY. However none of the DMI properties match exactly to 21BX or 21BY. These have to be either derived from product name 21BYZ9SNUS or some other dmi properties. This logic is not going to be very reliable, can differ across platforms. All of the qcom platforms use firmware-name from DT to get the full firmware path with name. I know this has scaling issues, but with the current state of things, its the only option I see. > put a filename in the ABI which has fun scaling issues. thanks, srini