<html><body><p>That process is no longer ran. All scu setup is done in kernel. We should probably remove that whole .c file from repository.<br><br><br>> On Mar 9, 2016, at 9:32 AM, OpenBMC Patches <openbmc-patches@stwcx.xyz> wrote:<br>> <br>> From: Adi Gangidi <adi.gangidi@rackspace.com><br>> <br>> Before This change:<br>> a) SCU 88 was double assigned, first to 0x01C000FF in line:97 and then to 0x01C0007F in line:100.<br>> b) Which meant (in both cases, some or all of) bits 7:0 were set to 1. That is: We were reading : PWMx or VPIGx instead of GPIONx (GPIONx gives us the PCIe inventory status, where x is bit number)<br>> <br>> After This Change:<br>> a) Got rid of the double assignment by commenting out the second SCU88 assignment<br>> b) Bits (7:0) of SCU 88 are set to 0 . (According to Page 111 of data sheet these have to be set to 0 for us to to read GPION0 to GPIO N7 which indicate if PCIe device is present )<br>> <br>> Description of pins 7:0 of SCU 88:<br>> 7 RW Enable PWM7 or VPIG7 function pin (SCU90[5:4]=0x2 select Video pin)<br>> 6 RW Enable PWM6 or VPIG6 function pin (SCU90[5:4]=0x2 select Video pin)<br>> 5 RW Enable PWM5 or VPIG5 function pin (SCU90[5:4]!=0 select Video pin)<br>> 4 RW Enable PWM4 or VPIG4 function pin (SCU90[5:4]!=0 select Video pin)<br>> 3 RW Enable PWM3 or VPIG3 function pin (SCU90[5:4]!=0 select Video pin)<br>> 2 RW Enable PWM2 or VPIG2 function pin (SCU90[5:4]!=0 select Video pin)<br>> 1 RW Enable PWM1 or VPIG1 function pin (SCU90[5:4]=0x3 select Video pin)<br>> 0 RW Enable PWM0 or VPIG0 function pin (SCU90[5:4]=0x3 select Video pin)<br>> ---<br>> objects/control_bmc_barreleye.c | 4 ++--<br>> 1 file changed, 2 insertions(+), 2 deletions(-)<br>> <br>> diff --git a/objects/control_bmc_barreleye.c b/objects/control_bmc_barreleye.c<br>> index ddfde88..0afc58d 100644<br>> --- a/objects/control_bmc_barreleye.c<br>> +++ b/objects/control_bmc_barreleye.c<br>> @@ -94,10 +94,10 @@ void reg_init()<br>> bmcreg = memmap(mem_fd,GPIO_BASE);<br>> devmem(bmcreg+0x84,0x00fff0c0); //Enable UART1<br>> devmem(bmcreg+0x80,0xCB000000);<br>> - devmem(bmcreg+0x88,0x01C000FF);<br>> + devmem(bmcreg+0x88,0x01C00000); //Changed Assigned value to SCU 88 for PCie inventory to work<br>> devmem(bmcreg+0x8c,0xC1C000FF);<br>> devmem(bmcreg+0x90,0x003FA009);<br>> - devmem(bmcreg+0x88,0x01C0007F);<br>> + // devmem(bmcreg+0x88,0x01C0007F); // Commented out to prevent dual assignment of SCU88<br>> <br>> <br>> bmcreg = memmap(mem_fd,COM_BASE);<br>> -- <br>> 2.7.1<br>> <br>> <br>> _______________________________________________<br>> openbmc mailing list<br>> openbmc@lists.ozlabs.org<br>> <a href="https://lists.ozlabs.org/listinfo/openbmc">https://lists.ozlabs.org/listinfo/openbmc</a><br><BR>
</body></html>