[LinuxBIOS] sunw ultra40

Lu, Yinghai yinghai.lu at amd.com
Fri Oct 13 23:21:37 CEST 2006


Before I can released the code, I was instructed to remove these
comments by management or vendor.

Maybe we need to change enable_serial to enable_dev?

YH

-----Original Message-----
From: Ronald G Minnich [mailto:rminnich at lanl.gov] 
Sent: Friday, October 13, 2006 2:10 PM
To: Ronald G Minnich
Cc: Lu, Yinghai; LinuxBIOS
Subject: Re: [LinuxBIOS] sunw ultra40

ok, are the workings of this function a Deep Secret or can we add
comments?



static void sio_setup(void)
{

         unsigned value;
         uint32_t dword;
         uint8_t byte;


         pci_write_config32(PCI_DEV(0, CK804_DEVN_BASE+1, 0), 0xac, 
0x047f0400);

         byte = pci_read_config32(PCI_DEV(0, CK804_DEVN_BASE+1 , 0),
0x7b);
         byte |= 0x20;
         pci_write_config8(PCI_DEV(0, CK804_DEVN_BASE+1 , 0), 0x7b,
byte);

         dword = pci_read_config32(PCI_DEV(0, CK804_DEVN_BASE+1 , 0),
0xa0);
         dword |= (1<<29)|(1<<0);
         pci_write_config32(PCI_DEV(0, CK804_DEVN_BASE+1 , 0), 0xa0,
dword);

#if  1
         lpc47b397_enable_serial(SUPERIO_GPIO_DEV,
SUPERIO_GPIO_IO_BASE);

         value =  lpc47b397_gpio_offset_in(SUPERIO_GPIO_IO_BASE, 0x77);
         value &= 0xbf;
         lpc47b397_gpio_offset_out(SUPERIO_GPIO_IO_BASE, 0x77, value);
#endif

}

Also, in this function, this happens:
         lpc47b397_enable_serial(SUPERIO_GPIO_DEV,
SUPERIO_GPIO_IO_BASE);

but in the real main this happens:
	lpc47b397_enable_serial(SERIAL_DEV, TTYS0_BASE);
         uart_init();
         console_init();


So the sio_setup, to me, looks bogus. Why would enable_serial use the 
GPIO dev and BASE?

What's up here?

ron








More information about the coreboot mailing list