[coreboot-gerrit] Change in ...coreboot[master]: drivers/smmstore: Fix some issues

[Arthur Heymans (Code Review)]

Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30432 )

Change subject: drivers/smmstore: Fix some issues
......................................................................

Patch Set 8:

(2 comments)

https://review.coreboot.org/#/c/30432/6/src/drivers/smmstore/store.c 
File src/drivers/smmstore/store.c:

https://review.coreboot.org/#/c/30432/6/src/drivers/smmstore/store.c@187 
PS6, Line 187: 	if (scan_end(&end) == CB_ERR)
> We're doing lookup_store() inside of scan_end(). […]

We don't seem to have functions to return cbfsfiles as writable region?

https://review.coreboot.org/#/c/30432/8/src/drivers/smmstore/store.c 
File src/drivers/smmstore/store.c:

https://review.coreboot.org/#/c/30432/8/src/drivers/smmstore/store.c@94 
PS8, Line 94: -1
> CB_ERR (here, and rest of file?)

The interface is defined above and the function calling it also uses plain numbers.