On Mon, Nov 26, 2012 at 02:23:56PM -0600, Kent Yoder wrote: > On Wed, Nov 21, 2012 at 11:50:24PM +0100, Peter Huewe wrote: > > Am Mittwoch, 21. November 2012, 21:56:45 schrieb Jason Gunthorpe: > > > This seems to be preferred these days. > > > > > > Signed-off-by: Jason Gunthorpe <jguntho...@obsidianresearch.com> > > > > Looks good to me > > Reviewed-by: Peter Huewe <peterhu...@gmx.de> > > Applied. I also rolled in an update to tpm_acpi.c to this commit: > > diff --git a/drivers/char/tpm/tpm_acpi.c b/drivers/char/tpm/tpm_acpi.c > index 56051d0..64420b3 100644 > +++ b/drivers/char/tpm/tpm_acpi.c > @@ -33,13 +33,13 @@ struct acpi_tcpa { > u16 platform_class; > union { > struct client_hdr { > - u32 log_max_len __attribute__ ((packed)); > - u64 log_start_addr __attribute__ ((packed)); > + u32 log_max_len __packed; > + u64 log_start_addr __packed; > } client; > struct server_hdr { > u16 reserved; > - u64 log_max_len __attribute__ ((packed)); > - u64 log_start_addr __attribute__ ((packed)); > + u64 log_max_len __packed; > + u64 log_start_addr __packed; > } server; > }; > };
Thanks, I looked at those as well, but I couldn't grok the intent. What does __packed on a member even do?? Certainly client_hdr should be moved to the struct, maybe server_hdr too? Jason -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/