From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Boyd Subject: Re: [RESEND PATCH v4] clk: stm32h7: Add stm32h743 clock driver Date: Thu, 29 Jun 2017 17:20:08 -0700 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 Return-path: Content-Disposition: inline In-Reply-To: <7c0c3861-cf61-a9e2-cd5d-2ae30b8f32fa@st.com> Sender: linux-clk-owner@vger.kernel.org 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" List-Id: devicetree@vger.kernel.org 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