* Re: USB: serial: add symbol serial driver
[not found] <200903261929.n2QJTS5L009700@hera.kernel.org>
@ 2009-03-26 19:42 ` Alan Cox
2009-04-06 5:37 ` Greg KH
0 siblings, 1 reply; 3+ messages in thread
From: Alan Cox @ 2009-03-26 19:42 UTC (permalink / raw)
To: Linux Kernel Mailing List; +Cc: torvalds, greg
On Thu, 26 Mar 2009 19:29:28 GMT
Linux Kernel Mailing List <linux-kernel@vger.kernel.org> wrote:
Thank you for not running this past the tty layer maintainer.
> + /*
> + * Force low_latency on so that our tty_push actually forces the data
> + * through, otherwise it is scheduled, and with high data rates (like
> + * with OHCI) data can get lost.
> + */
> + if (tty)
> + tty->low_latency = 1;
The above is bogus cargo cult programming copied from old drivers and
should be removed.
Otherwise looks fine
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: USB: serial: add symbol serial driver
2009-03-26 19:42 ` USB: serial: add symbol serial driver Alan Cox
@ 2009-04-06 5:37 ` Greg KH
2009-04-06 9:14 ` Alan Cox
0 siblings, 1 reply; 3+ messages in thread
From: Greg KH @ 2009-04-06 5:37 UTC (permalink / raw)
To: Alan Cox; +Cc: Linux Kernel Mailing List, torvalds
On Thu, Mar 26, 2009 at 07:42:28PM +0000, Alan Cox wrote:
> On Thu, 26 Mar 2009 19:29:28 GMT
> Linux Kernel Mailing List <linux-kernel@vger.kernel.org> wrote:
>
> Thank you for not running this past the tty layer maintainer.
Sorry about that.
> > + /*
> > + * Force low_latency on so that our tty_push actually forces the data
> > + * through, otherwise it is scheduled, and with high data rates (like
> > + * with OHCI) data can get lost.
> > + */
> > + if (tty)
> > + tty->low_latency = 1;
>
> The above is bogus cargo cult programming copied from old drivers and
> should be removed.
It's all over the usb-serial drivers, should I just remove it from all
of them?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: USB: serial: add symbol serial driver
2009-04-06 5:37 ` Greg KH
@ 2009-04-06 9:14 ` Alan Cox
0 siblings, 0 replies; 3+ messages in thread
From: Alan Cox @ 2009-04-06 9:14 UTC (permalink / raw)
To: Greg KH; +Cc: Linux Kernel Mailing List, torvalds
> > The above is bogus cargo cult programming copied from old drivers and
> > should be removed.
>
> It's all over the usb-serial drivers, should I just remove it from all
> of them?
I think so yes. USB folks used it to avoid overrunning the old flip
buffer system and with proper buffering that isn't needed. Now its only
relevance is if you need very low latency responses which is normally an
application layer choice.
Alan
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2009-04-06 9:14 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <200903261929.n2QJTS5L009700@hera.kernel.org>
2009-03-26 19:42 ` USB: serial: add symbol serial driver Alan Cox
2009-04-06 5:37 ` Greg KH
2009-04-06 9:14 ` Alan Cox
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.