The `set_mode_a()` and `set_mode_b()` functions use a local variable
`setup` which is initialized to the value of parameter `Setup` which is
not used further.  Get rid of the local variable and rename the
parameter to `setup`.

Signed-off-by: Ian Abbott <abbo...@mev.co.uk>
---
 drivers/staging/comedi/drivers/s626.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/comedi/drivers/s626.c 
b/drivers/staging/comedi/drivers/s626.c
index fcbeb28..12ebc85 100644
--- a/drivers/staging/comedi/drivers/s626.c
+++ b/drivers/staging/comedi/drivers/s626.c
@@ -1958,12 +1958,11 @@ static uint16_t get_mode_b(struct comedi_device *dev, 
struct enc_private *k)
  * ClkPol, ClkEnab, IndexSrc, IndexPol, LoadSrc.
  */
 static void set_mode_a(struct comedi_device *dev, struct enc_private *k,
-                      uint16_t Setup, uint16_t DisableIntSrc)
+                      uint16_t setup, uint16_t DisableIntSrc)
 {
        struct s626_private *devpriv = dev->private;
        uint16_t cra;
        uint16_t crb;
-       uint16_t setup = Setup; /*  Cache the Standard Setup. */
 
        /* Initialize CRA and CRB images. */
        /* Preload trigger is passed through. */
@@ -2037,12 +2036,11 @@ static void set_mode_a(struct comedi_device *dev, 
struct enc_private *k,
 }
 
 static void set_mode_b(struct comedi_device *dev, struct enc_private *k,
-                      uint16_t Setup, uint16_t DisableIntSrc)
+                      uint16_t setup, uint16_t DisableIntSrc)
 {
        struct s626_private *devpriv = dev->private;
        uint16_t cra;
        uint16_t crb;
-       uint16_t setup = Setup; /* Cache the Standard Setup. */
 
        /* Initialize CRA and CRB images. */
        /* IndexSrc field is restricted to ENC_X or IndxPol. */
-- 
1.8.3.2

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

Reply via email to