From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754020AbdLHOIU (ORCPT ); Fri, 8 Dec 2017 09:08:20 -0500 Received: from mail-wm0-f65.google.com ([74.125.82.65]:37060 "EHLO mail-wm0-f65.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753608AbdLHOIO (ORCPT ); Fri, 8 Dec 2017 09:08:14 -0500 X-Google-Smtp-Source: AGs4zMbdV7WNHIwHwtNXOtLtV5Gp8+8noI9QW4zQ379ZcDrl7zZ8cW69+gSc1KRbB7xCl3Z9kJY1Kg== Subject: Re: [PATCH v9 4/6] clocksource: stm32: add clocksource support To: Benjamin Gaignard Cc: Rob Herring , Mark Rutland , Russell King - ARM Linux , Maxime Coquelin , Alexandre Torgue , Thomas Gleixner , Arnd Bergmann , devicetree@vger.kernel.org, Linux ARM , Linux Kernel Mailing List , Benjamin Gaignard References: <20171208113250.359-1-benjamin.gaignard@st.com> <20171208113250.359-5-benjamin.gaignard@st.com> From: Daniel Lezcano Message-ID: <810270c4-c6b4-ee05-2481-ced52f7d2685@linaro.org> Date: Fri, 8 Dec 2017 15:08:09 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/12/2017 15:04, Benjamin Gaignard wrote: > 2017-12-08 14:46 GMT+01:00 Daniel Lezcano : >> On 08/12/2017 12:32, Benjamin Gaignard wrote: >>> The stm32 timer hardware is currently only used as a clock event device, >>> but it can be utilized as a clocksource as well. >>> >>> Implement this by enabling the free running counter in the hardware block >>> and converting the clock event part from a count down event timer to a >>> comparator based timer. >> >> >> Split this patch in two: >> - periodic support >> - clocksource support > > No because the way of implementing periodic support depend of how clocksource > part is implemented on the hardware. > I don't want to code functions that I will completely remove just after Sorry, I don't get it. Can you elaborate please ? -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog