linux-serial.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* uartlite - fix poll support
@ 2010-08-25 12:55 Michal Simek
  2010-08-25 12:55 ` [PATCH] serial: uartlite: Remove obscure loop from ulite_get_poll_char Michal Simek
  0 siblings, 1 reply; 4+ messages in thread
From: Michal Simek @ 2010-08-25 12:55 UTC (permalink / raw)
  To: linux-kernel; +Cc: linux-serial, gregkh

Hi,

I forget to CC Peter Korsgaard (sorry for that again) and he found one obscure thing
in my first patch. This second patch removes it. It will be the best
to add those two patches together but I am not sure if is possible. It depends
on Greg maintaining process. (If is possible I would like to ask for it).

Peter: I added your Signed-off-by line I hope that you are OK with it.

Greg: If there are no problems with it, please add to your tty tree or add together
with the first patch.

Thanks,
Michal

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH] serial: uartlite: Remove obscure loop from ulite_get_poll_char
  2010-08-25 12:55 uartlite - fix poll support Michal Simek
@ 2010-08-25 12:55 ` Michal Simek
  2010-09-02  2:00   ` Greg KH
  0 siblings, 1 reply; 4+ messages in thread
From: Michal Simek @ 2010-08-25 12:55 UTC (permalink / raw)
  To: linux-kernel; +Cc: linux-serial, gregkh, Michal Simek, Peter Korsgaard

The previous patch
"serial: Add CONSOLE_POLL support for uartlite"
(sha1 id 52ea4a7ef12f286bd68d13a93daaa688f56a1e14)
contained obcure loop in ulite_get_poll_char function.

Signed-off-by: Michal Simek <monstr@monstr.eu>
Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
 drivers/serial/uartlite.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/serial/uartlite.c b/drivers/serial/uartlite.c
index c0fae0d..c4bf54b 100644
--- a/drivers/serial/uartlite.c
+++ b/drivers/serial/uartlite.c
@@ -325,7 +325,7 @@ static int ulite_verify_port(struct uart_port *port, struct serial_struct *ser)
 #ifdef CONFIG_CONSOLE_POLL
 static int ulite_get_poll_char(struct uart_port *port)
 {
-	while (!(ioread32be(port->membase + ULITE_STATUS)
+	if (!(ioread32be(port->membase + ULITE_STATUS)
 						& ULITE_STATUS_RXVALID))
 		return NO_POLL_CHAR;
 
-- 
1.5.5.6


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] serial: uartlite: Remove obscure loop from ulite_get_poll_char
  2010-08-25 12:55 ` [PATCH] serial: uartlite: Remove obscure loop from ulite_get_poll_char Michal Simek
@ 2010-09-02  2:00   ` Greg KH
  2010-09-02  5:24     ` Michal Simek
  0 siblings, 1 reply; 4+ messages in thread
From: Greg KH @ 2010-09-02  2:00 UTC (permalink / raw)
  To: Michal Simek; +Cc: linux-kernel, linux-serial, gregkh, Peter Korsgaard

On Wed, Aug 25, 2010 at 02:55:04PM +0200, Michal Simek wrote:
> The previous patch
> "serial: Add CONSOLE_POLL support for uartlite"
> (sha1 id 52ea4a7ef12f286bd68d13a93daaa688f56a1e14)
> contained obcure loop in ulite_get_poll_char function.

I've merged this with the original patch so as to not have a problem.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] serial: uartlite: Remove obscure loop from ulite_get_poll_char
  2010-09-02  2:00   ` Greg KH
@ 2010-09-02  5:24     ` Michal Simek
  0 siblings, 0 replies; 4+ messages in thread
From: Michal Simek @ 2010-09-02  5:24 UTC (permalink / raw)
  To: Greg KH; +Cc: linux-kernel, linux-serial, gregkh, Peter Korsgaard

Greg KH wrote:
> On Wed, Aug 25, 2010 at 02:55:04PM +0200, Michal Simek wrote:
>> The previous patch
>> "serial: Add CONSOLE_POLL support for uartlite"
>> (sha1 id 52ea4a7ef12f286bd68d13a93daaa688f56a1e14)
>> contained obcure loop in ulite_get_poll_char function.
> 
> I've merged this with the original patch so as to not have a problem.


Thanks and sorry for the problem,
Michal

-- 
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2010-09-02  5:24 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-08-25 12:55 uartlite - fix poll support Michal Simek
2010-08-25 12:55 ` [PATCH] serial: uartlite: Remove obscure loop from ulite_get_poll_char Michal Simek
2010-09-02  2:00   ` Greg KH
2010-09-02  5:24     ` Michal Simek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).