]> asedeno.scripts.mit.edu Git - linux.git/commitdiff
mfd: rc5t583: Fix warning messages
authorVenu Byravarasu <vbyravarasu@nvidia.com>
Thu, 23 Aug 2012 06:51:16 +0000 (12:21 +0530)
committerSamuel Ortiz <sameo@linux.intel.com>
Sun, 23 Sep 2012 18:51:33 +0000 (20:51 +0200)
Two variables named master_int and sleepseq_val, were just
declared without initialization. Pointers to these variables
were passed to mfd read function. After that these variables
were used for some of the logical operations.

Though logically there is nothing wrong, compiler is complaining
that the variables may be used uninitialized.

Hence fixing these warning messages by initializing them.

Signed-off-by: Venu Byravarasu <vbyravarasu@nvidia.com>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
drivers/mfd/rc5t583-irq.c
drivers/mfd/rc5t583.c

index fa6f80fad5f17aa5fe3e9f12c6efc98705ee8789..fe00cdd6f83da134ebde854613388df07706cfca 100644 (file)
@@ -255,7 +255,7 @@ static irqreturn_t rc5t583_irq(int irq, void *data)
 {
        struct rc5t583 *rc5t583 = data;
        uint8_t int_sts[RC5T583_MAX_INTERRUPT_MASK_REGS];
-       uint8_t master_int;
+       uint8_t master_int = 0;
        int i;
        int ret;
        unsigned int rtc_int_sts = 0;
index 3a8fa88567b18385461fbed0bbc9ff30e8d0fffc..d6b50728ea32fbe7178e7579837049755f7da454 100644 (file)
@@ -85,7 +85,7 @@ static int __rc5t583_set_ext_pwrreq1_control(struct device *dev,
        int id, int ext_pwr, int slots)
 {
        int ret;
-       uint8_t sleepseq_val;
+       uint8_t sleepseq_val = 0;
        unsigned int en_bit;
        unsigned int slot_bit;