Patch set 8:Code-Review -1
5 comments:
Patch Set #8, Line 23: /* Enable function 5 (PS/2 AUX) */
looks like superio stuff and should be moved to src/superio/51nb/
Is it a common superio, like ITE?
Patch Set #8, Line 43: 0x00200000
is that the offset of the IFD ec region?
File src/ec/51nb/Makefile.inc:
Patch Set #8, Line 25: $(51NB_EC_INSERT) $(obj)/coreboot.pre \
is the IFD ec region in use?
Patch Set #8, Line 20: #define SETUP_COMMAND 0x4e
probably mainboard specific
File src/mainboard/51nb/x210/hda_verb.c:
no need for this file. use soc/intel/common/block/hda/hda.c instead
To view, visit change 32531. To unsubscribe, or for help writing mail filters, visit settings.