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=-7.2 required=3.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,NICE_REPLY_A, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no 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 0CA1CC433DB for ; Wed, 31 Mar 2021 11:20:21 +0000 (UTC) Received: from desiato.infradead.org (desiato.infradead.org [90.155.92.199]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 504876196C for ; Wed, 31 Mar 2021 11:20:20 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 504876196C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding :Content-Type:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:Cc:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=D9gC6iW/b3tDv34RLn3D6NbN4XnNNwlEzC37TcyXQXY=; b=VxrHw5mmGEM0Cxa9XCkZkukAC EjoDmjYZLVQlXvvehbhjEzUeRn1f+dSCnjrAI4uprc4ckjh5HGREVQZsJxgxqL3m2jg+310aFjE6c AfUb6GD0hkgLYGbBEIyejEq3SwnSBqzpk1B3cu9MGeU9Dw4T8ReZhv1jcM92AacHhoAHxpqnQ82UD eMT5i/ZwPR0P5h/GdJP9sW0KBD19uIKQnE/UBGNsBLMnYDTul3tdC//adtHipUs2YNH5wcnB2XpTC kprSNFqV6aPQxk5p5ZLAbCrIjCQfoROkX+1YpzxRulZ6qaj9632QLxNnCDPTZAygEZIMnq6BUFKz5 IqH8P26Ew==; Received: from localhost ([::1] helo=desiato.infradead.org) by desiato.infradead.org with esmtp (Exim 4.94 #2 (Red Hat Linux)) id 1lRYrv-006QAz-4j; Wed, 31 Mar 2021 11:18:39 +0000 Received: from mail-wm1-x333.google.com ([2a00:1450:4864:20::333]) by desiato.infradead.org with esmtps (Exim 4.94 #2 (Red Hat Linux)) id 1lRYju-006NBG-Sz for linux-arm-kernel@lists.infradead.org; Wed, 31 Mar 2021 11:10:28 +0000 Received: by mail-wm1-x333.google.com with SMTP id 12so9949410wmf.5 for ; Wed, 31 Mar 2021 04:10:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=SCIPrEQXmkFK9b+U3x+rs0/SFIu2Lw+iwpbKzvEC54I=; b=Xh5U3k4+K7YqbYZoenAUQ01BUgB9Wnn5PfYMDThE0qXD3ifUPW0Tr1A2BeWybn5Lh+ 9sSEhRzu1FBLW3tinYQ0zHuO6xXpBMpVWo+LtWD4USs1HERKGKse3i5/gAbCh8Ra+RAD etdd5KwOOYhMObwE6XVN72GogGGN7vydloH8t3Cj+yx3rR7rXxsMG1xhMWdnMEWCZWRA TdNwcOk+14fQPSIm5VTs4wTgKMhaIXYhL76C9U5C2ZJUlieFFF/qRpO1VLShm6oyqCTd V5+JH4dONYk7GnqYZSjbt6SpqxZhIGV0l35bQeLW9goudqt7yZvJH9qmqpBJRIp9fhfe CkAw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=SCIPrEQXmkFK9b+U3x+rs0/SFIu2Lw+iwpbKzvEC54I=; b=THNJguKJkBdRkliOkVOQ4jO3g1z8xB9k5BWtbA1fXcsTvtMgk7YudeVxbrtKX+k76i Yskd6R4Ij8lsVwDht5lxge8vjxJdvRVPKwFOj2d9fsz42V7dbJsGNBMPhGSIjT99+J+D sYD8T0DcuVBvS/RLsoMrIQLyp7SgavLmB8FjsiKTMu6/TjNIS9QLuz+lXXH4yee/Rp1Q 7MUeC/iYpkVkJ7yTNylF72eUL2pS3ztu1xmowegTUkKE32UIe21gms0/kCeSW4PaYgAE FjQ9/gSe0L5Hdwjxj5EvBHl4ZjTuwThDId5dyOl2HykBtJhlQHvI6guBKH7uP5Vy1DSY cguA== X-Gm-Message-State: AOAM531JOMqpb/2Yg8wuKyGegKb2asRh8dIDC5shATbMztl8nenA7hMH +/7u6b9k3ttnOxCgKdGPhC4= X-Google-Smtp-Source: ABdhPJwN9aeNLQFXf4YXYRsuBlLKXWnbvpsOy8BY8Eepw1f7SbXkgtblU15dkqlvX/z7NBnwoAckWA== X-Received: by 2002:a1c:7905:: with SMTP id l5mr2709597wme.181.1617189015735; Wed, 31 Mar 2021 04:10:15 -0700 (PDT) Received: from ?IPv6:2003:ea:8f1f:bb00:953:1e7:27d8:7327? (p200300ea8f1fbb00095301e727d87327.dip0.t-ipconnect.de. [2003:ea:8f1f:bb00:953:1e7:27d8:7327]) by smtp.googlemail.com with ESMTPSA id p16sm5139149wrt.54.2021.03.31.04.10.14 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 31 Mar 2021 04:10:15 -0700 (PDT) Subject: Re: [PATCH v4 0/3] PCI: Disable parity checking To: Bjorn Helgaas Cc: Leon Romanovsky , Russell King - ARM Linux , David Miller , Jakub Kicinski , nic_swsd@realtek.com, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, netdev@vger.kernel.org, Bjorn Helgaas References: <20210330174318.1289680-1-helgaas@kernel.org> From: Heiner Kallweit Message-ID: <3ed9ff5a-3984-1abd-c857-dd5ca942fcde@gmail.com> Date: Wed, 31 Mar 2021 13:08:56 +0200 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.9.0 MIME-Version: 1.0 In-Reply-To: <20210330174318.1289680-1-helgaas@kernel.org> Content-Language: en-US X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210331_121023_420762_252D6486 X-CRM114-Status: GOOD ( 19.02 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 30.03.2021 19:43, Bjorn Helgaas wrote: > From: Bjorn Helgaas > > I think this is essentially the same as Heiner's v3 posting, with these > changes: > > - Added a pci_disable_parity() interface in pci.c instead of making a > public pci_quirk_broken_parity() because quirks.c is only compiled when > CONFIG_PCI_QUIRKS=y. > > - Removed the setting of dev->broken_parity_status because it's really > only used by EDAC error reporting, and if we disable parity error > reporting, we shouldn't get there. This change will be visible in the > sysfs "broken_parity_status" file, but I doubt that's important. > > I dropped Leon's reviewed-by because I fiddled with the code. Similarly I > haven't added your signed-off-by, Heiner, because I don't want you blamed > for my errors. But if this looks OK to you I'll add it. > > v1: https://lore.kernel.org/r/a6f09e1b-4076-59d1-a4e3-05c5955bfff2@gmail.com > v2: https://lore.kernel.org/r/bbc33d9b-af7c-8910-cdb3-fa3e3b2e3266@gmail.com > - reduce scope of N2100 change to using the new PCI core quirk > v3: https://lore.kernel.org/r/992c800e-2e12-16b0-4845-6311b295d932@gmail.com/ > - improve commit message of patch 2 > > v4: > - add pci_disable_parity() (not conditional on CONFIG_PCI_QUIRKS) > - remove setting of dev->broken_parity_status > > > Bjorn Helgaas (1): > PCI: Add pci_disable_parity() > > Heiner Kallweit (2): > IB/mthca: Disable parity reporting > ARM: iop32x: disable N2100 PCI parity reporting > > arch/arm/mach-iop32x/n2100.c | 8 ++++---- > drivers/net/ethernet/realtek/r8169_main.c | 14 -------------- > drivers/pci/pci.c | 17 +++++++++++++++++ > drivers/pci/quirks.c | 13 ++++--------- > include/linux/pci.h | 1 + > 5 files changed, 26 insertions(+), 27 deletions(-) > LGTM. Thanks! _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel