From:   Richard van Schagen <richard@routerhints.com>
To:     Vladimir Oltean <vladimir.oltean@nxp.com>
CC:     Frank Wunderlich <frank-w@public-files.de>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        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-kernel@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>,
        "linux-mediatek@lists.infradead.org" 
        <linux-mediatek@lists.infradead.org>
Subject: Re: [PATCH net] net: dsa: mt7530: don't change PVC_EG_TAG when CPU
 port becomes VLAN-aware
Thread-Topic: [PATCH net] net: dsa: mt7530: don't change PVC_EG_TAG when CPU
 port becomes VLAN-aware
Thread-Index: AQHZPktX+C8b3oVL10yg7TqKJRr3Yw==
Date:   Sat, 11 Feb 2023 19:02:20 +0000
Message-ID: <F60ED8DB-C0DF-4C28-8D40-A91DE698C165@routerhints.com>
References: <20230205140713.1609281-1-vladimir.oltean@nxp.com>
 <1C64196D-371F-41AC-B357-41100DC66C2D@public-files.de>
 <20230211183143.2jb26kljgpwyc2jb@skbuf>
In-Reply-To: <20230211183143.2jb26kljgpwyc2jb@skbuf>
Accept-Language: nl-NL, en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?GH/XS1UDkzlOh+TIr85v3GUugzkq1iIjFGYwe7zYpsdXNL834SvV+yho9Kpx?=
 =?us-ascii?Q?k8Okxx1c51Gv3yQx3RsCKzhx3U2lVg5ismH5JNs9LnAOr9UkTR5EqtGXALHi?=
 =?us-ascii?Q?UZchcohqKNIfyYKyNFvDaNoamCijm9tKGc3742kzC4NZKjHwAA1yqB9lHh/p?=
 =?us-ascii?Q?ML7EHR96F/js/ragZyYZpN8wq781A6KvtB7EAOY4ZWzlTJPKGV5x+sTaIRww?=
 =?us-ascii?Q?PD7LaeeFrlMDcw+OIkMZbG9/v6chY6gmbE7Lgd5xW9ZTxguBUPGdLZTmZ727?=
 =?us-ascii?Q?LMHctlZCizxhbFzc3ZrNzU59tJaDVzPnrWRU/rt1DsBfvVsyNTGM+kK+SHWq?=
 =?us-ascii?Q?gau1kqURVCtVb+m4osBiOpmSZ3wopOf7lIl5jGytf8a3VcQNkOoZRHRh75j3?=
 =?us-ascii?Q?BqahFn62FNPktSkKKft7PZDSlacPDpauRzXQiNzgCmODPNTVZB/90aypjae5?=
 =?us-ascii?Q?zI5jDCI49H2uCvjPJ+nfsUFobTPn6qyD0h9a1mbV3kQf8u7NZodRYV+RDjAj?=
 =?us-ascii?Q?KpF3SFuxs1mVFaMK5O0DYpx/K5NUxPOdYKsqUtliSlEu/bvV4qoAdLOpym6c?=
 =?us-ascii?Q?bh4aBXA+uNF3WhOit7bf4+ua8fAtPg+VGWfb1dr3UXu2YwUMJ96lc19jvGsk?=
 =?us-ascii?Q?+d96ZIHKefLUxVThhsGH/2v0LYhiYZnf95jD63im6ijGE0y0ZTpA8QKtA9tc?=
 =?us-ascii?Q?Pw9BBivh0xGTrZXl/3J3w/TJvs4TXkpmPdG2eIFzSYNbwci8+XRfpj1nq1Op?=
 =?us-ascii?Q?zQI1XCtlCHp0ce3Tag8JUST+GDsTBiK/f4mzf3gzj1AAMxJLfBMRwEMndmFt?=
 =?us-ascii?Q?hlilCEO8yPaeRqBIN9ifDPePKYfNZ0m+7ymJjr9zK5RloPS+g6HuCKBtb/hP?=
 =?us-ascii?Q?Z1rg3BU8E6sdxi+qmr6e8gFitNrB1fKWHpN/xSrjgUW+2OAseMKy3euXOlpB?=
 =?us-ascii?Q?RgYAAORwpgnAFrSnURfDIUBDO0IPBCfBG33KIb1rNc3RbZGlauqPS9vn3iqV?=
 =?us-ascii?Q?F0NHpzf81Bbz9kYnw+PgrQ8BGCcckk+H8tjlashk48VSHH8oMwvGuhrbqfoA?=
 =?us-ascii?Q?UY6rO/ps5pKWfJYkCiHZCUeKrmMzKdTXaE0ld79TwfYhaNN76Be7FzfsldHd?=
 =?us-ascii?Q?13VmpSNj/MDqSqCcfGEX5tdwyDxXjjbHSsD28mDfMGbQymATVzHOVOtlhNFs?=
 =?us-ascii?Q?XadU3QwFnR5DzxKHnBXebnpE1bAXLKKOFfK1i9XH4BNoZZs/9Qm8urjqUFXM?=
 =?us-ascii?Q?3DZECW/uEtwOr1OpmyW+oIxkg5KDBhvXDVpzP/BHRKbnN2INdL2KBjfPp0lj?=
 =?us-ascii?Q?h85oLnwix+Izr48P5RRemgpGsd5qJeT9YKd2do9XjWKrUl6Mty2XuDLbZRdL?=
 =?us-ascii?Q?7lgwp07ljctcvQTZuFOiZ7M1Fb0iwvt8ldN0Qt3BCIOQ22Pkkda3YhMLVu+0?=
 =?us-ascii?Q?XKo31DLWVyBOfLFMyA+DAQUflV+tqrWB2oI+Nd0J8csgxfxi5xEnguosYoQq?=
 =?us-ascii?Q?EHSbx6CfW3iaZGL7O9I/AL5ns/tdslZQy2PU3g6p74Vf+gH8hO0W6SzNH3Hb?=
 =?us-ascii?Q?ssX8fdXc9I7C47gm+1RMqZk+o282XPqjkOOGcmuXQEJwUrvRp7W4YxWdRqBW?=
 =?us-ascii?Q?NUeBzeGa8IWrrzTmRNAbuXzWMjZckDNg+tD6Wul5mhFGr/6NInQZ4PDMvaA9?=
 =?us-ascii?Q?cVp7wg=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <EE9E92790DE5E9418533758236EFF904@eurprd04.prod.outlook.com>
Content-Transfer-Encoding: 8BIT
MIME-Version: 1.0
X-OriginatorOrg: routerhints.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VE1PR04MB7454.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 075087a2-45ae-488b-e1e1-08db0c628093
X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Feb 2023 19:02:20.5935
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 28838f2d-4c9a-459e-ada0-2a4216caa4fd
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: rdVAaBoDrp35L3ubshPkgK8Bt7hmrnY9gyf/pc3gvAMhH8dOlnuU6gwQu6RNmFVkw2J8UuVom6Egb8PEMPqgKw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR04MB6839
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> On 11 Feb 2023, at 19:31, Vladimir Oltean <vladimir.oltean@nxp.com> wrote:
> 
> On Sat, Feb 11, 2023 at 07:04:14PM +0100, Frank Wunderlich wrote:
>> Hi,
>> 
>> Can this be applied to next too?
>> 
>> It looks like discussion about different issues in mt7530 driver prevents it picking it up.
>> regards Frank
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git/commit/?id=0b6d6425103a676e2b6a81f3fd35d7ea4f9b90ec

Sorry, that patch is wrong. In the shared tree with Arinc I have a different version. One without any need to set / change anything on the CPU port.

Let me check with Arinc before I send that one.

Richard