Received-SPF: pass (google.com: domain of linux-kernel+bounces-173092-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <763a2640-2170-4b62-86db-7d7f8438c585@intel.com>
Date: Wed, 8 May 2024 12:58:11 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v3 1/2] net: ethernet: mediatek: split tx and rx fields in
 mtk_soc_data struct
Content-Language: en-US
To: Daniel Golle <daniel@makrotopia.org>, Felix Fietkau <nbd@nbd.name>, "Sean
 Wang" <sean.wang@mediatek.com>, Mark Lee <Mark-MC.Lee@mediatek.com>, "Lorenzo
 Bianconi" <lorenzo@kernel.org>, "David S. Miller" <davem@davemloft.net>,
	"Eric Dumazet" <edumazet@google.com>, Jakub Kicinski <kuba@kernel.org>, Paolo
 Abeni <pabeni@redhat.com>, Matthias Brugger <matthias.bgg@gmail.com>,
	AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>,
	<netdev@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>, <linux-mediatek@lists.infradead.org>
References: <70a799b1f060ec2f57883e88ccb420ac0fb0abb5.1715164770.git.daniel@makrotopia.org>
From: Przemek Kitszel <przemyslaw.kitszel@intel.com>
In-Reply-To: <70a799b1f060ec2f57883e88ccb420ac0fb0abb5.1715164770.git.daniel@makrotopia.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MUR2UjN4amhrbmxZUGJTNTQ3VmJXa0pKTGhuVk5YOWExWVY2Ump3bGhmdU45?=
 =?utf-8?B?dHB6QzU1TGltRVZCYXZsd01TUGZvY0oyQkxrUzlpbUMvV0dUSDBxM0tUa2pE?=
 =?utf-8?B?S2VaYWdHOVplMC9LeEdHVXhCTVgrT1ErWDJseWxaRm5nWVY1Ry9OUCtnM3dS?=
 =?utf-8?B?a05YY2J4cSs0TDNtakpNV0dER0xaUU5sakNXcEpqR25PWjNaRUx4TGhrRVRS?=
 =?utf-8?B?TXpCZ255NXpxZWorYVl3REZacjlkc1ZadllLK2VUK3c3bVg2RXdRcFFsaHVi?=
 =?utf-8?B?MDFLWVIzQlM0OXRjVDJLL3NHWFBTOGlZaGxzeUNKV0I3MWMyMDRPejlJZUZL?=
 =?utf-8?B?MXZxaDU1MXJyWi9HZDY3cGhXMk5KMTMrWGJHOEROQWlLcU5FU3B3MWR0OEcw?=
 =?utf-8?B?MWxHK2JhTGN3SXJabXIzRTkzdytCTnpsU01qZXhyY2JXYiswb205QU8rQkUr?=
 =?utf-8?B?ZU9nOGZidHNMaTFURDB3MUtPUDBlZWVSS2pWclozT3pjRjJBT3orcnhldVJl?=
 =?utf-8?B?R0hHWFR6UzRhYXZNL0g2N3JtRGhlSTFxeUg1T3dESnVFY1pDSDl2Nm9OTDF0?=
 =?utf-8?B?NllkOVVQeDFrQzZaTGRoSXk2WGdvK1p6WnAxZG55VnovdDhTV2tRYnVqWW10?=
 =?utf-8?B?M0pmQVBseFBzUnYxRWVrdlpTU2FIN0Y2U0ZCWWNXUVBlUjhNL2ZYNG1TZExP?=
 =?utf-8?B?SlJJV0FLZmRXQUNGRzhrSlJnY25mZE1UbUs1d2NOSGprc2FrcXhjcDZvT0RK?=
 =?utf-8?B?bFJvYW1UVzQzZVpqNUtzS1hSRDRrZmJ1Nm9pRGxmcjRqSUEweGdHa1VnYlQ5?=
 =?utf-8?B?WUVkS3FHMkw0Rk5NQ0FuNWxaajRHaENPT1lGNE13YU82UzNQVFBrNnFFUDUv?=
 =?utf-8?B?eXM5SXRWQ3IvNkhiOWE5K3BCY3V6RjVLdTJIVTJOMXF0SDFlSzJBaDE3OERJ?=
 =?utf-8?B?bFk4RXBQQTdOWWxTVHZRanp5d1hlQkhWUVFmUkdtM3FLSmdyTndYMXVhZEQy?=
 =?utf-8?B?eVkyZC8rMGpLZjM1clduak5NSGhXdS9paHBWUFFybVpSM0hJRzc0QWxVKytR?=
 =?utf-8?B?T2hSRnczN0VJSGR0Vk50YWVDdWdNdktsdTNrRnJyMEk1RHpnQkp3RThaTFJ3?=
 =?utf-8?B?Z3NDYVBLemNDZGYzbSs4djBldm5ndFNmMk1sZHlyd0pldnBSZnlvM1A1ZW9G?=
 =?utf-8?B?dHBYWnNVdjJoRmIvVTdLaWVGcUVTTWdnVG5nZWVTV1dVaGdmNGVWN3doRUg4?=
 =?utf-8?B?ZWxaRkhIYU1FRnFtZzNIWTIvZzhXRmZZY1hwNjExbUVSS1plbEg5cWdEM3VK?=
 =?utf-8?B?WmdIanZZcUhQL3RNcElEN3NwVlVrYW11UklHY1ZjRTcwaCtSQTFEQXN3L3NI?=
 =?utf-8?B?clYzMHBvUmptSEdlbFUyOWJrVE9JdFZXSGRCRmgzZmVDY2ZSUFJ6MTNpU1dq?=
 =?utf-8?B?bnVpeXpXUzVnRm9mRWtFUExGbVdRNUxPOHlmMUNUeXhWV2RiY2dGeG1ZcEpa?=
 =?utf-8?B?d0UvcStSY0ExOVVmZTJuZ1FvRWwxKy9hbURJVGdkWkxnUmZFV0Z5QXQ3ZUhF?=
 =?utf-8?B?UnZLVDl1TkEreTNJOGN3UzYrQmFJckYycUJnR0tKaTZ5Z3NrTDl4OU56Y3JG?=
 =?utf-8?B?WUNIUUtxWEZza2dPZEx4cHlYTmliTklpc21xeUxaYWxwUHBTY3RqcVJiZGpz?=
 =?utf-8?B?Nkg4S2d2OVR5UUduTXRIMmVXZkdYcDIwa3ZPdk1yT1NKVWlPL3VVSUlPSHBr?=
 =?utf-8?B?Z0VBUVdKK3JDUjY3QkVDYnZYUGJ5WU5aL29NVlhCTE91akpaUnVTNWloOTkr?=
 =?utf-8?B?TUNpTnY1YmtGNllBNGNJNE0wcUJxMlgzWS9kYjhrSXJ1aWNnNDBNRW91bFZ2?=
 =?utf-8?B?Z3ZwSWJwMTRVMHJIMFdvdDB5RWc2cGdsZDlPZmVlQ2xlbFFIaGtobnN5ZTdQ?=
 =?utf-8?B?bmpWeldNTjVzRithclJEa1lURnpsWXBnc3JLWjd4VXQ3MEJOelZ2OHRLV01Z?=
 =?utf-8?B?MXR4ZVhSMkVXQU1KM2xHemxidlI5MmM1eGdXM01EdHVSWDJDSE1sVDhYUS9B?=
 =?utf-8?B?bU1iNDJLNGxna0NsL1JSczJxOUYrWjBSYVB5Y1FmRDRJTDl4ZXZxT2Y1TjZa?=
 =?utf-8?B?WXpzT2lXL2pkWVNvak02Y3lWUWdsN0NKQ2lXTVhCbm1iZHg3WnhlWitVYStK?=
 =?utf-8?B?eVE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 626d38c1-b83b-46c7-d20b-08dc6f4dc384
