staging: comedi: s626: #if 0 out the "SAA7146 BUG WORKAROUND"
authorH Hartley Sweeten <hartleys@visionengravers.com>
Mon, 24 Sep 2012 20:37:52 +0000 (13:37 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 26 Sep 2012 16:37:32 +0000 (09:37 -0700)
Until it's determined if this workaround can be removed, block
out the code with an #if 0/#endif and remove the individual
comments on each line.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/s626.c

index cd09c37570bc7d1d92e27fd389ffa99538a3ec08..4315892740c994bb51fa013b3b45521df8e0a37e 100644 (file)
@@ -2459,16 +2459,10 @@ static int s626_allocate_dma_buffers(struct comedi_device *dev)
 
 static void s626_initialize(struct comedi_device *dev)
 {
-/*   uint8_t   PollList; */
-/*   uint16_t  AdcData; */
-/*   uint16_t  StartVal; */
-/*   uint16_t  index; */
-/*   unsigned int data[16]; */
        dma_addr_t pPhysBuf;
        uint16_t chan;
        int i;
 
-
        /*  enab DEBI and audio pins, enable I2C interface. */
        MC_ENABLE(P_MC1, MC1_DEBI | MC1_AUDIO | MC1_I2C);
        /*  Configure DEBI operating mode. */
@@ -2568,38 +2562,50 @@ static void s626_initialize(struct comedi_device *dev)
        /*  RPS program performs no explicit mem writes. */
        WR7146(P_RPS1_TOUT, 0); /*  Disable RPS timeouts. */
 
-       /* SAA7146 BUG WORKAROUND.  Initialize SAA7146 ADC interface
-        * to a known state by invoking ADCs until FB BUFFER 1
-        * register shows that it is correctly receiving ADC data.
-        * This is necessary because the SAA7146 ADC interface does
-        * not start up in a defined state after a PCI reset.
+#if 0
+       /*
+        * SAA7146 BUG WORKAROUND
+        *
+        * Initialize SAA7146 ADC interface to a known state by
+        * invoking ADCs until FB BUFFER 1 register shows that it
+        * is correctly receiving ADC data. This is necessary
+        * because the SAA7146 ADC interface does not start up in
+        * a defined state after a PCI reset.
         */
 
-/*     PollList = EOPL;                // Create a simple polling */
-/*                             // list for analog input */
-/*                             // channel 0. */
-/*     ResetADC( dev, &PollList ); */
-
-/*     s626_ai_rinsn(dev,dev->subdevices,NULL,data); //( &AdcData ); // */
-/*                                                     //Get initial ADC */
-/*                                                     //value. */
-
-/*     StartVal = data[0]; */
-
-/*     // VERSION 2.01 CHANGE: TIMEOUT ADDED TO PREVENT HANGED EXECUTION. */
-/*     // Invoke ADCs until the new ADC value differs from the initial */
-/*     // value or a timeout occurs.  The timeout protects against the */
-/*     // possibility that the driver is restarting and the ADC data is a */
-/*     // fixed value resulting from the applied ADC analog input being */
-/*     // unusually quiet or at the rail. */
-
-/*     for ( index = 0; index < 500; index++ ) */
-/*       { */
-/*     s626_ai_rinsn(dev,dev->subdevices,NULL,data); */
-/*     AdcData = data[0];      //ReadADC(  &AdcData ); */
-/*     if ( AdcData != StartVal ) */
-/*             break; */
-/*       } */
+       {
+       uint8_t PollList;
+       uint16_t AdcData;
+       uint16_t StartVal;
+       uint16_t index;
+       unsigned int data[16];
+
+       /* Create a simple polling list for analog input channel 0 */
+       PollList = EOPL;
+       ResetADC(dev, &PollList);
+
+       /* Get initial ADC value */
+       s626_ai_rinsn(dev, dev->subdevices, NULL, data);
+       StartVal = data[0];
+
+       /*
+        * VERSION 2.01 CHANGE: TIMEOUT ADDED TO PREVENT HANGED EXECUTION.
+        *
+        * Invoke ADCs until the new ADC value differs from the initial
+        * value or a timeout occurs.  The timeout protects against the
+        * possibility that the driver is restarting and the ADC data is a
+        * fixed value resulting from the applied ADC analog input being
+        * unusually quiet or at the rail.
+        */
+       for (index = 0; index < 500; index++) {
+               s626_ai_rinsn(dev, dev->subdevices, NULL, data);
+               AdcData = data[0];
+               if (AdcData != StartVal)
+                       break;
+       }
+
+       }
+#endif /* SAA7146 BUG WORKAROUND */
 
        /*  end initADC */