Werner Zeh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41280 )
Change subject: security/tpm/tspi: Fix handling of white space delimited list ......................................................................
Patch Set 9: Code-Review+2
Patch Set 9:
Patch Set 9: Code-Review+1
Patch Set 9: Code-Review+2
So now that there is a patch for strtok in the tree should we move forward to strtok()? Or would you lik eto keep this for now and provide a follow-up patch once strtok() is merged?
Oh, BTW: now that you have removed the Kconfig change form this patch you should stick with the comma as separator in order to stay in line with the help text. You can than switch back to a whitespace in the Kconfig patch.
I added Kconfig back in patch set 9. Since the previous implementation never worked even for comma-separated, do you think it is required to make changes in two separate patches?
Ups, I had my view on Patchset 8, sorry. All clear than, let's get this patch in.