linux-kernel - Re: [PATCH 4/4] Add support for SUNIX Multi-I/O board

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Date:   Wed, 3 Apr 2019 04:40:53 +0100
From:   Lee Jones <lee.jones@...aro.org>
To:     Morris Ku 古文俊 <morris_ku@...ix.com>
Cc:     linux-kernel@...r.kernel.org
Subject: Re: [PATCH 4/4] Add support for SUNIX Multi-I/O board

Please do not drop the list from your recipients.

On Tue, 02 Apr 2019, Morris Ku 古文俊 wrote:

> Hello , Lee ,
> 
> 
> Sunix multi-io card is a composite device,
> contains serial and parallel port , before we
> trying to place the driver in the /drivers/char,
> but maintainer suggests go to /driver/mfd,so we move to here.
> this driver is special,it supports more than one interface,
> so we combine to a single driver.

The MFD subsystem is designed to register child devices.  MFD drivers
do not contain any true functionality themselves.  If this device does
multiple things, you can split the functionality into their relevant
subsystems and register each of them from here.

There are 10's of examples available.  Please take a look around some
of the existing drivers to see how their functionality is split.

> On Tue, 19 Mar 2019, Morris Ku wrote:
> 
> > Driver for SUNIX Multi-I/O card.Based on parport_pc.c, ppdev.c
> > and lp.c by Linus Torvalds, Theodore Ts'o.
> 
> Parallel port drivers should live in 'drivers/parport' and
> 'drivers/char'.  LP drivers should live in 'drivers/char'.
> 
> Please them there.
> 
> > Signed-off-by: Morris Ku <saumah@...il.com>
> > ---
> >  mfd/sunix/snx_ieee1284.c     | 144 +++++++
> >  mfd/sunix/snx_ieee1284_ops.c | 258 +++++++++++++
> >  mfd/sunix/snx_lp.c           | 717 +++++++++++++++++++++++++++++++++++
> >  mfd/sunix/snx_lp.h           | 119 ++++++
> >  mfd/sunix/snx_parallel.c     | 397 +++++++++++++++++++
> >  mfd/sunix/snx_ppdev.c        | 454 ++++++++++++++++++++++
> >  mfd/sunix/snx_ppdev.h        |  15 +
> >  mfd/sunix/snx_share.c        | 629 ++++++++++++++++++++++++++++++
> >  8 files changed, 2733 insertions(+)
> >  create mode 100644 mfd/sunix/snx_ieee1284.c
> >  create mode 100644 mfd/sunix/snx_ieee1284_ops.c
> >  create mode 100644 mfd/sunix/snx_lp.c
> >  create mode 100644 mfd/sunix/snx_lp.h
> >  create mode 100644 mfd/sunix/snx_parallel.c
> >  create mode 100644 mfd/sunix/snx_ppdev.c
> >  create mode 100644 mfd/sunix/snx_ppdev.h
> >  create mode 100644 mfd/sunix/snx_share.c
> 

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog