* [PATCH] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 6:20 ` sehro rautenkranz
0 siblings, 0 replies; 22+ messages in thread
From: sehro rautenkranz @ 2018-09-25 6:20 UTC (permalink / raw)
To: lidza-louina
Cc: markh, driverdev-devel, devel, linux-kernel, sehro rautenkranz
Replace Kconfig help header and text to match other modules. Issue found
by checkpatch.
Signed-off-by: sehro rautenkranz <sehro@sehro.org>
---
drivers/staging/dgnc/Kconfig | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
index 032c2a795238..c845b357b53d 100644
--- a/drivers/staging/dgnc/Kconfig
+++ b/drivers/staging/dgnc/Kconfig
@@ -2,5 +2,9 @@ config DGNC
tristate "Digi Neo and Classic PCI Products"
default n
depends on TTY && PCI
- ---help---
- Driver for the Digi International Neo and Classic PCI based product line.
+ help
+ Say Y here to enable support for the Digi International Neo and
+ Classic PCI based product line.
+
+ To compile this driver as a module, say M here: the module will be
+ called dgnc
--
2.19.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 6:20 ` sehro rautenkranz
0 siblings, 0 replies; 22+ messages in thread
From: sehro rautenkranz @ 2018-09-25 6:20 UTC (permalink / raw)
To: lidza-louina
Cc: markh, devel, driverdev-devel, linux-kernel, sehro rautenkranz
Replace Kconfig help header and text to match other modules. Issue found
by checkpatch.
Signed-off-by: sehro rautenkranz <sehro@sehro.org>
---
drivers/staging/dgnc/Kconfig | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
index 032c2a795238..c845b357b53d 100644
--- a/drivers/staging/dgnc/Kconfig
+++ b/drivers/staging/dgnc/Kconfig
@@ -2,5 +2,9 @@ config DGNC
tristate "Digi Neo and Classic PCI Products"
default n
depends on TTY && PCI
- ---help---
- Driver for the Digi International Neo and Classic PCI based product line.
+ help
+ Say Y here to enable support for the Digi International Neo and
+ Classic PCI based product line.
+
+ To compile this driver as a module, say M here: the module will be
+ called dgnc
--
2.19.0
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH] staging: dgnc: Fix Kconfig help header and text
2018-09-25 6:20 ` sehro rautenkranz
@ 2018-09-25 15:34 ` Randy Dunlap
-1 siblings, 0 replies; 22+ messages in thread
From: Randy Dunlap @ 2018-09-25 15:34 UTC (permalink / raw)
To: sehro rautenkranz, lidza-louina
Cc: markh, driverdev-devel, devel, linux-kernel
On 9/24/18 11:20 PM, sehro rautenkranz wrote:
> Replace Kconfig help header and text to match other modules. Issue found
> by checkpatch.
>
> Signed-off-by: sehro rautenkranz <sehro@sehro.org>
> ---
Hi,
Documentation/process/coding-style.rst says:
Lines under a ``config`` definition
are indented with one tab, while help text is indented an additional two
spaces.
> drivers/staging/dgnc/Kconfig | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
> index 032c2a795238..c845b357b53d 100644
> --- a/drivers/staging/dgnc/Kconfig
> +++ b/drivers/staging/dgnc/Kconfig
> @@ -2,5 +2,9 @@ config DGNC
> tristate "Digi Neo and Classic PCI Products"
> default n
> depends on TTY && PCI
> - ---help---
> - Driver for the Digi International Neo and Classic PCI based product line.
> + help
> + Say Y here to enable support for the Digi International Neo and
> + Classic PCI based product line.
> +
> + To compile this driver as a module, say M here: the module will be
> + called dgnc
>
thanks,
--
~Randy
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 15:34 ` Randy Dunlap
0 siblings, 0 replies; 22+ messages in thread
From: Randy Dunlap @ 2018-09-25 15:34 UTC (permalink / raw)
To: sehro rautenkranz, lidza-louina
Cc: markh, devel, driverdev-devel, linux-kernel
On 9/24/18 11:20 PM, sehro rautenkranz wrote:
> Replace Kconfig help header and text to match other modules. Issue found
> by checkpatch.
>
> Signed-off-by: sehro rautenkranz <sehro@sehro.org>
> ---
Hi,
Documentation/process/coding-style.rst says:
Lines under a ``config`` definition
are indented with one tab, while help text is indented an additional two
spaces.
> drivers/staging/dgnc/Kconfig | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
> index 032c2a795238..c845b357b53d 100644
> --- a/drivers/staging/dgnc/Kconfig
> +++ b/drivers/staging/dgnc/Kconfig
> @@ -2,5 +2,9 @@ config DGNC
> tristate "Digi Neo and Classic PCI Products"
> default n
> depends on TTY && PCI
> - ---help---
> - Driver for the Digi International Neo and Classic PCI based product line.
> + help
> + Say Y here to enable support for the Digi International Neo and
> + Classic PCI based product line.
> +
> + To compile this driver as a module, say M here: the module will be
> + called dgnc
>
thanks,
--
~Randy
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH] staging: dgnc: Fix Kconfig help header and text
2018-09-25 15:34 ` Randy Dunlap
@ 2018-09-25 15:53 ` sehro rautenkranz
-1 siblings, 0 replies; 22+ messages in thread
From: sehro rautenkranz @ 2018-09-25 15:53 UTC (permalink / raw)
Cc: sehro, Lidza Louina, Mark Hounschell, Greg Kroah-Hartman,
driverdev-devel, devel, linux-kernel
Replace Kconfig help header and text to match other modules. Issue found
my checkpatch.
Spacing adjusted to match Documentation/process/coding-style.rst. Issue
reported by rdunlap@infradead.org.
Signed-off-by: sehro rautenkranz <sehro@sehro.org>
---
drivers/staging/dgnc/Kconfig | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
index c845b357b53d..7361d6cd73d4 100644
--- a/drivers/staging/dgnc/Kconfig
+++ b/drivers/staging/dgnc/Kconfig
@@ -3,8 +3,8 @@ config DGNC
default n
depends on TTY && PCI
help
- Say Y here to enable support for the Digi International Neo and
- Classic PCI based product line.
+ Say Y here to enable support for the Digi International Neo and
+ Classic PCI based product line.
- To compile this driver as a module, say M here: the module will be
- called dgnc
+ To compile this driver as a module, say M here: the module will be
+ called dgnc
--
2.19.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 15:53 ` sehro rautenkranz
0 siblings, 0 replies; 22+ messages in thread
From: sehro rautenkranz @ 2018-09-25 15:53 UTC (permalink / raw)
Cc: sehro, Lidza Louina, Mark Hounschell, Greg Kroah-Hartman,
driverdev-devel, devel, linux-kernel
Replace Kconfig help header and text to match other modules. Issue found
my checkpatch.
Spacing adjusted to match Documentation/process/coding-style.rst. Issue
reported by rdunlap@infradead.org.
Signed-off-by: sehro rautenkranz <sehro@sehro.org>
---
drivers/staging/dgnc/Kconfig | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
index c845b357b53d..7361d6cd73d4 100644
--- a/drivers/staging/dgnc/Kconfig
+++ b/drivers/staging/dgnc/Kconfig
@@ -3,8 +3,8 @@ config DGNC
default n
depends on TTY && PCI
help
- Say Y here to enable support for the Digi International Neo and
- Classic PCI based product line.
+ Say Y here to enable support for the Digi International Neo and
+ Classic PCI based product line.
- To compile this driver as a module, say M here: the module will be
- called dgnc
+ To compile this driver as a module, say M here: the module will be
+ called dgnc
--
2.19.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH] staging: dgnc: Fix Kconfig help header and text
2018-09-25 15:53 ` sehro rautenkranz
@ 2018-09-25 16:00 ` Randy Dunlap
-1 siblings, 0 replies; 22+ messages in thread
From: Randy Dunlap @ 2018-09-25 16:00 UTC (permalink / raw)
To: sehro rautenkranz
Cc: devel, Mark Hounschell, Lidza Louina, driverdev-devel,
linux-kernel, Greg Kroah-Hartman
On 9/25/18 8:53 AM, sehro rautenkranz wrote:
> Replace Kconfig help header and text to match other modules. Issue found
> my checkpatch.
by checkpatch.
>
> Spacing adjusted to match Documentation/process/coding-style.rst. Issue
> reported by rdunlap@infradead.org.
Reported-by: Randy Dunlap <rdunlap@infradead.org>
>
> Signed-off-by: sehro rautenkranz <sehro@sehro.org>
> ---
> drivers/staging/dgnc/Kconfig | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
> index c845b357b53d..7361d6cd73d4 100644
> --- a/drivers/staging/dgnc/Kconfig
> +++ b/drivers/staging/dgnc/Kconfig
> @@ -3,8 +3,8 @@ config DGNC
> default n
> depends on TTY && PCI
> help
> - Say Y here to enable support for the Digi International Neo and
> - Classic PCI based product line.
> + Say Y here to enable support for the Digi International Neo and
> + Classic PCI based product line.
>
> - To compile this driver as a module, say M here: the module will be
> - called dgnc
> + To compile this driver as a module, say M here: the module will be
> + called dgnc
That indentation is still not correct. :(
--
~Randy
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 16:00 ` Randy Dunlap
0 siblings, 0 replies; 22+ messages in thread
From: Randy Dunlap @ 2018-09-25 16:00 UTC (permalink / raw)
To: sehro rautenkranz
Cc: devel, Mark Hounschell, Greg Kroah-Hartman, driverdev-devel,
linux-kernel, Lidza Louina
On 9/25/18 8:53 AM, sehro rautenkranz wrote:
> Replace Kconfig help header and text to match other modules. Issue found
> my checkpatch.
by checkpatch.
>
> Spacing adjusted to match Documentation/process/coding-style.rst. Issue
> reported by rdunlap@infradead.org.
Reported-by: Randy Dunlap <rdunlap@infradead.org>
>
> Signed-off-by: sehro rautenkranz <sehro@sehro.org>
> ---
> drivers/staging/dgnc/Kconfig | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
> index c845b357b53d..7361d6cd73d4 100644
> --- a/drivers/staging/dgnc/Kconfig
> +++ b/drivers/staging/dgnc/Kconfig
> @@ -3,8 +3,8 @@ config DGNC
> default n
> depends on TTY && PCI
> help
> - Say Y here to enable support for the Digi International Neo and
> - Classic PCI based product line.
> + Say Y here to enable support for the Digi International Neo and
> + Classic PCI based product line.
>
> - To compile this driver as a module, say M here: the module will be
> - called dgnc
> + To compile this driver as a module, say M here: the module will be
> + called dgnc
That indentation is still not correct. :(
--
~Randy
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH] staging: dgnc: Fix Kconfig help header and text
2018-09-25 16:00 ` Randy Dunlap
@ 2018-09-25 16:22 ` sehro rautenkranz
-1 siblings, 0 replies; 22+ messages in thread
From: sehro rautenkranz @ 2018-09-25 16:22 UTC (permalink / raw)
Cc: sehro rautenkranz, Lidza Louina, Mark Hounschell,
Greg Kroah-Hartman, driverdev-devel, devel, linux-kernel
Replace Kconfig help header and text to match other modules. Issue found
my checkpatch.
Spacing adjusted to match Documentation/process/coding-style.rst. Issue
reported by rdunlap@infradead.org. (try 2)
Signed-off-by: sehro rautenkranz <sehro@sehro.org>
---
drivers/staging/dgnc/Kconfig | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
index 032c2a795238..0f7537eb27d8 100644
--- a/drivers/staging/dgnc/Kconfig
+++ b/drivers/staging/dgnc/Kconfig
@@ -1,6 +1,10 @@
config DGNC
- tristate "Digi Neo and Classic PCI Products"
- default n
- depends on TTY && PCI
- ---help---
- Driver for the Digi International Neo and Classic PCI based product line.
+ tristate "Digi Neo and Classic PCI Products"
+ default n
+ depends on TTY && PCI
+ help
+ Say Y here to enable support for the Digi International Neo and
+ Classic PCI based product line.
+
+ To compile this driver as a module, say M here: the module will be
+ called dgnc
--
2.19.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 16:22 ` sehro rautenkranz
0 siblings, 0 replies; 22+ messages in thread
From: sehro rautenkranz @ 2018-09-25 16:22 UTC (permalink / raw)
Cc: sehro rautenkranz, Lidza Louina, Mark Hounschell,
Greg Kroah-Hartman, driverdev-devel, devel, linux-kernel
Replace Kconfig help header and text to match other modules. Issue found
my checkpatch.
Spacing adjusted to match Documentation/process/coding-style.rst. Issue
reported by rdunlap@infradead.org. (try 2)
Signed-off-by: sehro rautenkranz <sehro@sehro.org>
---
drivers/staging/dgnc/Kconfig | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
index 032c2a795238..0f7537eb27d8 100644
--- a/drivers/staging/dgnc/Kconfig
+++ b/drivers/staging/dgnc/Kconfig
@@ -1,6 +1,10 @@
config DGNC
- tristate "Digi Neo and Classic PCI Products"
- default n
- depends on TTY && PCI
- ---help---
- Driver for the Digi International Neo and Classic PCI based product line.
+ tristate "Digi Neo and Classic PCI Products"
+ default n
+ depends on TTY && PCI
+ help
+ Say Y here to enable support for the Digi International Neo and
+ Classic PCI based product line.
+
+ To compile this driver as a module, say M here: the module will be
+ called dgnc
--
2.19.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH] staging: dgnc: Fix Kconfig help header and text
2018-09-25 16:22 ` sehro rautenkranz
@ 2018-09-25 18:46 ` Greg Kroah-Hartman
-1 siblings, 0 replies; 22+ messages in thread
From: Greg Kroah-Hartman @ 2018-09-25 18:46 UTC (permalink / raw)
To: sehro rautenkranz
Cc: Lidza Louina, Mark Hounschell, driverdev-devel, devel, linux-kernel
On Tue, Sep 25, 2018 at 10:22:24AM -0600, sehro rautenkranz wrote:
> Replace Kconfig help header and text to match other modules. Issue found
> my checkpatch.
>
> Spacing adjusted to match Documentation/process/coding-style.rst. Issue
> reported by rdunlap@infradead.org. (try 2)
The "try 2" needs go to below the --- line.
Also, you need to properly version your patches so that I have a clue
as to which is the "latest" one.
> Signed-off-by: sehro rautenkranz <sehro@sehro.org>
You forgot Randy's Reported-by.
Also, properly Capitalize your name.
v3?
thanks,
greg k-h
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 18:46 ` Greg Kroah-Hartman
0 siblings, 0 replies; 22+ messages in thread
From: Greg Kroah-Hartman @ 2018-09-25 18:46 UTC (permalink / raw)
To: sehro rautenkranz
Cc: Mark Hounschell, Lidza Louina, driverdev-devel, linux-kernel, devel
On Tue, Sep 25, 2018 at 10:22:24AM -0600, sehro rautenkranz wrote:
> Replace Kconfig help header and text to match other modules. Issue found
> my checkpatch.
>
> Spacing adjusted to match Documentation/process/coding-style.rst. Issue
> reported by rdunlap@infradead.org. (try 2)
The "try 2" needs go to below the --- line.
Also, you need to properly version your patches so that I have a clue
as to which is the "latest" one.
> Signed-off-by: sehro rautenkranz <sehro@sehro.org>
You forgot Randy's Reported-by.
Also, properly Capitalize your name.
v3?
thanks,
greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] staging: dgnc: Fix Kconfig help header and text
2018-09-25 18:46 ` Greg Kroah-Hartman
@ 2018-09-25 19:49 ` Randy Dunlap
-1 siblings, 0 replies; 22+ messages in thread
From: Randy Dunlap @ 2018-09-25 19:49 UTC (permalink / raw)
To: Greg Kroah-Hartman, sehro rautenkranz
Cc: Lidza Louina, Mark Hounschell, driverdev-devel, devel, linux-kernel
On 9/25/18 11:46 AM, Greg Kroah-Hartman wrote:
> On Tue, Sep 25, 2018 at 10:22:24AM -0600, sehro rautenkranz wrote:
>> Replace Kconfig help header and text to match other modules. Issue found
>> my checkpatch.
>>
>> Spacing adjusted to match Documentation/process/coding-style.rst. Issue
>> reported by rdunlap@infradead.org. (try 2)
>
> The "try 2" needs go to below the --- line.
>
> Also, you need to properly version your patches so that I have a clue
> as to which is the "latest" one.
>
>> Signed-off-by: sehro rautenkranz <sehro@sehro.org>
>
> You forgot Randy's Reported-by.
>
> Also, properly Capitalize your name.
>
> v3?
and the help text is still not indented correctly:
use one tab + 2 spaces for help text.
--
~Randy
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 19:49 ` Randy Dunlap
0 siblings, 0 replies; 22+ messages in thread
From: Randy Dunlap @ 2018-09-25 19:49 UTC (permalink / raw)
To: Greg Kroah-Hartman, sehro rautenkranz
Cc: Mark Hounschell, Lidza Louina, driverdev-devel, linux-kernel, devel
On 9/25/18 11:46 AM, Greg Kroah-Hartman wrote:
> On Tue, Sep 25, 2018 at 10:22:24AM -0600, sehro rautenkranz wrote:
>> Replace Kconfig help header and text to match other modules. Issue found
>> my checkpatch.
>>
>> Spacing adjusted to match Documentation/process/coding-style.rst. Issue
>> reported by rdunlap@infradead.org. (try 2)
>
> The "try 2" needs go to below the --- line.
>
> Also, you need to properly version your patches so that I have a clue
> as to which is the "latest" one.
>
>> Signed-off-by: sehro rautenkranz <sehro@sehro.org>
>
> You forgot Randy's Reported-by.
>
> Also, properly Capitalize your name.
>
> v3?
and the help text is still not indented correctly:
use one tab + 2 spaces for help text.
--
~Randy
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
* [PATCH v3] staging: dgnc: Fix Kconfig help header and text
2018-09-25 19:49 ` Randy Dunlap
@ 2018-09-25 23:41 ` Sehro Rautenkranz
-1 siblings, 0 replies; 22+ messages in thread
From: Sehro Rautenkranz @ 2018-09-25 23:41 UTC (permalink / raw)
To: sehro
Cc: Lidza Louina, Mark Hounschell, Greg Kroah-Hartman,
driverdev-devel, devel, linux-kernel
Replace Kconfig help header and text to match other modules. Issue found
by checkpatch.
Spacing adjusted to match Documentation/process/coding-style.rst. Issue
reported by rdunlap@infradead.org.
Reported-by: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Sehro Rautenkranz <sehro@sehro.org>
---
drivers/staging/dgnc/Kconfig | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
index 032c2a795238..9f4f52c9c27d 100644
--- a/drivers/staging/dgnc/Kconfig
+++ b/drivers/staging/dgnc/Kconfig
@@ -1,6 +1,10 @@
config DGNC
- tristate "Digi Neo and Classic PCI Products"
- default n
- depends on TTY && PCI
- ---help---
- Driver for the Digi International Neo and Classic PCI based product line.
+ tristate "Digi Neo and Classic PCI Products"
+ default n
+ depends on TTY && PCI
+ help
+ Say Y here to enable support for the Digi International Neo and
+ Classic PCI based product line.
+
+ To compile this driver as a module, say M here: the module will be
+ called dgnc
--
2.19.0
^ permalink raw reply related [flat|nested] 22+ messages in thread
* [PATCH v3] staging: dgnc: Fix Kconfig help header and text
@ 2018-09-25 23:41 ` Sehro Rautenkranz
0 siblings, 0 replies; 22+ messages in thread
From: Sehro Rautenkranz @ 2018-09-25 23:41 UTC (permalink / raw)
To: sehro
Cc: devel, Mark Hounschell, Lidza Louina, driverdev-devel,
linux-kernel, Greg Kroah-Hartman
Replace Kconfig help header and text to match other modules. Issue found
by checkpatch.
Spacing adjusted to match Documentation/process/coding-style.rst. Issue
reported by rdunlap@infradead.org.
Reported-by: Randy Dunlap <rdunlap@infradead.org>
Signed-off-by: Sehro Rautenkranz <sehro@sehro.org>
---
drivers/staging/dgnc/Kconfig | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
index 032c2a795238..9f4f52c9c27d 100644
--- a/drivers/staging/dgnc/Kconfig
+++ b/drivers/staging/dgnc/Kconfig
@@ -1,6 +1,10 @@
config DGNC
- tristate "Digi Neo and Classic PCI Products"
- default n
- depends on TTY && PCI
- ---help---
- Driver for the Digi International Neo and Classic PCI based product line.
+ tristate "Digi Neo and Classic PCI Products"
+ default n
+ depends on TTY && PCI
+ help
+ Say Y here to enable support for the Digi International Neo and
+ Classic PCI based product line.
+
+ To compile this driver as a module, say M here: the module will be
+ called dgnc
--
2.19.0
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply related [flat|nested] 22+ messages in thread
* Re: [PATCH v3] staging: dgnc: Fix Kconfig help header and text
2018-09-25 23:41 ` Sehro Rautenkranz
(?)
@ 2018-09-28 19:59 ` Lidza Louina
2018-10-01 12:09 ` Mark Hounschell
-1 siblings, 1 reply; 22+ messages in thread
From: Lidza Louina @ 2018-09-28 19:59 UTC (permalink / raw)
To: Sehro Rautenkranz
Cc: Mark Hounschell, Greg Kroah-Hartman, driverdev-devel, devel,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1715 bytes --]
I haven't done work on this driver in a long time. I looked up the devices,
and they seem to already have an existing Linux driver:
https://www.digi.com/products/usb-and-serial-connectivity/serial-cards/digineo
and https://www.digi.com/support/productdetail?pid=1694
I've contacted the company to see if the driver is still needed in staging.
I'll send an update once they get back to me.
Lidza
On Tue, Sep 25, 2018 at 7:42 PM Sehro Rautenkranz <sehro@sehro.org> wrote:
> Replace Kconfig help header and text to match other modules. Issue found
> by checkpatch.
>
> Spacing adjusted to match Documentation/process/coding-style.rst. Issue
> reported by rdunlap@infradead.org.
>
> Reported-by: Randy Dunlap <rdunlap@infradead.org>
> Signed-off-by: Sehro Rautenkranz <sehro@sehro.org>
> ---
> drivers/staging/dgnc/Kconfig | 14 +++++++++-----
> 1 file changed, 9 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/staging/dgnc/Kconfig b/drivers/staging/dgnc/Kconfig
> index 032c2a795238..9f4f52c9c27d 100644
> --- a/drivers/staging/dgnc/Kconfig
> +++ b/drivers/staging/dgnc/Kconfig
> @@ -1,6 +1,10 @@
> config DGNC
> - tristate "Digi Neo and Classic PCI Products"
> - default n
> - depends on TTY && PCI
> - ---help---
> - Driver for the Digi International Neo and Classic PCI based
> product line.
> + tristate "Digi Neo and Classic PCI Products"
> + default n
> + depends on TTY && PCI
> + help
> + Say Y here to enable support for the Digi International Neo and
> + Classic PCI based product line.
> +
> + To compile this driver as a module, say M here: the module will
> be
> + called dgnc
> --
> 2.19.0
>
> --
Lidza
[-- Attachment #2: Type: text/html, Size: 2621 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3] staging: dgnc: Fix Kconfig help header and text
2018-09-28 19:59 ` Lidza Louina
@ 2018-10-01 12:09 ` Mark Hounschell
2018-10-01 16:49 ` Lidza Louina
0 siblings, 1 reply; 22+ messages in thread
From: Mark Hounschell @ 2018-10-01 12:09 UTC (permalink / raw)
To: Lidza Louina, Sehro Rautenkranz
Cc: Greg Kroah-Hartman, driverdev-devel, devel, linux-kernel
On 9/28/18 3:59 PM, Lidza Louina wrote:
> I haven't done work on this driver in a long time. I looked up the
> devices, and they seem to already have an existing Linux driver:
> https://www.digi.com/products/usb-and-serial-connectivity/serial-cards/digineo and
> https://www.digi.com/support/productdetail?pid=1694
>
> I've contacted the company to see if the driver is still needed in
> staging. I'll send an update once they get back to me.
>
> Lidza
>
Good luck with that. Digi obsoleted ALL their PCI serial cards several
months ago. They would not even respond to requests from me to include
their firmware for their PCI dgap cards into the kernel firmware
package. The card for the dgnc driver is a pci-e card that may still be
supported by them however. Also there does appear to be an already in
kernel driver for these "PCI-e" cards too. It actually works as I have a
couple of these cards. The module name is "jsm".
03:00.0 Serial controller: Digi International Device 00f0 (rev 02)
(prog-if 02 [16550])
Subsystem: Digi International Device 00f0
Control: I/O- Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop-
ParErr- Stepping- SERR- FastB2B- DisINTx-
Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort-
<TAbort- <MAbort- >SERR- <PERR- INTx-
Interrupt: pin A routed to IRQ 29
Region 0: Memory at f7200000 (32-bit, non-prefetchable) [size=4K]
Capabilities: [40] Power Management version 2
Flags: PMEClk+ DSI- D1- D2- AuxCurrent=0mA
PME(D0-,D1-,D2-,D3hot+,D3cold-)
Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME-
Kernel driver in use: jsm
Kernel modules: jsm
Regards
Mark
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3] staging: dgnc: Fix Kconfig help header and text
2018-10-01 12:09 ` Mark Hounschell
@ 2018-10-01 16:49 ` Lidza Louina
2018-10-01 17:38 ` Mark Hounschell
2018-10-02 22:09 ` Greg Kroah-Hartman
0 siblings, 2 replies; 22+ messages in thread
From: Lidza Louina @ 2018-10-01 16:49 UTC (permalink / raw)
To: markh
Cc: Sehro Rautenkranz, Greg Kroah-Hartman, driverdev-devel, devel,
linux-kernel
[-- Attachment #1: Type: text/plain, Size: 2058 bytes --]
On Mon, Oct 1, 2018 at 8:09 AM Mark Hounschell <markh@compro.net> wrote:
> On 9/28/18 3:59 PM, Lidza Louina wrote:
> > I haven't done work on this driver in a long time. I looked up the
> > devices, and they seem to already have an existing Linux driver:
> >
> https://www.digi.com/products/usb-and-serial-connectivity/serial-cards/digineo and
>
> > https://www.digi.com/support/productdetail?pid=1694
> >
> > I've contacted the company to see if the driver is still needed in
> > staging. I'll send an update once they get back to me.
> >
> > Lidza
> >
>
> Good luck with that. Digi obsoleted ALL their PCI serial cards several
> months ago. They would not even respond to requests from me to include
> their firmware for their PCI dgap cards into the kernel firmware
> package. The card for the dgnc driver is a pci-e card that may still be
> supported by them however. Also there does appear to be an already in
> kernel driver for these "PCI-e" cards too. It actually works as I have a
> couple of these cards. The module name is "jsm".
>
>
> 03:00.0 Serial controller: Digi International Device 00f0 (rev 02)
> (prog-if 02 [16550])
> Subsystem: Digi International Device 00f0
> Control: I/O- Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop-
> ParErr- Stepping- SERR- FastB2B- DisINTx-
> Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort-
> <TAbort- <MAbort- >SERR- <PERR- INTx-
> Interrupt: pin A routed to IRQ 29
> Region 0: Memory at f7200000 (32-bit, non-prefetchable) [size=4K]
> Capabilities: [40] Power Management version 2
> Flags: PMEClk+ DSI- D1- D2- AuxCurrent=0mA
> PME(D0-,D1-,D2-,D3hot+,D3cold-)
> Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME-
> Kernel driver in use: jsm
> Kernel modules: jsm
>
>
> Regards
> Mark
>
Thanks for the heads up Mark. I've totally neglected this driver and should
have caught this earlier. I'm okay with removing it from staging. We can
bring it back if they need it.
Lidza
--
Lidza
[-- Attachment #2: Type: text/html, Size: 2917 bytes --]
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3] staging: dgnc: Fix Kconfig help header and text
2018-10-01 16:49 ` Lidza Louina
@ 2018-10-01 17:38 ` Mark Hounschell
2018-10-02 22:09 ` Greg Kroah-Hartman
1 sibling, 0 replies; 22+ messages in thread
From: Mark Hounschell @ 2018-10-01 17:38 UTC (permalink / raw)
To: Lidza Louina
Cc: Sehro Rautenkranz, Greg Kroah-Hartman, driverdev-devel, devel,
linux-kernel
On 10/1/18 12:49 PM, Lidza Louina wrote:
> On Mon, Oct 1, 2018 at 8:09 AM Mark Hounschell <markh@compro.net
> <mailto:markh@compro.net>> wrote:
>
> On 9/28/18 3:59 PM, Lidza Louina wrote:
> > I haven't done work on this driver in a long time. I looked up the
> > devices, and they seem to already have an existing Linux driver:
> >
> https://www.digi.com/products/usb-and-serial-connectivity/serial-cards/digineo and
>
> > https://www.digi.com/support/productdetail?pid=1694
> >
> > I've contacted the company to see if the driver is still needed in
> > staging. I'll send an update once they get back to me.
> >
> > Lidza
> >
>
> Good luck with that. Digi obsoleted ALL their PCI serial cards several
> months ago. They would not even respond to requests from me to include
> their firmware for their PCI dgap cards into the kernel firmware
> package. The card for the dgnc driver is a pci-e card that may still be
> supported by them however. Also there does appear to be an already in
> kernel driver for these "PCI-e" cards too. It actually works as I
> have a
> couple of these cards. The module name is "jsm".
>
>
> 03:00.0 Serial controller: Digi International Device 00f0 (rev 02)
> (prog-if 02 [16550])
> Subsystem: Digi International Device 00f0
> Control: I/O- Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop-
> ParErr- Stepping- SERR- FastB2B- DisINTx-
> Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast
> >TAbort-
> <TAbort- <MAbort- >SERR- <PERR- INTx-
> Interrupt: pin A routed to IRQ 29
> Region 0: Memory at f7200000 (32-bit, non-prefetchable)
> [size=4K]
> Capabilities: [40] Power Management version 2
> Flags: PMEClk+ DSI- D1- D2- AuxCurrent=0mA
> PME(D0-,D1-,D2-,D3hot+,D3cold-)
> Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME-
> Kernel driver in use: jsm
> Kernel modules: jsm
>
>
> Regards
> Mark
>
>
> Thanks for the heads up Mark. I've totally neglected this driver and
> should have caught this earlier. I'm okay with removing it from staging.
> We can bring it back if they need it.
>
It may actually be my fault that Greg-KH added the dgnc driver to
staging. Shortly after Digi stopped supporting their PCI cards, I had
some contact with Greg-KH and sent him all the Digi GPL drivers that I
was using at the time for possible inclusion. Digi was no longer
maintaining them. I did not know at the time about the jsm driver
already being there.
The dgnc driver supports the following PCI cards:
#define PCI_DEVICE_CLASSIC_4_DID 0x0028
#define PCI_DEVICE_CLASSIC_8_DID 0x0029
#define PCI_DEVICE_CLASSIC_4_422_DID 0x00D0
#define PCI_DEVICE_CLASSIC_8_422_DID 0x00D1
The jsm driver supports all of these plus their newer PCI-e cards:
#define PCI_DEVICE_ID_CLASSIC_4 0x0028
#define PCI_DEVICE_ID_CLASSIC_8 0x0029
#define PCI_DEVICE_ID_CLASSIC_4_422 0x00D0
#define PCI_DEVICE_ID_CLASSIC_8_422 0x00D1
#define PCI_DEVICE_ID_NEO_4 0x00B0
#define PCI_DEVICE_ID_NEO_1_422 0x00CC
#define PCI_DEVICE_ID_NEO_1_422_485 0x00CD
#define PCI_DEVICE_ID_NEO_2_422_485 0x00CE
#define PCIE_DEVICE_ID_NEO_8 0x00F0
#define PCIE_DEVICE_ID_NEO_4 0x00F1
#define PCIE_DEVICE_ID_NEO_4RJ45 0x00F2
#define PCIE_DEVICE_ID_NEO_8RJ45 0x00F3
So I think it is safe to remove the dgnc driver from staging. I also
think the PCI-e versions defined above are actually still available from
Digi as current products.
Regards
Mark
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3] staging: dgnc: Fix Kconfig help header and text
2018-10-01 16:49 ` Lidza Louina
@ 2018-10-02 22:09 ` Greg Kroah-Hartman
2018-10-02 22:09 ` Greg Kroah-Hartman
1 sibling, 0 replies; 22+ messages in thread
From: Greg Kroah-Hartman @ 2018-10-02 22:09 UTC (permalink / raw)
To: Lidza Louina
Cc: markh, Sehro Rautenkranz, driverdev-devel, devel, linux-kernel
On Mon, Oct 01, 2018 at 12:49:50PM -0400, Lidza Louina wrote:
> On Mon, Oct 1, 2018 at 8:09 AM Mark Hounschell <markh@compro.net> wrote:
>
> > On 9/28/18 3:59 PM, Lidza Louina wrote:
> > > I haven't done work on this driver in a long time. I looked up the
> > > devices, and they seem to already have an existing Linux driver:
> > >
> > https://www.digi.com/products/usb-and-serial-connectivity/serial-cards/digineo and
> >
> > > https://www.digi.com/support/productdetail?pid=1694
> > >
> > > I've contacted the company to see if the driver is still needed in
> > > staging. I'll send an update once they get back to me.
> > >
> > > Lidza
> > >
> >
> > Good luck with that. Digi obsoleted ALL their PCI serial cards several
> > months ago. They would not even respond to requests from me to include
> > their firmware for their PCI dgap cards into the kernel firmware
> > package. The card for the dgnc driver is a pci-e card that may still be
> > supported by them however. Also there does appear to be an already in
> > kernel driver for these "PCI-e" cards too. It actually works as I have a
> > couple of these cards. The module name is "jsm".
> >
> >
> > 03:00.0 Serial controller: Digi International Device 00f0 (rev 02)
> > (prog-if 02 [16550])
> > Subsystem: Digi International Device 00f0
> > Control: I/O- Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop-
> > ParErr- Stepping- SERR- FastB2B- DisINTx-
> > Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort-
> > <TAbort- <MAbort- >SERR- <PERR- INTx-
> > Interrupt: pin A routed to IRQ 29
> > Region 0: Memory at f7200000 (32-bit, non-prefetchable) [size=4K]
> > Capabilities: [40] Power Management version 2
> > Flags: PMEClk+ DSI- D1- D2- AuxCurrent=0mA
> > PME(D0-,D1-,D2-,D3hot+,D3cold-)
> > Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME-
> > Kernel driver in use: jsm
> > Kernel modules: jsm
> >
> >
> > Regards
> > Mark
> >
>
> Thanks for the heads up Mark. I've totally neglected this driver and should
> have caught this earlier. I'm okay with removing it from staging. We can
> bring it back if they need it.
Ok, I'll go delete it, thanks.
greg k-h
^ permalink raw reply [flat|nested] 22+ messages in thread
* Re: [PATCH v3] staging: dgnc: Fix Kconfig help header and text
@ 2018-10-02 22:09 ` Greg Kroah-Hartman
0 siblings, 0 replies; 22+ messages in thread
From: Greg Kroah-Hartman @ 2018-10-02 22:09 UTC (permalink / raw)
To: Lidza Louina
Cc: markh, devel, driverdev-devel, linux-kernel, Sehro Rautenkranz
On Mon, Oct 01, 2018 at 12:49:50PM -0400, Lidza Louina wrote:
> On Mon, Oct 1, 2018 at 8:09 AM Mark Hounschell <markh@compro.net> wrote:
>
> > On 9/28/18 3:59 PM, Lidza Louina wrote:
> > > I haven't done work on this driver in a long time. I looked up the
> > > devices, and they seem to already have an existing Linux driver:
> > >
> > https://www.digi.com/products/usb-and-serial-connectivity/serial-cards/digineo and
> >
> > > https://www.digi.com/support/productdetail?pid=1694
> > >
> > > I've contacted the company to see if the driver is still needed in
> > > staging. I'll send an update once they get back to me.
> > >
> > > Lidza
> > >
> >
> > Good luck with that. Digi obsoleted ALL their PCI serial cards several
> > months ago. They would not even respond to requests from me to include
> > their firmware for their PCI dgap cards into the kernel firmware
> > package. The card for the dgnc driver is a pci-e card that may still be
> > supported by them however. Also there does appear to be an already in
> > kernel driver for these "PCI-e" cards too. It actually works as I have a
> > couple of these cards. The module name is "jsm".
> >
> >
> > 03:00.0 Serial controller: Digi International Device 00f0 (rev 02)
> > (prog-if 02 [16550])
> > Subsystem: Digi International Device 00f0
> > Control: I/O- Mem+ BusMaster- SpecCycle- MemWINV- VGASnoop-
> > ParErr- Stepping- SERR- FastB2B- DisINTx-
> > Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=fast >TAbort-
> > <TAbort- <MAbort- >SERR- <PERR- INTx-
> > Interrupt: pin A routed to IRQ 29
> > Region 0: Memory at f7200000 (32-bit, non-prefetchable) [size=4K]
> > Capabilities: [40] Power Management version 2
> > Flags: PMEClk+ DSI- D1- D2- AuxCurrent=0mA
> > PME(D0-,D1-,D2-,D3hot+,D3cold-)
> > Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=0 PME-
> > Kernel driver in use: jsm
> > Kernel modules: jsm
> >
> >
> > Regards
> > Mark
> >
>
> Thanks for the heads up Mark. I've totally neglected this driver and should
> have caught this earlier. I'm okay with removing it from staging. We can
> bring it back if they need it.
Ok, I'll go delete it, thanks.
greg k-h
_______________________________________________
devel mailing list
devel@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
^ permalink raw reply [flat|nested] 22+ messages in thread
end of thread, other threads:[~2018-10-02 22:09 UTC | newest]
Thread overview: 22+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-25 6:20 [PATCH] staging: dgnc: Fix Kconfig help header and text sehro rautenkranz
2018-09-25 6:20 ` sehro rautenkranz
2018-09-25 15:34 ` Randy Dunlap
2018-09-25 15:34 ` Randy Dunlap
2018-09-25 15:53 ` sehro rautenkranz
2018-09-25 15:53 ` sehro rautenkranz
2018-09-25 16:00 ` Randy Dunlap
2018-09-25 16:00 ` Randy Dunlap
2018-09-25 16:22 ` sehro rautenkranz
2018-09-25 16:22 ` sehro rautenkranz
2018-09-25 18:46 ` Greg Kroah-Hartman
2018-09-25 18:46 ` Greg Kroah-Hartman
2018-09-25 19:49 ` Randy Dunlap
2018-09-25 19:49 ` Randy Dunlap
2018-09-25 23:41 ` [PATCH v3] " Sehro Rautenkranz
2018-09-25 23:41 ` Sehro Rautenkranz
2018-09-28 19:59 ` Lidza Louina
2018-10-01 12:09 ` Mark Hounschell
2018-10-01 16:49 ` Lidza Louina
2018-10-01 17:38 ` Mark Hounschell
2018-10-02 22:09 ` Greg Kroah-Hartman
2018-10-02 22:09 ` Greg Kroah-Hartman
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.