[coreboot-gerrit] Change in ...coreboot[master]: superio/ite/it8613e: add support for ITE IT8613E

[Krystian Hebel (Code Review)]

Krystian Hebel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31617 )

Change subject: superio/ite/it8613e: add support for ITE IT8613E
......................................................................

Patch Set 8:

(1 comment)

https://review.coreboot.org/#/c/31617/4/src/superio/ite/it8613e/superio.c 
File src/superio/ite/it8613e/superio.c:

https://review.coreboot.org/#/c/31617/4/src/superio/ite/it8613e/superio.c@74 
PS4, Line 74: 0x0fff
> I checked the IT8772E datasheet and there the simple I/O BAR is 8 bytes long: […]

OK, I think I misunderstood how these values are used in coreboot.
I was looking at IT8783 code and specification [1]. Does it mean that there is error in code for IT8783?

1: http://www.ite.com.tw/uploads/product_download/IT8783_A_V0.5_062510.pdf