From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 1 Jul 2016 08:04:00 +0200 From: Jean-Francois Moine To: Maxime Ripard Cc: Emilio Lopez , Chen-Yu Tsai , Stephen Boyd , Michael Turquette , linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-sunxi@googlegroups.com Subject: Re: [PATCH 1/3] clk: sunxi: Add a driver for the CCU Message-Id: <20160701080400.fc6070646ce7c98fe0e63bb5@free.fr> In-Reply-To: <20160630211635.GE5485@lukather> References: <20160628204502.GG5550@lukather> <20160629101256.93895e6ff9184efa340f69dd@free.fr> <20160630211635.GE5485@lukather> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 List-ID: On Thu, 30 Jun 2016 23:16:35 +0200 Maxime Ripard wrote: > > - in case a driver requests a reset, this last one should exist. > > But, this reset may point to a void one (reg =3D null) when the real > > reset has been moved to the prepare/unprepare of the associated clock. >=20 > And what if this driver wants to be reset, by calling > reset_control_reset? You realize you're going against the semantics of > the reset and clock APIs, and what the hardware expose here right? If a driver really wants a reset, the reset stuff is not included in the clock prepare. --=20 Ken ar c'henta=F1 | ** Breizh ha Linux atav! ** Jef | http://moinejf.free.fr/