Received-SPF: pass (google.com: domain of linux-kernel+bounces-30839-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:40f1:3f00::1 as permitted sender) client-ip=2604:1380:40f1:3f00::1;
Message-ID:
 <AS8P193MB1285B34B71F3143FA9B0A053E4702@AS8P193MB1285.EURP193.PROD.OUTLOOK.COM>
Date: Fri, 19 Jan 2024 08:15:31 +0100
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2] net: stmmac: Wait a bit for the reset to take effect
To: Jose Abreu <Jose.Abreu@synopsys.com>, Andrew Lunn <andrew@lunn.ch>
Cc: Alexandre Torgue <alexandre.torgue@foss.st.com>,
 "David S. Miller" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>,
 Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
 Maxime Coquelin <mcoquelin.stm32@gmail.com>,
 "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
 "linux-stm32@st-md-mailman.stormreply.com"
 <linux-stm32@st-md-mailman.stormreply.com>,
 "linux-arm-kernel@lists.infradead.org"
 <linux-arm-kernel@lists.infradead.org>,
 "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
 Jiri Pirko <jiri@nvidia.com>, Andrew Morton <akpm@linux-foundation.org>
References: <AS8P193MB1285DECD77863E02EF45828BE4A1A@AS8P193MB1285.EURP193.PROD.OUTLOOK.COM>
 <AS8P193MB1285EEAFE30C0DE7B201D33CE46C2@AS8P193MB1285.EURP193.PROD.OUTLOOK.COM>
 <6e33c038-45a7-4941-b5d9-ce5704e13d48@lunn.ch>
 <AS8P193MB128591BBF397DC664D7D860EE4722@AS8P193MB1285.EURP193.PROD.OUTLOOK.COM>
 <DM4PR12MB50883D41B18E8627FBDF5E32D3722@DM4PR12MB5088.namprd12.prod.outlook.com>
