Re: [JANITOR-PATCH] rtc: switch to unlocked_ioctl

[Andi Kleen <ak@suse.de>]

On Tuesday 04 March 2008 23:58:01 Jiri Slaby wrote:
> Harvey Harrison napsal(a):
> > Make the lock/unlock_kernel explicit.
> > 
> > Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
> > ---
> >  arch/arm/common/rtctime.c |    7 ++++---
> >  1 files changed, 4 insertions(+), 3 deletions(-)
> > 
> > diff --git a/arch/arm/common/rtctime.c b/arch/arm/common/rtctime.c
> > index f53bca4..2062f92 100644
> > --- a/arch/arm/common/rtctime.c
> > +++ b/arch/arm/common/rtctime.c
> > @@ -173,8 +173,7 @@ static unsigned int rtc_poll(struct file *file, poll_table *wait)
> >  	return data != 0 ? POLLIN | POLLRDNORM : 0;
> >  }
> >  
> > -static int rtc_ioctl(struct inode *inode, struct file *file, unsigned int cmd,
> > -		     unsigned long arg)
> > +static int rtc_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
> 
> long? I guess you need to get a cross-compiler...

Yes please do these transformations only for code that you can compile.

One easy way to do that is to do an allyesconfig build once and save the log (or 
fetch http://halobates.de/allyes/i386-2.6.24-rc6-git12 and 
http://halobates.de/allyes/x86_64-2.6.24-rc6-git12 as an older example) 
and always check if the file you're changing is compiled in the log.

Then after changing do at least a make allyesconfig ; make path/to/file.o

-Andi