diff mbox

arm64: dts: rockchip: vdd_log is not an i2c slave

Message ID 20180218154359.9362-1-vicencb@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Vicente Bergas Feb. 18, 2018, 3:43 p.m. UTC
The vdd_log power supply is controlled by a PWM pin, not by i2c register access.
There is a boot message that reports an error about not being able to bring that supply up.
Also, the eMMC is not detected and /dev/mmcblk2 does not exist.

References:
[1]
Schematic page 16, search for LOG_DVS_PWM
https://dl.vamrs.com/products/sapphire_excavator/RK_SAPPHIRE_SOCBOARD_RK3399_LPDDR3D178P232SD8_V12_20161109HXS.pdf
(alternate link)
http://opensource.rock-chips.com/images/f/f0/RK_Sapphire_RK3399.rar

[2]
arch/arm64/boot/dts/rockchip/rk3399-firefly.dts

Signed-off-by: Vicente Bergas <vicencb@gmail.com>
---
 arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)

Comments

Heiko Stübner Feb. 19, 2018, 6:01 a.m. UTC | #1
Am Sonntag, 18. Februar 2018, 16:43:59 CET schrieb Vicente Bergas:
> The vdd_log power supply is controlled by a PWM pin, not by i2c register access.
> There is a boot message that reports an error about not being able to bring that supply up.
> Also, the eMMC is not detected and /dev/mmcblk2 does not exist.
> 
> References:
> [1]
> Schematic page 16, search for LOG_DVS_PWM
> https://dl.vamrs.com/products/sapphire_excavator/RK_SAPPHIRE_SOCBOARD_RK3399_LPDDR3D178P232SD8_V12_20161109HXS.pdf
> (alternate link)
> http://opensource.rock-chips.com/images/f/f0/RK_Sapphire_RK3399.rar
> 
> [2]
> arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
> 
> Signed-off-by: Vicente Bergas <vicencb@gmail.com>

I've adapted the commit message a bit and applied the patch
for 4.17

For the emmc, that is a separate issue and you'll need to look at the logs
to see if there is anything shown. Especially device-numbering
(your mmcblk2) is not guaranteed, so this could very well also be
mmcblk0 or mmcblk1


Thanks
Heiko
Vicente Bergas Feb. 19, 2018, 5:34 p.m. UTC | #2
On Mon, Feb 19, 2018 at 7:01 AM, Heiko Stuebner <heiko@sntech.de> wrote:
> Am Sonntag, 18. Februar 2018, 16:43:59 CET schrieb Vicente Bergas:
>> The vdd_log power supply is controlled by a PWM pin, not by i2c register access.
>> There is a boot message that reports an error about not being able to bring that supply up.
>> Also, the eMMC is not detected and /dev/mmcblk2 does not exist.
>>
>> References:
>> [1]
>> Schematic page 16, search for LOG_DVS_PWM
>> https://dl.vamrs.com/products/sapphire_excavator/RK_SAPPHIRE_SOCBOARD_RK3399_LPDDR3D178P232SD8_V12_20161109HXS.pdf
>> (alternate link)
>> http://opensource.rock-chips.com/images/f/f0/RK_Sapphire_RK3399.rar
>>
>> [2]
>> arch/arm64/boot/dts/rockchip/rk3399-firefly.dts
>>
>> Signed-off-by: Vicente Bergas <vicencb@gmail.com>
>
> I've adapted the commit message a bit and applied the patch
> for 4.17
>
> For the emmc, that is a separate issue and you'll need to look at the logs
> to see if there is anything shown. Especially device-numbering
> (your mmcblk2) is not guaranteed, so this could very well also be
> mmcblk0 or mmcblk1
>
>
> Thanks
> Heiko

Sorry, I did not mention it because I thought it was implicit:
 both issues, the one related to the vdd_log and the other related to
eMMC, are fixed with this patch.
Anyways, thanks for applying it!

Regards,
  Vicente.
diff mbox

Patch

diff --git a/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi b/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
index 0f873c897d0d..46c95ccfd5cb 100644
--- a/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
+++ b/arch/arm64/boot/dts/rockchip/rk3399-sapphire.dtsi
@@ -143,6 +143,17 @@ 
 		regulator-always-on;
 		vin-supply = <&vcc_sys>;
 	};
+
+	vdd_log: vdd-log {
+		compatible = "pwm-regulator";
+		pwms = <&pwm2 0 25000 1>;
+		regulator-name = "vdd_log";
+		regulator-always-on;
+		regulator-boot-on;
+		regulator-min-microvolt = <800000>;
+		regulator-max-microvolt = <1400000>;
+		vin-supply = <&vcc_sys>;
+	};
 };
 
 &cpu_l0 {
@@ -421,17 +432,6 @@ 
 			regulator-off-in-suspend;
 		};
 	};
-
-	vdd_log: vdd-log {
-		compatible = "pwm-regulator";
-		pwms = <&pwm2 0 25000 1>;
-		regulator-name = "vdd_log";
-		regulator-always-on;
-		regulator-boot-on;
-		regulator-min-microvolt = <800000>;
-		regulator-max-microvolt = <1400000>;
-		vin-supply = <&vcc_sys>;
-	};
 };
 
 &i2c3 {