From nobody Mon Apr 27 11:50:47 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 51035C43334 for ; Mon, 13 Jun 2022 23:22:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S244254AbiFMXWq (ORCPT ); Mon, 13 Jun 2022 19:22:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55274 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1344104AbiFMXWi (ORCPT ); Mon, 13 Jun 2022 19:22:38 -0400 Received: from mail-pl1-x635.google.com (mail-pl1-x635.google.com [IPv6:2607:f8b0:4864:20::635]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 231D71277B for ; Mon, 13 Jun 2022 16:22:26 -0700 (PDT) Received: by mail-pl1-x635.google.com with SMTP id o17so6352593pla.6 for ; Mon, 13 Jun 2022 16:22:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=chromium.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=B3RxiogwxQ3efUX3+sbp3cwVCT5XzvkaPhVLnHLET4Y=; b=BqiMiN+lQO+8l/tGBuOxJn2dFR3aFYKX4uFwvVPTVw9Tj0NCQMlDy4L3Ic/ONqIl31 wzplWXTVffQhgKN0pb8Qe6+UaeYgrIuAGQgjchacCAgw3eH84QGONtmqN+2GuLf5sKqy 8neAm2iWJ61No+MZXjc82pQNWaZZjB6WB5IMQ= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=B3RxiogwxQ3efUX3+sbp3cwVCT5XzvkaPhVLnHLET4Y=; b=4FJMYVWPI8q92DcR+I1vOPAVyLrDur9nDfRKDD+FQpAbGjkqm9RK5GF6O0XjT1ye1W /zd6zL+O/hVsi8q+O0t5cufRBK5SZOylxkVAHaUh5qBI+S3lmjSGWVfMUJ/blmEuRrMI Zivoxz6RM+fCi5+DE9MjOlnlPnJVufliUfv7yYi5ngFTsGTl5Nur/1QX7g6nE7+Iwwi2 J7pFfEoxw7ahvigo1qX/FqhYOR7eJLIeZjYFjEoWdcw0cdY/jKoNoNhUMmuGQdPHVVCc Qfn4vFtMi3qdhJ5kyN0kAxV433jle78Xka+muNxJoAAjF3YDazvb8T+uDRnYi+iPgSnO dTUA== X-Gm-Message-State: AJIora/3ALnuxkR9KMljRsgyaIJOoh8jycAk556huBYz5dURpKZ4G9Ng vbDB/9m1IGuRmeUVWp8/8MrUbK2/cK5law== X-Google-Smtp-Source: AGRyM1uhlZCnyIP4x2P9IFkCataXFtmzmEbxJHxrC4AW0Y0NWP6mb3j4cyOf8LdmLE/f++Ma2sk8Zg== X-Received: by 2002:a17:902:d50b:b0:165:2aa0:4b22 with SMTP id b11-20020a170902d50b00b001652aa04b22mr1358214plg.131.1655162545532; Mon, 13 Jun 2022 16:22:25 -0700 (PDT) Received: from smtp.gmail.com ([2620:15c:202:201:82ae:1136:b202:95da]) by smtp.gmail.com with ESMTPSA id s9-20020a63af49000000b003f5d7f0ad6asm6057407pgo.48.2022.06.13.16.22.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Jun 2022 16:22:25 -0700 (PDT) From: Stephen Boyd To: Jonathan Cameron Cc: Aashish Sharma , linux-kernel@vger.kernel.org, patches@lists.linux.dev, linux-iio@vger.kernel.org, Gwendal Grignou , kernel test robot Subject: [PATCH] iio:proximity:sx9324: Check ret value of device_property_read_u32_array() Date: Mon, 13 Jun 2022 16:22:24 -0700 Message-Id: <20220613232224.2466278-1-swboyd@chromium.org> X-Mailer: git-send-email 2.36.1.476.g0c4daa206d-goog MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Type: text/plain; charset="utf-8" From: Aashish Sharma 0-day reports: drivers/iio/proximity/sx9324.c:868:3: warning: Value stored to 'ret' is never read [clang-analyzer-deadcode.DeadStores] Put an if condition to break out of switch if ret is non-zero. Signed-off-by: Aashish Sharma Fixes: a8ee3b32f5da ("iio:proximity:sx9324: Add dt_binding support") Reported-by: kernel test robot [swboyd@chromium.org: Reword commit subject, add fixes tag] Signed-off-by: Stephen Boyd Reviewed-by: Gwendal Grignou --- drivers/iio/proximity/sx9324.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/iio/proximity/sx9324.c b/drivers/iio/proximity/sx9324.c index 70c37f664f6d..63fbcaa4cac8 100644 --- a/drivers/iio/proximity/sx9324.c +++ b/drivers/iio/proximity/sx9324.c @@ -885,6 +885,9 @@ sx9324_get_default_reg(struct device *dev, int idx, break; ret =3D device_property_read_u32_array(dev, prop, pin_defs, ARRAY_SIZE(pin_defs)); + if (ret) + break; + for (pin =3D 0; pin < SX9324_NUM_PINS; pin++) raw |=3D (pin_defs[pin] << (2 * pin)) & SX9324_REG_AFE_PH0_PIN_MASK(pin); base-commit: f2906aa863381afb0015a9eb7fefad885d4e5a56 --=20 https://chromeos.dev