Linux Serial subsystem development
 help / color / mirror / Atom feed
From: Sebastian Reichel <sre@kernel.org>
To: "H. Nikolaus Schaller" <hns@goldelico.com>
Cc: One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>,
	Oleksij Rempel <linux@rempel-privat.de>,
	Rob Herring <robh@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Marcel Holtmann <marcel@holtmann.org>,
	Jiri Slaby <jslaby@suse.com>, Pavel Machek <pavel@ucw.cz>,
	Peter Hurley <peter@hurleysoftware.com>,
	NeilBrown <neil@brown.name>, Arnd Bergmann <arnd@arndb.de>,
	Linus Walleij <linus.walleij@linaro.org>,
	"open list:BLUETOOTH DRIVERS" <linux-bluetooth@vger.kernel.org>,
	"linux-serial@vger.kernel.org" <linux-serial@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [RFC PATCH 0/3] UART slave device bus
Date: Tue, 23 Aug 2016 00:42:49 +0200	[thread overview]
Message-ID: <20160822224247.vdh3plda6dxelvzw@earth> (raw)
In-Reply-To: <EA228ADD-F03A-49C0-9A0B-E768D7A7A466@goldelico.com>

[-- Attachment #1: Type: text/plain, Size: 4881 bytes --]

Hi,

On Mon, Aug 22, 2016 at 11:23:26PM +0200, H. Nikolaus Schaller wrote:
> > Am 22.08.2016 um 22:39 schrieb Sebastian Reichel <sre@kernel.org>:
> > 
> > Hi,
> > 
> > On Sun, Aug 21, 2016 at 09:50:57AM +0200, H. Nikolaus Schaller wrote:
> >>> Am 20.08.2016 um 15:34 schrieb One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>:
> >>>> What it is not about are UART/RS232 converters connected through USB or virtual
> >>>> serial ports created for WWAN modems (e.g. /dev/ttyACM, /dev/ttyHSO). Or BT devices
> >>>> connected through USB (even if they also run HCI protocol).
> >>> 
> >>> It actually has to be about both because you will find the exact same
> >>> device wired via USB SSIC/HSIC to a USB UART or via a classic UART. Not is
> >>> it just about embedded boards.
> >> 
> >> Not necessarily.
> >> 
> >> We often have two interface options for exactly the sam sensor chips. They can be connected
> >> either through SPI or I2C. Which means that there is a core driver for the chip and two different
> >> transport glue components (see e.g. iio/accel/bmc150).
> >> 
> >> This does not require I2C to be able to handle SPI or vice versa or provide a common API.
> > 
> > I don't understand this comparison. I2C and SPI are different
> > protocols,
> 
> Yes, they are different on protocol level, but on both you transfer blocks of data from/to a slave device
> which usually can be addressed. And for some chips they are just two slightly alternative serial interfaces.
> 
> > while native UART and USB-connected UART are both UART.
> 
> I see what you mean, but kernel divides between directly connected UART and USB-connected UART.
> 
> drivers/usb/serial/ vs. drivers/tty/serial/
> 
> to implement two different groups of UARTs. Although on user space level they are harmonized again.
> This is why I compare with i2c and spi. But each such comparison is not perfect.
> 
> Anyways, to me it looks as if everybody wants to make the solution work for usb-uarts as well
> (although I still would like to see a real world use-case).
> 
> > 
> >> And most Bluetooth devices I know have either UART or a direct
> >> USB interface. So in the USB case there is no need to connect
> >> it through some USB-UART bridge and treat it as an UART at all.
> > 
> > I think having support for USB-UART dongles is useful for
> > driver development and testing on non-embedded HW.
> 
> Hm. I assume you mean the Bluetooth situation where both, embedded UART
> connected chips and USB dongles are available.

No. I mean I have some serial device, which is connected to the
embedded UART, but I also have a standalone version. For driver
development I can just use my standalone serial device, connect
it to an USB-UART and develop the driver on non embedded HW.
Then I can use the same driver on my embedded platform and it
works, since it uses the same API.

For e.g. I2C this works perfectly fine. I already did this with
the I2C interface exposed on my notebook's VGA port.

> I am not a specialist for such things, but  I think you have three
> options to connect bluetooth:
> 
> a) SoC-UART <-> BT-Chip-UART-port
> b) USB-UART (FT232, PL2303 etc.) <-> BT-Chip-UART-port
> c) USB <-> BT-Chip-USB-port (not UART involved at all)
>
> Case c) IMHO means you anyways need a special USB driver for the BT-Chip connected
> through USB and plugging it into a non-embedded USB port does not automatically
> show it as a tty interface. So you can't use it for testing the UART drivers.
> 
> BTW: the Wi2Wi W2CBW003 chip comes in two firmware variants: one for UART and
> one for USB. So they are also not exchangeable.