Content-Language: en-US
From: Bernd Edlinger <bernd.edlinger@hotmail.de>
In-Reply-To: <DM4PR12MB50883D41B18E8627FBDF5E32D3722@DM4PR12MB5088.namprd12.prod.outlook.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: AS8P193MB1285:EE_|PAXP193MB1438:EE_
X-MS-Office365-Filtering-Correlation-Id: de6019a3-c598-4282-c1ab-08dc18be496f
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	pzaqQdZil9BdMK/aDQI1zzeb2wLDSo4/KKsIg7pOX46pqonlQc4sr1cvHbJ9PE1omsuqEaWS611KE1GXYH8Fe7/0aGwm0qhh9KZLaYv4NihMgXeP/YxBpX9KWAQSTxU2/MRzn75e57dPeW1/rtqSFk8reaE5DBYtlOQ23SQ91U7mtBEOOv6IqDnAd0itAVIjYtYMsH7S7V08Srpgp6k/sLf3oLPOSH9wC0T9jAHdOpxja4RNHG3RBfSWHzEeEss+NNxu4zwG6wZJmNscbw6sCz8pH6qmcWLXyB8dF1aFXiQqDXo7bkPZIGV/n7a6TWrVszb73p5ueQ89gjqxV/bE7PRZQUGn4VXC45WichlvHRabqOAMRulvgUq8/rsO7wedjnbTdSqJq1svHrCjkhvvhb0y41qq732nb3iEO59sENRYF7cqdkJXv8Sa/8VZjugPRUxudBb9OBaY1651Uyau5qKiXVBOUlPPZR7+HVBN1nvsQcR96uy5ibXcdnMw19lVtGxjIXqIb41wHOSaH+obMVGXXCYAhkYGGjL3vQEbt3EJq2xBieH063Msjsj4tATo
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?V0JxWVRKalBHVVNIcnFaSSt5eU4vYkczSVNKSzRibHliUWttUTVLMDBUeUt1?=
 =?utf-8?B?RFJ1SlN0a2NleVJ1RlBDQnhFNjgzUUpRdDhvM1llZUtmeXlMdkZlSkV0UUxl?=
 =?utf-8?B?K0pEQlZDVFZjUWxDTUVWcjgyd1hVKzJWNUhhMzAzak9vZnJkTVFSZEJjMGpU?=
 =?utf-8?B?N3BkRUYzVFpKaHRXTWl0QW9KbG5vNndBWHdCM1lRSHR1T0ZSNERQSUhVRmth?=
 =?utf-8?B?L1Fmc25oNklsMlN5MlkwaTVFeFc5Wm8wWEd0SGwyRDUrQS94blZ2R253QzVh?=
 =?utf-8?B?NmxuWStHVENzV2dHc3NlTXBLUTVwRlNzM25MbDU3eGFRTFRHVUpJSXpsWHdN?=
 =?utf-8?B?SWFBRnpkM2lzN2NTckpVYW5WZURNZndvN2g0ZUFSMVQ0MXRYSnlNYXZBaUEz?=
 =?utf-8?B?VkNQUEk2NDBWL0ZzWlNkcmpJaFFtN1dIRVN4bW14T1J2Y2NKQ0p1elpmNDVz?=
 =?utf-8?B?NTJuWlhRVm9HVTBuMHVqMVFXSi9lajVkc05mekpldXh3aVdHcndsbHJNejlh?=
 =?utf-8?B?Z1ZWUVRzZ1ZENXNHRDliQWNaYktLOS9SdTBxMEh5b1pzUTdPR3JUdE12aUFM?=
 =?utf-8?B?UitIWEtYQllUdXAvdFU5YUIwNGZOWGo2czNOL1pldkZCWWg3Ujk4QmVJYjRR?=
 =?utf-8?B?OHVUMmF6YUluL1lZR25KYmhTblpOLzVrV3dtOUtLT2hKTElXTk92dFp2Q1p4?=
 =?utf-8?B?cEtteXNLWDlwcW9wRmt1cXI2V2FaOHdXcE02SGdwY0RqNThrQTBRSzhLZ2Jq?=
 =?utf-8?B?cjM4QWhpWjg4WjlKdUxjeVFvZ00xc2JZSVRVQzZQT3NHVVpOQ2dkeDlIVUw3?=
 =?utf-8?B?NVNPUU9NdVZ3Ulh4SGVqelVsM2RPMldkU084OUhUbDB2ZVlCdHhodGQvQkU2?=
 =?utf-8?B?MVpqalBzS0xBMlRCUHJzNW9uSzhveEI4bEwxakVHSjRUc2R3cjUrUjFjR1RJ?=
 =?utf-8?B?MVp6MlUvY2lXWTR4ais1cWRUTmJFTnhCNVlZdkdhUE9IU2FlbTBmQVU3TTBW?=
 =?utf-8?B?c09xVFdEV2VITFdRVFlPUmlWMEVXeU5FNE55Ri9YYTA2aG8reFo5Tlhqbmxa?=
 =?utf-8?B?ODREUXl6Q0xTTk0rSFhUT1RhcUlxNkZ6ZzZnNGIvSzk0UXd3cmhsaC9DK2tI?=
 =?utf-8?B?SFhsYjFwaDJQY1kxbnZRNHVGaDBoM2dUaERwRW56YTB5dlVFU0pYcmxiYnV3?=
 =?utf-8?B?OWloR09IK2NKb05wZzJobWdhWGFNSTZSL0t4bGphdW1YczJRcVkzaS9KR201?=
 =?utf-8?B?bVhmcE1lY0NUZERnTmN0TFJFRkg2MEVMNGM2SnNCeDFVNTFNd0RHQ2FrNGJ2?=
 =?utf-8?B?K0MvRk1NeFJ3VGQ3WUVjN0l3TGtKZjlLUTJJcjIvNVhiQUZNSFhCeW5WcVN1?=
 =?utf-8?B?MTJxVXU5cjRNYWovU25PSmY5UWxpNGQ5akl1dk50ZWZGbHp5SnFCMm10YXBC?=
 =?utf-8?B?d2pjK2xLak9kZURUVG9ZZkNTSHBwRHNvM0VTNFVVUU92TkNzT1hCa2ZXQVBF?=
 =?utf-8?B?cy9KN3VMRVRMSXlpRThoK2d6a2pPcnI0ZkhMN3dtVlpJZU9wNW9ZemxnVTJU?=
 =?utf-8?B?a0V1Qk1BUlJ1WjBBSHdqVU1MT3FNMGQrZnUrb1ROVys5QTNZQWd6NnRDZlI5?=
 =?utf-8?B?RWFYenpBT1dMMTNxdm5hNW9veHpxajc5eEpxaEhLSmRqMVVLbS8yK2ZoYk5i?=
 =?utf-8?B?V0plNldSQ0gyNWtWZDhsUkxhWUg5dlVLRzNLckNUU2xScm9KNVUwQ3VPTitp?=
 =?utf-8?Q?q4E0JMXABU6R4kTpgk=3D?=
X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-80ceb.templateTenant
X-MS-Exchange-CrossTenant-Network-Message-Id: de6019a3-c598-4282-c1ab-08dc18be496f
X-MS-Exchange-CrossTenant-AuthSource: AS8P193MB1285.EURP193.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Jan 2024 07:14:33.8063
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa
X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg:
	00000000-0000-0000-0000-000000000000
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXP193MB1438

On 1/17/24 17:55, Jose Abreu wrote:
> From: Bernd Edlinger <bernd.edlinger@hotmail.de>
> Date: Wed, Jan 17, 2024 at 16:48:22
> 
>> I don't know at all.  And actually, I am more concerned that other registers
>> might be unreliable within the first microsecond after reset is de-asserted.
> 
> Are you guaranteeing that the documented PoR time is achieved before reading registers?
> 

Yes, that is the idea, why I added the udelay directly after releasing the reset,
thus simply delaying the execution of the stmmac_hw_init function, and not directly
where the synopsys_id register is accessed.


Thanks
Bernd.