regulator:max77686: Correct GPIOs for BUCK2 DVS setting
authorLukasz Majewski <l.majewski@samsung.com>
Thu, 31 Oct 2013 15:52:55 +0000 (16:52 +0100)
committerChanho Park <chanho61.park@samsung.com>
Tue, 18 Nov 2014 02:45:02 +0000 (11:45 +0900)
Before this patch SELB and DVS pins were wrongly defined in the DTS.

As a result MAX77686 always used BUCK2_DVS1 register (the DVS pins were
set all to 0). However the default DVS index (default_dvs_idx) was set to 1.

Therefore, the default value of BUCK2_DVS1 register (1.1V) was always
setup. Wrongly the driver was changing BUCK2_DVS2 register. Due to that
voltage was NOT changed at all.

The problem appeared when BOOST was ported to v3.10-mobile. It requires
around 1.3V, but default voltage is set to 1.1V (0x28). Too small voltage
resulted in OOPs at random places.

As a side note:
The MAX77686 is NOT using DVS with GPIO. It changes the value of current
voltage with I2C utilization.

Tested at: REDWOOD - Exynos4412 - rev1.

Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
arch/arm/boot/dts/exynos4412-slp_pq.dts

index 5c3bedd..e05f3fa 100644 (file)
                        reg = <0x09>;
                        #clock-cells = <1>;
 
-                       max77686,dvs_gpios = <&gpf3 1 0>,
+                       max77686,selb_gpios = <&gpf3 1 0>,
                                             <&gpf3 2 0>,
                                             <&gpf3 3 0>;
 
-                       max77686,selb_gpios = <&gpm3 0 0>,
+                       max77686,dvs_gpios = <&gpm3 0 0>,
                                              <&gpm3 1 0>,
                                              <&gpm3 2 0>;