From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Gross Subject: Re: [PATCH 1/2] tty: serial: msm_serial regression fix data corruption Date: Thu, 5 May 2016 18:52:26 -0500 Message-ID: <20160505235226.GD6533@hector.attlocal.net> References: <571BAC8B.80001@gmail.com> <571BAD78.4020609@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-ob0-f175.google.com ([209.85.214.175]:33283 "EHLO mail-ob0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756135AbcEEXw2 (ORCPT ); Thu, 5 May 2016 19:52:28 -0400 Received: by mail-ob0-f175.google.com with SMTP id x1so44240788obt.0 for ; Thu, 05 May 2016 16:52:27 -0700 (PDT) Content-Disposition: inline In-Reply-To: <571BAD78.4020609@gmail.com> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Frank Rowand Cc: David Brown , Greg Kroah-Hartman , Jiri Slaby , "linux-arm-msm@vger.kernel.org" , linux-soc@vger.kernel.org, linux-serial@vger.kernel.org, Linux Kernel list , ivan.ivanov@linaro.org On Sat, Apr 23, 2016 at 10:14:32AM -0700, Frank Rowand wrote: > From: Frank Rowand > > Commit 3a878c430fd6 ("tty: serial: msm: Add TX DMA support") regression. > The calculation of tx_count was moved from the old msm_handle_tx(), > now renamed msm_handle_tx_pio(), to the new msm_handle_tx(). The > move left out one size test. > > The regression seen on the qcom-apq8074-dragonboard is dropped > characters and corrupted characters (values greater than 0x7f) > when DMA is not enabled. > > Signed-off-by: Frank Rowand This allowed me to transfer files over the console without any CRC errors. Zmodem transfer at 115k worked great. Tested-by: Andy Gross