Am Freitag, 6. Januar 2006 20:33 schrieb Alan Stern:
> On Fri, 6 Jan 2006, Oliver Neukum wrote:
>
> > Am Freitag, 6. Januar 2006 19:38 schrieb Russell King:
> > > On Fri, Jan 06, 2006 at 07:15:43PM +0100, Oliver Neukum wrote:
> > > > Hi,
> > > >
> > > > is there any architecture for which packed is required in structures
> > > > like this:
> > > >
> > > > /* All standard descriptors have these 2 fields at the beginning */
> > > > struct usb_descriptor_header {
> > > > __u8 bLength;
> > > > __u8 bDescriptorType;
> > > > };
> > >
> > > sizeof(struct usb_descriptor_header) will be 4 on ARM. If this
> > > concerns you, you need to pack the structure thusly:
> > >
> > > struct usb_descriptor_header {
> > > __u8 bLength;
> > > __u8 bDescriptorType;
> > > } __attribute__((packed));
> >
> > I had feared so. The usb include files are incorrect for arm in this case.
> > Did I overdo it?
>
> Perhaps you are overreacting. Even though the total size of the structure
> may be 4, I bet that bLength and bDescriptorType still occupy just the
> first two bytes of the structure. Assuming this is so, there's nothing
> wrong with the existing headers.
Data will be copied directly adjacent to this structure.
Regards
Oliver
-------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc. Do you grep through log files
for problems? Stop! Download the new AJAX search engine that makes
searching your log files as easy as surfing the web. DOWNLOAD SPLUNK!
http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
