Date:   Mon, 6 Feb 2023 22:33:35 +0200
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     =?utf-8?B?QXLEsW7DpyDDnE5BTA==?= <arinc.unal@arinc9.com>
Cc:     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-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: <20230206203335.6uxfiylftyktut5u@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>
 <5e474cb7-446c-d44a-47f6-f679ae121335@arinc9.com>
 <f297c2c4-6e7c-57ac-2394-f6025d309b9d@arinc9.com>
 <f297c2c4-6e7c-57ac-2394-f6025d309b9d@arinc9.com>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <f297c2c4-6e7c-57ac-2394-f6025d309b9d@arinc9.com>
 <f297c2c4-6e7c-57ac-2394-f6025d309b9d@arinc9.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bzRFVy9mRGxBL2pLNisvbHNQWE5NVllmaWVhMUhnb01IUm1QRUd5RXlOeHl5?=
 =?utf-8?B?ZlcraUIyeUMvV1hpNXZFZHp2K0F1eUU1dUJjRU4zR1puNmJaRUx5cXZ3dVFs?=
 =?utf-8?B?b1ZJZnd5VVZFVmVSV3l3cDRYdTRYNTNDNjhwblhyZWk1eHFoQ1IxYnQ0cHRh?=
 =?utf-8?B?VFg2WGZnS3NMN2xmY3A2WWNVWWNWalhIYWNnendFdnNmckV1N2czVVhVS1RX?=
 =?utf-8?B?ZVBVcG9BdHRSaHlHTjY1WVNKQVBTZlZsYjVFcnJxM3lMME9rWGh5aGJMYTZa?=
 =?utf-8?B?THVpaDM2UStpVFg5SUN1ODQwRWlZYVlibElEdy9SRkEwU0pCYWttb21IUVcw?=
 =?utf-8?B?WStjUkkwYXZDeE9TUjdTMjRCeDJiaXBKaVp1aVR0VDZ2Nnl0dzJkYk5wT1pr?=
 =?utf-8?B?d25oa2hJYk11dS9FSnloY1dsbzVzcGRFSC9hdzFoV3FOZkxRYXRZaVJ2YVlX?=
 =?utf-8?B?N2V5MkNoTmxWSjEyZzdmUDg4c3IyTmFFZEp5SGhmWjRwU0RlSFpyR0hDd0Er?=
 =?utf-8?B?S1ZpNmRzZE5LbnY3eGxuUHQ2eWxiNWMrK2ZSbkR1c0pPQmJXMmZJK3ZqN1NJ?=
 =?utf-8?B?OXhVZW5XTGVqR2I0TzdyaDlUNnoyT21wRGlYNFNwUUpadElzUHJuTkNVOFhP?=
 =?utf-8?B?bUZkSlZ6NktiWGc2MTFaTUlvMXNuNTV6NGl3a0VJck5PeDB4TlE0WWFXcU9V?=
 =?utf-8?B?K1J0TnVRUzRlRitObzA2UUZGSGxSeEIxWTMzQWljR1dnSHo5M1BFdlVWZzA1?=
 =?utf-8?B?RkE3RFo1dXh5SUdkRm5VNEpzTWRoVzNGRmpzc0thWTRJSXY3TTEwSVd3WldE?=
 =?utf-8?B?NVF0NFZTT1dvbkZ2MVJ0aDY5WURZa0tSc052QllmeHFLdnBoNURGMmxuRjBp?=
 =?utf-8?B?MlJQVG5oY0IrNUNHMDU4b0tuY0FQK2Vobm1mSUdyNWJSTUIzWEJtbHZZcVF0?=
 =?utf-8?B?c29kYWdYSUZ6Y1hkU2JBODUwSGRBb2wxMHB5WjZqcGt4SERUVG9xUVN4WnBM?=
 =?utf-8?B?aHJuVWw4Z0VUK2NKSVBQZkQvbHRWOVFqUlcwNXhSUGFyK2grbjNFYUoyZTNm?=
 =?utf-8?B?bVE4M2xIZUtjM0d6OVpuNmVzZ3ZvcmpVenNsaGxSQ3NRUFdSaUMzQ1N4STBr?=
 =?utf-8?B?YUtvZjhCSW4zamdsL054N0dGdGJTOVkwd00xUWR2d3BQd0JiU0srWUJxTU1W?=
 =?utf-8?B?ZkgrUXZUZXhPdnprVG1mM21zd3lPRC9kNmVzOTRmY1dIakU5a2xCTm00eVJm?=
 =?utf-8?B?NUxxRXh0YUMzQXkrNWI4VnZmdjR1QXJ6Uml4NDkvb3lMZG5iZ2dXSVAwRGN1?=
 =?utf-8?B?d0VMTlNuUkRwa20zcUFGcWptbWVPQzdrZHU3V2RSWE8vWWh1ZkZoK0xaUGhX?=
 =?utf-8?B?cE1od2tpdTM2MVU5aFBSa0N0SXRNUTdUcmN1c0lmRHZLUUhXOGxXWFVEWm5I?=
 =?utf-8?B?VUFOUXVud2YvRGNoZjdqUDVXbVNhTGMrUzB1VEJwcjNqOTFTVUVXWTE1MTRP?=
 =?utf-8?B?MDR3OFowbFB5N0x1eHJoaGxnVGlRVktvblBOQlBtVElrdGJmVHBUUmJEeWx1?=
 =?utf-8?B?anR6R2hYUTlOZjg2VVZjVHNKVWovL2ZNMndWNDlWYktmd0hOd0ZHdVp2T0xP?=
 =?utf-8?B?dHZ1bnhwR0liYXppT1FpTzh2U2NFeUJEcVFUQnlYWFY3MmtMYkpoTENRd2Rz?=
 =?utf-8?B?TUdySXJIQTU0YmpWVUp6Q25nMkhra0hGSG9FVXBKMWt4akl6NXg0WkVvK0RU?=
 =?utf-8?B?WkhxcHl0VDdmWkFlQlJiNlRTRTBZUXlyUU9yRU1GbGZmb3l6Tk1Vb2FKOGlT?=
 =?utf-8?B?dWVUL3paVWxwanZwSThGZFM4b3IxaE1NRFBqY0dPcklYd21OOHgxdkp3VndJ?=
 =?utf-8?B?VUZ2YUdFb1lUSnpRTGM4TUE4SDJyT0hRWG93YWhEYTBhRDlYOGNvMnRhbUNa?=
 =?utf-8?B?dU0wcC84aVQrcC9SOHFiZ1R1Vlp1YVVaSSs3dnRQeS9DM2Zpd0hOdGxzdkk4?=
 =?utf-8?B?YUR4UEtEMkJhVk9XeEg5bDdFN0VUdHhOQnpzaW5sQlBJU2hZM3R5aUpFbnFW?=
 =?utf-8?B?cmZTeGpyQzUxUUhKcHhWT3d2V21ibmpKWDVUbmsvS04vS3Z4M0VJUkUzOEcr?=
 =?utf-8?B?bVczOC9GbUxkWStUdFNWMEI4Ris4YTVjMnBYemR2ZzloU0VlQ1QrS0EwUS9Y?=
 =?utf-8?B?MVE9PQ==?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3547a517-c27e-4c43-a0a0-08db08816e5e
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Feb 2023 20:33:40.1368
 (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: 5cye8bTLlbcXxB6fA7wUHlzk9OLchBCu1g1jpkg7B+hFmEgclAPWtuEQiim2Yp7UnFRFTWh/GtG0SQ28OQPgKQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBAPR04MB7304
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Feb 06, 2023 at 10:41:47PM +0300, Arınç ÜNAL wrote:
> One last thing. Specific to MT7530 switch in MT7621 SoC, if port@5 is the
> only CPU port defined on the devicetree, frames sent from DSA master appears
> malformed on the user port. Packet capture on computer connected to the user
> port is attached.
> 
> The ARP frames on the pcapng file are received on the DSA master, I captured
> them with tcpdump, and put it in the attachments. Then I start pinging from
> the DSA master and the malformed frames appear on the pcapng file.
> 
> It'd be great if you could take a look this final issue.

What phy-mode does port@5 use when it doesn't work? What about the DSA master?