From: Julia Lawall <julia.lawall-L2FTfq7BK8M@public.gmane.org>
To: Codrut Grosu
<codrut.cristian.grosu-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: perex-/Fr2/VpizcU@public.gmane.org,
tiwai-IBi9RG/b67k@public.gmane.org,
lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org,
swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org,
thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
Julia.Lawall-L2FTfq7BK8M@public.gmane.org,
nicoleotsuka-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org,
alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH] ASoC: tegra: Remove unnecessary 'Block comments should align the * on each line' message
Date: Sat, 25 Feb 2017 19:57:01 +0100 (CET) [thread overview]
Message-ID: <alpine.DEB.2.20.1702251955110.2179@hadrien> (raw)
In-Reply-To: <1488048686-5964-1-git-send-email-codrut.cristian.grosu-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
The subject line is not very accurate. You don't remove the message, in
the sense of picking it up and putting it somewhere else. You adjust the
code in some way such that the error no longer occurs. So it would be
better to say what adjustment you have done. Something like line up *s in
block comments.
julia
On Sat, 25 Feb 2017, Codrut Grosu wrote:
> This was reported by checkpatch.pl
>
> Signed-off-by: Codrut Grosu <codrut.cristian.grosu-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
> sound/soc/tegra/tegra_alc5632.c | 2 +-
> sound/soc/tegra/tegra_rt5640.c | 2 +-
> sound/soc/tegra/tegra_rt5677.c | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/sound/soc/tegra/tegra_alc5632.c b/sound/soc/tegra/tegra_alc5632.c
> index 0509902..1f648f9 100644
> --- a/sound/soc/tegra/tegra_alc5632.c
> +++ b/sound/soc/tegra/tegra_alc5632.c
> @@ -1,5 +1,5 @@
> /*
> -* tegra_alc5632.c -- Toshiba AC100(PAZ00) machine ASoC driver
> + * tegra_alc5632.c -- Toshiba AC100(PAZ00) machine ASoC driver
> *
> * Copyright (C) 2011 The AC100 Kernel Team <ac100-oU9gvf+ajcSyy5GWsZ7R2qxOck334EZe@public.gmane.org>
> * Copyright (C) 2012 - NVIDIA, Inc.
> diff --git a/sound/soc/tegra/tegra_rt5640.c b/sound/soc/tegra/tegra_rt5640.c
> index 93a3568..cf706d0 100644
> --- a/sound/soc/tegra/tegra_rt5640.c
> +++ b/sound/soc/tegra/tegra_rt5640.c
> @@ -1,5 +1,5 @@
> /*
> -* tegra_rt5640.c - Tegra machine ASoC driver for boards using RT5640 codec.
> + * tegra_rt5640.c - Tegra machine ASoC driver for boards using RT5640 codec.
> *
> * Copyright (c) 2013, NVIDIA CORPORATION. All rights reserved.
> *
> diff --git a/sound/soc/tegra/tegra_rt5677.c b/sound/soc/tegra/tegra_rt5677.c
> index ebf58d0..0ed2eca 100644
> --- a/sound/soc/tegra/tegra_rt5677.c
> +++ b/sound/soc/tegra/tegra_rt5677.c
> @@ -1,5 +1,5 @@
> /*
> -* tegra_rt5677.c - Tegra machine ASoC driver for boards using RT5677 codec.
> + * tegra_rt5677.c - Tegra machine ASoC driver for boards using RT5677 codec.
> *
> * Copyright (c) 2014, The Chromium OS Authors. All rights reserved.
> *
> 1.9.1
>
>
prev parent reply other threads:[~2017-02-25 18:57 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-02-25 18:51 [PATCH] ASoC: tegra: Remove unnecessary 'Block comments should align the * on each line' message Codrut Grosu
[not found] ` <1488048686-5964-1-git-send-email-codrut.cristian.grosu-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-02-25 18:57 ` Julia Lawall [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=alpine.DEB.2.20.1702251955110.2179@hadrien \
--to=julia.lawall-l2ftfq7bk8m@public.gmane.org \
--cc=alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org \
--cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=codrut.cristian.grosu-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=nicoleotsuka-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=perex-/Fr2/VpizcU@public.gmane.org \
--cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=tiwai-IBi9RG/b67k@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox