* [PATCH] comedi: drivers: ni_pcimio: Remove the code enclosed by #if 0 and its #endif
@ 2023-03-10 14:01 Sumitra Sharma
2023-03-10 15:49 ` Deepak R Varma
0 siblings, 1 reply; 2+ messages in thread
From: Sumitra Sharma @ 2023-03-10 14:01 UTC (permalink / raw)
To: abbotti, hsweeten, outreachy; +Cc: linux-kernel
Reported by checkpatch.pl:
WARNING: Consider removing the code enclosd by this #if 0 and its #endif
at line 482, 501, 570.
Signed-off-by: Sumitra Sharma <sumitraartsy@gmail.com>
---
drivers/comedi/drivers/ni_pcimio.c | 48 ------------------------------
1 file changed, 48 deletions(-)
diff --git a/drivers/comedi/drivers/ni_pcimio.c b/drivers/comedi/drivers/ni_pcimio.c
index 0b055321023d..1603095f2ed9 100644
--- a/drivers/comedi/drivers/ni_pcimio.c
+++ b/drivers/comedi/drivers/ni_pcimio.c
@@ -479,43 +479,6 @@ static const struct ni_board_struct ni_boards[] = {
.ao_speed = 250,
.caldac = { ad8804, ad8804 },
},
-#if 0
- /* The 6115 boards probably need their own driver */
- [BOARD_PCI6115] = { /* .device_id = 0x2ed0, */
- .name = "pci-6115",
- .n_adchan = 4,
- .ai_maxdata = 0x0fff,
- .ai_fifo_depth = 8192,
- .gainlkup = ai_gain_611x,
- .ai_speed = 100,
- .n_aochan = 2,
- .ao_maxdata = 0xffff,
- .ao_671x = 1,
- .ao_fifo_depth = 2048,
- .ao_speed = 250,
- .reg_611x = 1,
- /* XXX */
- .caldac = { ad8804_debug, ad8804_debug, ad8804_debug },
- },
-#endif
-#if 0
- [BOARD_PXI6115] = { /* .device_id = ????, */
- .name = "pxi-6115",
- .n_adchan = 4,
- .ai_maxdata = 0x0fff,
- .ai_fifo_depth = 8192,
- .gainlkup = ai_gain_611x,
- .ai_speed = 100,
- .n_aochan = 2,
- .ao_maxdata = 0xffff,
- .ao_671x = 1,
- .ao_fifo_depth = 2048,
- .ao_speed = 250,
- .reg_611x = 1,
- /* XXX */
- .caldac = { ad8804_debug, ad8804_debug, ad8804_debug },
- },
-#endif
[BOARD_PCI6711] = {
.name = "pci-6711",
.n_aochan = 4,
@@ -567,17 +530,6 @@ static const struct ni_board_struct ni_boards[] = {
.reg_type = ni_reg_6711,
.caldac = { ad8804_debug },
},
-#if 0
- [BOARD_PXI6731] = { /* .device_id = ????, */
- .name = "pxi-6731",
- .n_aochan = 4,
- .ao_maxdata = 0xffff,
- .ao_fifo_depth = 8192,
- .ao_range_table = &range_bipolar10,
- .reg_type = ni_reg_6711,
- .caldac = { ad8804_debug },
- },
-#endif
[BOARD_PCI6733] = {
.name = "pci-6733",
.n_aochan = 8,
--
2.25.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] comedi: drivers: ni_pcimio: Remove the code enclosed by #if 0 and its #endif
2023-03-10 14:01 [PATCH] comedi: drivers: ni_pcimio: Remove the code enclosed by #if 0 and its #endif Sumitra Sharma
@ 2023-03-10 15:49 ` Deepak R Varma
0 siblings, 0 replies; 2+ messages in thread
From: Deepak R Varma @ 2023-03-10 15:49 UTC (permalink / raw)
To: Sumitra Sharma; +Cc: abbotti, hsweeten, outreachy, linux-kernel
On Fri, Mar 10, 2023 at 06:01:19AM -0800, Sumitra Sharma wrote:
> Reported by checkpatch.pl:
>
> WARNING: Consider removing the code enclosd by this #if 0 and its #endif
> at line 482, 501, 570.
>
> Signed-off-by: Sumitra Sharma <sumitraartsy@gmail.com>
> ---
> drivers/comedi/drivers/ni_pcimio.c | 48 ------------------------------
> 1 file changed, 48 deletions(-)
Hi Sumitra,
Nice first attempt. However, you should be working in the drivers/staging
directory for now.
Also, specific to the following change, the device definitions may be useful to
people who have those devices and can be added to the driver by simply removing
the #if 0 condition. Hence, these commented blocks are likely retained due to
same reason.
Thank you,
Deepak.
>
> diff --git a/drivers/comedi/drivers/ni_pcimio.c b/drivers/comedi/drivers/ni_pcimio.c
> index 0b055321023d..1603095f2ed9 100644
> --- a/drivers/comedi/drivers/ni_pcimio.c
> +++ b/drivers/comedi/drivers/ni_pcimio.c
> @@ -479,43 +479,6 @@ static const struct ni_board_struct ni_boards[] = {
> .ao_speed = 250,
> .caldac = { ad8804, ad8804 },
> },
> -#if 0
> - /* The 6115 boards probably need their own driver */
> - [BOARD_PCI6115] = { /* .device_id = 0x2ed0, */
> - .name = "pci-6115",
> - .n_adchan = 4,
> - .ai_maxdata = 0x0fff,
> - .ai_fifo_depth = 8192,
> - .gainlkup = ai_gain_611x,
> - .ai_speed = 100,
> - .n_aochan = 2,
> - .ao_maxdata = 0xffff,
> - .ao_671x = 1,
> - .ao_fifo_depth = 2048,
> - .ao_speed = 250,
> - .reg_611x = 1,
> - /* XXX */
> - .caldac = { ad8804_debug, ad8804_debug, ad8804_debug },
> - },
> -#endif
> -#if 0
> - [BOARD_PXI6115] = { /* .device_id = ????, */
> - .name = "pxi-6115",
> - .n_adchan = 4,
> - .ai_maxdata = 0x0fff,
> - .ai_fifo_depth = 8192,
> - .gainlkup = ai_gain_611x,
> - .ai_speed = 100,
> - .n_aochan = 2,
> - .ao_maxdata = 0xffff,
> - .ao_671x = 1,
> - .ao_fifo_depth = 2048,
> - .ao_speed = 250,
> - .reg_611x = 1,
> - /* XXX */
> - .caldac = { ad8804_debug, ad8804_debug, ad8804_debug },
> - },
> -#endif
> [BOARD_PCI6711] = {
> .name = "pci-6711",
> .n_aochan = 4,
> @@ -567,17 +530,6 @@ static const struct ni_board_struct ni_boards[] = {
> .reg_type = ni_reg_6711,
> .caldac = { ad8804_debug },
> },
> -#if 0
> - [BOARD_PXI6731] = { /* .device_id = ????, */
> - .name = "pxi-6731",
> - .n_aochan = 4,
> - .ao_maxdata = 0xffff,
> - .ao_fifo_depth = 8192,
> - .ao_range_table = &range_bipolar10,
> - .reg_type = ni_reg_6711,
> - .caldac = { ad8804_debug },
> - },
> -#endif
> [BOARD_PCI6733] = {
> .name = "pci-6733",
> .n_aochan = 8,
> --
> 2.25.1
>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-03-10 15:56 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-03-10 14:01 [PATCH] comedi: drivers: ni_pcimio: Remove the code enclosed by #if 0 and its #endif Sumitra Sharma
2023-03-10 15:49 ` Deepak R Varma
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox