[coreboot-gerrit] Change in coreboot[master]: sb/intel/bd82x6x/lpc: Setup default LPC decode ranges

[Felix Held (Code Review)]

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

Change subject: sb/intel/bd82x6x/lpc: Setup default LPC decode ranges
......................................................................

Patch Set 5: Code-Review+2

(2 comments)

https://review.coreboot.org/c/coreboot/+/33549/4/src/mainboard/roda/rv11/var... 
File src/mainboard/roda/rv11/variants/rw11/romstage.c:

https://review.coreboot.org/c/coreboot/+/33549/4/src/mainboard/roda/rv11/var... 
PS4, Line 35: COMB_LPC_EN
> > if we always enable com B, this could also be dropped […]

Done

https://review.coreboot.org/c/coreboot/+/33549/4/src/southbridge/intel/bd82x... 
File src/southbridge/intel/bd82x6x/early_pch.c:

https://review.coreboot.org/c/coreboot/+/33549/4/src/southbridge/intel/bd82x... 
PS4, Line 264: 		| KBC_LPC_EN | MC_LPC_EN;
> > | LPT_LPC_EN and | COMB_LPC_EN ? […]

Done