From mboxrd@z Thu Jan 1 00:00:00 1970 From: Grygorii Strashko Subject: Re: [4.4-rc][PATCH v2] ARM: dts: am4372: fix clock source for arm twd and global timers Date: Tue, 8 Dec 2015 19:41:39 +0200 Message-ID: <56671653.4030002@ti.com> References: <1448898998-32749-1-git-send-email-grygorii.strashko@ti.com> <20151203163551.GQ23396@atomide.com> <5660843E.3000801@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5660843E.3000801-l0cyMroinI0@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Tony Lindgren , t-kristo-l0cyMroinI0@public.gmane.org, Felipe Balbi Cc: nsekhar-l0cyMroinI0@public.gmane.org, linux-omap-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Mark Rutland List-Id: devicetree@vger.kernel.org Hi Tony, Felipe, Tero, On 12/03/2015 08:04 PM, Grygorii Strashko wrote: > On 12/03/2015 06:35 PM, Tony Lindgren wrote: >> * Grygorii Strashko [151130 07:58]: >>> ARM TWD and Global timer are clocked by PERIPHCLK which is MPU_CLK/2. >>> But now they are clocked by dpll_mpu_m2_ck == MPU_CLK and, as result. >>> Timekeeping core misbehaves. For example, execution of command >>> "sleep 5" will take 10 sec instead of 5. >>> >>> Hence, fix it by adding mpu_periphclk ("fixed-factor-clock") and use >>> it for clocking ARM TWD and Global timer (same way as on OMAP4). >>> >>> Cc: Tony Lindgren >>> Cc: Felipe Balbi >>> Cc: Tero Kristo >>> Fixes:commit 8cbd4c2f6a99 ("arm: boot: dts: am4372: add ARM timers and SCU nodes") >>> Signed-off-by: Grygorii Strashko >> >> Is this fix alone enough or do you still need to disable other the ARM >> timers in the board specific dts file? >> > > This one is independent and it's required for -rc, because ARM TWD timer will be selected > now for am43xx even if http://www.spinics.net/lists/arm-kernel/msg459649.html > is not merged yet ;) in case of omap2plus_defconfig build. > > Are there any comments objection? Could this patch be merged as part of -rc cycle? -- regards, -grygorii -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html