Attention is currently required from: Kyösti Mälkki, Patrick Georgi, Patrick Rudolph.

3 comments:

Patchset:
- Patch Set #4:
  Thanks for the patch!
  Jenkins fail looks temporary, should go away next time you rebase.
File ich_descriptors.c:
- Patch Set #4, Line 1024:
```
if (content->ISL <= 80)
			return CHIPSET_C620_SERIES_LEWISBURG
```
  I am a bit confused, what happens with support for LEWISBURG ? Why are you just removing it?
  I remember there were lots of debates two years ago, I tried to follow the chain of patches but there are lots of abandoned ones. I can understand this line might still need a fix. But it would be in a separate commit anyway?
  And if it's possible to fix, would be great to fix, instead of removing ?
- Patch Set #4, Line 490: to be compatible with 500 Series PCH below
  Maybe "to align with the SPI programming guide, which also conveniently makes it compatible with 500 Series PCH below" ?
  If SPI guide mentions 6 masters, you would keep it at 6 ?

To view, visit change 78186. To unsubscribe, or for help writing mail filters, visit settings.