On Thu, 29 Mar 2007, Tony Lindgren wrote:

You left out part of my suggestion (setting the pointer to NULL):

> > > +#ifndef CONFIG_HIGHMEM
> > >                   io->urbs [i]->transfer_buffer =
> > >                           page_address (sg [i].page) + sg [i].offset;
> > 
> > Insert here:
> > 
> > +#else
> > +                   io->urbs[i]->transfer_buffer = NULL;

^^^ You didn't do this.

> > 
> > > +#endif
> > > +         } else {
> > >                   len = sg [i].length;
> > > +                 io->urbs [i]->transfer_buffer =
> > > +                         page_address (sg [i].page) + sg [i].offset;
> > >           }

Alan Stern


-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to