Date:   Tue, 7 Feb 2023 14:39:52 +0200
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Paolo Abeni <pabeni@redhat.com>
Cc:     =?utf-8?B?QXLEsW7DpyDDnE5BTA==?= <arinc.unal@arinc9.com>,
        netdev@vger.kernel.org, "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Sean Wang <sean.wang@mediatek.com>,
        Landen Chao <Landen.Chao@mediatek.com>,
        DENG Qingfang <dqfext@gmail.com>, Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Matthias Brugger <matthias.bgg@gmail.com>,
        AngeloGioacchino Del Regno 
        <angelogioacchino.delregno@collabora.com>,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-mediatek@lists.infradead.org,
        Frank Wunderlich <frank-w@public-files.de>,
        erkin.bozoglu@xeront.com, richard@routerhints.com
Subject: Re: [PATCH net] net: dsa: mt7530: don't change PVC_EG_TAG when CPU
 port becomes VLAN-aware
Message-ID: <20230207123952.r2r7tnama3vcr7vt@skbuf>
References: <20230205140713.1609281-1-vladimir.oltean@nxp.com>
 <3649b6f9-a028-8eaf-ac89-c4d0fce412da@arinc9.com>
 <20230205203906.i3jci4pxd6mw74in@skbuf>
 <b055e42f-ff0f-d05a-d462-961694b035c1@arinc9.com>
 <20230205235053.g5cttegcdsvh7uk3@skbuf>
 <116ff532-4ebc-4422-6599-1d5872ff9eb8@arinc9.com>
 <20230206174627.mv4ljr4gtkpr7w55@skbuf>
 <c4c90e6576f1bc4ef9d634edda5862c5f003ae3c.camel@redhat.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <c4c90e6576f1bc4ef9d634edda5862c5f003ae3c.camel@redhat.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?q+ywLf6gXz/tevE8nGQytpDNq+aZzTEVvv6z0DWc6FKCoUJdE+4w2fQ8nBlr?=
 =?us-ascii?Q?UjAu09ZQ6LTyDWloOmV442fpIexqLGlU15qLs6bPCPDlUw9WOmfQRIatCc7o?=
 =?us-ascii?Q?gzEs1o6VkDMxsd3w3Bn0nisaBQZhWjDWEgYVHt95Mx3y5U8DVmxxffi76fb3?=
 =?us-ascii?Q?6WSW11cofz4HzSLCIGmNd010YmUL0unXDY4l91/LGiuF34MSdmk5cggzdxwU?=
 =?us-ascii?Q?xfKFSauV3DAHNjwSGpA2sUc3VboGXFYxnBDq+KlRM3+vKthBMNoau0XhaEC7?=
 =?us-ascii?Q?8RJJshUb9B8eT4Ct0YFVtPA2JqARpa83u2eM43xs2jrC+5P+LUVBGnV4wofB?=
 =?us-ascii?Q?hghIYcVB8xFlHoWFo+whfT/O5dnldwtVph7hRzEjUrwSh12fAGAdivJPD/0V?=
 =?us-ascii?Q?RO0NIk9pwLW98swgEb6Vl7fRfLmixJKMSpICqCa/a8RZRoDYCWzejz5d0gIq?=
 =?us-ascii?Q?xNBSZkSluQMqv/b/3dxAAK64WDSDmPpKgYAOeIWQlVn1dWtFa3xnb9JHqITj?=
 =?us-ascii?Q?6P0KDffMSnxo+Rae4Q/zllZIucmsHa/FiDnxaSl/YQlt7o5B9CijAnjZNKEw?=
 =?us-ascii?Q?J6fn7sl6QhUahSixmbidWUcSD1lbjKFsB9aGZlKsNQaYOjn97ZleoaeM8nud?=
 =?us-ascii?Q?ii20Eu4ji4sXuTaFFP/Cm6szcZ96pJEXX0Ukkn5PTE5nd3XYSOavKI7k5x5A?=
 =?us-ascii?Q?nRtSwh7ScxwocelFQayZjNIXSeHRlHOLLv9SxfnOOFZoOgbMvevR1kpMbtHB?=
 =?us-ascii?Q?kvNF7Cs6s5O8oLfYrRwXoOe4vC3ftYoNh/LCMRJ6LLTxXb0rQKEcscDoGCZg?=
 =?us-ascii?Q?a8G8YOSzIQ0eDVxtJQY4AAeso8qecuoNdgQ3WZQNsXcwWWq0dcGH6LJWi/tb?=
 =?us-ascii?Q?UHhopjU/cHulKG3gpY35mib4c/oVGwP1jmwA59F+Zlyyjpmqa8t40wBmeKsA?=
 =?us-ascii?Q?dZGPr7T97fuV86LJcXkXSy5MyNC0B+VkFGXJ25f3gPIXPEn9XXzikvu1gYpn?=
 =?us-ascii?Q?B7ydHqyw86tp97EhS27FP8AC8EzNvgmmLm4+F468DKtVYnxR/G3ux4y4QzbJ?=
 =?us-ascii?Q?7nsxEL7XqEguCEREDBiPGDqiLDbRY0VjAdeh59J0hC3oJG2Dt/oxpp4wlFav?=
 =?us-ascii?Q?MNs1rnWG889ky+2EdE7+TW9sSwJTzJLBUpwvKXGJrEu4W7ty5GtMaNDxov5u?=
 =?us-ascii?Q?gkRU1BS1DZZa2f1jwuDAvUOvBqZEQJgX9JrjQ94zx5jTz0Jfy30cQWNngSNK?=
 =?us-ascii?Q?ddYxK8Ic9ruqT9twQcPK6s2PdWMNvXHfTTqrJsCZ1/0TgXrUCelKhNdrwSQq?=
 =?us-ascii?Q?qzQaZEzCs0cN+SnVTimnk/3Em0tFRzH3SEj6B9DdleMgIv0ZqFBU+akJ1O2A?=
 =?us-ascii?Q?Q8x7xWEeifwWHZ5IQY81u/r6Fam1beHK38FkRK4wM3J6fxMf0xsyP2SMtoXG?=
 =?us-ascii?Q?8Ie4o26tq6kMsZIzvqrYzpDXZh7iw1v1gaq4rublwVa9W7XdlTulurR++PoR?=
 =?us-ascii?Q?6b38hndw+FUPCNQg1i4bpvFeaHeEW86l76yuRFQfhGWH6S2KlQbd8EZn5G2E?=
 =?us-ascii?Q?gn1UirT/fbvEkwuYPovGGGQ/qGaN+klC8XdkXvCs3HRjOVNMvUoMjvyBi9AD?=
 =?us-ascii?Q?pg=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 76407186-5e67-4c7e-afe3-08db09086bb9
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Feb 2023 12:39:57.8526
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: q+lQYMnre43lqmQakZilXTX7LgwJnieh6pPVttuJpzSNYPzLM5VtMPE/fzlFXEp4cKafSpcPvQ6qTy7MGkiMFA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB7021
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Paolo,

On Tue, Feb 07, 2023 at 11:56:13AM +0100, Paolo Abeni wrote:
> Thank you Vladimir for the quick turn-around! 
> 
> For future case, please avoid replying with new patches - tag area
> included - to existing patch/thread, as it confuses tag propagation,
> thanks!

Ah, yes, I see (and thanks for fixing it up).

Although I need to ask, since I think I made legitimate use of the tools
given to me. What should I have done instead? Post an RFC patch (even
though I didn't know whether it worked or not) in a thread separate to
the debugging session? I didn't want to diverge from the thread reporting
the issue. Maybe we should have started a new thread, decoupled from the
patch?