From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751392AbbCTVDR (ORCPT ); Fri, 20 Mar 2015 17:03:17 -0400 Received: from down.free-electrons.com ([37.187.137.238]:52782 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751151AbbCTVDP (ORCPT ); Fri, 20 Mar 2015 17:03:15 -0400 Date: Fri, 20 Mar 2015 21:56:55 +0100 From: Maxime Ripard To: Daniel Lezcano , Hans de Goede , Chen-Yu Tsai Cc: Thomas Gleixner , 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: <20150320205655.GJ4255@lukather> References: <1425550856-12928-1-git-send-email-maxime.ripard@free-electrons.com> <20150319225344.GF4255@lukather> <550C28EC.2040800@linaro.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="SdaPbLtAangIkrMZ" Content-Disposition: inline In-Reply-To: <550C28EC.2040800@linaro.org> 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 --SdaPbLtAangIkrMZ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Fri, Mar 20, 2015 at 03:04:28PM +0100, Daniel Lezcano wrote: > On 03/19/2015 11:53 PM, Maxime Ripard wrote: > >Hi Daniel, > > > >On Thu, Mar 05, 2015 at 11:20:51AM +0100, Maxime Ripard wrote: > >>Hi, > >> > >>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. > >> > >>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. > >> > >>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. > >> > >>Maxime > >> > >>Changes from v2: > >> - Rebased on top of v4.0-rc1 > >> - Removed the local_irq_save/restore around clockevents_update_freq > >> > >>Changes from v1: > >> - Changed the interrupt name to its previous value > >> > >>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. >=20 > Ok, I applied the patch for v4.0-rc5 but I had to fix a conflict and chan= ge > the changelog. It looks fine. Note that the rest of the serie should also be merged, this is just a temporary measure for 4.0. > Mind to test it works ? >=20 > git.linaro.org/people/daniel.lezcano/linux.git clockevents/v4.0-rc4 Unfortunately, I won't have access to my boards for ~10 days due to travel to ELC. Chen-Yu, Hans, could you test this and see if it works? Thanks, Maxime --=20 Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com --SdaPbLtAangIkrMZ Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJVDImXAAoJEBx+YmzsjxAg9hQP/3pp2TxgNFSv4etl8jc8h1Hu ro6/nYjsbS4ieuxQHGgR5ulQ7vj3tvgQy3JZX8Vfqe7aV71aqV5R2motf/Si9T6M YpPjW9YQNGE/nbx/8ecFvAtrkQJ6W7Km5hEmjoSa8jrJBIYAm24GgCrjp3ZIg5Vk sAAVJI+3+sVmee8uP0T6K/pmACBE7Ll5NApBwPAOpZjQYADxPGxpurOW30be3yJj oJ5pPwQf/rjcQtoxJXkQpZFrBa4faBmRsGy8K6EhOwip4AzWeI2U2cbCNkIVA4SP 7alR5ybU3dW7gWvwASuILERLYZ6zS/C2QUhhDCYfDg3+QWK0OxQzesMn7ddpirZu Dg/jf5JrRNvMEAgpPl1kWXWJ0PojZz3eI8dMlyFPPqXHnlfKwK27q/m7Noh5t+xC /VVKKC7ryTb4NC2CNLMUhlNyI0TjXPfhEyn2WrSvv/Ef12mUPrMVVkDJlEOYDdz+ 6hkxrrhpV0X0yCQzuHJZrjcCcVkvMzmMCeCQNdCl2UPmsck8A/B1EZ6yZBe/C7IK szxyFQoHuHp5smPIYIW/U+ThwsbgYoITqqawrG+00V9zc7SDUVP1Ec9kHGPvQKMQ 5F9EpHnCxrwRp6/x6WRBiOvEGaAXY+D/Mq4uh6R3dNxZsAc438HBRLbswm58IWfa fCZAl7b6ylutQbf1p6MO =lzvE -----END PGP SIGNATURE----- --SdaPbLtAangIkrMZ--