From patchwork Wed Feb 3 14:37:23 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Alex Elder X-Patchwork-Id: 375425 Delivered-To: patch@linaro.org Received: by 2002:a02:b18a:0:0:0:0:0 with SMTP id t10csp392381jah; Wed, 3 Feb 2021 06:40:17 -0800 (PST) X-Google-Smtp-Source: ABdhPJwfjtz6z/yZ+Q0of+mDmpGf8N7BO+h8mAaf2Sj0ArC/+VAIT8sgKu/odb8TSJ/XfSd+h713 X-Received: by 2002:a50:fe04:: with SMTP id f4mr3279629edt.63.1612363217614; Wed, 03 Feb 2021 06:40:17 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1612363217; cv=none; d=google.com; s=arc-20160816; b=BJwcOy83lQs1Wa5VMHkuzwvrnCOwGhaYJC2F3MwNJZ8Upiye4IXPTuPFHzfaozWxe9 /38v248/NZbVhgoZ7GntTqsrm2eTvU9Hbd5kJkyBkzFdb14XsXgiRygcOfSUVAdrxzE8 7MD/RWJcktDkW+MHHDtjByjnf/kw6xM48zpoAWxnuyybdus6TefQqPjpxm/XqqZ14+y2 YTnlGk84mTjKvuD6EoqfFTn5fnUacgHWM3U/UgSIOkUN3z4XiVQfss049wS0ULdX7Pvr Yny1+qtuzpv79g8dYbuJe4kn+MA8ocVyKZSX9RI57Zr5MZjIZmMK5F5LfFuh4GscSH/6 yhSg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=XsFAitZq+M4tO2JmgFSy4L0QDeaBFSgSKoOpXPki0CA=; b=UBxgq0SqJzsrHXeNB5NeMk2FBPzDzPBnqw5jGgcPHuRxAEoNviOFs9RP/DPPFKouHa D4lfSzI72l5dyDqYpfFFS7lrErYNJ2xLBxNR/HXsjxTyXCQza4vgFu2AGwsVD/doh89U X6rZRZPTwVY4QCdBr5cN+XNRI+vKwriy0P/mom9OdpUlQBxHI2JFyx5xfKBFw9x/5MY7 qaH63uzlDFM6AGnGyqrAsBOwMssI8leX63zdCtXRA3IdtttHQx6NpqYZbesoWERM2Jsj GFD8f+oVcSqaiIKsGDxZ2nbTf10bxp/amIFtc6LY/IQcBqO2AXEuDkyZnAu6krnZkYm3 u55g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=MqGiJDs0; spf=pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id di1si1242156edb.489.2021.02.03.06.40.16; Wed, 03 Feb 2021 06:40:17 -0800 (PST) Received-SPF: pass (google.com: domain of netdev-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=@linaro.org header.s=google header.b=MqGiJDs0; spf=pass (google.com: domain of netdev-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233210AbhBCOjA (ORCPT + 7 others); Wed, 3 Feb 2021 09:39:00 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57010 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232245AbhBCOiH (ORCPT ); Wed, 3 Feb 2021 09:38:07 -0500 Received: from mail-io1-xd2e.google.com (mail-io1-xd2e.google.com [IPv6:2607:f8b0:4864:20::d2e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6C59DC061788 for ; Wed, 3 Feb 2021 06:37:27 -0800 (PST) Received: by mail-io1-xd2e.google.com with SMTP id y19so25457958iov.2 for ; Wed, 03 Feb 2021 06:37:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=XsFAitZq+M4tO2JmgFSy4L0QDeaBFSgSKoOpXPki0CA=; b=MqGiJDs0nR5mYzkyjKBPRmovzjcjk/nQU/ZwSGNMFfmcgDts0xs7NZ0A0xW4MxeElW 6pymBnBN4xmUHYWS8L86gailn/M8sBnAt0z0iu8OP0SSynYvCCTw8oze9TTy/X0ZAwAJ ZUdouE/ph4LhEyNlE1qIkx2Mv/WjfZoZ98mTEF09hpWEVjxFva3F1OoEwopMoDtlyIzn TKnpvfDWgveVynlVyZpxmtqqLvXSS/l1jqLncFKNpFfAmTaArzdBhjeyY2KTwwb6AlVC 5vKjIwNusRnFmFJS3fNzN5nxEiPkNJkITQuk1FsQOpye5Rdb3SqXa6ksrZkbbjX1v9hO xLLA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=XsFAitZq+M4tO2JmgFSy4L0QDeaBFSgSKoOpXPki0CA=; b=ZE6y/gt5Gn9pHBCc/bpO8JFBLeTeV2HKv5X5QmRf9nHUn/BOJi/WYzdhd6Af3Ja/Mz 40JBUuRnbUzXE3hTChiRe0bVe/bXQR5+84WRPf6w6IpWWTnw28lUj/6MJ0Sd0h4mB6wk 0oyTdYm3b9VoOl2dbs8zcGvGnJSr5s7Zx7ClLK8iSOH0AiXNSHmQXR5rcSMVQEB79YWh Mb/9OVBIATkI/WJJ7EmP1dglOMFljxs66UTQpKwa1dLMctbsffTBX+H8Etlf2vbHs8ZJ OFEy2llkCPFnbAicHO0xGSO9LdzJGI3DLF8Akk0OA1RVc6JPuGw3HzH+3xIofpHI91Tr pd4g== X-Gm-Message-State: AOAM532EBt3VFGZl1eBw3R8Ao3HhqirDTDi5nnnjpN3I1yUw4wqjH23U PAp9GdOW6y0LR/9NE0rPUoYUVw== X-Received: by 2002:a6b:5505:: with SMTP id j5mr2635815iob.148.1612363046927; Wed, 03 Feb 2021 06:37:26 -0800 (PST) Received: from beast.localdomain (c-73-185-129-58.hsd1.mn.comcast.net. [73.185.129.58]) by smtp.gmail.com with ESMTPSA id i78sm487816ild.50.2021.02.03.06.37.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 03 Feb 2021 06:37:26 -0800 (PST) From: Alex Elder To: davem@davemloft.net, kuba@kernel.org Cc: dan.carpenter@oracle.com, elder@kernel.org, evgreen@chromium.org, bjorn.andersson@linaro.org, cpratapa@codeaurora.org, subashab@codeaurora.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH net] net: ipa: set error code in gsi_channel_setup() Date: Wed, 3 Feb 2021 08:37:23 -0600 Message-Id: <20210203143723.17881-1-elder@linaro.org> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org In gsi_channel_setup(), we check to see if the configuration data contains any information about channels that are not supported by the hardware. If one is found, we abort the setup process, but the error code (ret) is not set in this case. Fix this bug. Reported-by: Dan Carpenter Signed-off-by: Alex Elder --- drivers/net/ipa/gsi.c | 1 + 1 file changed, 1 insertion(+) -- 2.20.1 diff --git a/drivers/net/ipa/gsi.c b/drivers/net/ipa/gsi.c index 34e5f2155d620..b77f5fef7aeca 100644 --- a/drivers/net/ipa/gsi.c +++ b/drivers/net/ipa/gsi.c @@ -1710,6 +1710,7 @@ static int gsi_channel_setup(struct gsi *gsi) if (!channel->gsi) continue; /* Ignore uninitialized channels */ + ret = -EINVAL; dev_err(gsi->dev, "channel %u not supported by hardware\n", channel_id - 1); channel_id = gsi->channel_count;