16 Jan
                
                    2013
                
            
            
                16 Jan
                
                '13
                
            
            
            
        
    
                12:49 p.m.
            
        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. And normally we use cpu_to_xxx() or xxx_to_cpu() macros for endian conversions.
Takashi