All of lore.kernel.org
 help / color / mirror / Atom feed
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

             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.