[PATCH] cpukit: Add nxp-sc16is752 serial device driver

Sebastian Huber sebastian.huber at embedded-brains.de
Mon Sep 19 16:13:25 UTC 2016


Hello Joel,

----- Joel Sherrill <joel.sherrill at gmail.com> schrieb:
> Sebastian,
> 
> This is yet another style of serial port driver in recent history and
> the software tax hasn't been paid on the last round of changes.

this is not a new style of serial port drivers.  It uses the new Termios device driver model which I changed in the last patch set to use IMFS generic nodes.  This particular serial device (NXP SC16IS752) is attached via an I2C or SPI bus.  So, this is highly board/application specific.  Adding such a device to a particular BSP would end up in a configuration nightmare.

> Merging this will result in three different styles of serial port drivers
> by my count.
> 
> I am not opposed to improvement but this is rapidly heading
> down the "old vs new PowerPC interrupt" path and that sucked
> for years.

With the recent change for the new Termios device driver model its easier to move the existing console drivers, since there is no need to maintain major/minor numbers anymore.  The drawback is that you have to use IMFS as the base file system.  However, its just a matter of doing it to strip down the IMFS to have similar size demands than the current devfs.

> 
> --joel
> 
> 
> On Mon, Sep 19, 2016 at 8:57 AM, Gedare Bloom <gedare at rtems.org> wrote:
> 
> > It will be good to have some documentation (BSP/driver manual) about
> > how to write these kind of termios drivers. Usually without it we will
> > see someone just copy-paste this one and modify until they get it
> > right... So lacking a plain english doc, this "first" implementation
> > should provide enough comments to make it easy to copy-paste-modify.
> >
> > On Mon, Sep 19, 2016 at 4:11 AM, Sebastian Huber
> > <sebastian.huber at embedded-brains.de> wrote:
> > > Please note that this is the first Termios device driver that is
> > independent
> > > of a BSP, CPU architecture and console driver. It is currently work in
> > > progress and lacks support for interrupts. We will add this probably in
> > one
> > > or two weeks.
> > >
> > > --
> > > Sebastian Huber, embedded brains GmbH
> > >
> > > Address : Dornierstr. 4, D-82178 Puchheim, Germany
> > > Phone   : +49 89 189 47 41-16
> > > Fax     : +49 89 189 47 41-09
> > > E-Mail  : sebastian.huber at embedded-brains.de
> > > PGP     : Public key available on request.
> > >
> > > Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.
> > >
> > >
> > > _______________________________________________
> > > devel mailing list
> > > devel at rtems.org
> > > http://lists.rtems.org/mailman/listinfo/devel
> > _______________________________________________
> > devel mailing list
> > devel at rtems.org
> > http://lists.rtems.org/mailman/listinfo/devel
> >

-- 
Sebastian Huber, embedded brains GmbH

Address : Dornierstr. 4, D-82178 Puchheim, Germany
Phone   : +49 89 189 47 41-16
Fax     : +49 89 189 47 41-09
E-Mail  : sebastian.huber at embedded-brains.de
PGP     : Public key available on request.

Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.



More information about the devel mailing list