This is a note to let you know that I've just added the patch titled

    regulator: tps65910: Add missing breaks in switch/case

to the 3.0-stable tree which can be found at:
    
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     regulator-tps65910-add-missing-breaks-in-switch-case.patch
and it can be found in the queue-3.0 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <sta...@kernel.org> know about it.


>From d04156bca629740a661fd0738cd69ba1f08b2b20 Mon Sep 17 00:00:00 2001
From: Axel Lin <axel....@gmail.com>
Date: Sun, 10 Jul 2011 21:44:09 +0800
Subject: regulator: tps65910: Add missing breaks in switch/case

From: Axel Lin <axel....@gmail.com>

commit d04156bca629740a661fd0738cd69ba1f08b2b20 upstream.

Also add a default case in tps65910_list_voltage_dcdc to silence
'volt' may be used uninitialized in this function warning.

Signed-off-by: Axel Lin <axel....@gmail.com>
Acked-by: Mark Brown <broo...@opensource.wolfsonmicro.com>
Signed-off-by: Liam Girdwood <l...@slimlogic.co.uk>
Cc: Johan Hovold <jhov...@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gre...@suse.de>

---
 drivers/regulator/tps65910-regulator.c |    7 +++++++
 1 file changed, 7 insertions(+)

--- a/drivers/regulator/tps65910-regulator.c
+++ b/drivers/regulator/tps65910-regulator.c
@@ -759,8 +759,13 @@ static int tps65910_list_voltage_dcdc(st
                mult = (selector / VDD1_2_NUM_VOLTS) + 1;
                volt = VDD1_2_MIN_VOLT +
                                (selector % VDD1_2_NUM_VOLTS) * VDD1_2_OFFSET;
+               break;
        case TPS65911_REG_VDDCTRL:
                volt = VDDCTRL_MIN_VOLT + (selector * VDDCTRL_OFFSET);
+               break;
+       default:
+               BUG();
+               return -EINVAL;
        }
 
        return  volt * 100 * mult;
@@ -898,9 +903,11 @@ static __devinit int tps65910_probe(stru
        case TPS65910:
                pmic->get_ctrl_reg = &tps65910_get_ctrl_register;
                info = tps65910_regs;
+               break;
        case TPS65911:
                pmic->get_ctrl_reg = &tps65911_get_ctrl_register;
                info = tps65911_regs;
+               break;
        default:
                pr_err("Invalid tps chip version\n");
                return -ENODEV;


Patches currently in stable-queue which might be from axel....@gmail.com are

queue-3.0/regulator-tps65910-add-missing-breaks-in-switch-case.patch
queue-3.0/tty-add-spi-prefix-for-spi-modalias.patch
queue-3.0/asoc-soc-jack-fix-checking-return-value-of-request_any_context_irq.patch

_______________________________________________
stable mailing list
stable@linux.kernel.org
http://linux.kernel.org/mailman/listinfo/stable

Reply via email to