On Tuesday, April 15, 2014 11:58 PM, Chase Southwood wrote:
> We can remove this function from the boardinfo and move the code from
> hwdrv_apci1564.c into addi_apci_1564.c since it is the only reset function
> used by the driver. The function was also messy and failed to reset a few
> registers, these issues were fixed on the move.
>
> Signed-off-by: Chase Southwood <chase.southw...@yahoo.com>
> ---
> 2: *Changed order of register accesses for digital input registers back to
> original ordering.
> *Removed read of digital output status register and reordered the accesses
> of digital output registers to agree with the order found in addi_apci_2032.
> *Fixed copy/paste error in the counter register reset lines.
>
> Ian, I addressed your comments as above.
> Chase
>  .../comedi/drivers/addi-data/hwdrv_apci1564.c      | 28 ---------------
>  drivers/staging/comedi/drivers/addi_apci_1564.c    | 40 
> +++++++++++++++++++++-
>  2 files changed, 39 insertions(+), 29 deletions(-)
>
> diff --git a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c 
> b/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c
> index 4db3bea..45617cf 100644
> --- a/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c
> +++ b/drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c
> @@ -576,31 +576,3 @@ static void apci1564_interrupt(int irq, void *d)
>       }
>       return;
>  }
> -
> -static int apci1564_reset(struct comedi_device *dev)
> -{
> -     struct addi_private *devpriv = dev->private;
> -
> -     /* disable the interrupts */
> -     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DI_IRQ_REG);
> -     /* Reset the interrupt status register */
> -     inl(devpriv->i_IobaseAmcc + APCI1564_DI_INT_STATUS_REG);
> -     /* Disable the and/or interrupt */
> -     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DI_INT_MODE1_REG);
> -     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DI_INT_MODE2_REG);
> -     devpriv->b_DigitalOutputRegister = 0;
> -     ui_Type = 0;
> -     /* Resets the output channels */
> -     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DO_REG);
> -     /* Disables the interrupt. */
> -     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DO_INT_CTRL_REG);
> -     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_WDOG_RELOAD_REG);
> -     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_TIMER_REG);
> -     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_TIMER_CTRL_REG);
> -
> -     outl(0x0, dev->iobase + APCI1564_TCW_CTRL_REG(APCI1564_COUNTER1));
> -     outl(0x0, dev->iobase + APCI1564_TCW_CTRL_REG(APCI1564_COUNTER2));
> -     outl(0x0, dev->iobase + APCI1564_TCW_CTRL_REG(APCI1564_COUNTER3));
> -     outl(0x0, dev->iobase + APCI1564_TCW_CTRL_REG(APCI1564_COUNTER4));
> -     return 0;
> -}
> diff --git a/drivers/staging/comedi/drivers/addi_apci_1564.c 
> b/drivers/staging/comedi/drivers/addi_apci_1564.c
> index 11aa0bd..546dad7 100644
> --- a/drivers/staging/comedi/drivers/addi_apci_1564.c
> +++ b/drivers/staging/comedi/drivers/addi_apci_1564.c
> @@ -4,6 +4,7 @@
>  #include "../comedidev.h"
>  #include "comedi_fc.h"
>  #include "amcc_s5933.h"
> +#include "addi_watchdog.h"
>  
>  #include "addi-data/addi_common.h"
>  
> @@ -22,7 +23,6 @@ static const struct addi_board apci1564_boardtypes[] = {
>               .i_DoMaxdata            = 0xffffffff,
>               .i_Timer                = 1,
>               .interrupt              = apci1564_interrupt,
> -             .reset                  = apci1564_reset,

You can't do this yet.

The addi_acpi_1564 driver needs to be separated from addi_common.c before you
can start ripping the callbacks out of this struct.

If you compile test this you will see:
  CC [M]  drivers/staging/comedi/drivers/addi_apci_1564.o
In file included from drivers/staging/comedi/drivers/addi_apci_1564.c:11:0:
drivers/staging/comedi/drivers/addi-data/hwdrv_apci1564.c:581:12: warning: 
'apci1564_reset' defined but not used [-Wunused-function]

>               .di_config              = apci1564_di_config,
>               .di_bits                = apci1564_di_insn_bits,
>               .do_config              = apci1564_do_config,
> @@ -34,6 +34,44 @@ static const struct addi_board apci1564_boardtypes[] = {
>       },
>  };
>  
> +static int apci1564_reset(struct comedi_device *dev)
> +{
> +     struct addi_private *devpriv = dev->private;
> +
> +     ui_Type = 0;
> +
> +     /* Disable the input interrupts and reset status register */
> +     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DI_IRQ_REG);
> +     inl(devpriv->i_IobaseAmcc + APCI1564_DI_INT_STATUS_REG);
> +     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DI_INT_MODE1_REG);
> +     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DI_INT_MODE2_REG);
> +
> +     /* Reset the output channels and disable interrupts */
> +     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DO_IRQ_REG);

This register is read only. Bit 0 is set if there is a digital output IRQ 
pending.

> +     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DO_REG);
> +     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_DO_INT_CTRL_REG);
> +
> +     /* Reset the watchdog registers */
> +     addi_watchdog_reset(devpriv->i_IobaseAmcc + APCI1564_WDOG_REG);
> +
> +     /* Reset the timer registers */
> +     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_TIMER_CTRL_REG);
> +     outl(0x0, devpriv->i_IobaseAmcc + APCI1564_TIMER_RELOAD_REG);
> +
> +     /* Reset the counter registers */
> +     outl(0x0, dev->iobase + APCI1564_TCW_CTRL_REG(APCI1564_COUNTER1));
> +     outl(0x0, dev->iobase + APCI1564_TCW_CTRL_REG(APCI1564_COUNTER2));
> +     outl(0x0, dev->iobase + APCI1564_TCW_CTRL_REG(APCI1564_COUNTER3));
> +     outl(0x0, dev->iobase + APCI1564_TCW_CTRL_REG(APCI1564_COUNTER4));
> +
> +     outl(0x0, dev->iobase + APCI1564_TCW_RELOAD_REG(APCI1564_COUNTER1));
> +     outl(0x0, dev->iobase + APCI1564_TCW_RELOAD_REG(APCI1564_COUNTER2));
> +     outl(0x0, dev->iobase + APCI1564_TCW_RELOAD_REG(APCI1564_COUNTER3));
> +     outl(0x0, dev->iobase + APCI1564_TCW_RELOAD_REG(APCI1564_COUNTER4));

It's probably not necessary to clear the reload values since you have disabled 
all the timers.
But, it shouldn't hurt anything.

> +
> +     return 0;
> +}
> +
>  static int apci1564_auto_attach(struct comedi_device *dev,
>                               unsigned long context)
>  {

Regards,
Hartley

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to