linux-kernel - Re: [PATCH] Fix broken compilation of Cris serial driver

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20090408140003.GK8115@axis.com>
Date:	Wed, 8 Apr 2009 16:00:04 +0200
From:	Jesper Nilsson <Jesper.Nilsson@...s.com>
To:	Alexey Dobriyan <adobriyan@...il.com>
Cc:	Claudio Scordino <claudio@...dence.eu.com>,
	Mikael Starvik <mikael.starvik@...s.com>,
	Alan Cox <alan@...rguk.ukuu.org.uk>,
	Hinko Kocevar <hinko.kocevar@...rtapot.si>,
	Janez Cufer <janez.cufer@...rtapot.si>,
	Andrew Morton <akpm@...ux-foundation.org>,
	Linux Kernel <linux-kernel@...r.kernel.org>,
	dev-etrax <dev-etrax@...s.com>
Subject: Re: [PATCH] Fix broken compilation of Cris serial driver

On Wed, Apr 08, 2009 at 02:51:22PM +0200, Alexey Dobriyan wrote:
> On Wed, Apr 08, 2009 at 12:57:01PM +0200, Jesper Nilsson wrote:
> > On Wed, Apr 08, 2009 at 12:07:16PM +0200, Claudio Scordino wrote:
> > >     compilation of cris serial driver on 2.6.30-rc1 is broken since 
> > > commit number 0f043a81ebe84be3576667f04fdda481609e3816 removed the 
> > > read_proc field from the tty_driver structure.
> > > 
> > > The patch in attachment may fix the problem. Please check it very 
> > > carefully: I don't have hardware to make any test.
> 
> > > -	.read_proc = rs_read_proc,
> > >  	.tiocmget = rs_tiocmget,
> > >  	.tiocmset = rs_tiocmset
> > >  };
> > > @@ -4490,6 +4490,7 @@ rs_init(void)
> > >  	if (tty_register_driver(driver))
> > >  		panic("Couldn't register serial driver\n");
> > >  	/* do some initializing for the separate ports */
> > > +	driver->proc_entry->read_proc = rs_read_proc;
> > 
> > Is this the recommended way to convert proc handling?
> 
> No.

Ok, that's what I thought.

> > I find no other serial driver that does it this way...
> 
> ->read_proc effectively went away.

...so what do we use as a replacement, is it .proc_fops
or anything else?

/^JN - Jesper Nilsson
-- 
               Jesper Nilsson -- jesper.nilsson@...s.com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/