X-MS-Exchange-CrossTenant-AuthSource: MN6PR11MB8102.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 May 2024 10:58:16.4969
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: mE0htnNnjhHe7DPpP+OGxQAQucS3tdVdVz4pZ34O9jSEGZr2OtK0R+Yk0rRCfi/L00DNlLJSi2LcGWB9I53Oqp26Qg8SlfqartrU8dfXKBA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR11MB7345
X-OriginatorOrg: intel.com

On 5/8/24 12:43, Daniel Golle wrote:
> From: Lorenzo Bianconi <lorenzo@kernel.org>
> 
> Split tx and rx fields in mtk_soc_data struct. This is a preliminary
> patch to roll back to ADMAv1 for MT7986 and MT7981 SoC in order to fix a
> hw hang if the device receives a corrupted packet when using ADMAv2.0.
> 
> Fixes: 197c9e9b17b1 ("net: ethernet: mtk_eth_soc: introduce support for mt7986 chipset")
> Signed-off-by: Lorenzo Bianconi <lorenzo@kernel.org>
> Signed-off-by: Daniel Golle <daniel@makrotopia.org>
> ---
> v2: improve commit message
> v3: resend because messages were accidentally sent inline
> 
>   drivers/net/ethernet/mediatek/mtk_eth_soc.c | 210 ++++++++++++--------
>   drivers/net/ethernet/mediatek/mtk_eth_soc.h |  29 +--
>   2 files changed, 139 insertions(+), 100 deletions(-)
> 

next time please collect provided tags
for this series:
Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>