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 X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 514D6C433DF for ; Tue, 9 Jun 2020 19:21:45 +0000 (UTC) Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6AF9920734 for ; Tue, 9 Jun 2020 19:21:44 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=alsa-project.org header.i=@alsa-project.org header.b="Sl+IFCqJ"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=ti.com header.i=@ti.com header.b="f+J4vZsc" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6AF9920734 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=ti.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 211E11614; Tue, 9 Jun 2020 21:20:52 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 211E11614 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1591730502; bh=7AyHFdjonTrFn2j/MwHO0y9u2frYNk7ouqrWlBP4c1c=; h=Subject:To:References:From:Date:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=Sl+IFCqJPidHhignfusDOCefj3SYfh66ctKOJLUn7f117wTtBy1xBcU8XnihJzdli qIuuMkA45hRxx4zKxCkRXub1sqMMgHZSHl/8xOPBglDreZSSUfT1O4O2wAkPGlhPD6 1EK0xZA8hlbb0NROLzKE96ZkkWSXTfeG13YlVpcs= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 9AEBDF80278; Tue, 9 Jun 2020 21:20:51 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id D958AF8028C; Tue, 9 Jun 2020 21:20:49 +0200 (CEST) Received: from fllv0016.ext.ti.com (fllv0016.ext.ti.com [198.47.19.142]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 77780F80124 for ; Tue, 9 Jun 2020 21:20:35 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 77780F80124 Authentication-Results: alsa1.perex.cz; dkim=pass (1024-bit key) header.d=ti.com header.i=@ti.com header.b="f+J4vZsc" Received: from lelv0265.itg.ti.com ([10.180.67.224]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 059JKX1d088349; Tue, 9 Jun 2020 14:20:33 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1591730433; bh=ZPzxZJaQ2aInc4QHSdxUXpf6Q0oMRP3/URsi//CF6L8=; h=Subject:To:CC:References:From:Date:In-Reply-To; b=f+J4vZscyfmvcnEPlE8ig6eRWF1Mcldf0lezx2j9IV/V5OBx3Ljdp9TGELuebGnHz dCAWGlw5iYb3/Ra5aHxyNdTzGkvhcgzq0KfB2sPYWD1NCuKxy2wuXY/pNVKQLmF0Rk Vz7SQKuam4lgPahMqc0JKaErMDosf5dxuZDlBNgw= Received: from DFLE103.ent.ti.com (dfle103.ent.ti.com [10.64.6.24]) by lelv0265.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 059JKXDc063661 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Tue, 9 Jun 2020 14:20:33 -0500 Received: from DFLE113.ent.ti.com (10.64.6.34) by DFLE103.ent.ti.com (10.64.6.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3; Tue, 9 Jun 2020 14:20:33 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DFLE113.ent.ti.com (10.64.6.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3 via Frontend Transport; Tue, 9 Jun 2020 14:20:33 -0500 Received: from [10.250.65.13] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 059JKU5W083640; Tue, 9 Jun 2020 14:20:31 -0500 Subject: Re: [RFC PATCH 1/2] dt-bindings: tas2562: Add firmware support for tas2563 To: Mark Brown References: <20200609172841.22541-1-dmurphy@ti.com> <20200609172841.22541-2-dmurphy@ti.com> <20200609173143.GN4583@sirena.org.uk> <20200609175852.GQ4583@sirena.org.uk> <414a2d73-6d09-1e76-59c8-4943c0e8f720@ti.com> <20200609184734.GS4583@sirena.org.uk> From: Dan Murphy Message-ID: <014b85b5-677b-569a-4eb2-74526d3f00bc@ti.com> Date: Tue, 9 Jun 2020 14:20:29 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.0 MIME-Version: 1.0 In-Reply-To: <20200609184734.GS4583@sirena.org.uk> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Cc: robh@kernel.org, alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, tiwai@suse.com, lgirdwood@gmail.com X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" Mark On 6/9/20 1:47 PM, Mark Brown wrote: > On Tue, Jun 09, 2020 at 01:06:50PM -0500, Dan Murphy wrote: > >> I could make a default as you suggested to include i2c address and bus in >> the name.  But the TAS2563 does not need the firmware to operate and the >> 2562 does not have a DSP. > That's fine, the driver can just use the compatible string to check this > and not offer any of the DSP related stuff (it should do this regardless > of the method used here). I'm guessing the regmap configs should also > be different. The driver does check the compatible to determine if DSP loading is available for the device. The driver also checks to see if the firmware file is declared in the DT. So it has to pass 2 checks to even load and parse the firmware to present the controls for the programs and configs. >> What if there was an ALSA control instead that passed in the firmware name >> from the user space instead of using the DT? >> Then the control can load and parse the firmware and wait for the user to >> select the program. >> This would solve a user from having ot update the DT to use a firmware. > That's really not very idiomatic for how Linux does stuff and seems to > pretty much guarantee issues with hotplugging controls and ordering - > you'd need special userspace to start up even if it was just a really > simple DSP config doing only speaker correction or something. I'm not > sure what the advantage would be - what problem is this solving over > static names? IMO having a static name is the problem. It is an inflexible design.  Besides the firmware-name property seems to be used in other drivers to declare firmwares for the boards. But if no one is complaining or submitting patches within the codecs to be more flexible with firmware then I can just hard code the name like other drivers do. Dan