[coreboot-gerrit] Change in coreboot[master]: mainboard/volteer: Update Aux settings for Port 0

[Brandon Breitenstein (Code Review)]

Brandon Breitenstein has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40245 )

Change subject: mainboard/volteer: Update Aux settings for Port 0
......................................................................

Patch Set 6:
> Patch Set 6:
> > Patch Set 6:
> > > Patch Set 6:
> > >
> > > Hi Brandon, Where did you find definition of NF6 for GPP_E10 and GPP_E13?  I don't see an NF6 defined for either of those GPIOs in the "607038-intel-500-chipset-pch-gpio-implementsummary-rev1p2.xlsx" document.  Does GPP_E22 and GPP_E23 also have an NF6 that serves this same purpose?
> >
> > It is specified here that any GPIO that is used for AUX has to meet certain criteria and be set to NF6 https://docs.google.com/document/d/1nhWN1rzMFXBOb6XVSvQeNwPanKTq_Iai_kLdKfFi...
>
> I can't view that document.  I requested permission yesterday, but it hasn't been granted yet.

I pinged Aline to grant the permission. I assume you are inquiring here for Port 1 with Sku 1? I'm working on that implementation as well currently the same settings but with GPIOS E22 and E23 isn't showing the same behavior I'm going to do some more debug on that today and tomorrow