Michael Niewöhner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36622 )
Change subject: drivers/fsp2_0: drop support for FSP-T ......................................................................
Patch Set 4:
Patch Set 4:
Patch Set 4:
Patch Set 4:
Patch Set 4:
Patch Set 4:
Patch Set 4: Code-Review-1
FSP-T already set as optional. Some of our customers are using FSP-T, it has to be there.
Could you please state why? What does FSP-T do that coreboot cannot?
I understand the concern Arthur.
There are some customer who still like to make use of FSP-T due to various reason.
- native CAR implementation majorly focusing on NEM enhance
- Some customers like to use FSP-T with intel boot guard solution, where else native CAR what we implemented is not applicable with bootguard
Can you give more details on that NEM part, please?
here you go https://software.intel.com/en-us/articles/introduction-to-cache-allocation-t...
On 2: is there any chance, Intel can release more details here so we can (maybe) add support for it?
So this is supported by fsp-t but not cb native, yet? Is it just writing some registers/msrs or is more needed to get CAT working?
-
Michael Niewöhner (Code Review)