[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <9cc733b9-005d-4587-bc08-971f39334663@lunn.ch>
Date: Wed, 8 May 2024 13:57:12 +0200
From: Andrew Lunn <andrew@...n.ch>
To: Rengarajan.S@...rochip.com
Cc: Bryan.Whitehead@...rochip.com, davem@...emloft.net,
linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
pabeni@...hat.com, richardcochran@...il.com, edumazet@...gle.com,
UNGLinuxDriver@...rochip.com, kuba@...nel.org
Subject: Re: [PATCH net-next v1] net: microchip: lan743x: Reduce PTP timeout
on HW failure
> Hi Andrew, based on the customer experience they felt that there might
> be cases where the 20-sec delay can cause the issue(reporting the HW to
> be dead). For boards with defects/failure on few occasions it was found
> that resetting the chip can lead to successful resolution; however,
> since we need to wait for 20 sec for chip reset, we found that reducing
> the timeout to 1 sec would be optimal.
O.K. This should of been part of the commit message, since with this
comment the change becomes meaningful. Please try to ensure the commit
message explains "Why?"
Andrew
Powered by blists - more mailing lists