Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752139AbdF3AUW (ORCPT ); Thu, 29 Jun 2017 20:20:22 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:38028 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751601AbdF3AUU (ORCPT ); Thu, 29 Jun 2017 20:20:20 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 8CDC860A96 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=sboyd@codeaurora.org Date: Thu, 29 Jun 2017 17:20:08 -0700 From: Stephen Boyd To: Gabriel FERNANDEZ Cc: Rob Herring , Mark Rutland , Russell King , Maxime Coquelin , Alexandre TORGUE , Michael Turquette , Nicolas Pitre , Arnd Bergmann , "daniel.thompson@linaro.org" , "andrea.merello@gmail.com" , "radoslaw.pietrzyk@gmail.com" , Lee Jones , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "linux-clk@vger.kernel.org" , Ludovic BARRE , Olivier BIDEAU , Amelie DELAUNAY , "gabriel.fernandez.st@gmail.com" Subject: Re: [RESEND PATCH v4] clk: stm32h7: Add stm32h743 clock driver Message-ID: <20170630002008.GD22780@codeaurora.org> References: <1496818794-14771-1-git-send-email-gabriel.fernandez@st.com> <20170621220703.GI4493@codeaurora.org> <110cf67f-e1d6-1f99-953c-6b9639bb4910@st.com> <20170628155956.GA5316@codeaurora.org> <7c0c3861-cf61-a9e2-cd5d-2ae30b8f32fa@st.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <7c0c3861-cf61-a9e2-cd5d-2ae30b8f32fa@st.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1002 Lines: 30 On 06/29, Gabriel FERNANDEZ wrote: > > > On 06/28/2017 05:59 PM, Stephen Boyd wrote: > > On 06/27, Gabriel FERNANDEZ wrote: > >> > >> On 06/22/2017 12:07 AM, Stephen Boyd wrote: > >>> readl_poll_timeout? > >>> > >> if i use readl_poll_timeout (wich use 'ktime_get()') it can be > >> operational only after the selection of clocksource ? (device_initcall). > >> And then if a driver turn on a clock before, it could blocked the linux > >> console ? > >> > > Ok. I wonder if we could add some sort of starting check to > > readl_poll_timeout() that tests system_state for booting vs. > > scheduling? That should be sufficient to handle this case? > > > Oops i think i understood my problem... > i used readl_poll_timeout in atomic context. > I have to move my code in the .prepare ops. > > If you are ok with that i will send a v5 > There's readl_poll_timeout_atomic() for those modes. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project