[coreboot-gerrit] [S] Change in coreboot[main]: vendorcode/google/chromeos: Add API to read factory config

[Tim Van Patten (Code Review)]

Attention is currently required from: Subrata Banik.

Tim Van Patten has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79737?usp=email )

Change subject: vendorcode/google/chromeos: Add API to read factory config
......................................................................

Patch Set 8:

(1 comment)

File src/vendorcode/google/chromeos/tpm_factory_config.c:

https://review.coreboot.org/c/coreboot/+/79737/comment/4c5a07ad_5a1f5d31 :
PS8, Line 7: int64_t
> > I don't think it's a good idea to make this signed everywhere, it's not a signed number, it's a bi […]

Alternatively, follow the same convention as `tlcl_cr50_get_factory_config()` and return a result enum value while taking a `uint64_t *` whose underlying value is updated by `tlcl_cr50_get_factory_config()`.

```
tpm_result_t tlcl_cr50_get_factory_config(uint64_t *factory_config);
```

Plenty of ways to skin the cat.