From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Subject: Re: [RFC RESEND] serial: 8250: fix regression in 8250 uart driver Date: Sat, 02 Jul 2016 14:58:59 +0300 Message-ID: <1467460739.30123.443.camel@linux.intel.com> References: <575F7325.3030907@kernel.org> <1465897220.30123.26.camel@linux.intel.com> <1465920046.30123.83.camel@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Thorsten Leemhuis , dinh.linux@anniebear.net Cc: Dinh Nguyen , heikki.krogerus@linux.intel.com, peter@hurleysoftware.com, Greg Kroah-Hartman , linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org List-Id: linux-serial@vger.kernel.org On Sat, 2016-07-02 at 13:53 +0200, Thorsten Leemhuis wrote: > On 14.06.2016 18:00, Andy Shevchenko wrote: > > On Tue, 2016-06-14 at 10:27 -0500, dinh.linux@anniebear.net wrote: > > > > On Mon, 2016-06-13 at 21:59 -0500, Dinh Nguyen wrote: > > > > I suppose this one, i.e. commit dd4e91d538b3 ("dmaengine: slave > > > > means at > > > > least one of DMA_SLAVE, DMA_CYCLIC"), solves the issue. > > > Hmm...that commit didn't seem to fix my issue. > > [=E2=80=A6] > > > Does this mean that the 8250_dw.c driver can no longer use DMA? > >=20 > > It means that previously the use of DMA engine whatever was used is > > not > > correct in your case. There is no issues in UART driver, the issue > > is in > > DMA controller driver. > >=20 > > I'm working on a quirk that will allow user to enforce PIO mode for > > Rx/Tx or both. In that case if DMA controller can't be fixed you > > still > > might use DMA for Tx. >=20 > What's the status of this issue? Apparently I'm not going to fix DMA controller engine if it even would be fixed (I don't know what one you use), but I will continue with the mentioned quirk as soon as I finish one higher priority task. Sorry for a delay. > Just wondering, as this issue is listed > in my regression reports for 4.7 and it seems nothing much happened > for > more than a week now =E2=80=93 which is a bad sign as the 4.7 release= seems > only > a week or two away. --=20 Andy Shevchenko Intel Finland Oy