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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5985FC04A6A for ; Wed, 2 Aug 2023 14:40:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:CC:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=t9sQKd7WlH3NCfmAt97WCJN/ef5YMAp15IiKp5AMQbw=; b=yycMpJ2B615Uic PwYgFH4uGfSKlWkyOeWwka6ltb1ZrcXW7LVlKrGJCwwGcT+muACKHIPqWBzCRtNBjpmRiAWx+NRvI aR7a0aEmKuPNJT61uhvUd6XIPz1QuHDXzm2Xq5puGQtBE/E0HS9YaXUUjl8qxzn1wrGiVA5r+lE1B 7DlJfCf681m9kAwLLQ0zYiONnIUTbsEFbDfuBXK1nOF9s/6mpIlngV6YhncOTYsGTczhNvRY3hLRF yLdiOi9SSKX4/cVCRkcsXoZ6apqertqwMZ3ciAUX1u8xjN6i23PBpX7TiwoWRqGVHYoneI9UAwy9r eP1/JbgAEAwczNJYCLYg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qRD1A-0056Wb-2r; Wed, 02 Aug 2023 14:40:04 +0000 Received: from lelv0142.ext.ti.com ([198.47.23.249]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qRD17-0056VQ-25 for linux-arm-kernel@lists.infradead.org; Wed, 02 Aug 2023 14:40:03 +0000 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by lelv0142.ext.ti.com (8.15.2/8.15.2) with ESMTP id 372EdmaR009457; Wed, 2 Aug 2023 09:39:48 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1690987188; bh=fMKK42wAH8Nnsvw63GKfzyP6R9L7nXqVg3xURLy8OK4=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=DzngAnTH9zKrAGnE/DM4ePAfcPfTZSGU+16bl3VimKR7DVl5IQ1m3MAOuLjAWkGK6 5CJNOSUL6iqqqILXnnISEAy7GK2rqPUhjzy2QcHtqgKOqjUXgNwBVOfHcU7VmYQFOb xdWAzKwO/kFwev2IGF8ayMGnq4ruMGsk4TP/e+Jo= Received: from DLEE101.ent.ti.com (dlee101.ent.ti.com [157.170.170.31]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 372EdmVW118309 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 2 Aug 2023 09:39:48 -0500 Received: from DLEE115.ent.ti.com (157.170.170.26) by DLEE101.ent.ti.com (157.170.170.31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Wed, 2 Aug 2023 09:39:48 -0500 Received: from fllv0039.itg.ti.com (10.64.41.19) by DLEE115.ent.ti.com (157.170.170.26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Wed, 2 Aug 2023 09:39:48 -0500 Received: from [172.24.227.6] (ileaxei01-snat.itg.ti.com [10.180.69.5]) by fllv0039.itg.ti.com (8.15.2/8.15.2) with ESMTP id 372EdiEH024344; Wed, 2 Aug 2023 09:39:44 -0500 Message-ID: Date: Wed, 2 Aug 2023 20:09:43 +0530 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 4/5] arm64: dts: ti: k3-am62a7-sk: Enable audio on AM62A Content-Language: en-US To: Nishanth Menon , Jai Luthra CC: Vignesh Raghavendra , Tero Kristo , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Catalin Marinas , Will Deacon , , , , Jayesh Choudhary References: <20230731-mcasp_am62a-v1-0-8bd137ffa8f1@ti.com> <20230731-mcasp_am62a-v1-4-8bd137ffa8f1@ti.com> <52pbbqnp46h33gymoydnjtxoo3dsb6wnytvjnmomtjdtwck536@ewhb2rngomr2> <20230802133502.zjvf7sslmcuayg5z@defog> From: Devarsh Thakkar In-Reply-To: <20230802133502.zjvf7sslmcuayg5z@defog> X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230802_074001_829263_6A90335B X-CRM114-Status: GOOD ( 21.65 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi Jai, Nishant, On 02/08/23 19:05, Nishanth Menon wrote: > On 17:10-20230802, Jai Luthra wrote: >> Hi Devarsh, >> >> On Aug 02, 2023 at 16:15:12 +0530, Devarsh Thakkar wrote: >>> Hi Jai, >>> >>> Thanks for the patch. >>> >>> On 31/07/23 18:14, Jai Luthra wrote: >>>> Add nodes for audio codec and sound card, enable the audio serializer >>>> (McASP1) under use and update pinmux. >>>> >>>> Link: https://www.ti.com/lit/zip/sprr459 >>>> Signed-off-by: Jai Luthra >>>> Reviewed-by: Jayesh Choudhary >>>> --- >>>> arch/arm64/boot/dts/ti/k3-am62a7-sk.dts | 77 +++++++++++++++++++++++++++++++++ >>>> 1 file changed, 77 insertions(+) >>>> >>>> diff --git a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts >>>> index 752c2f640f63..5f68d2eefe0f 100644 >>>> --- a/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts >>>> +++ b/arch/arm64/boot/dts/ti/k3-am62a7-sk.dts >>>> @@ -125,6 +125,41 @@ led-0 { >>>> default-state = "off"; >>>> }; >>>> }; >>>> + >>>> + tlv320_mclk: clk-0 { >>>> + #clock-cells = <0>; >>>> + compatible = "fixed-clock"; >>>> + clock-frequency = <12288000>; >>>> + }; >>>> + >>>> + codec_audio: sound { >>>> + compatible = "simple-audio-card"; >>>> + simple-audio-card,name = "AM62Ax-SKEVM"; >>> >>> In my opinion better to give the codec name instead of board name here. >> >> I agree, maybe calling it "sk-am62a-tlv320aic3106" would be the most >> clear option. >> >> Running a quick ripgrep on next tree: >> >> $ rg "simple-audio-card,name" arch/*/boot/dts/ >> >> I see a healthy mix of using both board and/or codec name here - with TI >> almost always using the board name. Maybe we can change the convention, >> but it would be a good idea to at least update SK-AM62 as well to use >> the new convention. >> >> Is it okay with you if it is handled as a separate series? > Yes agreed, the naming can be taken care of as separate series as it also applies to other TI board (AM62x), we can brainstorm on the naming separately and not block this patch. Reviewed-by: Devarsh Thakkar Regards Devarsh > Will this cleanup of existing board break any userspace? If so, NO and > follow existing "board" convention - I'd like to maintain consistency, > even if that is not exactly clean! If not, cleanup in a later series > is fine, but please make sure to follow through this week - with this > patch following the convention of choice. > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel