From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg Kroah-Hartman Date: Sat, 25 Jun 2016 17:01:53 +0000 Subject: Re: [PATCH v2 2/4] serial: sh-sci: Stop transfers in sci_shutdown() Message-Id: <20160625170153.GA9544@kroah.com> List-Id: References: <1466780356-12623-1-git-send-email-geert+renesas@glider.be> <1466780356-12623-3-git-send-email-geert+renesas@glider.be> <20160625160939.GA29593@kroah.com> <20160625161051.GA30995@kroah.com> In-Reply-To: <20160625161051.GA30995@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Geert Uytterhoeven Cc: Jiri Slaby , Magnus Damm , Yoshihiro Shimoda , linux-serial@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-sh@vger.kernel.org, uclinux-h8-devel@lists.sourceforge.jp On Sat, Jun 25, 2016 at 09:10:51AM -0700, Greg Kroah-Hartman wrote: > On Sat, Jun 25, 2016 at 09:09:39AM -0700, Greg Kroah-Hartman wrote: > > On Fri, Jun 24, 2016 at 04:59:14PM +0200, Geert Uytterhoeven wrote: > > > Make sure the transmitter and receiver are stopped when shutting down > > > the port, and related interrupts are disabled. > > > > > > Without this: > > > - New input data may be received into the RX FIFO, possibly > > > triggering a new RX DMA completion, > > > - Transfers will still be enabled on a subsequent startup of the UART, > > > before the UART's FIFOs have been reset, causing reading of stale > > > data. > > > > > > Inspired by a patch in the BSP by Koji Matsuoka > > > . > > > > > > Signed-off-by: Geert Uytterhoeven > > > --- > > > v2: > > > - Do not clear clock source bits, since the serial console relies on > > > them, as reported by Shimoda-san. > > > > > > v1 was extracted from "[PATCH/RFC v3 0/4] serial: sh-sci: Add DT DMA > > > support". The issues with the serial console seen before on > > > r8a7740/armadillo and sh73a0/kzm9g seem to be gone. Changes after > > > resurrection: > > > - Write zero to also disable related interrupts, as suggested by > > > Laurent Pinchart, > > > - Enhanced patch description. > > > --- > > > drivers/tty/serial/sh-sci.c | 5 +++++ > > > 1 file changed, 5 insertions(+) > > > > This patch does not apply to my tree :( > > But I don't have access to my 2-factor token for about 24 hours, so my > tree will not be public until after then, sorry about that, I'll email > you when it gets synced up to kernel.org... It's synced now, sorry for the noise. greg k-h