2008-12-03 15:23:06

by Jiri Slaby

[permalink] [raw]
Subject: [PATCH 1/1 #2] Char: n_tty, include module.h

Hi,

this is a ldisc-n_tty-export-all-n_tty-ldisc-methods.patch fix.

--

Since we are using EXPORT_SYMBOL, we should include linux/module.h

Signed-off-by: Jiri Slaby <[email protected]>
Cc: Alan Cox <[email protected]>
Cc: Rodolfo Giometti <[email protected]>
---
drivers/char/n_tty.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/char/n_tty.c b/drivers/char/n_tty.c
index b441c09..8e6c591 100644
--- a/drivers/char/n_tty.c
+++ b/drivers/char/n_tty.c
@@ -41,6 +41,7 @@
#include <linux/timer.h>
#include <linux/ctype.h>
#include <linux/mm.h>
+#include <linux/module.h>
#include <linux/string.h>
#include <linux/slab.h>
#include <linux/poll.h>
--
1.6.0.4


2008-12-03 15:35:59

by Alan

[permalink] [raw]
Subject: Re: [PATCH 1/1 #2] Char: n_tty, include module.h

On Wed, 3 Dec 2008 16:21:51 +0100
Jiri Slaby <[email protected]> wrote:

> Hi,
>
> this is a ldisc-n_tty-export-all-n_tty-ldisc-methods.patch fix.

NAK and NAK the patch needing it.

Once PPS is in this can be revisted by adding the almost identical N_PPS
ldisc into n_tty with a couple of different handlers (notably some of the
termios flags need to be handled differently)

2008-12-03 20:34:59

by Andrew Morton

[permalink] [raw]
Subject: Re: [PATCH 1/1 #2] Char: n_tty, include module.h

On Wed, 3 Dec 2008 15:35:37 +0000
Alan Cox <[email protected]> wrote:

> On Wed, 3 Dec 2008 16:21:51 +0100
> Jiri Slaby <[email protected]> wrote:
>
> > Hi,
> >
> > this is a ldisc-n_tty-export-all-n_tty-ldisc-methods.patch fix.
>
> NAK and NAK the patch needing it.

Late :(

> Once PPS is in

PPS can't go in without this?

Perhaps I can drop pps-serial-clients-support.patch, dunno.

> this can be revisted by adding the almost identical N_PPS
> ldisc into n_tty with a couple of different handlers (notably some of the
> termios flags need to be handled differently)

2008-12-03 21:10:56

by Alan

[permalink] [raw]
Subject: Re: [PATCH 1/1 #2] Char: n_tty, include module.h

> > NAK and NAK the patch needing it.
>
> Late :(
>
> > Once PPS is in
>
> PPS can't go in without this?

PPS stands alone without the tty parts. Once the core PPS bits are
accepted (Ingo seems to be silent on that ?) I'll redo the N_PPS tty
bits.

2008-12-04 08:41:21

by Rodolfo Giometti

[permalink] [raw]
Subject: Re: [PATCH 1/1 #2] Char: n_tty, include module.h

On Wed, Dec 03, 2008 at 12:06:43PM -0800, Andrew Morton wrote:
> On Wed, 3 Dec 2008 15:35:37 +0000
> Alan Cox <[email protected]> wrote:
>
> > On Wed, 3 Dec 2008 16:21:51 +0100
> > Jiri Slaby <[email protected]> wrote:
> >
> > > Hi,
> > >
> > > this is a ldisc-n_tty-export-all-n_tty-ldisc-methods.patch fix.
> >
> > NAK and NAK the patch needing it.
>
> Late :(
>
> > Once PPS is in
>
> PPS can't go in without this?

No, PPS works perfectly without that patch.

> Perhaps I can drop pps-serial-clients-support.patch, dunno.

I think you can drop the last fix.

Ciao,

Rodolfo

--

GNU/Linux Solutions e-mail: [email protected]
Linux Device Driver [email protected]
Embedded Systems phone: +39 349 2432127
UNIX programming skype: rodolfo.giometti