Hi Marko,

On 28/11/2017 1:22 PM, Marko Kovacevic wrote:

--snip--

int
-guest_channel_host_connect(const char *path, unsigned lcore_id)
+guest_channel_host_connect(const char *path, unsigned int lcore_id)

--snip--

I'd suggest adding the following into the commit message. It would help explain why you're doing all the "unsigned" to "unsigned int" changes up front rather than including them in the other patches.

"Since this patch-set attempts to clean up the power library, and there are many instances of "unsigned" caught by checkpatch, it was decided to clean these up first rather than have them included in the later patches in the patch set. And would also minimise this type of error being caught by checkpatch in the future."

Regards,
Dave.

Reply via email to