[coreboot-gerrit] Change in ...coreboot[master]: superio/ite: Add it8528e

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30957 )

Change subject: superio/ite: Add it8528e
......................................................................

Patch Set 5:

(4 comments)

https://review.coreboot.org/#/c/30957/4//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/30957/4//COMMIT_MSG@7 
PS4, Line 7: and use
> Can be removed?

Done

https://review.coreboot.org/#/c/30957/4//COMMIT_MSG@9 
PS4, Line 9: * Add SuperIO IT8528E
> Could you extend the message with the discussion in the comments, that it’s actually an EC?

Done

https://review.coreboot.org/#/c/30957/4//COMMIT_MSG@10 
PS4, Line 10:
> Could you please also add the datasheet name/URL?

Done

https://review.coreboot.org/#/c/30957/4//COMMIT_MSG@13 
PS4, Line 13: The serial works without CONFIG_CONSOLE_SERIAL.
> Is that good or bad? Or do you mean, that it works under the OS?

Done