From: Jamie Iles Subject: [PATCH 01/16] crypto: omap-aes: don't treat NULL clk as an error Date: Tue, 11 Jan 2011 12:43:38 +0000 Message-ID: <1294749833-32019-2-git-send-email-jamie@jamieiles.com> References: <1294749833-32019-1-git-send-email-jamie@jamieiles.com> Cc: Jamie Iles , Dmitry Kasatkin , linux-crypto@vger.kernel.org To: linux-kernel@vger.kernel.org Return-path: In-Reply-To: <1294749833-32019-1-git-send-email-jamie@jamieiles.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-crypto.vger.kernel.org clk_get() returns a struct clk cookie to the driver and some platforms may return NULL if they only support a single clock. clk_get() has only failed if it returns a ERR_PTR() encoded pointer. Cc: Dmitry Kasatkin Cc: linux-crypto@vger.kernel.org Signed-off-by: Jamie Iles --- drivers/crypto/omap-aes.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/omap-aes.c b/drivers/crypto/omap-aes.c index 799ca51..24d0f3f 100644 --- a/drivers/crypto/omap-aes.c +++ b/drivers/crypto/omap-aes.c @@ -17,6 +17,7 @@ #include #include #include +#include #include #include #include @@ -830,9 +831,9 @@ static int omap_aes_probe(struct platform_device *pdev) /* Initializing the clock */ dd->iclk = clk_get(dev, "ick"); - if (!dd->iclk) { + if (IS_ERR(dd->iclk)) { dev_err(dev, "clock intialization failed.\n"); - err = -ENODEV; + err = PTR_ERR(dd->iclk); goto err_res; } -- 1.7.3.4