From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH] ARM: dts: am57xx-idk: Add DCAN support Date: Mon, 6 Mar 2017 10:41:13 -0800 Message-ID: <20170306184113.GN20572@atomide.com> References: <8981d49e7011a7246ff7dc6a9a9da0a56dfe5621.1487670384.git.nsekhar@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Sekhar Nori Cc: Mark Rutland , Device Tree Mailing List , Schuyler Patton , Kishon Vijay Abraham I , Rob Herring , Franklin S Cooper Jr , Linux OMAP Mailing List , Linux ARM Mailing List , "Quadros, Roger" List-Id: devicetree@vger.kernel.org * Sekhar Nori [170221 02:13]: > On Tuesday 21 February 2017 03:30 PM, Sekhar Nori wrote: > > From: Schuyler Patton > > > > AM571x IDK and the AM572x IDK use CAN1 interface. > > This patch enables it for both boards. > > > > Tested on AM572x IDK using cansequence. > > > > Signed-off-by: Schuyler Patton > > Signed-off-by: Kishon Vijay Abraham I > > Signed-off-by: Franklin S Cooper Jr > > [nsekhar@ti.com: move to use DRA7XX_CORE_IOPAD()) > > Signed-off-by: Sekhar Nori > > I should have noted that although CAN does function on the IDK with this > patch, there are "omap_hwmod: dcan1: _wait_target_disable failed" > messages that come up due to lack of workaround for errata i893[1] > > Roger does have some patches to workaround that errata. If you want to > wait for those patches to be submitted and accepted first, it should be > fine as well. OK let's wait a bit on this then to avoid introducing extra warnings. Regards, Tony > [1] http://www.ti.com/lit/er/sprz436b/sprz436b.pdf