Attention is currently required from: Subrata Banik, Nico Huber, Paul Menzel, Edward O'Callaghan, Angel Pons, Anastasia Klimchuk.
3 comments:
Commit Message:
Here is the timeout calculation for any hardware sequencing operation:
Worst Case Operational Delay =
(Maximum Time consumed by a SPI operation + Any marginal
adjustment)
Timeout Recommendation for Hardware Sequencing Operation =
((Worst Case Operational Delay) * (#No. Of SPI Master - 1) +
Current Operational latency)
Assume, on Intel platform with 2 SPI master like, Host CPU and CSE, the
Timeout Calculation for SPI Write Operation would look like as below:
Maximum Time consumed by a SPI Operation = 2 seconds
(for SPI Erase Operation as per Winbond Data Sheet)
Worst Case Operational Delay = (2 seconds + 50 milliseconds) =
2 seconds 50 milliseconds
Timeout Recommendation for Hardware Seq Operation =
(2 seconds 50 milliseconds) * (2 - 1) + 15 milliseconds
(for SPI Write Operation as per Winbond Data Sheet)
= 2 seconds 65 milliseconds
Doing the calculation with these numbers can be a bit confusing. I'd
do it with the estimated worst-case numbers right away (i.e. 6 masters,
5s).Shorting some things might make it more readable, e.g. `2.05s` instead
of `2 seconds 50 milliseconds`, also shorter identifiers and removing
unnecessary parentheses can help legibility.
ACK
File ichspi.c:
Patch Set #12, Line 1281: operation
`operations`? because multiple masters would preform multiple operations?
ACK
Patch Set #12, Line 1282: introduces
`introduce` without s, I guess
ACK
To view, visit change 62867. To unsubscribe, or for help writing mail filters, visit settings.