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:
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
Is there a bug for it? I came up with an implementation, but didn't realize someone else was working on it. No need for both of us to be implementing it, so I will step back and let you finish driving this one unless you'd prefer me to drive this. I'd still like access to that document, however.
No bug on this as we don't have many SKU 1 and are treating it as a bring up item for SKU 1. The main issue I see with this is we need to be able to dynamically set this or unset it based on SKU and I'm not sure I see the path forward there.