18 Oct
2022
18 Oct
'22
6:15 p.m.
On 2022-10-17 10:49 PM, Pierre-Louis Bossart wrote:
All Intel drivers for cAVS platforms contain a sequence for S3 resume which doesn't seem justified nor necessary. Forensic Git investigation in internal repositories did not provide any rationale for the implementation, and tests show no impact when those sequences are removed.
This sequence was identified as problematic during a large HDaudio cleanup where all programming sequences were revisited before extensions are added.
This is a good finding as discussed earlier in the review. Tested-by is already there by the series-wide reviewed-by won't hurt:
Reviewed-by: Cezary Rojewski cezary.rojewski@intel.com