Hi Aaro,

On Mon, Nov 16, 2015 at 10:14:05PM +0200, Aaro Koskinen wrote:
> On Mon, Nov 16, 2015 at 06:20:51AM +0200, Baruch Siach wrote:
> > On Mon, Nov 16, 2015 at 02:10:53AM +0200, Aaro Koskinen wrote:
> > > @@ -144,6 +147,9 @@ int nandwrite_main(int argc UNUSED_PARAM, char **argv)
> > >   fd = xopen(argv[0], IS_NANDWRITE ? O_RDWR : O_RDONLY);
> > >   xioctl(fd, MEMGETINFO, &meminfo);
> > >  
> > > + if ((opts & OPT_n) && ioctl(fd, MTDFILEMODE, MTD_FILE_MODE_RAW))
> > > +         bb_simple_perror_msg_and_die("MTDFILEMODE");
> > 
> > Why not just use xioctl instead of open coding it?
> 
> xioctl() takes pointer argument.

So how about

   xioctl(fd, MTDFILEMODE, (void *)MTD_FILE_MODE_RAW);

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - [email protected] - tel: +972.2.679.5364, http://www.tkos.co.il -
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to