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=-11.0 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,INCLUDES_CR_TRAILER,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS 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 3D203C4361B for ; Sat, 19 Dec 2020 18:00:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 08AFB2389F for ; Sat, 19 Dec 2020 18:00:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727571AbgLSSAA (ORCPT ); Sat, 19 Dec 2020 13:00:00 -0500 Received: from mail.kernel.org ([198.145.29.99]:58354 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727208AbgLSR77 (ORCPT ); Sat, 19 Dec 2020 12:59:59 -0500 Date: Sat, 19 Dec 2020 09:59:17 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1608400758; bh=pdO7yN27dVbxPgRegNWOCdEQpv/5y8Wa0WcvQGFzoF0=; h=From:To:Cc:Subject:In-Reply-To:References:From; b=holaY5/ZCCy1tufiJqgn51KOF2wx6Aj5L6gpdupdp1Aef7D3Qz+DFS+1NsIWj5dI6 4Y+bmaLuRfbn+jJzt71J8teHp8fKi8Fue6d4IHYtCOo3DCAPxFoAosOqoG6ryJWC8r nj9LVLzSlpiAkHHG+gMXCSSekXcFU6DRuilthVueCPRd5rUQrtqshhm7oxpQN3Y7se fRu6yNjjXZdvvsqcv2rNY+383gExHrktmnsZe7rMcIKcHvSuGfiqLtBQpifnpvhhHg 5MfeDGukiurVImv2FKWdjJKjDQREOanCavrUmQ7Fam0n/k6Z4YFMuFzPu8KOpvBqY6 xgocci6UgjGUg== From: Jakub Kicinski To: Cc: , , , , , , , , , , Subject: Re: [PATCH net v3] net: mvpp2: Fix GoP port 3 Networking Complex Control configurations Message-ID: <20201219095917.67401234@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com> In-Reply-To: <1608208648-13710-1-git-send-email-stefanc@marvell.com> References: <1608208648-13710-1-git-send-email-stefanc@marvell.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, 17 Dec 2020 14:37:28 +0200 stefanc@marvell.com wrote: > From: Stefan Chulski > > During GoP port 2 Networking Complex Control mode of operation configurations, > also GoP port 3 mode of operation was wrongly set. > Patch removes these configurations. > GENCONF_CTRL0_PORTX naming also fixed. Testing the stable backport it looks like this addition change will be problematic. Not to mention it goes against the "fixes should be minimal" rule. Could you please send just a one liner which removes the offending ORing in of the bad bit? We can do the rename soon after in net-next, the trees are merged pretty much every week so it won't be a long wait. > Cc: stable@vger.kernel.org > Fixes: f84bf386f395 ("net: mvpp2: initialize the GoP") > Signed-off-by: Stefan Chulski