Re: [patch 6/6] staging: speakup: Migrate acntsa, bns, dectlk and txprt to ttyio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Okash Khawaja, on ven. 24 mars 2017 19:40:42 +0000, wrote:
> On Fri, Mar 24, 2017 at 08:24:22PM +0100, Samuel Thibault wrote:
> > Okash Khawaja, on ven. 24 mars 2017 19:21:54 +0000, wrote:
> > > On Sun, Feb 26, 2017 at 03:59:30AM +0100, Samuel Thibault wrote:
> > > > and then in speakup_apollo.c, instead of 
> > > > 
> > > > 	outb(UART_MCR_DTR, speakup_info.port_tts + UART_MCR);
> > > > 	outb(UART_MCR_DTR | UART_MCR_RTS,
> > > > 			speakup_info.port_tts + UART_MCR);
> > > 
> > > Is there a specific reason for keeping DTR set all the time? Or is it
> > > just a defensive measure?
> > 
> > I guess the device requires it. That's normal RS232 signaling anyway.
> > With tiocmset, however, you won't have to care: the generic code will
> > already have enabled DTR, and you will only want to clear and set RTS.
> 
> I noticed that spk_serial_init sets it and other places keep it set. By
> generic code, do you mean that when migrated to TTY, we don't have to
> explicitly set DTR?

Yes, it'll already be enabled by the layers below.

Samuel
_______________________________________________
Speakup mailing list
Speakup@xxxxxxxxxxxxxxxxx
http://linux-speakup.org/cgi-bin/mailman/listinfo/speakup




[Index of Archives]     [Linux for the Blind]     [Fedora Discussioin]     [Linux Kernel]     [Yosemite News]     [Big List of Linux Books]
  Powered by Linux