From patchwork Mon Dec 11 12:14:27 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Mark Brown X-Patchwork-Id: 121389 Delivered-To: patch@linaro.org Received: by 10.140.22.227 with SMTP id 90csp2713312qgn; Mon, 11 Dec 2017 04:15:02 -0800 (PST) X-Google-Smtp-Source: ACJfBoto7wm1ZdVdMJFk6r6LE7iMmK1Z9XsqDnvumtvkIOfldCfjvEJ8ZpTC+FRZK6Qrwz6M+Xso X-Received: by 10.223.190.135 with SMTP id i7mr187662wrh.91.1512994502171; Mon, 11 Dec 2017 04:15:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1512994502; cv=none; d=google.com; s=arc-20160816; b=Whc7Lvv+VoIdXmycd+yh+AOIgtk8qWysdkXSCfDE9nWtbB8PrPia6x/zOzWsbp6Wsm W2GQIW3PTO6Fbe3d+bKRbfRsF5UlY0lp7Sq1T2nz8DGI0jMiKOOJ8ZLFx1HOO8i5D40x rSoR1M9xXZfV9vGk2K7crdlzeu3iq/oRiEStcvGEHrTiAcTgpaCi7CdEL9OvaC8wzVFz oTJ8CyD6lAGJmsHyPaOhM+yTh3jXx8vjttcNqoB+nA/l8mX0vNtZDmbSpa7Y40rcRnpc QNIxW5gxNi5eUf3UDE/i9WG4oOsAgm3DtntKj6TBGj/yOmK58EOJOn5GNiAXgBQHhZan yW0A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:content-transfer-encoding:mime-version :list-subscribe:list-help:list-post:list-archive:list-unsubscribe :list-id:precedence:subject:cc:date:message-id:in-reply-to:to:from :dkim-signature:delivered-to:arc-authentication-results; bh=O4pSJZHQpKcRal8OJY9Hdg8866GrtC8PrIZ8rTnwoNY=; b=0AjXtCdm25P8yw5pyLsSDR3BCpeoQ/irURxEHLE6sykkwgkxGHKFNLjYPO1DvovxXI wbOtaY4gRCDzNfMMoHY7iGudcQGNuOYkOZhy8IaFxOtriXFr5NGOxKS8clLq9Rl/37wi i7ZBNeD00bw7B4PjE8Izh9/Elp39QUL7EKY/PlmpcZcAX4S6bVP8zxCpq4e/csUCQPs5 aOoOV9XO4pa2JxEP3pzUFjlY2pho3PVYVOrJFkr3oBnPgwtbM7pQSJYsfLsnal9Kg/sU YnfXHWonRttyeIh2Dcg7XOZwoT5R72on2Lfew2qWNRM/FCcM6SYGxC1a2LZhJenLUExH x6Iw== ARC-Authentication-Results: i=1; mx.google.com; dkim=neutral (body hash did not verify) header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=RlrbnmBn; spf=pass (google.com: domain of alsa-devel-bounces@alsa-project.org designates 77.48.224.243 as permitted sender) smtp.mailfrom=alsa-devel-bounces@alsa-project.org Return-Path: Received: from alsa0.perex.cz (alsa0.perex.cz. [77.48.224.243]) by mx.google.com with ESMTP id p40si10569098wrb.224.2017.12.11.04.15.00; Mon, 11 Dec 2017 04:15:02 -0800 (PST) Received-SPF: pass (google.com: domain of alsa-devel-bounces@alsa-project.org designates 77.48.224.243 as permitted sender) client-ip=77.48.224.243; Authentication-Results: mx.google.com; dkim=neutral (body hash did not verify) header.i=@sirena.org.uk header.s=20170815-heliosphere header.b=RlrbnmBn; spf=pass (google.com: domain of alsa-devel-bounces@alsa-project.org designates 77.48.224.243 as permitted sender) smtp.mailfrom=alsa-devel-bounces@alsa-project.org Received: from alsa0.perex.cz (localhost [127.0.0.1]) by alsa0.perex.cz (Postfix) with ESMTP id 066262675A4; Mon, 11 Dec 2017 13:15:00 +0100 (CET) X-Original-To: alsa-devel@alsa-project.org Delivered-To: alsa-devel@alsa-project.org Received: by alsa0.perex.cz (Postfix, from userid 1000) id 8C751267608; Mon, 11 Dec 2017 13:14:58 +0100 (CET) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on mail1.perex.cz X-Spam-Level: X-Spam-Status: No, score=0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS, SPF_PASS, T_RP_MATCHES_RCVD autolearn=disabled version=3.4.0 Received: from heliosphere.sirena.org.uk (heliosphere.sirena.org.uk [172.104.155.198]) by alsa0.perex.cz (Postfix) with ESMTP id ED11D26758C for ; Mon, 11 Dec 2017 13:14:31 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sirena.org.uk; s=20170815-heliosphere; h=Date:Message-Id:In-Reply-To: Subject:Cc:To:From:Sender:Reply-To:MIME-Version:Content-Type: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:References: List-Id:List-Help:List-Unsubscribe:List-Subscribe:List-Post:List-Owner: List-Archive; bh=UmbvdF72MrheV1YEnmoq4GXskqWp45wKfcfsRmE1WXw=; b=RlrbnmBnvsDN MceeBqLL65rq0eKpKHSQjQq9lOtKnfNzPU2u2p8Jh/RsLzvFQ5E3hISc/jYEOs7q+TH3SvGi0yHM9 A6+8saeKKx5OGB5zyU2RdTokFD+cpaDdpCwMqDnT+heweift8ZfTLUk7hO5GOC48RxNB2jI4LlNMa U2QPk=; Received: from debutante.sirena.org.uk ([2001:470:1f1d:6b5::3] helo=debutante) by heliosphere.sirena.org.uk with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1eOMyZ-0002Kn-WE; Mon, 11 Dec 2017 12:14:28 +0000 Received: from broonie by debutante with local (Exim 4.90_RC3) (envelope-from ) id 1eOMyZ-0004f5-IZ; Mon, 11 Dec 2017 12:14:27 +0000 From: Mark Brown To: Dan Carpenter In-Reply-To: <20171209115203.pdtdfnmzwz6zpjqs@mwanda> Message-Id: Date: Mon, 11 Dec 2017 12:14:27 +0000 Cc: alsa-devel@alsa-project.org, kernel-janitors@vger.kernel.org, Liam Girdwood , Takashi Iwai , Mark Brown , Arvind Yadav Subject: [alsa-devel] Applied "ASoC: nuc900: Fix platform_get_irq() error checking some more" to the asoc tree X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org The patch ASoC: nuc900: Fix platform_get_irq() error checking some more has been applied to the asoc tree at https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted. You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed. If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced. Please add any relevant lists and maintainers to the CCs when replying to this mail. Thanks, Mark >From cd430a244cd5d3ca0f4053718eabdf42bc12c517 Mon Sep 17 00:00:00 2001 From: Dan Carpenter Date: Sat, 9 Dec 2017 14:52:03 +0300 Subject: [PATCH] ASoC: nuc900: Fix platform_get_irq() error checking some more The error handling doesn't work here because "nuc900_audio->irq_num" is unsigned. Also we should be checking for < 0 and not <= 0 but I believe that's harmless. The platform_get_irq() comments don't talk about the return values... Fixes: fa8cc38165c2 ("ASoC: nuc900: Fix platform_get_irq's error checking") Signed-off-by: Dan Carpenter Signed-off-by: Mark Brown --- sound/soc/nuc900/nuc900-ac97.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) -- 2.15.1 _______________________________________________ Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel diff --git a/sound/soc/nuc900/nuc900-ac97.c b/sound/soc/nuc900/nuc900-ac97.c index 5e4fbd2d3479..71fce7c85c93 100644 --- a/sound/soc/nuc900/nuc900-ac97.c +++ b/sound/soc/nuc900/nuc900-ac97.c @@ -345,11 +345,10 @@ static int nuc900_ac97_drvprobe(struct platform_device *pdev) goto out; } - nuc900_audio->irq_num = platform_get_irq(pdev, 0); - if (nuc900_audio->irq_num <= 0) { - ret = nuc900_audio->irq_num < 0 ? nuc900_audio->irq_num : -EBUSY; + ret = platform_get_irq(pdev, 0); + if (ret < 0) goto out; - } + nuc900_audio->irq_num = ret; nuc900_ac97_data = nuc900_audio;