From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andy Shevchenko Date: Wed, 31 Aug 2016 16:11:01 +0000 Subject: Re: [patch] tty/serial/8250: unlock on error in serial8250_register_8250_port() Message-Id: <1472659861.4887.413.camel@linux.intel.com> List-Id: References: <20160628091522.GA7720@mwanda> <20160831135758.GA13298@kroah.com> In-Reply-To: <20160831135758.GA13298@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Greg Kroah-Hartman , Dan Carpenter Cc: Yegor Yefremov , Jiri Slaby , Anton Wuerfel , Phillip Raffeck , Peter Hurley , Wan Ahmad Zainie , Masahiro Yamada , "Maciej S. Szmigiero" , linux-serial@vger.kernel.org, kernel-janitors@vger.kernel.org On Wed, 2016-08-31 at 15:57 +0200, Greg Kroah-Hartman wrote: > On Tue, Jun 28, 2016 at 12:15:22PM +0300, Dan Carpenter wrote: > > > > We need to unlock before returning here. > > > > Fixes: 4ef03d328769 ('tty/serial/8250: use mctrl_gpio helpers') > > I ended up reverting this patch, perhaps this is the problem that > people > were having with this patch? Nope, I tried this fix, didn't help. Mika put the details in his message I mentioned in the patch: Link: https://lkml.kernel.org/r/20160809130229.GN1729@lahna.fi.intel.com -- Andy Shevchenko Intel Finland Oy