[coreboot-gerrit] [M] Change in coreboot[master]: cpu/intel/sandybridge: Add `hyper_threading` option

[Felix Singer (Code Review)]

Attention is currently required from: Nico Huber.

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29669 )

Change subject: cpu/intel/sandybridge: Add `hyper_threading` option
......................................................................

Patch Set 18:

(1 comment)

File src/cpu/intel/model_206ax/bootblock.c:

https://review.coreboot.org/c/coreboot/+/29669/comment/2035e550_f989082f 
PS18, Line 87: 	if (need_reset) {
How about inverting the condition so that the trivial case comes first? This way we also save the indentations.

```
if (!need_reset) {
  RCBA32_OR(SOFT_RESET_LOCK, 1);
  return;
}

/* Set current requests */
RCBA32(SOFT_RESET_DATA) = soft_reset;
....
```