On Thu, Jan 18, 2024 at 10:40:44AM -0500, Jacob Lott wrote:Checkpatch is not pointing out any issues when I run it:
The current help text is short and triggers aPlease wrap your lines at 72 columns.
warning from checkpatch.pl. This patch adds more
details to the help text which should provide better information
for whether or not to enable the driver.
Signed-off-by: Jacob Lott <jklott.git@xxxxxxxxx>No tab? Didn't checkpatch complain about this when you ran it on this
---
drivers/staging/rts5208/Kconfig | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/rts5208/Kconfig b/drivers/staging/rts5208/Kconfig
index b864023d3ccb..76a083fbe401 100644
--- a/drivers/staging/rts5208/Kconfig
+++ b/drivers/staging/rts5208/Kconfig
@@ -3,7 +3,11 @@ config RTS5208
tristate "Realtek PCI-E Card Reader RTS5208/5288 support"
depends on PCI && SCSI
help
- Say Y here to include driver code to support the Realtek
- PCI-E card reader rts5208/rts5288.
+ Choose Y here to enable support for the Realtek PCI-E card reader RTS5208/5288.
patch?
+ This driver facilitates communication between the Linux kernel and the RealtekWrap your lines properly here at 80 columns.
+ PCI-E card reader.
- If this driver is compiled as a module, it will be named rts5208.
+ If you opt to compile this driver as a module, it will be named rts5208. Selecting
+ N will exclude this driver from the kernel build. Choose option Y if your system includes
+ the Realtek PCI-E card reader rts5208/rts5288. When in doubt, it is generally safe
+ to select N.
thanks,
greg k-h