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=-10.4 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING, NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=ham 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 AFBD1C4338F for ; Thu, 5 Aug 2021 12:49:27 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 8F4C6601FC for ; Thu, 5 Aug 2021 12:49:27 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241489AbhHEMtj (ORCPT ); Thu, 5 Aug 2021 08:49:39 -0400 Received: from thoth.sbs.de ([192.35.17.2]:47409 "EHLO thoth.sbs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S235161AbhHEMtj (ORCPT ); Thu, 5 Aug 2021 08:49:39 -0400 Received: from mail2.sbs.de (mail2.sbs.de [192.129.41.66]) by thoth.sbs.de (8.15.2/8.15.2) with ESMTPS id 175CnB0j001791 (version=TLSv1.2 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 5 Aug 2021 14:49:11 +0200 Received: from [167.87.0.185] ([167.87.0.185]) by mail2.sbs.de (8.15.2/8.15.2) with ESMTP id 175CnAiP027338; Thu, 5 Aug 2021 14:49:10 +0200 Subject: Re: [PATCH v2 0/6] CAN: Add support for CAN in AM65,J721e and AM64 To: Aswath Govindraju , Nishanth Menon Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Tero Kristo , Rob Herring , Marc Kleine-Budde , Lokesh Vutla , Faiz Abbas References: <20210726101012.26983-1-a-govindraju@ti.com> <20210802124509.z23lfg7xxkqaakbo@truce> <36011cee-f617-02ed-7446-c297ba65e6bf@ti.com> <35b0e2dc-0591-0a2f-8e3d-4177792a13fa@siemens.com> From: Jan Kiszka Message-ID: Date: Thu, 5 Aug 2021 14:49:10 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.12.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05.08.21 14:44, Aswath Govindraju wrote: > Hi Jan, > > On 05/08/21 6:10 pm, Jan Kiszka wrote: >> On 05.08.21 14:32, Aswath Govindraju wrote: >>> Hi, >>> >>> On 02/08/21 6:15 pm, Nishanth Menon wrote: >>>> On 15:40-20210726, Aswath Govindraju wrote: >>>>> The following series of patches add support for CAN in SoC's AM65, J721e >>>>> and AM64. >>>>> >>>>> This patch series is dependent on [1] and [2] and I have requested for an >>>>> immutable tag from the Marc Kleine-Budde(maintainer of net tree). >>>>> >>>>> [1] - https://lore.kernel.org/patchwork/patch/1423048/ >>>>> [2] - https://www.spinics.net/lists/linux-can/msg08108.html >>>>> >>>>> changes since v1 - >>>>> - changed the message ram configuration to use the maximum value >>>>> in each field, for better performance. >>>>> >>>>> Aswath Govindraju (3): >>>>> arm64: dts: ti: am654-base-board: Disable mcan nodes >>>>> arm64: dts: ti: k3-am64-main: Add support for MCAN >>>>> arm64: dts: ti: k3-am642-evm/sk: Add support for main domain mcan >>>>> nodes in EVM and disable them on SK >>>>> >>>>> Faiz Abbas (3): >>>>> arm64: dts: ti: k3-am65-mcu: Add Support for MCAN >>>>> arm64: dts: ti: k3-j721e: Add support for MCAN nodes >>>>> arm64: dts: ti: k3-j721e-common-proc-board: Add support for mcu_mcan >>>>> nodes >>>> >>>> I noticed in my checkup: https://pastebin.ubuntu.com/p/zSk39M943N/ >>>> warnings with dtbs_check, I think the bindings need a little more help >>>> here (please also notice the iot platform warnings getting introduced). >>>> >>> >>> Jan, >>> >>> Can you please point me to the schematics for iot platform? Also, is CAN >>> subsystem brought out on the iot platform boards? >> >> Schematics aren't publicly available. If you have specific questions, I >> may help, though. >> >> CAN is not in use yet (not connected AFAIK). >> > > Thank you for clarifying this. I will disable the CAN DT nodes in > k3-am65-iot2050-common.dtsi in my respin. > That is not a problem for the current design, so you can go ahead with it. Current defconfig - though not yet upstream based - can be found at [1]. The CAN subsystem is enabled, but not the AM65x CAN driver. Jan [1] https://github.com/siemens/meta-iot2050/tree/master/recipes-kernel/linux/files -- Siemens AG, T RDA IOT Corporate Competence Center Embedded Linux