Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36130 )
Change subject: soc/intel/broadwell: Fix 'dead increment' ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/36130/1/src/soc/intel/broadwell/sat... File src/soc/intel/broadwell/sata.c:
https://review.coreboot.org/c/coreboot/+/36130/1/src/soc/intel/broadwell/sat... PS1, Line 78: reg16 = 0; /* Disable alternate ID */ : reg16 = 1 << 5; /* BWG step 12 */
I suppose what scan-build is really complaining about is that the variable is first set to 0, then t […]
The documentation I have states that the register's default value is zero. It also states that this register is 32 bit wide.
In any case, this "Disable alternate ID" line is because bits 7 and 6 allow changing the PCI ID of the SATA controller when in RAID mode.