Yes, let's ignore option c). I'm talking about UART only. If the
chip has native USB support, then that's a different driver. Note,
that for more complex drivers it may become possible to use the same
high-level driver via regmap at some point. Not sure if this kind of
HW exists, though.

> Variant b) is IMHO of no practical relevance (but I may be wrong)
> because it would mean to add some costly FT232 or PL2302 chip
> where a different firmware variant works with direct USB
> connection.

Well for some chips there is not native USB support. But my scenario
was about development. Let's say I have a serial-chip and I want to
develop a driver for it. It would be nice if I can develop the
driver with a USB-UART and then use it on my embedded system.

There are usb-serial devices, which could benefit from support
btw. I would find it really useful, if the Dangerous Prototype's
Bus Pirate would expose native /dev/i2c and /dev/spi and it's
based on FT232.

> So to me it looks as if you need to develop different low-level
> drivers anyways.

No. You say, that option b) is irrelevant and assume, that every
serial chip also has native USB support.

-- Sebastian

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  parent reply	other threads:[~2016-08-22 22:42 UTC|newest]

Thread overview: 103+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-18  1:14 [RFC PATCH 0/3] UART slave device bus Rob Herring
2016-08-18  1:14 ` [RFC PATCH 1/3] uart bus: Introduce new bus for UART slave devices Rob Herring
2016-08-18  1:14 ` [RFC PATCH 2/3] tty: serial_core: make tty_struct optional Rob Herring
2016-08-18 10:50   ` Pavel Machek
2016-08-18  1:14 ` [RFC PATCH 3/3] tty: serial_core: add uart controller registration Rob Herring
     [not found] ` <20160818011445.22726-1-robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2016-08-18 10:22   ` [RFC PATCH 0/3] UART slave device bus Greg Kroah-Hartman
2016-08-18 10:30     ` Marcel Holtmann
2016-08-18 10:53       ` Greg Kroah-Hartman
2016-08-18 13:53         ` Rob Herring
2016-08-18 13:15     ` Rob Herring
     [not found]       ` <20160818160449.328b2eec@lxorguk.ukuu.org.uk>
2016-08-18 18:33         ` Rob Herring
2016-08-19 11:03           ` One Thousand Gnomes
     [not found]         ` <20160818160449.328b2eec-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org>
2016-08-25 16:40           ` Rob Herring
2016-08-26 13:12             ` One Thousand Gnomes
2016-08-18 10:39 ` H. Nikolaus Schaller
2016-08-18 10:47   ` Pavel Machek
2016-08-18 10:54     ` H. Nikolaus Schaller
2016-08-18 10:57       ` Greg Kroah-Hartman
     [not found]         ` <20160818105759.GA642-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2016-08-18 11:14           ` H. Nikolaus Schaller
2016-08-18 14:40           ` One Thousand Gnomes
2016-08-18 11:27     ` H. Nikolaus Schaller
2016-08-18 10:49   ` Marcel Holtmann
2016-08-18 10:55     ` Greg Kroah-Hartman
2016-08-18 11:01       ` Marcel Holtmann
2016-08-18 11:24         ` Greg Kroah-Hartman
     [not found]           ` <20160818112435.GA20876-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2016-08-18 11:42             ` Pavel Machek
2016-08-18 11:51             ` Marcel Holtmann
2016-08-18 11:10       ` Pavel Machek
2016-08-18 11:18         ` H. Nikolaus Schaller
2016-08-18 11:49           ` Marcel Holtmann
2016-08-18 12:16             ` H. Nikolaus Schaller
2016-08-18 11:47         ` Marcel Holtmann
2016-08-18 13:01           ` Pavel Machek
2016-08-18 11:02     ` H. Nikolaus Schaller
2016-08-18 11:41       ` Marcel Holtmann
     [not found]         ` <07FF6D1B-7B9B-441C-AFB9-E06AC5F469E2-kz+m5ild9QBg9hUCZPvPmw@public.gmane.org>
