Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753700AbbK3IsX (ORCPT ); Mon, 30 Nov 2015 03:48:23 -0500 Received: from sabertooth02.qualcomm.com ([65.197.215.38]:25660 "EHLO sabertooth02.qualcomm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753257AbbK3IsU (ORCPT ); Mon, 30 Nov 2015 03:48:20 -0500 X-IronPort-AV: E=McAfee;i="5700,7163,8000"; a="103034418" X-IronPort-AV: E=Sophos;i="5.20,364,1444719600"; d="scan'208";a="1045253146" From: Kalle Valo To: Brent Taylor CC: , , , Subject: Re: [PATCH v2] ath6kl: Use vmalloc for loading firmware using api1 method and use kvfree References: <1444972259-24260-1-git-send-email-motobud@gmail.com> <1448774038-26984-1-git-send-email-motobud@gmail.com> Date: Mon, 30 Nov 2015 10:48:03 +0200 In-Reply-To: <1448774038-26984-1-git-send-email-motobud@gmail.com> (Brent Taylor's message of "Sat, 28 Nov 2015 23:13:58 -0600") Message-ID: <87zixvlvp8.fsf@kamboji.qca.qualcomm.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" X-Originating-IP: [10.80.80.8] X-ClientProxiedBy: NASANEXM01E.na.qualcomm.com (10.85.0.31) To euamsexm01a.eu.qualcomm.com (10.251.127.40) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 840 Lines: 24 Brent Taylor writes: > Signed-off-by: Brent Taylor > > 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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/