From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tony Lindgren Subject: Re: [PATCH 2/2] ARM: dts: dra7x-evm: Prevent glitch on DCAN1 pinmux Date: Sun, 12 Jul 2015 23:29:06 -0700 Message-ID: <20150713062906.GA26485@atomide.com> References: <1436279277-3386-1-git-send-email-rogerq@ti.com> <1436279277-3386-3-git-send-email-rogerq@ti.com> <559EBEC8.1070609@pengutronix.de> <55A2BD97.8080006@pengutronix.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from muru.com ([72.249.23.125]:34579 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751028AbbGMG3K (ORCPT ); Mon, 13 Jul 2015 02:29:10 -0400 Content-Disposition: inline In-Reply-To: <55A2BD97.8080006@pengutronix.de> Sender: linux-can-owner@vger.kernel.org List-ID: To: Marc Kleine-Budde Cc: Roger Quadros , wg@grandegger.com, jay.schroeder@garmin.com, linux-can@vger.kernel.org, linux-omap@vger.kernel.org, linux-kernel@vger.kernel.org * Marc Kleine-Budde [150712 12:22]: > On 07/09/2015 08:34 PM, Marc Kleine-Budde wrote: > > On 07/07/2015 04:27 PM, Roger Quadros wrote: > >> Driver core sets "default" pinmux on on probe and CAN driver > >> sets "sleep" pinmux during register. This causes a small window > >> where the CAN pins are in "default" state with the DCAN module > >> being disabled. > >> > >> Change the "default" state to be like sleep so this glitch is > >> avoided. Add a new "active" state that is used by the driver > >> when CAN is actually active. > > > > Who is taking care of this patch? I'm applying 1/2 (v2) to linux-can, > > should I take this patch, too? > > I've included this patch in my pull request. That's fine thanks there should not be any merge conflicts. For things going in during the merge windows we need to worry about the dts merge conflicts. Usually not a problem for fixes. Regards, Tony