From: Daniel Grimshaw <grimshaw@linux.vnet.ibm.com>
To: gregkh@linuxfoundation.org
Cc: benchan@chromium.org, rafael.j.wysocki@intel.com, tj@kernel.org,
devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org,
trivial@kernel.org
Subject: [PATCH] staging: gdm72xx: Split Kconfig descriptions into more lines
Date: Thu, 25 Jun 2015 15:36:46 -0700 [thread overview]
Message-ID: <558C827E.1020105@linux.vnet.ibm.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 468 bytes --]
I am a high school student trying to become familiar with linux kernel
development. This is my first patch.
checkpatch.pl throws a warning that config WIMAX_GDM72XX_QOS, config
WIMAX_GDM72XX_K_MODE, and config WIMAX_GDM72XX_USB do not have enough
of a description. By splitting the current description into multiple
lines, the warning is removed.
The module still worked with these changes when tested.
Signed-off-by: Daniel Grimshaw <grimshaw@linux.vnet.ibm.com>
[-- Attachment #2: patchfile --]
[-- Type: text/plain, Size: 1047 bytes --]
diff --git a/drivers/staging/gdm72xx/Kconfig b/drivers/staging/gdm72xx/Kconfig
index bf11a7f..bfbe3d6 100644
--- a/drivers/staging/gdm72xx/Kconfig
+++ b/drivers/staging/gdm72xx/Kconfig
@@ -14,14 +14,19 @@ config WIMAX_GDM72XX_QOS
bool "Enable QoS support"
default n
help
- Enable Quality of Service support based on the data protocol of
+ Enable Quality of
+ Service support based
+ on the data protocol of
transmitting packets.
config WIMAX_GDM72XX_K_MODE
bool "Enable K mode"
default n
help
- Enable support for proprietary functions for KT (Korea Telecom).
+ Enable support
+ for proprietary
+ functions for KT
+ (Korea Telecom).
config WIMAX_GDM72XX_WIMAX2
bool "Enable WiMAX2 support"
@@ -37,7 +42,9 @@ config WIMAX_GDM72XX_USB
bool "USB interface"
depends on (USB = y || USB = WIMAX_GDM72XX)
help
- Select this option if the WiMAX module interfaces with the host
+ Select this option
+ if the WiMAX module
+ interfaces with the host
processor via USB.
config WIMAX_GDM72XX_SDIO
next reply other threads:[~2015-06-25 22:36 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-25 22:36 Daniel Grimshaw [this message]
2015-06-26 7:37 ` [PATCH] staging: gdm72xx: Split Kconfig descriptions into more lines Dan Carpenter
2015-06-26 11:29 ` Luis de Bethencourt
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=558C827E.1020105@linux.vnet.ibm.com \
--to=grimshaw@linux.vnet.ibm.com \
--cc=benchan@chromium.org \
--cc=devel@driverdev.osuosl.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=rafael.j.wysocki@intel.com \
--cc=tj@kernel.org \
--cc=trivial@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.