From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 109EFC433E0 for ; Sat, 6 Feb 2021 04:21:43 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id D845764F06 for ; Sat, 6 Feb 2021 04:21:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229824AbhBFEUd (ORCPT ); Fri, 5 Feb 2021 23:20:33 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37376 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229767AbhBFCbp (ORCPT ); Fri, 5 Feb 2021 21:31:45 -0500 Received: from mail-io1-xd35.google.com (mail-io1-xd35.google.com [IPv6:2607:f8b0:4864:20::d35]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8456BC0611BC for ; Fri, 5 Feb 2021 14:11:07 -0800 (PST) Received: by mail-io1-xd35.google.com with SMTP id f8so8781937ion.4 for ; Fri, 05 Feb 2021 14:11:07 -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=VdNQsrR6L4Zb5IpKYZydONt7isMKUPDajZ0BlWteITU=; b=GJef85PVibNB32cjYhF6sjtPaoih2XjB+VLDpYmn0I0+42AAj0kuRQ7CSqlsAcMlyT vaODY3I56ik030rqwqW2HZDgEpC7shYpMVNhor41GRtpZWRS1oiHivgEfnu/hzuYbb8h Oq4voH7IXpE0YiGfsrXH+9smDa9U8Jr/LhR/2sKb0oUNbZxVqrHzJAFjytg8Cl36l9/O kA5MTxeyfxVjOZD6X0VZ/n2ffz9Jc9EbZAZxMp8ksWQiEO8H2VUnmsdQrVCz8/W3Y8DV DNYaHoW74RzmiHGnOsTfOlFWnuOUDQHImSnZst1WOcEIzzJfs8I3X7bq2dG0DGB7wWtV XREA== 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=VdNQsrR6L4Zb5IpKYZydONt7isMKUPDajZ0BlWteITU=; b=EVU1Ex/vJrZ3VvMUGerUWSaEUqkx6C9x70LR4dpG/CpgX1poPh7oTA8EQIDtYbczeh PcqQNSWPsh0j1EhKBOvvj0m/JcWE4uQA2TVswg4FVIyGELAG+Yf+UOwoPD7zVLO3Nj8a B3lZk713E+zyMPe/XaDnoUOPqq0thvB53GoXh1Rs8X179a4s9pS+013XJipenfHkN/3r df8D4HNnuZ3gGudT3zWzwcD5sItMi6HWgnxxQBedZAC5a8qubQMK6JVygfiLbuvQX7Yt lgRyuDaaQtBLLqnM/Z4UaRhkZx6r3uD5FHhPJYi5Ev46xzqdFcTjK3uwsnrCHuzoujGb GTEA== X-Gm-Message-State: AOAM531InnLs0hWOw2Y7m6fKPNehb8+pJUv0GIpcG9bv1QzUiabnVaBO V/ikqUJ5syPUQZjG28rzbYwXHQ== X-Google-Smtp-Source: ABdhPJxerHpEay6NlWEwO5TQ9neasKrO20VZimkAH87y5dIuzHoFViU5NymJo1bu29WrzWzxj/5fzA== X-Received: by 2002:a5d:9552:: with SMTP id a18mr6136048ios.154.1612563066961; Fri, 05 Feb 2021 14:11:06 -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 m15sm4647171ilh.6.2021.02.05.14.11.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 05 Feb 2021 14:11:06 -0800 (PST) From: Alex Elder To: davem@davemloft.net, kuba@kernel.org Cc: 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-next v2 0/7] net: ipa: a mix of small improvements Date: Fri, 5 Feb 2021 16:10:53 -0600 Message-Id: <20210205221100.1738-1-elder@linaro.org> X-Mailer: git-send-email 2.20.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Version 2 of this series restructures a couple of the changed functions (in patches 1 and 2) to avoid blocks of indented code by returning early when possible, as suggested by Jakub. The description of the first patch was changed as a result, to better reflect what the updated patch does. It also fixes one spot I identified when updating the code, where gsi_channel_stop() was doing the wrong thing on error. The original description for this series is below. -Alex This series contains a sort of unrelated set of code cleanups. The first two are things I wanted to do in a series that updated some NAPI code recently. I didn't want to change things in a way that affected existing testing so I set these aside for later (i.e., now). The third makes a change to event ring handling that's similar to what was done a while back for channels. There's little benefit to cacheing the current state of an event ring, so with this we'll just fetch the state from hardware whenever we need it. The fourth patch removes the definitions of two unused symbols. The fifth replaces a count that is always 0 or 1 with a Boolean. The sixth removes a build-time validation check that doesn't really provide benefit. And the last one fixes a problem (in two spots) that could cause a build-time check to fail "bogusly". -Alex Alex Elder (7): net: ipa: move mutex calls into __gsi_channel_stop() net: ipa: synchronize NAPI only for suspend net: ipa: do not cache event ring state net: ipa: remove two unused register definitions net: ipa: use a Boolean rather than count when replenishing net: ipa: get rid of status size constraint net: ipa: avoid field overflow drivers/net/ipa/gsi.c | 69 ++++++++++++++++++++-------------- drivers/net/ipa/gsi.h | 1 - drivers/net/ipa/gsi_reg.h | 10 ----- drivers/net/ipa/ipa_endpoint.c | 38 +++++++++---------- drivers/net/ipa/ipa_reg.h | 22 +++++++---- 5 files changed, 73 insertions(+), 67 deletions(-) -- 2.20.1