17 Jan
2013
17 Jan
'13
6:58 a.m.
On Wed, 16 Jan 2013 12:49:37 +0100 Takashi Iwai tiwai@suse.de wrote:
At Wed, 16 Jan 2013 17:28:32 +0900, Yoichi Yuasa wrote:
+static void bdsp_check_sin_control_sel(struct aec_bdsp_info *bdsp, u8 *data) +{
- u32 addr;
- u32 size;
- addr = htonl(*(u32 *) data);
- size = htonl(*(u32 *) (data + 5));
It's an unaligned access.
The size is unaligned. I'll add a comment.
And normally we use cpu_to_xxx() or xxx_to_cpu() macros for endian conversions.
I'll update them.
Thank you for your comment.
Yoichi