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 451DDC4167B for ; Thu, 7 Dec 2023 13:54:44 +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=JqBn4FkNonnxJdP03/BzQZlroeC/hiA8aZwdxXrNqBA=; b=pUeH3yYy97xbzE 8nK3iDb0hcM6Usw1NqpB799Lx3xksbvWxKB9dgvC+l0pH7gwgt3oNd3ZJc7GBxLcdAOYvi2lGDdSS YeCSg57own+lJ98Ka5M4nQ0pUFJUn6Ds9jrGJRVtsfvKMCl9aoyygBf14eUcoGxT39bt98LlxBGZb EqNophBJaQvFRW0o6HY1NmaGg2NXiUmdpyIIF+Jz5Zz2mqlqb5tbsXRDLdVasjq3QbW8ucVe+jBsN CznCY8friuyUFLEFoipkPxTMenEs7TLJw5xQpdtokRMELiOwKjcVdedri+WL8+LqJ1CIG0Gg3g9UQ usPhVTWRVv+0kakHxR6Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rBEpU-00Cx2t-0Q; Thu, 07 Dec 2023 13:54:16 +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 1rBEpR-00Cx2U-34 for linux-arm-kernel@lists.infradead.org; Thu, 07 Dec 2023 13:54:15 +0000 Received: from fllv0034.itg.ti.com ([10.64.40.246]) by lelv0142.ext.ti.com (8.15.2/8.15.2) with ESMTP id 3B7Ds45R083785; Thu, 7 Dec 2023 07:54:04 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1701957244; bh=W5gXtXfZlSK11aQjTNxP7QVpz+cyU+aQUS0zTnnUPpc=; h=Date:Subject:To:CC:References:From:In-Reply-To; b=p65KeSRApGwvhso3ak+P27MaIukNysRQ+TWyldNFdhLd3e+2vNmvzLDLrZIW2RTdP R4GOOVToI0o/TJIOPn2UT7r89WrjM1tSQ31nOW58+3udob8dC1HYk1rixTWwRYcUJ3 LeMV409vzXHzyJw4bEE3hDQFy0JyqZc0zaCsnS0c= Received: from DLEE102.ent.ti.com (dlee102.ent.ti.com [157.170.170.32]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 3B7Ds4L5082652 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Thu, 7 Dec 2023 07:54:04 -0600 Received: from DLEE115.ent.ti.com (157.170.170.26) by DLEE102.ent.ti.com (157.170.170.32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Thu, 7 Dec 2023 07:54:03 -0600 Received: from lelvsmtp6.itg.ti.com (10.180.75.249) 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; Thu, 7 Dec 2023 07:54:03 -0600 Received: from [10.249.135.225] ([10.249.135.225]) by lelvsmtp6.itg.ti.com (8.15.2/8.15.2) with ESMTP id 3B7Ds04n032059; Thu, 7 Dec 2023 07:54:00 -0600 Message-ID: <6f2c1c23-bed6-48be-8407-030f2993ec8a@ti.com> Date: Thu, 7 Dec 2023 19:23:59 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 2/3] arm64: dts: ti: k3-am642-evm: add ICSSG1 Ethernet support Content-Language: en-US To: Nishanth Menon CC: MD Danish Anwar , Vignesh Raghavendra , Conor Dooley , Krzysztof Kozlowski , Rob Herring , , , , Tero Kristo , , References: <20231207081917.340167-1-danishanwar@ti.com> <20231207081917.340167-3-danishanwar@ti.com> <20231207131818.3n7z64ve6izatlvs@unchanged> <20231207134343.ufiy2owik5kn3y2r@degrease> From: "Anwar, Md Danish" In-Reply-To: <20231207134343.ufiy2owik5kn3y2r@degrease> 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-20231207_055414_074360_25184D57 X-CRM114-Status: GOOD ( 17.95 ) 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 On 12/7/2023 7:13 PM, Nishanth Menon wrote: > On 18:58-20231207, Anwar, Md Danish wrote: > [...] >>>> + >>>> memory@80000000 { >>>> bootph-all; >>>> device_type = "memory"; >>>> @@ -229,6 +234,70 @@ transceiver2: can-phy1 { >>>> max-bitrate = <5000000>; >>>> standby-gpios = <&exp1 9 GPIO_ACTIVE_HIGH>; >>>> }; >>>> + >>>> + icssg1_eth: icssg1-eth { >>>> + compatible = "ti,am642-icssg-prueth"; >>>> + pinctrl-names = "default"; >>>> + pinctrl-0 = <&icssg1_rgmii1_pins_default>; >>>> + >>>> + sram = <&oc_sram>; >>>> + ti,prus = <&pru1_0>, <&rtu1_0>, <&tx_pru1_0>, <&pru1_1>, <&rtu1_1>, <&tx_pru1_1>; >>>> + firmware-name = "ti-pruss/am65x-sr2-pru0-prueth-fw.elf", >>>> + "ti-pruss/am65x-sr2-rtu0-prueth-fw.elf", >>>> + "ti-pruss/am65x-sr2-txpru0-prueth-fw.elf", >>>> + "ti-pruss/am65x-sr2-pru1-prueth-fw.elf", >>>> + "ti-pruss/am65x-sr2-rtu1-prueth-fw.elf", >>>> + "ti-pruss/am65x-sr2-txpru1-prueth-fw.elf"; >>> >>> Umm... am65x??? is that a typo? I'd rather keep it am64x here and drop >>> that sr2 thing. Tomorrow there will be a custom bug on am64 and then we >>> will have to respin this again. >>> >> >> No Nishant, this is not a typo. Both AM64x and AM65x use the same ICSSG >> firmwares. We only have am65x-sr2-* firmwares and they are used by both >> AM64x and AM65x and that is why I have kept the firmware-name here in dt >> same as the files that we load on the pru cores. >> > > SoCs are different. The hardware as a result is different as well. In > fact, you do have a different compatible to distinguish the two. Some > day, there will be an erratum that is different and we will be stuck > with abi breakage across distros. So, unless you can explain why this > scenario will never occur, I don't buy the argument this will survive > long term. > Agreed, this property was introduced for this purpose only. Today am65x and am64x share the same firmware however in future the firmwares might change and that is why we have this property. Currently this property is not used in driver and firmware name is defined in the driver (with the below structure) which is used for both am64x and am65x. I will rename the firmware names here to am64x-sr2* in v2. In future when we have different firmwares for different SoCs, we can stop using the below structure and use the firmware-name property from dt. static struct icssg_firmwares icssg_emac_firmwares[] = { { .pru = "ti-pruss/am65x-sr2-pru0-prueth-fw.elf", .rtu = "ti-pruss/am65x-sr2-rtu0-prueth-fw.elf", .txpru = "ti-pruss/am65x-sr2-txpru0-prueth-fw.elf", }, { .pru = "ti-pruss/am65x-sr2-pru1-prueth-fw.elf", .rtu = "ti-pruss/am65x-sr2-rtu1-prueth-fw.elf", .txpru = "ti-pruss/am65x-sr2-txpru1-prueth-fw.elf", } }; -- Thanks and Regards, Md Danish Anwar _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel