On Fri, 16 Oct 2015, Takashi Iwai wrote:
The changes look OK, the only point to be fixed would be the use of __le32 for a variable as kbuild bot suggested.
Ok, however I interpreted the kbuild warning as that the output type of cpu_to_le32() is the same as the input type, so the resulting 'unsigned int' might not necessarily fit into the target u32 variable. So when the input type is an unsigned int, it should be cast to an u32 to be the same as the output type.
I looked around in various parts of the kernel code to see how cpu_to_le32() was used, and most cases have an u32 as the argument, and when not, the argument is simply cast to u32, i.e.
unsigned int length; u32 packet_length = cpu_to_le32((u32)length);
However, there are way too few description in the changelog. The patches 1-4 are only code shuffling, so it's OK (though, better to mention more explicitly that there is no functional change). But patches 5-6 are the core part, and this doesn't contain *why* this is needed. The irony is that you put so many good texts in the cover letter.
Same problem as with the single usb packet length fix I submitted a few days ago then. :-)
So, please make the changelogs more readable and enjoyable instead of the cover letter texts. The cover letter won't be included in the commits, after all.
Ok, I'll rework the changelogs over the next few days and resubmit the set.
Thanks!
/Ricard