From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f44.google.com (mail-wm1-f44.google.com [209.85.128.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 16FD52E3B1C for ; Tue, 2 Sep 2025 06:33:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756794823; cv=none; b=am7gYhvBQPFrcJdINMeOAPYlHPf6uOwqVD2WbhKpCtSmFubYTfBO0hbv3p09o2RHu9+Enz7P0oQTVUYBuy5Hot+4RIvpGkjTm70/MEKTCOVWA9PFvlHszpM8TX7M4BruMA3NeXPHn9/bWaGaZPRDONS574MUmwqgbM7OPXDi8Qg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756794823; c=relaxed/simple; bh=MKo8n/x52MPOw7BrUfoyfV5mi9Zpix6Gd7w11RHjJxo=; h=Date:From:To:Cc:Subject:Message-ID:MIME-Version:Content-Type: Content-Disposition; b=cgilO481htm6EX8X3WYtxm2yt2XeWc2uP80AhKltAadostUJYL6XIS/Wzt86VPEdaAsQNE0K0cKwDleu1zF6qVUKLLgUJgjQmQ5OOw3t2wS+FHvdzHGtpj7jEpjrE7MPiKfWbhiwiCvi4RbCKE5xiT1yjwzGWx/mEqkw8JNuJDc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=OW+QmvWd; arc=none smtp.client-ip=209.85.128.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="OW+QmvWd" Received: by mail-wm1-f44.google.com with SMTP id 5b1f17b1804b1-45b804ed966so14791485e9.2 for ; Mon, 01 Sep 2025 23:33:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1756794820; x=1757399620; darn=vger.kernel.org; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :from:to:cc:subject:date:message-id:reply-to; bh=losT/kfGGLPsjGybv1p41hnyyy7EMl3u0Mtb0w2e7UQ=; b=OW+QmvWdPyijoDUNn/b6n9Ok/x3duEO0BLJ+bq6MsDq4cZuhp8QIIBqWuafnw6jumn mkuk5CZ1/MedVVi/YehIWBuEwKsrGWfG826AW460YFbgVI9MTT7xBJgAkNBadJCeLPgI 8rsol1Q8JADindc25txhwyLzc7X0GSJFU3PTU5SDyKaFLJjM8lX7G8RbI946QOFGdMvP 5QabQ2NmU4nilDM17n35MyVLjsJ5NxQws/+FV/2wOQkoaSBGnsk3BJ7G6OrKiweQjZiT BALBeJN4oZCvCN9Ayq//rtMh57PatRxSoptS05nE1D88A8oevznx8cvt4JyI1+8GpbLW ZYFg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1756794820; x=1757399620; h=content-disposition:mime-version:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=losT/kfGGLPsjGybv1p41hnyyy7EMl3u0Mtb0w2e7UQ=; b=l78RAY54sxcsL5kmFEPmRM/Vazv23Vqf4l0N21+HgqPzOOYgWUvgLdbtk9RAiZKtIK evbXL53teiQqujOw55THYyhOVdvlqnTYqCLCDCJTdR9RsXT6WvNKUzrG+iTUlBBu8xuC aroU74ZyZPhN2dqpWunZ8BdDM7W5NP7cDmOh5AP1RqrY15Nk+QPwKx8e8XLcVVfH3Lri RVbsNDkSym79rphHYV0/ZHL82IQS7Ka3PCyTlPWJX9GtqoCb7W+v3Lu9JQw98WRe9CwS kGCHqkrF1kDM2FGJsVDMfcf5iwQI1PGn2AxtYPTGQTVxylAP/SjAdo1Mdo1/poQ0lVUp 9vOg== X-Forwarded-Encrypted: i=1; AJvYcCXeaDcmBuxENWMUapzSRAcYhO6ENZf3tZemvXUB/+qIEvnqc1/6zi3dWPEzsoOwfIbc9/YyPK3vUzo=@vger.kernel.org X-Gm-Message-State: AOJu0YwV4pc0PNCbcY/kzza2Q0qtZTIs8jJ1jXjV4z3QRtwNjcJUq62u t+TMx8flK+5b5XHOCr0dQGmnX/cGwFCXqQL9uhbX48tYLAKSX25eeqmhPwtjf5Iq3n/TLWDPN1v ZUTAW X-Gm-Gg: ASbGncsvKl2+fYrQ80btcUVvPkaHD5BQwhyeusokTTkmgMTplBKd1KQk0VoLaYaYnBP V3FepkQ6QtyfPHgYkWCTKUiMxEY6qoZBGRRzvrGTSyQS1nlb/d5l1VJB0Gv1nhG5I6KqmN3p744 s8qxhFBuhhpk9d/gDETxn5IJYyAna9rOO9lh/K6LwNCkrpKt3UreftorEvnG83AnLE/PMBv7PEg iz0Wm4ucVqcxTL2gm5Lxt8C84QxH5hUfXQlDTgUkqfX3WKVICnOvGRxrn//Rn8y9B6vFPqrGZwS /9qx81cXaWZWdcC6g/wypO1/RKYu0Z0BXUmSffxZYr796dxVMMz/UjBb6PANlMM+UFPsQzwgP4Y b+JrhEPF+1Bff2XMwULRwMfjnybF7Q/V9bwjrdQ== X-Google-Smtp-Source: AGHT+IGTC4jvhc5wtr9AKkKmGa0tiEpKXwEYpezAIz5zGfXKE+mLDpBMK8tQiURoKbOUDB0H5J7XFg== X-Received: by 2002:a05:600c:4fc1:b0:45b:6163:c031 with SMTP id 5b1f17b1804b1-45b9353ed8dmr24223305e9.24.1756794820456; Mon, 01 Sep 2025 23:33:40 -0700 (PDT) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with UTF8SMTPSA id 5b1f17b1804b1-45b6f0c6dc1sm265828005e9.1.2025.09.01.23.33.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Sep 2025 23:33:40 -0700 (PDT) Date: Tue, 2 Sep 2025 09:33:36 +0300 From: Dan Carpenter To: Varadarajan Narayanan Cc: Bjorn Andersson , Michael Turquette , Stephen Boyd , linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org Subject: [PATCH] clk: qcom: common: Fix NULL vs IS_ERR() check in qcom_cc_icc_register() Message-ID: Precedence: bulk X-Mailing-List: linux-clk@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline X-Mailer: git-send-email haha only kidding The devm_clk_hw_get_clk() function doesn't return NULL, it returns error pointers. Update the checking to match. Fixes: 8737ec830ee3 ("clk: qcom: common: Add interconnect clocks support") Signed-off-by: Dan Carpenter --- drivers/clk/qcom/common.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/clk/qcom/common.c b/drivers/clk/qcom/common.c index 37c3008e6c1b..121591886774 100644 --- a/drivers/clk/qcom/common.c +++ b/drivers/clk/qcom/common.c @@ -277,8 +277,8 @@ static int qcom_cc_icc_register(struct device *dev, icd[i].slave_id = desc->icc_hws[i].slave_id; hws = &desc->clks[desc->icc_hws[i].clk_id]->hw; icd[i].clk = devm_clk_hw_get_clk(dev, hws, "icc"); - if (!icd[i].clk) - return dev_err_probe(dev, -ENOENT, + if (IS_ERR(icd[i].clk)) + return dev_err_probe(dev, PTR_ERR(icd[i].clk), "(%d) clock entry is null\n", i); icd[i].name = clk_hw_get_name(hws); } -- 2.47.2