From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753559AbbCSWzI (ORCPT ); Thu, 19 Mar 2015 18:55:08 -0400 Received: from down.free-electrons.com ([37.187.137.238]:35041 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751653AbbCSWzF (ORCPT ); Thu, 19 Mar 2015 18:55:05 -0400 Date: Thu, 19 Mar 2015 23:53:44 +0100 From: Maxime Ripard To: Daniel Lezcano , Thomas Gleixner Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v3 0/5] clocksource: sun5i: Support parent clock rate changes Message-ID: <20150319225344.GF4255@lukather> References: <1425550856-12928-1-git-send-email-maxime.ripard@free-electrons.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="l+goss899txtYvYf" Content-Disposition: inline In-Reply-To: <1425550856-12928-1-git-send-email-maxime.ripard@free-electrons.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --l+goss899txtYvYf Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi Daniel, On Thu, Mar 05, 2015 at 11:20:51AM +0100, Maxime Ripard wrote: > Hi, >=20 > The Allwinner HS timers have the AHB clock as their parent > clock. Since this clock is shared with other devices, we could very > well have another driver requesting a rate change of that clock, > making our timer change frequency at the same time. >=20 > This is especially true on the A31, where the DMA controller needs to > do such a rate change, making the HS timer unreliable at the time on > the A31. >=20 > This serie makes some cleanups and implements clock notifiers to be > able to reflect such rate changes and make sure that the timer is > always working. >=20 > Maxime >=20 > Changes from v2: > - Rebased on top of v4.0-rc1 > - Removed the local_irq_save/restore around clockevents_update_freq >=20 > Changes from v1: > - Changed the interrupt name to its previous value >=20 > Maxime Ripard (5): > clocksource: sun5i: Switch to request_irq > clocksource: sun5i: Use of_io_request_and_map > clocksource: sun5i: Remove sched_clock > clocksource: sun5i: Refactor the current code > clocksource: sun5i: Add clock notifiers Have these patches been merged? If not, it woulde be great if the third one ("clocksource: sun5i: Remove sched_clock") was merged for 4.0. The sched_clock we use on some system is this timer's, and since we started using cpufreq, the cpu clock (that is one of the timer's clock indirect parent) now changes, along with the actual sched_clock rate. We can safely remove the sched_clock on those systems, since we have other reliable sched_clock in the system. Thanks, Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com --l+goss899txtYvYf Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVC1N4AAoJEBx+YmzsjxAg37UP/2CuaWTnQi74tPXq6fHk9REt siRRrY4reWPnO0PKSRBzLC3QIvSBJfpuVbW1C2qP+FXTa4lQoskkqYIXeBKeQ2C4 h6bGS640+IiwXAacTzMI8c1W9k3uq1SN7NZEAr228SI5V131oQM+nD23KN1VZIzy 7GO82UdxRuqoOfNWuwylAzK/mWQ9MxWv2+8oKhI36bZO2p4WP35TSZySwH9uDuBi rNsrDBtu7r9/K3bsb59zg+NgSsFYRy6o2aRflXPG3P8DTEPgYvmeB5F17ZfeSnWq 3p5/EjXpcPRpbq1ZAGR1svLUudLvbDj5Z58L+LdptF84r/YsHY0pkJrwKNvdjdVh QgFvVtKEDWhwngt0q4Oo2FC6khV9yEUgApOoWjGvwzf/aZUSWB5KAyrHeLRzjSKN AxUXUK2cUJ6sDtEukRAOXojDwyBLBbdGLFcYb+DchzUBrUosZVI8lNSBEpHidrCk SOWZfYP1uhWq/44AraPczeCQrXA4tj+bYFETUNwL3+h96QMYzqRtduMaI5mUOC1T naooo9ylCxTpTZFmj+FaorEqzywQ7EK2zJUOfQZVL4PzBE3oUNVsh1sAoK5YfDXB MwLIGYZpDIohQGAG0o+ox3q5kdAWYHSTgu1RoClPJFRXI75Bxr71+zL5/KacFTvf vUERte6u21jygWwA18De =SKR1 -----END PGP SIGNATURE----- --l+goss899txtYvYf--