Re: [PATCH v2] ath6kl: Use vmalloc for loading firmware using api1 method and use kvfree

From: Kalle Valo
Date: Mon Nov 30 2015 - 03:48:26 EST


Brent Taylor <motobud@xxxxxxxxx> writes:

> Signed-off-by: Brent Taylor <motobud@xxxxxxxxx>
>
> ath6kl: Use vmalloc for loading firmware using api1 method and free using kvfree
> ---
> Changes v1 -> v2:
> - simplify memory allocation
> - use kvfree

Why? The commit log should _always_ answer that. Are you fixing a bug
(what bug exactly?), is this just cleanup or what?

And the commit log is wrongly formatted anyway, the Signed-off-by line
should be the last and there should be no "ath6kl:" string in the commit
log (just in the title). Use 'git log' to find examples.

--
Kalle Valo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/