Re: [PATCH] include linux/fs.h in linux/cdev.h for struct inode

2007-01-19 Thread Jan Engelhardt
>Subject: [PATCH] include linux/fs.h in linux/cdev.h for struct inode NAK. Better is this: Add missing struct predeclarations, otherwise we may get x.c:2: warning: ‘struct inode’ declared inside parameter list x.c:2: warning: its scope is only this definition or declaration, which is probably

Re: [PATCH] include linux/fs.h in linux/cdev.h for struct inode

2007-01-19 Thread Alexey Dobriyan
On Fri, Jan 19, 2007 at 12:54:07PM -0600, Noah Watkins wrote: > --- a/include/linux/cdev.h > +++ b/include/linux/cdev.h > @@ -5,6 +5,7 @@ > #include > #include > #include > +#include > > struct cdev { > struct kobject kobj; It is not for "struct inode", but to a pointer to struct ino

Re: [PATCH] include linux/fs.h in linux/cdev.h for struct inode

2007-01-19 Thread Randy Dunlap
On Fri, 19 Jan 2007 12:54:07 -0600 Noah Watkins wrote: > --- > include/linux/cdev.h |1 + > 1 files changed, 1 insertions(+), 0 deletions(-) > > diff --git a/include/linux/cdev.h b/include/linux/cdev.h > index f309b00..b53e2a0 100644 > --- a/include/linux/cdev.h > +++ b/include/linux/cdev.h