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=-16.4 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 5B9B1C4338F for ; Mon, 26 Jul 2021 07:55:27 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 1CF6E60C51 for ; Mon, 26 Jul 2021 07:55:27 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 1CF6E60C51 Authentication-Results: mail.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=ti.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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:MIME-Version:Date: Message-ID:References:CC:To:From:Subject:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=5WAmZU6ToZP0AqUbsmu6eWSi/WA9HL+aNhqwcvSJPKs=; b=MmJlvhmaKgHUqNuCx2xPpsMW8y baCrsj02bY2F25LsAmX/f2c/L+3DMtQGZe7mBSTlX2w1unRVO8STIlAjynMu+7cZ6QlpqXZvtkCgQ +QOjfcxGeJbOJfKewZfhchj0Uja6qtb200dglc64a19ukNbVAioBtUtXT2aemDPMEiGjPcyNE/8UR BvOR8VT5j9vJa8HJSXk6zW7XPyxOZoU3D19i0hVuW+TcC2FHP3bgLRa8FUNsfEXvXotnM8lg5wthC VCPQQYPBzWBB5iq9xGNFds3AvT7mJgwlNa8siZ1XoXQann3q6qO0WgIfmjPHa/25MXeqr1x+c39/L cS3m6ujQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1m7vGj-00A0N8-6r; Mon, 26 Jul 2021 07:43:21 +0000 Received: from fllv0015.ext.ti.com ([198.47.19.141]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1m7vGf-00A0MB-Gy for linux-arm-kernel@lists.infradead.org; Mon, 26 Jul 2021 07:43:19 +0000 Received: from lelv0265.itg.ti.com ([10.180.67.224]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 16Q7hEDb040963; Mon, 26 Jul 2021 02:43:14 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1627285394; bh=Z7jD0mQAoy3ojk+iiHUn5mhLG0G8KU5yeTjQ/OUuIHc=; h=Subject:From:To:CC:References:Date:In-Reply-To; b=rCaB7D0oplf2khih2XUOj5Lad1KWQq/J9d8A238aNupC4FA2uYhARrkeY+ql68CUI W7LZBO8cr/OX0kOME9wpHGxSJB1smNF2e0RjV+PQCmTkdCZzx9w1DoejEH4T5Za3p4 bzmASsw9oLH+3myDJHjS/gt5MWTqPonpt+1dElmM= Received: from DLEE100.ent.ti.com (dlee100.ent.ti.com [157.170.170.30]) by lelv0265.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 16Q7hE3L036458 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Mon, 26 Jul 2021 02:43:14 -0500 Received: from DLEE104.ent.ti.com (157.170.170.34) by DLEE100.ent.ti.com (157.170.170.30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Mon, 26 Jul 2021 02:43:14 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DLEE104.ent.ti.com (157.170.170.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2 via Frontend Transport; Mon, 26 Jul 2021 02:43:14 -0500 Received: from [10.250.148.136] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 16Q7h9gu048924; Mon, 26 Jul 2021 02:43:10 -0500 Subject: Re: [PATCH 3/6] arm64: dts: ti: k3-j721e: Add support for MCAN nodes From: Aswath Govindraju To: Marc Kleine-Budde CC: , , , Nishanth Menon , Tero Kristo , Rob Herring , Lokesh Vutla References: <20210720141642.24999-1-a-govindraju@ti.com> <20210720141642.24999-4-a-govindraju@ti.com> <20210720145451.6eby7pbfocyyfi4e@pengutronix.de> <41ff222b-e751-1586-b774-4e55bfe5e017@ti.com> Message-ID: <65368cf6-afad-cb08-1b27-883e8e7eafef@ti.com> Date: Mon, 26 Jul 2021 13:13:08 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: <41ff222b-e751-1586-b774-4e55bfe5e017@ti.com> Content-Language: en-US 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-20210726_004317_706971_33E31742 X-CRM114-Status: GOOD ( 21.34 ) 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 Marc, On 21/07/21 1:18 pm, Aswath Govindraju wrote: > Hi Marc, > > On 20/07/21 8:24 pm, Marc Kleine-Budde wrote: >> On 20.07.2021 19:46:39, Aswath Govindraju wrote: >>> From: Faiz Abbas >>> >>> Add support for 14 MCAN controllers in main domain and 2 MCAN controllers >>> present in mcu domain. All the MCAN controllers support classic CAN >>> messages as well as CAN_FD messages. >>> >>> Signed-off-by: Faiz Abbas >>> Signed-off-by: Aswath Govindraju >>> --- >>> arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 196 ++++++++++++++++++ >>> .../boot/dts/ti/k3-j721e-mcu-wakeup.dtsi | 28 +++ >>> 2 files changed, 224 insertions(+) >>> >>> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >>> index cf3482376c1e..4215b8e6785a 100644 >>> --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >>> +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >>> @@ -1940,4 +1940,200 @@ >>> bus_freq = <1000000>; >>> }; >>> }; >>> + >>> + main_mcan0: can@2701000 { >>> + compatible = "bosch,m_can"; >>> + reg = <0x00 0x02701000 0x00 0x200>, >>> + <0x00 0x02708000 0x00 0x8000>; >>> + reg-names = "m_can", "message_ram"; >>> + power-domains = <&k3_pds 156 TI_SCI_PD_EXCLUSIVE>; >>> + clocks = <&k3_clks 156 1>, <&k3_clks 156 0>; >>> + clock-names = "cclk", "hclk"; >>> + interrupts = , >>> + ; >>> + interrupt-names = "int0", "int1"; >>> + bosch,mram-cfg = <0x0 0 0 32 0 0 1 1>; >> >> Are you intentionally only enabling 1 TX buffer? >> > > I have used this configuration for testing. It can be increased to 32 if > required. Is it better to set it to the maximum number of buffers ? > I have now set all the parameters that can be configured, to the their max values. "bosch,mram-cfg = <0x0 128 64 64 64 64 32 32>;" Earlier while setting only one tx buffer I was unintentionally limiting it. As far as I was able to search, the only constraint in setting them to max values is the memory space allocated for message ram. As in this case there is enough memory for configuring the message ram with max values for all parameters, I see that memory space wouldn't be an issue. After setting the above mentioned configuration I was able to perform a few tests and they were passing. I will fix this configuration and send a respin for this series. Thanks, Aswath > Thanks, > Aswath > >> Marc >> > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel