Change in coreboot[master]: ec/starlabs: Add standardised ITE EC support

[Andy Pont (Code Review)]

Attention is currently required from: Sean Rhodes, Paul Menzel, Angel Pons, Felix Held.
Andy Pont has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58343 )

Change subject: ec/starlabs: Add standardised ITE EC support
......................................................................

Patch Set 44:

(2 comments)

File src/ec/starlabs/merlin/ec.c:

https://review.coreboot.org/c/coreboot/+/58343/comment/b2b36b61_ce432f30 
PS44, Line 14: u16 it_get_version(void)
This file seems to have developed a mismatched use of type definitions i.e we have u16 and uint8_t.  Need to stick with one or the other!

https://review.coreboot.org/c/coreboot/+/58343/comment/651692ad_2c123fe4 
PS44, Line 47: 	static u16 ite_get_chip_id(unsigned int port)
             : 	{
             : 		return (pnp_read_index(port, ITE_CHIPID1) << 8) |
             : 			pnp_read_index(port, ITE_CHIPID2);
             : 	}
             : 
This probably needs moving up to the top next to the it_get_version() function.