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 E40F9C4332F for ; Thu, 14 Dec 2023 11:58:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1572880AbjLNL6P (ORCPT ); Thu, 14 Dec 2023 06:58:15 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36014 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1444170AbjLNL6M (ORCPT ); Thu, 14 Dec 2023 06:58:12 -0500 Received: from madrid.collaboradmins.com (madrid.collaboradmins.com [46.235.227.194]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 76258124; Thu, 14 Dec 2023 03:58:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1702555096; bh=H4glu+KQRzxtv6e+eHc7mU495CXC1LwVvNxxpwKbl44=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=lpL7V9u562AAjhB9/7kMqY42xMlBcHExPNKxYRabxpeHTeNwKhav30nl9FFyV+5zx xeydlCjn0OeZRugoBsM+njnzzaQCfiveDzMInmVjeaQySD6tWAv8sc9+B5v0sGK+CT N9qhlJfhtnW0U1zTFDSh/vGsheTnF+ihaSkFFhSZ2H2YO9YHlBHGmvfgy3OvduV+CU jttkLN5R8Q+bEbW8t0Dkz6YP8p4ODQUGxvFHw1W/AvZlBOmF9EhehgX9vMd5gVgBEL WwzQM2z3qFg8xI3ESkYrsYFaj8agiJrDkNPN3F2rF6Vov02dWKpTdSe3ORrecY+Wni Fg1BHTsBa16VA== Received: from [100.115.223.179] (cola.collaboradmins.com [195.201.22.229]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: cristicc) by madrid.collaboradmins.com (Postfix) with ESMTPSA id 741BA37814A4; Thu, 14 Dec 2023 11:58:15 +0000 (UTC) Message-ID: Date: Thu, 14 Dec 2023 13:58:14 +0200 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 07/11] ASoC: SOF: core: Skip firmware test for undefined fw_name Content-Language: en-US To: =?UTF-8?Q?P=C3=A9ter_Ujfalusi?= , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Pierre-Louis Bossart , Bard Liao , Ranjani Sridharan , Daniel Baluta , Kai Vehmanen , Venkata Prasad Potturu , Alper Nebi Yasak , Syed Saba Kareem , Kuninori Morimoto , Marian Postevca , Vijendar Mukunda , V sujith kumar Reddy , Mastan Katragadda , Ajit Kumar Pandey Cc: linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org, sound-open-firmware@alsa-project.org, kernel@collabora.com References: <20231209205351.880797-1-cristian.ciocaltea@collabora.com> <20231209205351.880797-8-cristian.ciocaltea@collabora.com> <6e52c5a2-24d5-422a-9a40-a0053729c98e@linux.intel.com> <22ac83a7-287c-46ba-b981-cb810b911e5e@linux.intel.com> From: Cristian Ciocaltea In-Reply-To: <22ac83a7-287c-46ba-b981-cb810b911e5e@linux.intel.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/14/23 13:51, Péter Ujfalusi wrote: > > > On 14/12/2023 12:48, Péter Ujfalusi wrote: >> @@ -265,7 +276,9 @@ static void sof_print_profile_info(struct snd_sof_dev *sdev, >> "Using fallback IPC type %d (requested type was %d)\n", >> profile->ipc_type, ipc_type); >> >> - dev_info(dev, "Firmware paths/files for ipc type %d:\n", profile->ipc_type); >> + /* The firmware path only valid when generic loader is used */ >> + if (sof_platform_uses_generic_loader(sdev)) >> + dev_info(dev, "Firmware paths/files for ipc type %d:\n", profile->ipc_type); >> > > This is the correct section in here, sorry: > > - dev_info(dev, " Firmware file: %s/%s\n", profile->fw_path, profile->fw_name); > + /* The firmware path only valid when generic loader is used */ > + if (sof_platform_uses_generic_loader(sdev)) > + dev_info(dev, " Firmware file: %s/%s\n", profile->fw_path, profile->fw_name); > + > if (profile->fw_lib_path) No problem, thanks for the update!