Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <b48858f4-2904-fddf-fe9e-61100ba1a122@ti.com>
Date:   Mon, 24 Jul 2023 11:25:07 -0500
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.0
Subject: Re: [PATCH v1 1/2] arm64: dts: ti: k3-am62: Add MCU MCAN nodes
Content-Language: en-US
To:     Francesco Dolcini <francesco@dolcini.it>
CC:     Nishanth Menon <nm@ti.com>, Vignesh Raghavendra <vigneshr@ti.com>,
        Tero Kristo <kristo@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        <linux-arm-kernel@lists.infradead.org>,
        <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
        Francesco Dolcini <francesco.dolcini@toradex.com>
References: <20230724133612.37366-1-francesco@dolcini.it>
 <20230724133612.37366-2-francesco@dolcini.it>
 <42044653-cc22-131a-bf1b-e68ddfa0ff9d@ti.com>
 <ZL6lVGklNclkVu58@francesco-nb.int.toradex.com>
From:   Judith Mendez <jm@ti.com>
In-Reply-To: <ZL6lVGklNclkVu58@francesco-nb.int.toradex.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Francesco,

On 7/24/23 11:22 AM, Francesco Dolcini wrote:
> Hello Judith,
> 
> On Mon, Jul 24, 2023 at 11:06:24AM -0500, Judith Mendez wrote:
>> On 7/24/23 8:36 AM, Francesco Dolcini wrote:
>>> From: Judith Mendez <jm@ti.com>
>>>
>>> On AM62x there are no hardware interrupts routed to A53 GIC
>>> interrupt controller for MCU MCAN IPs, so MCU MCAN nodes were
>>> omitted from MCU dtsi.
>>>
>>> Timer polling was introduced in commits [1][2] so now add MCU MCAN nodes
>>> to the MCU dtsi for the Cortex A53.
>>>
>>> [1] b382380c0d2d ("can: m_can: Add hrtimer to generate software interrupt")
>>> [2] bb410c03b999 ("dt-bindings: net: can: Remove interrupt properties for MCAN")
>>>
>>> Signed-off-by: Judith Mendez <jm@ti.com>
>>> [fd: fixed labels to match datasheet numbering, revised commit message,
>>>        fixed reg/reg-names order]
>>> Signed-off-by: Francesco Dolcini <francesco.dolcini@toradex.com>
>>> ---
> 
> ...
> 
>> Thanks for sending this patch, will also send for AM62ax.
> 
> Please have a look at the couple of changes I did:
>   - corrected the label to be coherent with the datasheet and the other
>     peripheral names
>   - inverted "m_can" and "message_ram" regs/regs-names to fix a binding
>     checker error.
> 
> These might be relevant also for the AM62a.

Yes, already did. The changes look good to me, tested and everything 
works on AM62x. Did the same changes to AM62ax MCU dtsi. Thanks!

~ Judith