[PATCH][3/5][resend] floppy.c: Remove dead/commented out code

2007-03-19 Thread Jesper Juhl

This removes commented out/dead code from floppy.c

The code in question has been commented out for quite a while (at least as
far as I can see) and I see no reason for it to clutter up the source any
longer.
If, in the future, someone sees a real need to resurrect this code then
they can still find it in the revision history.
Let's kill the cruft :)

Signed-off-by: Jesper Juhl <[EMAIL PROTECTED]>
Acked-by: Alan Cox <[EMAIL PROTECTED]>
Acked-by: Bartlomiej Zolnierkiewicz <[EMAIL PROTECTED]>
---

diff --git a/drivers/block/floppy.c b/drivers/block/floppy.c
index dea7c4f..9a52ca6 100644
--- a/drivers/block/floppy.c
+++ b/drivers/block/floppy.c
@@ -774,8 +774,6 @@ static int disk_change(int drive)
current_type[drive] = NULL;
floppy_sizes[TOMINOR(drive)] = MAX_DISK_SIZE << 1;
}
-
-   /* USETF(FD_DISK_NEWCHANGE); */
return 1;
} else {
UDRS->last_checked = jiffies;
@@ -1325,7 +1323,6 @@ static void fdc_specify(void)
if (FDCS->need_configure && FDCS->version >= FDC_82072A) {
fdc_configure();
FDCS->need_configure = 0;
-   /* DPRINT("FIFO enabled\n"); */
}
 
switch (raw_cmd->rate & 0x03) {
@@ -1917,8 +1914,6 @@ static void floppy_shutdown(unsigned long data)
is_alive("floppy shutdown");
 }
 
-/* typedef void (*timeout_fn)(unsigned long); */
-
 /* start motor, check media-changed condition and write protection */
 static int start_motor(void (*function)(void))
 {
@@ -2564,19 +2559,6 @@ static void copy_buffer(int ssize, int max_sector, int 
max_sector_2)
 #endif
 }
 
-#if 0
-static inline int check_dma_crossing(char *start,
-   unsigned long length, char *message)
-{
-   if (CROSS_64KB(start, length)) {
-   printk("DMA xfer crosses 64KB boundary in %s %p-%p\n",
-  message, start, start + length);
-   return 1;
-   } else
-   return 0;
-}
-#endif
-
 /* work around a bug in pseudo DMA
  * (on some FDCs) pseudo DMA does not stop when the CPU stops
  * sending data.  Hence we need a different way to signal the
@@ -2768,7 +2750,6 @@ static int make_raw_rw_request(void)
 */
if (!direct || (indirect * 2 > direct * 3 &&
 *errors < DP->max_errors.read_track &&
-/* !TESTF(FD_NEED_TWADDLE) && */
 ((!probing
   || (DP->read_track & (1 << DRS->probed_format)) {
max_size = current_req->nr_sectors;
@@ -2782,9 +2763,6 @@ static int make_raw_rw_request(void)
   indirect, direct, fsector_t);
return 0;
}
-   /* check_dma_crossing(raw_cmd->kernel_data,
-  raw_cmd->length,
-  "end of make_raw_request [1]"); */
virtualdmabug_workaround();
return 2;
}
@@ -2834,8 +2812,6 @@ static int make_raw_rw_request(void)
raw_cmd->length = ((raw_cmd->length - 1) | (ssize - 1)) + 1;
raw_cmd->length <<= 9;
 #ifdef FLOPPY_SANITY_CHECK
-   /*check_dma_crossing(raw_cmd->kernel_data, raw_cmd->length,
-  "end of make_raw_request"); */
if ((raw_cmd->length < current_count_sectors << 9) ||
(raw_cmd->kernel_data != current_req->buffer &&
 CT(COMMAND) == FD_WRITE &&
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH][3/5][resend] floppy.c: Remove dead/commented out code

2007-03-19 Thread Jesper Juhl

This removes commented out/dead code from floppy.c

The code in question has been commented out for quite a while (at least as
far as I can see) and I see no reason for it to clutter up the source any
longer.
If, in the future, someone sees a real need to resurrect this code then
they can still find it in the revision history.
Let's kill the cruft :)

Signed-off-by: Jesper Juhl [EMAIL PROTECTED]
Acked-by: Alan Cox [EMAIL PROTECTED]
Acked-by: Bartlomiej Zolnierkiewicz [EMAIL PROTECTED]
---

diff --git a/drivers/block/floppy.c b/drivers/block/floppy.c
index dea7c4f..9a52ca6 100644
--- a/drivers/block/floppy.c
+++ b/drivers/block/floppy.c
@@ -774,8 +774,6 @@ static int disk_change(int drive)
current_type[drive] = NULL;
floppy_sizes[TOMINOR(drive)] = MAX_DISK_SIZE  1;
}
-
-   /* USETF(FD_DISK_NEWCHANGE); */
return 1;
} else {
UDRS-last_checked = jiffies;
@@ -1325,7 +1323,6 @@ static void fdc_specify(void)
if (FDCS-need_configure  FDCS-version = FDC_82072A) {
fdc_configure();
FDCS-need_configure = 0;
-   /* DPRINT(FIFO enabled\n); */
}
 
switch (raw_cmd-rate  0x03) {
@@ -1917,8 +1914,6 @@ static void floppy_shutdown(unsigned long data)
is_alive(floppy shutdown);
 }
 
-/* typedef void (*timeout_fn)(unsigned long); */
-
 /* start motor, check media-changed condition and write protection */
 static int start_motor(void (*function)(void))
 {
@@ -2564,19 +2559,6 @@ static void copy_buffer(int ssize, int max_sector, int 
max_sector_2)
 #endif
 }
 
-#if 0
-static inline int check_dma_crossing(char *start,
-   unsigned long length, char *message)
-{
-   if (CROSS_64KB(start, length)) {
-   printk(DMA xfer crosses 64KB boundary in %s %p-%p\n,
-  message, start, start + length);
-   return 1;
-   } else
-   return 0;
-}
-#endif
-
 /* work around a bug in pseudo DMA
  * (on some FDCs) pseudo DMA does not stop when the CPU stops
  * sending data.  Hence we need a different way to signal the
@@ -2768,7 +2750,6 @@ static int make_raw_rw_request(void)
 */
if (!direct || (indirect * 2  direct * 3 
 *errors  DP-max_errors.read_track 
-/* !TESTF(FD_NEED_TWADDLE)  */
 ((!probing
   || (DP-read_track  (1  DRS-probed_format)) {
max_size = current_req-nr_sectors;
@@ -2782,9 +2763,6 @@ static int make_raw_rw_request(void)
   indirect, direct, fsector_t);
return 0;
}
-   /* check_dma_crossing(raw_cmd-kernel_data,
-  raw_cmd-length,
-  end of make_raw_request [1]); */
virtualdmabug_workaround();
return 2;
}
@@ -2834,8 +2812,6 @@ static int make_raw_rw_request(void)
raw_cmd-length = ((raw_cmd-length - 1) | (ssize - 1)) + 1;
raw_cmd-length = 9;
 #ifdef FLOPPY_SANITY_CHECK
-   /*check_dma_crossing(raw_cmd-kernel_data, raw_cmd-length,
-  end of make_raw_request); */
if ((raw_cmd-length  current_count_sectors  9) ||
(raw_cmd-kernel_data != current_req-buffer 
 CT(COMMAND) == FD_WRITE 
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/