[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cnl: Enable VT-d

[Lijian Zhao (Code Review)]

Lijian Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32432 )

Change subject: soc/intel/cnl: Enable VT-d
......................................................................

Patch Set 3:
> Patch Set 3:
>
> (1 comment)
> > Enable by default will be okay? Or we can still disable that by default and leave mainboard to decide that's enabled or disabled?
>
> I would prefer to enable it and not to have an option
> at all until somebody asks for an option to disable it.
> Or does anybody already want to disable it?
>
> IMHO, the `VtdDisable` in `chip.h` should be (re)moved
> anyway: It's not a mainboard question but a question of
> the coreboot build and, thus, belongs into Kconfig.

Not I am aware of, just want to be more conservative here. But if no one is against it, I am okay to cast my vote.