Received: by 2002:a25:868d:0:0:0:0:0 with SMTP id z13csp1765195ybk; Mon, 11 May 2020 03:51:45 -0700 (PDT) X-Google-Smtp-Source: APiQypLoUU/HYheBndr2q3wrgTu9x9yR+Ka1o6q3rdcfZSycRhdkqg3pYsJ30T/CqBGeJQRSza8V X-Received: by 2002:a17:906:c7cc:: with SMTP id dc12mr6792023ejb.263.1589194305741; Mon, 11 May 2020 03:51:45 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1589194305; cv=none; d=google.com; s=arc-20160816; b=PPJj0wBIMztWIuV994tZNUvmyAKoqAYm3XAyrZhQiSpvGQYo1MfbXcoN+MN5h4XUwa ic2EWI1mQnHCizO+lPzcQMR8+Gk8jKeQT+Wa4JuavFE0GebbXTP1wnDBVmqvQNjbyOhK 19UrMy1g19PlLwBcBfCgLm/QYbjj62BQvf4dbphcwRArHjMIfo+82ac6jPA9/aCdSupp bHWuG3MLFOiscEeCz7+WMXyeXYczMjF8fENROBLE7MHtYqhAocmoD9lw9wdkD06Kfr9K gwc+IUTuNSQRgVpQ3TEfNEZnRrdI8GyMoI86hCt3cdLF6JgkEYUOOeaNWxx98VCvxQvY Hb3g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date:dkim-signature; bh=s4kNpPJ98fSBlw7AhLgayoeulxOpYffY+qXe77AKn80=; b=JUHZVjLpPOT6q5dD2Y44ClBrNxMw35vQ/TkeBvMfzT5eqBYWOJtbD+qW1I2/FR3exL Wr5VdAM6nXudy97/u9M2FBDA67cFPBVJrGclnrniiEmEpJ750Zw0uQaFNkxWJI72MdBA QMdq5l3CBMdij9REq2PRm7WF0jas5A2zm2lHeOa+jYWHv0qKZSf8AQ8rZMVbkjOtMv2q X5qWKnOjme48qLFjdzxPVXJjf3x9d92O6VI3tViZXoR/pjdfLfWCIl125rMmE06FAOJ2 Ka6hzeUVmTKu2H2wDhJqO6m3lIXPZuJNJmxYksb87lceVhwXUFU2qgYMzEsBSmV9hizT pk9g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=zf0CyIb3; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id n2si1587892edy.425.2020.05.11.03.51.23; Mon, 11 May 2020 03:51:45 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=zf0CyIb3; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729568AbgEKKt0 (ORCPT + 99 others); Mon, 11 May 2020 06:49:26 -0400 Received: from mail.kernel.org ([198.145.29.99]:45892 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728209AbgEKKtZ (ORCPT ); Mon, 11 May 2020 06:49:25 -0400 Received: from localhost (fw-tnat.cambridge.arm.com [217.140.96.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id A621F206D5; Mon, 11 May 2020 10:49:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1589194165; bh=15islGkeDrYc/THLIgzaeQ3iuhszhWj2T2ccvUjSv+Y=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=zf0CyIb35SsbLV8ORJUAwSCDMQY+UUxtoEUkcW8pZXArWQWhPNKat6tarU4oD/il1 KqX/CtZ66hNsE3k8Xi6oZRL4J2Zw2gBm3vf/CfdsmOxs0pCkpuSlSDHztYqOmZbZ/K 3c1g0EHonSsBJxXDY6m7m5MkbnnX9Fl6mC93Tbfc= Date: Mon, 11 May 2020 11:49:22 +0100 From: Mark Brown To: Sumit Semwal Cc: agross@kernel.org, bjorn.andersson@linaro.org, lgirdwood@gmail.com, robh+dt@kernel.org, nishakumari@codeaurora.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, kgunda@codeaurora.org, rnayak@codeaurora.org Subject: Re: [v2 4/4] regulator: qcom: labibb: Add SC interrupt handling Message-ID: <20200511104922.GD8216@sirena.org.uk> References: <20200508204200.13481-1-sumit.semwal@linaro.org> <20200508204200.13481-5-sumit.semwal@linaro.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="NKoe5XOeduwbEQHU" Content-Disposition: inline In-Reply-To: <20200508204200.13481-5-sumit.semwal@linaro.org> X-Cookie: TANSTAAFL User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --NKoe5XOeduwbEQHU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Sat, May 09, 2020 at 02:12:00AM +0530, Sumit Semwal wrote: > +static irqreturn_t labibb_sc_err_handler(int irq, void *_reg) > +{ > + int ret, count; > + u16 reg; > + u8 sc_err_mask; > + unsigned int val; > + struct labibb_regulator *labibb_reg = (struct labibb_regulator *)_reg; > + bool in_sc_err, reg_en, scp_done = false; > + > + if (irq == labibb_reg->sc_irq) > + reg = labibb_reg->base + REG_LABIBB_STATUS1; > + else > + return IRQ_HANDLED; Why would we be registering the interrupt handler when it's not valid? --NKoe5XOeduwbEQHU Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl65LbEACgkQJNaLcl1U h9A8Wgf+NWKmlAAltMsGck9iS25HVb+wWUuXPnOZWCe13+DjXCuc7QDmqpRwjzLy aYXbTXmU+6GCZHfZbazTY7bZzsq8NG8DhZ2Gzwuoh2PcUuKwBvFNuhX8rwAB5W4a d73uCvaIxfFjxbB0Xgkjw1QF89ijTK8N+uemNHs7wtLW8LxoPKO/37ajwTLWYqNf aDg44Y1e7PkhfGoGB2IiIZq/srn6TOYJmcK3FojllvVQEBi/Xsu2K40HSK+r2L8S xlHe++6Hf7byPabmH7VBLAU+YvhtxRBSfW6TrMFnCFNlczK+sIJhKJhmA4EJ+3Zc ixVK6i/oJptEZkUH0OVtgFd/UQe6ig== =FTe5 -----END PGP SIGNATURE----- --NKoe5XOeduwbEQHU--