Mark Brown wrote at Wednesday, November 23, 2011 2:32 PM:
On Wed, Nov 23, 2011 at 12:42:05PM -0700, Stephen Warren wrote:
v2: Rename flag from auto_nc_pins to fully_routed. Adjust description.
Signed-off-by: Stephen Warren swarren@nvidia.com
sound/soc/tegra/tegra_wm8903.c | 22 +--------------------- 1 files changed, 1 insertions(+), 21 deletions(-)
Things like your above "v2" line really ought to go here below the --- so they're automatically cut from the changelog by the tools.
That was deliberate; some advantages: * Documents the review process in git, potentially giving some insight into reasoning behind parts of the patch. * Allows confirmation that the correct version was checked in. (rarely an issue, but occasionally, albeit I haven't seen it in ASoC)
It's been mentioned a couple of times briefly on various lists before, and while I'm sure it's not standard practice or anything, at least there are some people supporting it. However, if you prefer, I'll try to remember to move the changelog for ASoC patches.