> + for (i = 0, csum = 0; i < hdr->data.size; i++)
> + csum += data[i];
> + for (sect = fw->data; (sect + sizeof(hdr)) < (fw->data + fw->size);) {
> + memcpy(&hdr, sect, sizeof(hdr));
> + hdr.data.size = cpu_to_le32(hdr.data.size);
hdr.data.size is little endian. Doing a for loop using a little endian
test seems wrong. Should this actually be le32_to_cpu()?
Andrew
