Re: [PATCH] cdc-acm: fix abnormal DATA RX issue for Mediatek Preloader.

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

 



On Do, 2018-12-13 at 10:43 +0100, Johan Hovold wrote:
> On Thu, Dec 13, 2018 at 11:27:56AM +0800, macpaul.lin@xxxxxxxxxxxx wrote:
> > From: Macpaul Lin <macpaul.lin@xxxxxxxxxxxx>
> > 
> > +	/* handle active handshake triggered by device */
> > +	if (quirks == DISABLE_ECHO)
> > +		acm_tty_driver->init_termios.c_lflag &= ~(ECHO);
> 
> You cannot change the driver init_termios like this as that will affect
> all cdc-acm devices that are probed later. If this is at all needed,
> this will have to be done at tty install time.

Right. How do with decide on a sensible default anyway?

	Regards
		Oliver




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux