Received: by 2002:a05:6359:c8b:b0:c7:702f:21d4 with SMTP id go11csp4971174rwb; Wed, 21 Sep 2022 01:00:33 -0700 (PDT) X-Google-Smtp-Source: AMsMyM5x/TKNeL8shzdgsHMw+5LRXXbcidZ3+L+YIuIxbb/RF2zCvPWE1o135jOilFOULpA7tEj1 X-Received: by 2002:a17:902:f604:b0:172:75a4:33ea with SMTP id n4-20020a170902f60400b0017275a433eamr3472301plg.7.1663747233553; Wed, 21 Sep 2022 01:00:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1663747233; cv=none; d=google.com; s=arc-20160816; b=jR0Sm7YFbBo6AnXUANW/6Y6Zyv16GSQ4nHANZ+dlliryJOwa4CWqgLf69qKQ6V1GjT xEFhBDAAZM/Eh3jaPwE+KHLCVgAzGOjYUjfV6pB+ag8kGm2Sx8i/UshW53FQ9mwi9571 JLEcqdaPioPndq0CV8uvN0g6q1cquXp0bm+IlqngRRUjXMgKzNy2QmQeD/6A/M3fD8Uy N0o6NLOnuOrLFdKYvPzQEAug2KlpxsAwggbxriptbHwBmqRS8wcRVNUgH728ON+fpQ6T TdQGPJRvHIj/FR0aKKYe3/6Fe2TsaLaYB54AFpvo0I/KShlXiKFY8EdYJ5mYc2eN+z1N u/4A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=SsmCCyDi74VwnxgmOAe/pUNyj8fElhmMBo+WxDPOa58=; b=R8r18GiwMOosYoIRvklWK++4nWRfwI0dC9Zq892W8+BHkR6d53jf/NZcDt+7cKIti4 HL4Qc2mbYQerxPuM3RGhca36ZlpBJX5bdzFTv1LoR9onFvdGxyYZxCkliaDi2GdAuPQN 9utDS1teXgiy3xToWjUXyY3hVaVEfq20YA8p50bIiO9Pbypr6gKKJTld3DAmHRvIWXHJ HuDTf/b373npB6QasJ12RLeYpc4J8QP7gYNuzc+FaCiA/9VTX6HAUeqDTAevMzjJjrDx 1R2fIAzpimkKeVoia7fZagRJxO6fyGP+HJ13o1i67bzLJdCOFRUWt0rDlG1jvGc3MXSR HgZw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@bgdev-pl.20210112.gappssmtp.com header.s=20210112 header.b=6Sn1ysZx; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 2-20020a170902c10200b0016c4fc2de44si1845379pli.456.2022.09.21.01.00.18; Wed, 21 Sep 2022 01:00:33 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@bgdev-pl.20210112.gappssmtp.com header.s=20210112 header.b=6Sn1ysZx; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229990AbiIUHcA (ORCPT + 99 others); Wed, 21 Sep 2022 03:32:00 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51596 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229783AbiIUHb5 (ORCPT ); Wed, 21 Sep 2022 03:31:57 -0400 Received: from mail-ej1-x62d.google.com (mail-ej1-x62d.google.com [IPv6:2a00:1450:4864:20::62d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id AF07E564EE for ; Wed, 21 Sep 2022 00:31:56 -0700 (PDT) Received: by mail-ej1-x62d.google.com with SMTP id 13so11624086ejn.3 for ; Wed, 21 Sep 2022 00:31:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bgdev-pl.20210112.gappssmtp.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date; bh=SsmCCyDi74VwnxgmOAe/pUNyj8fElhmMBo+WxDPOa58=; b=6Sn1ysZxrf3fCZ8WZSUG4RtP6Lg5Om3weZ4AQnx3wiKJjf0RGAKrXapRCaA2vEAON0 EB0ipwxXeJ8qw8cwPEYgQ00Jw37LrvafeNvzODLn1oFS+X+EV/tayNzxvMMhRNSRspr0 zNVJZ9TMih2FUDITgNkIjxl1aHnys+WZRRfozMwTcy/uWowGCURn10qFjqZpcLgHGngs 4KBO2jT3ZawoMquUabR3+oCsd6jZeWVc4/UQSEkSslKx7u0Qk/o1YITQtPR1wsEaoMH6 UeWCtXsm/y3ptaw9UC8aPXwolYH+qMQ8nWvjDNjiJEg7kb4kH+BWUqecivvP/VD/spOR bjrg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date; bh=SsmCCyDi74VwnxgmOAe/pUNyj8fElhmMBo+WxDPOa58=; b=nv9E4Lf9Bo05AjjUymA11qzVw66Ip9Zgj70QF6tVGr+nfRv/IOzr8N2MuT3753AErQ yy+WpJ5wIRLVaIJfbJDAXr+iXp4ZJ54ccFyrJS6LBoQlLW6JdImBOJEoyiftCVjEfdan s1i7u+T+PlkybCslEN+GZHhMtpecQTwdPhOb1slhhVNwvuT8sCtGNh4BpF1KSvHvox5a da44XgNO5QNv2mmkmarZeZrzlF+ObtnlgeQh9N/uPTrKQMw2bEaSYNdPE8KZRPQUkzjX nPr875NzZrfaoD68Yb9WRLuYw78wEKCig89iWE1pM44SsFK611jVX/lQV/Tfg4zgGMKl aS8w== X-Gm-Message-State: ACrzQf1wd27leKsO3Tj6KUgaOE2NiIS/RuUS3y3IFao0IrpXitXlTjtp nmVW0X2ikfWmpzg7DU2vb1Y0S6ltKrDcvYuOmqo77Q== X-Received: by 2002:a17:907:3da9:b0:77b:9672:3f7f with SMTP id he41-20020a1709073da900b0077b96723f7fmr19870688ejc.636.1663745515250; Wed, 21 Sep 2022 00:31:55 -0700 (PDT) MIME-Version: 1.0 References: <20220919031250.770285-1-dzm91@hust.edu.cn> In-Reply-To: <20220919031250.770285-1-dzm91@hust.edu.cn> From: Bartosz Golaszewski Date: Wed, 21 Sep 2022 09:31:44 +0200 Message-ID: Subject: Re: [PATCH v2] gpio: tqmx86: fix uninitialized variable girq To: Dongliang Mu Cc: Linus Walleij , Marc Zyngier , Dongliang Mu , linux-gpio@vger.kernel.org, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_NONE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 19, 2022 at 5:14 AM Dongliang Mu wrote: > > From: Dongliang Mu > > The commit 924610607f19 ("gpio: tpmx86: Move PM device over to > irq domain") adds a dereference of girq that may be uninitialized. > > Fix this by moving irq_domain_set_pm_device into if true branch > as suggested by Marc Zyngier. > > Fixes: 924610607f19 ("gpio: tpmx86: Move PM device over to irq domain") > Suggested-by: Marc Zyngier > Signed-off-by: Dongliang Mu > --- > v1->v2: modify fix method to moving irq_domain_set_pm_device into > if true branch as suggested by Marc Zyngier > drivers/gpio/gpio-tqmx86.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpio/gpio-tqmx86.c b/drivers/gpio/gpio-tqmx86.c > index fa4bc7481f9a..e739dcea61b2 100644 > --- a/drivers/gpio/gpio-tqmx86.c > +++ b/drivers/gpio/gpio-tqmx86.c > @@ -307,6 +307,8 @@ static int tqmx86_gpio_probe(struct platform_device *pdev) > girq->default_type = IRQ_TYPE_NONE; > girq->handler = handle_simple_irq; > girq->init_valid_mask = tqmx86_init_irq_valid_mask; > + > + irq_domain_set_pm_device(girq->domain, dev); > } > > ret = devm_gpiochip_add_data(dev, chip, gpio); > @@ -315,8 +317,6 @@ static int tqmx86_gpio_probe(struct platform_device *pdev) > goto out_pm_dis; > } > > - irq_domain_set_pm_device(girq->domain, dev); > - > dev_info(dev, "GPIO functionality initialized with %d pins\n", > chip->ngpio); > > -- > 2.35.1 > Applied, thanks! Bart