Received: by 2002:a25:23cc:0:0:0:0:0 with SMTP id j195csp1580229ybj; Fri, 8 May 2020 04:53:11 -0700 (PDT) X-Google-Smtp-Source: APiQypI5GQcvw4xa31goHkfi89Wd5ClmpTFXNiaqfkOeFrxuQj6xDnXyFvdQzEd7KLl/S+B+PayK X-Received: by 2002:a17:906:3607:: with SMTP id q7mr1447917ejb.313.1588938791132; Fri, 08 May 2020 04:53:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1588938791; cv=none; d=google.com; s=arc-20160816; b=UWWysKE2D9YGN9st8tlcd/N4WGr3c3g8BT/S8Q/VZ2twnBhHM+3YbfwJgeykMU0mvf s7yPbvml5FoSac2ZTxehHIvwPqHewsZa86ADyhENXIszk3nLo2x7Y8okgtMRh3/rlSvF X7o+ZwOa/fxIvGJ9Fe1GpkxELH7LJrsyvVf/WY5USdUYIQsGGSB5621uOFPwrt99Y+fp CxFyBcAt25KRPLBwdub1wU7Okqn7NUHXuRE2wt9Fyboc6dQPkahojhTv6OphAquUj/3z XTiw/AIBn+f23h0RSPq4Ds0kPbX1ynW9BcIPLljoMo0raqlwPo4bZUNYr41OGKezqIm2 EIVw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=4qyQfUnvReSWJegKwFJB387zPkWEG5HyQ3umWapdiy0=; b=JNjX/YZEhDC6gEitrD6p4SqMUhfLPnO3MJpHrBVpPyGZLr1DM4y8qKxYpVFWUKuHDN gtq5muxUAOfXEK0Rh9WzTpcloE0BiDDkLPz0vIcZaMxnQf3pxyNcnHpdcL01Rh0FtOrY YdvtkUVDvuhiWXENtpU1V52NSL5SOM7rWiDoT0HqO7jBZOdttCHsgLknfJyrYH58tWBr /7OfaJtRi/o+FKDjpioP/6QEZnrrpESc51dxHnXJd3cwlVw5b7eFjilzlszuo8WboulL LoFFAEZpIOkj5Idp8ADMgJlqiHW4Jg2lsyBvFwD7yDhyl3AitaD5QfxrOocPXIFI2A2u p53Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=nvsvQSGY; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a63si753203ede.399.2020.05.08.04.52.46; Fri, 08 May 2020 04:53:11 -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=nvsvQSGY; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726908AbgEHLtB (ORCPT + 99 others); Fri, 8 May 2020 07:49:01 -0400 Received: from mail.kernel.org ([198.145.29.99]:42836 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726689AbgEHLtB (ORCPT ); Fri, 8 May 2020 07:49:01 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (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 AAF2D206D5; Fri, 8 May 2020 11:49:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1588938541; bh=3XIhhm1Au/xGkfcAr3Z9Yi2x3ynKKFKgs0eZV/NQRgw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=nvsvQSGYKzYmyT1GogXbnmyne0OVwyEJLMmFQZoxw8cqsVmaiMwZ18Fv/ZovqttVX j0NvV2MUYhdZ+6GNLpdvDNoKHYFINt4Va8mIkvXzdL8ayp5zLkcKcIuMWyXMDDgKCS EnLtw9BgnueLU3C+JSD/ZF9KHLqAdNlAOOMgvLuo= Date: Fri, 8 May 2020 13:48:58 +0200 From: Greg KH To: Tang Bin Cc: stern@rowland.harvard.edu, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Zhang Shengju Subject: Re: [PATCH] USB: host: ehci: Add error handling in ehci_mxc_drv_probe() Message-ID: <20200508114858.GA4085349@kroah.com> References: <20200508114453.15436-1-tangbin@cmss.chinamobile.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20200508114453.15436-1-tangbin@cmss.chinamobile.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, May 08, 2020 at 07:44:53PM +0800, Tang Bin wrote: > The function ehci_mxc_drv_probe() does not perform sufficient error > checking after executing platform_get_irq(), thus fix it. > > Fixes: 7e8d5cd93fa ("USB: Add EHCI support for MX27 and MX31 based boards") > Signed-off-by: Zhang Shengju > Signed-off-by: Tang Bin > --- > drivers/usb/host/ehci-mxc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/usb/host/ehci-mxc.c b/drivers/usb/host/ehci-mxc.c > index a1eb5ee77..a0b42ba59 100644 > --- a/drivers/usb/host/ehci-mxc.c > +++ b/drivers/usb/host/ehci-mxc.c > @@ -50,6 +50,8 @@ static int ehci_mxc_drv_probe(struct platform_device *pdev) > } > > irq = platform_get_irq(pdev, 0); > + if (irq < 0) > + return irq; <= ?