All of lore.kernel.org
 help / color / mirror / Atom feed
From: Alan Cox <alan@lxorguk.ukuu.org.uk>
To: Jonathan Woithe <jwoithe@just42.net>
Cc: linux-serial@vger.kernel.org
Subject: Re: Comments requested: driver for Quatech PCI serial cards
Date: Wed, 14 Nov 2012 12:12:22 +0000	[thread overview]
Message-ID: <20121114121222.3e42860f@pyramind.ukuu.org.uk> (raw)
In-Reply-To: <20121114060724.GN30156@marvin.atrad.com.au>

> The driver supports a wide range of Quatech DSC/ESC/QSC cards, including the
> custom functionality which allow various aspects of the signal routing to be
> altered under software control.

This has actually been kicked around with another couple of cards. We
have a draft but never test implemented API for binding gpio numbers on
a device to signals for the tty layer.

>  - the name of the module.  Anything with "qt" in it will confuse people.
>    I'm thinking along the lines of serial-quatech or quatech-serial:
>    comments welcome.

qt is fine in kernel - just be careful of the qt usb module.

I think in the same place I'd start by just adding the needed identifiers
to the 8250_pci driver along with any needed init helper.

For the ioctls a lot appear to be just exposing values which as you say
would fit sysfs. The tty layer now (as of 3.7rc) usefully supports sysfs
nodes on a tty itself too.

The basic ones are already handled by 8250_pci.c (QSC100, DSC100,
ESC100D, ESC100M).

As far as I can see the issues are

1.	Clock multiplier feature

Supportable by flags in 8250.c and worst case by providing a custom
set_termios. No API needed.

2.	RS485/RS422 options

Supportable by adding TIOCRS485 handling/callout to 8250.c (needs doing
anyway)

The RS485 ioctl might need some extending but we've been gradually doing
this as we hit chips with more features in hardware so that's not a
problem beyond synchronizing with other platforms who can use the extra
flags.

Alan

  reply	other threads:[~2012-11-14 12:07 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-14  6:07 Comments requested: driver for Quatech PCI serial cards Jonathan Woithe
2012-11-14 12:12 ` Alan Cox [this message]
2012-11-14 23:10   ` Jonathan Woithe
2012-11-14 23:57     ` Alan Cox
2012-11-15  0:11       ` Jonathan Woithe
2012-11-15 15:53         ` Alan Cox
2012-11-15 22:23           ` Jonathan Woithe
2012-11-28  2:29           ` Jonathan Woithe
2012-11-28 13:24             ` Alan Cox
2012-11-28 13:41               ` Jonathan Woithe
2012-11-28 17:30                 ` Greg KH
2012-11-28 22:33                   ` Jonathan Woithe
2012-12-10  0:13                     ` Did quatech changes make 3.8? (was: Re: Comments requested: driver for Quatech PCI serial cards) Jonathan Woithe
2012-12-10  2:19                       ` Greg KH

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20121114121222.3e42860f@pyramind.ukuu.org.uk \
    --to=alan@lxorguk.ukuu.org.uk \
    --cc=jwoithe@just42.net \
    --cc=linux-serial@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.