From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stanislav Brabec Subject: Re: [PATCH] Serial: BUGFIX: uart_resume_port has an omitted condition. Date: Thu, 14 Oct 2010 21:45:57 +0200 Message-ID: <1287085557.10610.18.camel@utx.lan> References: <1286949486-2446-1-git-send-email-myungjoo.ham@samsung.com> <20101013062605.GB4393@suse.de> <1286969205.6082.700.camel@hammer.site> <20101014184511.GA17096@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-2 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from cantor2.suse.de ([195.135.220.15]:53845 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755961Ab0JNTqA (ORCPT ); Thu, 14 Oct 2010 15:46:00 -0400 In-Reply-To: <20101014184511.GA17096@kroah.com> Sender: linux-serial-owner@vger.kernel.org List-Id: linux-serial@vger.kernel.org To: Greg KH Cc: MyungJoo Ham , Greg KH , linux-kernel@vger.kernel.org, kyungmin.park@samsung.com, Alan Cox , Arnd Bergmann , Andr Goddard Rosa , Jason Wang , linux-serial@vger.kernel.org Greg KH wrote:=20 > > This disparity appeared deliberately in 4547be78. If you need > > no_console_suspend and the hardware lets the device in an undefined > > state after resume (i. e. PXA270), you need to call subset of the > > resume, even if the suspend counterpart was not called. Yes, I can > > imagine that it may be a source of problems. =20 > > It seems that support for no_console_suspend for all devices is bec= oming > > more complicated. I guess that a new driver calls (maybe "save_stat= e" > > and "resume_state") or support for no_console_suspend directly in > > drivers may be useful. >=20 > Ok, so I'm guessing that this patch is not to be applied then, correc= t? I just compared it with Jason Wang's patches. MyungJoo Ham's patch is exactly equal to the PATCH 1/2. So it already has my ACK and it is pending as: serial-core: skip call set_termios/console_start when no_console_suspen= d Related thread can be found in linux-serial archives from August 2010: [PATCH RESEND 0/2] two serial_core suspend/resume fixes To MyungJoo Ham: Could you verify, that Jason Wang's patch 2/2 does not break your system? My patch to fix PXA270 does not exist yet. I'll ask you in future for testing. --=20 Best Regards / S pozdravem, Stanislav Brabec software developer --------------------------------------------------------------------- SUSE LINUX, s. r. o. e-mail: sbrabec@suse.cz Lihovarsk=E1 1060/12 tel: +420 284 028 966, +49 911 740538747 190 00 Praha 9 fax: +420 284 028 951 Czech Republic http://www.suse.cz/ -- To unsubscribe from this list: send the line "unsubscribe linux-serial"= in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html