[lkml]   [2016]   [Oct]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] char/pcmcia: add scr24x_cs chip card interface driver
On Monday, October 10, 2016 7:33:45 PM CEST Greg Kroah-Hartman wrote:
> > +
> > +#define SCR24X_DEVS 8
> > +static DECLARE_BITMAP(scr24x_minors, SCR24X_DEVS);
> > +
> > +static struct class *scr24x_class;
> > +dev_t scr24x_devt;
> global variable? And why do you need a char device for this type of
> hardware? Isn't there already an existing interface for this device
> class?

It looks like this is modeled after the other two chip card drivers
in drivers/char/pcmcia/, which are also char devices. Looking at, it seems that other
drivers for this protocol are typically in user space when they are
for USB devices, so apparently no existing subsystem abstraction
in the kernel.

This reminds me that there is just one other driver in drivers/char/pcmcia
(synclink_cs), and that should be moved to drivers/tty/ with the other
synclink variants. If we do this, we could rename drivers/char/pcmcia
to something that implies "smartcard".


 \ /
  Last update: 2016-10-10 21:25    [W:0.079 / U:0.424 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site