Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Content-Type: text/plain; charset=utf-8
Mime-Version: 1.0 (Mac OS X Mail 9.3 \(3124\))
Subject: Re: [PATCH AUTOSEL for 4.4 062/115] ARM: dts: rockchip: disable arm-global-timer for rk3188
From:   Alexander Kochetkov <al.kochet@gmail.com>
In-Reply-To: <20180303223010.27106-62-alexander.levin@microsoft.com>
Date:   Mon, 5 Mar 2018 02:19:56 +0300
Cc:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "stable@vger.kernel.org" <stable@vger.kernel.org>,
        Daniel Lezcano <daniel.lezcano@linaro.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <208E5E7C-3737-44B0-ACEB-5C3F20516EB0@gmail.com>
References: <20180303223010.27106-1-alexander.levin@microsoft.com> <20180303223010.27106-62-alexander.levin@microsoft.com>
To:     Sasha Levin <Alexander.Levin@microsoft.com>
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

Hello, Sasha!

Following 2 patches must be applied together with the patch:
5e0a39d0f727b35c8b7ef56ba0724c8ceb006297 =
clocksource/drivers/rockchip_timer: Implement clocksource timer
627988a66aee3c845aa2f1f874a3ddba8adb89d9 ARM: dts: rockchip: Add timer =
entries to rk3188 SoC

Alexander.

> 4 =D0=BC=D0=B0=D1=80=D1=82=D0=B0 2018 =D0=B3., =D0=B2 1:31, Sasha =
Levin <Alexander.Levin@microsoft.com> =D0=BD=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=
=D0=BB(=D0=B0):
>=20
> From: Alexander Kochetkov <al.kochet@gmail.com>
>=20
> [ Upstream commit 500d0aa918a2ea6bb918fee8adcf27dc2912bcd1 ]
>=20
> The clocksource and the sched_clock provided by the arm_global_timer
> are quite unstable because their rates depend on the cpu frequency.
>=20
> On the other side, the arm_global_timer has a higher rating than the
> rockchip_timer, it will be selected by default by the time framework
> while we want to use the stable rockchip clocksource.
>=20
> Let's disable the arm_global_timer in order to have the rockchip
> clocksource selected by default.
>=20
> Signed-off-by: Alexander Kochetkov <al.kochet@gmail.com>
> Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
> Reviewed-by: Heiko Stuebner <heiko@sntech.de>
> Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
> ---
> arch/arm/boot/dts/rk3188.dtsi | 1 +
> 1 file changed, 1 insertion(+)
>=20
> diff --git a/arch/arm/boot/dts/rk3188.dtsi =
b/arch/arm/boot/dts/rk3188.dtsi
> index 6399942f1840..0e5142b6e914 100644
> --- a/arch/arm/boot/dts/rk3188.dtsi
> +++ b/arch/arm/boot/dts/rk3188.dtsi
> @@ -513,6 +513,7 @@
>=20
> &global_timer {
> 	interrupts =3D <GIC_PPI 11 0xf04>;
> +	status =3D "disabled";
> };
>=20
> &local_timer {
> --=20
> 2.14.1