[Qemu-devel] Re: hw/serial.c: Xmit fifo never used

2010-05-26 Thread Jan Kiszka
Frank Mehnert wrote: Hi, the xmit fifo of the serial device is never used. If qemu_chr_write() fails (interface currently not able to send characters) then the transmit_timer should be engaged to try to send the current character from the fifo again after some time. The code is }

Re: [Qemu-devel] Re: hw/serial.c: Xmit fifo never used

2010-05-26 Thread Frank Mehnert
On Wednesday 26 May 2010, Jan Kiszka wrote: Frank Mehnert wrote: I assume the correct condition would be '= 0', that is ... if ((s-tsr_retry = 0) (s-tsr_retry = MAX_XMIT_RETRY)) { s-tsr_retry++; ... Makes sense, patch welcome. Attached. Kind

[Qemu-devel] Re: hw/serial.c: Xmit fifo never used

2010-05-26 Thread Jan Kiszka
Frank Mehnert wrote: On Wednesday 26 May 2010, Jan Kiszka wrote: Frank Mehnert wrote: I assume the correct condition would be '= 0', that is ... if ((s-tsr_retry = 0) (s-tsr_retry = MAX_XMIT_RETRY)) { s-tsr_retry++; ... Makes sense, patch welcome.

Re: [Qemu-devel] Re: hw/serial.c: Xmit fifo never used

2010-05-26 Thread Stefano Stabellini
On Wed, 26 May 2010, Jan Kiszka wrote: Frank Mehnert wrote: On Wednesday 26 May 2010, Jan Kiszka wrote: Frank Mehnert wrote: I assume the correct condition would be '= 0', that is ... if ((s-tsr_retry = 0) (s-tsr_retry = MAX_XMIT_RETRY)) { s-tsr_retry++;

Re: [Qemu-devel] Re: hw/serial.c: Xmit fifo never used

2010-05-26 Thread Frank Mehnert
On Wednesday 26 May 2010, Stefano Stabellini wrote: I think the patch is correct. serial: fixed bug which prevented the use of the xmit fifo Signed-off-by: Frank Mehnert frank.mehn...@sun.com --- hw/serial.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/hw/serial.c