From mboxrd@z Thu Jan 1 00:00:00 1970 From: Charles Keepax Subject: Re: [PATCH v1 2/9] mfd: wm8994: Add support for MCLKn clock control Date: Wed, 18 Sep 2019 12:54:35 +0000 Message-ID: <20190918125435.GF10204@ediswmail.ad.cirrus.com> References: <20190918104634.15216-1-s.nawrocki@samsung.com> <20190918104634.15216-3-s.nawrocki@samsung.com> <49ef7282-3765-9149-285d-a4025417eb58@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <49ef7282-3765-9149-285d-a4025417eb58@samsung.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Sylwester Nawrocki Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, linux-samsung-soc@vger.kernel.org, b.zolnierkie@samsung.com, sbkim73@samsung.com, patches@opensource.cirrus.com, broonie@kernel.org, lgirdwood@gmail.com, krzk@kernel.org, robh+dt@kernel.org, lee.jones@linaro.org, linux-arm-kernel@lists.infradead.org, m.szyprowski@samsung.com List-Id: devicetree@vger.kernel.org On Wed, Sep 18, 2019 at 12:59:28PM +0200, Sylwester Nawrocki wrote: > On 9/18/19 12:46, Sylwester Nawrocki wrote: > > The WM1811/WM8994/WM8958 audio CODEC DT bindings specify two optional > > clocks: "MCLK1", "MCLK2". Add code for getting those clocks in the MFD > > part of the wm8994 driver so they can be further handled in the audio > > CODEC part. > > > > Signed-off-by: Sylwester Nawrocki > > --- > > > > + wm8994->mclk[WM8994_MCLK1].id = "MCLK1"; > > + wm8994->mclk[WM8994_MCLK2].id = "MCLK2"; > > + > > + ret = devm_clk_bulk_get_optional(wm8994->dev, ARRAY_SIZE(wm8994->mclk), > > + wm8994->mclk); > > + if (ret != 0) > > + return ret; Would be nice to print a message here as well, make it clear what failed in the log. Apart from that minor nit: Acked-by: Charles Keepax Thanks, Charles