LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Gaurav Dhingra <gauravdhingra.gxyd@gmail.com>
To: mgreer@animalcreek.com, johan@kernel.org, elder@kernel.org,
gregkh@linuxfoundation.org
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH] staging: greybus: Fix warning to limit chars per line
Date: Thu, 5 Apr 2018 12:47:19 +0530 [thread overview]
Message-ID: <9c77723c-dc3f-2a86-b7f3-be3b751fdfe2@gmail.com> (raw)
In-Reply-To: <20180403183246.GA23438@stallman>
Hi,
Can anyone please review this patchset? Since this is my first patch
set, so I felt it was okay to make change even in the comment, though
the patch set fixes the warning. Please let me know in anycase your view
on the patchset.
On Wednesday 04 April 2018 12:02 AM, Gaurav Dhingra wrote:
> Wrap comment to fix warning "prefer a maximum 75 chars per line"
>
> Signed-off-by: Gaurav Dhingra <gauravdhingra.gxyd@gmail.com>
> ---
> drivers/staging/greybus/audio_codec.h | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/staging/greybus/audio_codec.h b/drivers/staging/greybus/audio_codec.h
> index a1d5440..01838d9 100644
> --- a/drivers/staging/greybus/audio_codec.h
> +++ b/drivers/staging/greybus/audio_codec.h
> @@ -23,7 +23,9 @@ enum {
> NUM_CODEC_DAIS,
> };
>
> -/* device_type should be same as defined in audio.h (Android media layer) */
> +/* device_type should be same as defined in audio.h
> + * (Android media layer)
> + */
> enum {
> GBAUDIO_DEVICE_NONE = 0x0,
> /* reserved bits */
--
Gaurav Dhingra
(sent from Thunderbird email client)
next prev parent reply other threads:[~2018-04-05 7:17 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-03 18:32 Gaurav Dhingra
2018-04-05 7:17 ` Gaurav Dhingra [this message]
2018-04-05 12:37 ` Greg KH
2018-04-05 22:19 ` Mark Greer
2018-04-06 5:22 ` Viresh Kumar
2018-04-06 10:17 ` Gaurav Dhingra
2018-04-06 10:25 ` Viresh Kumar
2018-04-06 10:26 ` Viresh Kumar
2018-04-06 10:29 ` Gaurav Dhingra
2018-04-06 16:01 ` Mark Greer
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=9c77723c-dc3f-2a86-b7f3-be3b751fdfe2@gmail.com \
--to=gauravdhingra.gxyd@gmail.com \
--cc=elder@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=johan@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mgreer@animalcreek.com \
--subject='Re: [PATCH] staging: greybus: Fix warning to limit chars per line' \
/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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).