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  PHC 
Open Source and information security mailing list archives
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Mon, 8 Nov 2010 08:57:37 +0200
From:	Baruch Siach <>
To:	Greg KH <>
	Andrew Morton <>,
	Alex Gershgorin <>
Subject: Re: [PATCH] drivers/misc: Altera Cyclone active serial

Hi Greg,

On Sat, Nov 06, 2010 at 11:19:30AM -0700, Greg KH wrote:
> On Wed, Nov 03, 2010 at 04:21:35PM +0200, Baruch Siach wrote:
> > From: Alex Gershgorin <>
> > 
> > The active serial protocol can be used to program Altera Cyclone FPGA devices.
> > This driver uses the kernel gpio interface to implement the active serial
> > protocol.
> > 
> > Signed-off-by: Alex Gershgorin <>
> > Signed-off-by: Baruch Siach <>
> > ---


> > +static struct class *cyclone_as_class;
> Please don't create your own class just for a single driver.  Just use
> the misc class interface instead, as all you really want/need here is a
> character device node, right?

Searching for 'mist' under include/linux/ I couldn't find this "misc class 
interface". Can you enlighten me?

I did find the "miscdevices" interface in include/linux/miscdevice.h.  I also 
tried this one before going on to create a class of my own. However, this 
interface seems to be limited to singleton devices. Can I use 
MISC_DYNAMIC_MINOR to create multiple device nodes? Is there any example for 

> And as discussed at the Plumbers conference this past week, we don't
> want to add any new 'struct class' implementations to the kernel from
> now on, as it's the overall wrong thing to do.
> > --- /dev/null
> > +++ b/include/linux/cyclone_as.h
> Why do you need a .h file at all?

Look at the content of this file. I need to pass GPIO numbers from the 
platform code under arch/ to the driver. For this I use a platform_data 
struct, which must be visible to the platform code. Is there a better way to 
do this?


                                                     ~. .~   Tk Open Systems
   - - tel: +972.2.679.5364, -
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

Powered by blists - more mailing lists