2016-08-18 12:07           ` H. Nikolaus Schaller
2016-08-18 11:02 ` Pavel Machek
2016-08-18 13:07 ` Linus Walleij
2016-08-18 17:31   ` Marcel Holtmann
2016-08-18 14:25 ` One Thousand Gnomes
2016-08-18 15:14   ` H. Nikolaus Schaller
     [not found]     ` <20160818163809.1b2fcfe5@lxorguk.ukuu.org.uk>
2016-08-18 18:31       ` H. Nikolaus Schaller
2016-08-18 22:25   ` Rob Herring
2016-08-19 11:38     ` One Thousand Gnomes
2016-08-19 15:36       ` Sebastian Reichel
2016-08-18 20:29 ` Sebastian Reichel
2016-08-18 23:08   ` Rob Herring
2016-08-19  5:21     ` Sebastian Reichel
2016-08-19  7:29       ` H. Nikolaus Schaller
     [not found]         ` <ACEF800E-8DB3-4345-ADE2-15C99F3659D5-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2016-08-19  7:49           ` Oleksij Rempel
2016-08-19 17:50             ` H. Nikolaus Schaller
     [not found]               ` <53A846F1-33E5-48C3-B3A6-DB251661CDD5-xXXSsgcRVICgSpxsJD1C4w@public.gmane.org>
2016-08-19 20:19                 ` Oleksij Rempel
2016-08-20 13:34               ` One Thousand Gnomes
2016-08-21  7:50                 ` H. Nikolaus Schaller
2016-08-22 20:39                   ` Sebastian Reichel
2016-08-22 21:23                     ` H. Nikolaus Schaller
2016-08-22 21:43                       ` Arnd Bergmann
2016-08-22 22:42                       ` Sebastian Reichel [this message]
2016-08-22 22:52                         ` One Thousand Gnomes
2016-08-22 23:10                           ` Sebastian Reichel
2016-08-23  7:28                         ` H. Nikolaus Schaller
2016-08-27 12:01                       ` Michal Suchanek
2016-08-19 11:06         ` One Thousand Gnomes
2016-08-19 17:42           ` H. Nikolaus Schaller
2016-08-20 13:22             ` One Thousand Gnomes
     [not found]               ` <20160820142226.6121e76d-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org>
2016-08-21  7:50                 ` H. Nikolaus Schaller
2016-08-21 17:09                   ` One Thousand Gnomes
2016-08-21 18:23                     ` H. Nikolaus Schaller
2016-08-22  9:09                       ` One Thousand Gnomes
     [not found]                         ` <20160822100936.19fd809f-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org>
2016-08-22  9:33                           ` Marcel Holtmann
2016-08-19 11:03       ` One Thousand Gnomes
2016-08-19 14:44         ` Sebastian Reichel
2016-08-22 12:37 ` Arnd Bergmann
2016-08-22 13:38   ` Rob Herring
2016-08-22 15:24     ` Arnd Bergmann
2016-08-22 15:28       ` Marcel Holtmann
2016-08-22 15:46         ` Arnd Bergmann
2016-08-22 15:45       ` One Thousand Gnomes
2016-08-22 21:07         ` Marcel Holtmann
2016-08-22 21:35           ` One Thousand Gnomes
2016-08-22 22:03           ` Sebastian Reichel
2016-08-22 22:46             ` One Thousand Gnomes
2016-08-22 23:41               ` Sebastian Reichel
2016-08-24 12:14         ` Linus Walleij
2016-08-22 16:44       ` Rob Herring
2016-08-22 17:02         ` One Thousand Gnomes
     [not found]           ` <20160822180254.5c95af7c-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org>
2016-08-22 17:30             ` Rob Herring
2016-08-22 17:38               ` One Thousand Gnomes
2016-08-22 21:16                 ` Marcel Holtmann
2016-08-22 21:32                   ` One Thousand Gnomes
2016-08-22 22:00                     ` Pavel Machek
2016-08-22 22:54                       ` One Thousand Gnomes
2016-08-22 23:57                         ` Sebastian Reichel
2016-08-23  0:15                           ` One Thousand Gnomes
2016-08-23  0:57                             ` Sebastian Reichel
2016-08-24 13:57                               ` One Thousand Gnomes
     [not found]                                 ` <20160824145714.02bbebb3-qBU/x9rampVanCEyBjwyrvXRex20P6io@public.gmane.org>
2016-08-24 14:29                                   ` Marcel Holtmann
2016-08-23 11:42                             ` Marcel Holtmann
2016-08-22 23:02                       ` Sebastian Reichel
2016-08-22 20:00               ` Sebastian Reichel
2016-08-22 22:00                 ` Rob Herring
2016-08-22 22:18                   ` Sebastian Reichel
2016-08-23 21:04       ` Rob Herring

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=20160822224247.vdh3plda6dxelvzw@earth \
    --to=sre@kernel.org \
    --cc=arnd@arndb.de \
    --cc=gnomes@lxorguk.ukuu.org.uk \
    --cc=gregkh@linuxfoundation.org \
    --cc=hns@goldelico.com \
    --cc=jslaby@suse.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-bluetooth@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=linux@rempel-privat.de \
    --cc=marcel@holtmann.org \
    --cc=neil@brown.name \
    --cc=pavel@ucw.cz \
    --cc=peter@hurleysoftware.com \
    --cc=robh@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox