Re: PATCH: usb: descriptor structures have to be packed

Oliver Neukum <[email protected]>
Newsgroups gmane.linux.usb.devel
Message-ID <[email protected]>
Am Mittwoch, 7. Februar 2007 18:31 schrieb Pete Zaitcev:
> On Wed, 7 Feb 2007 14:39:28 +0100, Oliver Neukum <[email protected]> wrote:
> > Am Mittwoch, 7. Februar 2007 14:31 schrieb Sergei Organov:
> 
> > >   static inline unsigned usb_get_uint16(__u8 const *p)
> > >   {
> > >     return p[0] | (p[1] << 8);
> > >   }
> > 
> > It makes little sense to reinvent le16_to_cpu()
> 
> It would not, if le16_to_cpu took a pointer to u8. But as it is,
> functions are quite different.

That's what casts are for. However, why do you have no type information
in the first place? Untyped data is risky.

	Regards
		Oliver

-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.