From mboxrd@z Thu Jan 1 00:00:00 1970 From: Josh Wu Subject: Re: [PATCH 4/5] media: ov2640: add a master clock for sensor Date: Tue, 9 Dec 2014 11:03:40 +0800 Message-ID: <5486668C.3000808@atmel.com> References: <1418038147-13221-1-git-send-email-josh.wu@atmel.com> <1418038147-13221-5-git-send-email-josh.wu@atmel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-media-owner@vger.kernel.org To: Fabio Estevam Cc: linux-media , Laurent Pinchart , Mauro Carvalho Chehab , "linux-arm-kernel@lists.infradead.org" , Guennadi Liakhovetski , "devicetree@vger.kernel.org" List-Id: devicetree@vger.kernel.org Hi, Fabio On 12/8/2014 11:10 PM, Fabio Estevam wrote: > On Mon, Dec 8, 2014 at 9:29 AM, Josh Wu wrote: > >> + priv->master_clk = devm_clk_get(&client->dev, "xvclk"); >> + if (IS_ERR(priv->master_clk)) >> + return -EINVAL; > You should return PTR_ERR(priv->master_clk) instead. sure. I'll fix it in next version. Thanks. Best Regards, Josh Wu