[LinuxBIOS] compiler error isolated

Doug Bell nxtv3 at yahoo.com
Wed Sep 28 01:02:29 CEST 2005


The attached revised spd.c compiles as is. When you
have :

subtotal=dev+0;  // it works 
subtotal=dev;    // it fails ( internal compiler error
- rhs not used ) 

Amazing but true. What's going on here? 

--- Doug Bell <nxtv3 at yahoo.com> wrote:

> The attached spd.c contains functions called from my
> raminit.c's sdram_set_spd_registers(). Raminit.c is
> #included into auto.c. I've tried various
> workarounds
> including putting all the code in spd.c under one
> function, getting rid of the device_t ctl, etc. but
> compilation doesn't yet work. Ideas appreciated.
> 
> Thanks, dB
> 
> --- Doug Bell <nxtv3 at yahoo.com> wrote:
> 
> > Beyond that, there was one version in which I
> simply
> > switched the order of declaration of variables to
> > make
> > it compile:
> > 
> > // unsigned char row,col; // broken
> > unsigned char col,row; // works
> > 
> > In another test run........
> > //if ( total < c) total+=div; // failed
> > 
> > if ( total < c) subtotal+=div; // was ok
> > total=subtotal;
> > 
> > Ron / Eric  - What is the best way to proceed on
> > this?
> > What info would you want? I don't think I'm doing
> > anything bizarre, maybe 4 chars declared, no deep
> > levels of functions. 
> > 
> > --- Doug Bell <nxtv3 at yahoo.com> wrote:
> > 
> > > I had one version of code where it failed on a
> > line
> > > like:
> > > if (speed==100) speed=0;
> > > 
> > > but problem disappeared if I did :
> > > if (speed==100) speed2=0;
> > > 
> > > This may have been a factor in the first
> reporting
> > > of
> > > the problem as well.
> > > 
> > > -dB
> > > 
> > > --- Doug Bell <nxtv3 at yahoo.com> wrote:
> > > 
> > > > It seems to not be related to register
> > allocation,
> > > > how
> > > > many layers of functions,  or code size, but
> to
> > > > nested
> > > > if statements or variations thereof, e.g. if (
> > > > (condition a) && ( condition b) ) ....
> > > > 
> > > > 
> > > > /home/dbell # uname -a
> > > > Linux Doug 2.6.9 #1 Tue Jan 11 17:26:41 UTC
> 2005
> > > > i686
> > > > Intel(R) Pentium(R) 4 CPU 1500MHz GenuineIntel
> > > > GNU/Linux
> > > > 
> > > > 
> > > > Target is via epia-sp, under development but
> > close
> > > > to
> > > > an alpha release with some open items.
> > > > 
> > > > BTW, is there any documentation regarding what
> > > > limitations romcc has (e.g. multidimensional
> > > arrays
> > > > not allowed, etc )?   
> > > > 
> > > > Thanks, Doug
> > > > 
> > > > --- ron minnich <rminnich at gmail.com> wrote:
> > > > 
> > > > > On 9/26/05, Doug Bell <nxtv3 at yahoo.com>
> wrote:
> > > > > >
> > > > > > 0x8360d58 phi Internal compiler error:
> > Cannot
> > > > > > find block dominated by 0x82ba9a8
> > > > > >
> > > > > > Any ideas on workarounds / fixes here? Is
> am
> > I
> > > > > running
> > > > > > out of registers or some other problem?
> > > > > 
> > > > > 
> > > > > can you tell us more about the environment,
> > > > > uname -a
> > > > > distro, and linuxbios target?
> > > > > 
> > > > > ron
> > > > > > -- 
> > > > > LinuxBIOS mailing list
> > > > > LinuxBIOS at openbios.org
> > > > >
> > >
> http://www.openbios.org/mailman/listinfo/linuxbios
> > > > 
> > > > 
> > > > 
> > > > 		
> > > > __________________________________ 
> > > > Yahoo! Mail - PC Magazine Editors' Choice 2005
> 
> > > > http://mail.yahoo.com
> > > > 
> > > > -- 
> > > > LinuxBIOS mailing list
> > > > LinuxBIOS at openbios.org
> > > >
> > http://www.openbios.org/mailman/listinfo/linuxbios
> > > > 
> > > 
> > > 
> > > 
> > > 		
> > > __________________________________ 
> > > Yahoo! Mail - PC Magazine Editors' Choice 2005 
> > > http://mail.yahoo.com
> > > 
> > > -- 
> > > LinuxBIOS mailing list
> > > LinuxBIOS at openbios.org
> > >
> http://www.openbios.org/mailman/listinfo/linuxbios
> > > 
> > 
> > 
> > 
> > 		
> > __________________________________ 
> > Yahoo! Mail - PC Magazine Editors' Choice 2005 
> > http://mail.yahoo.com
> > 
> 
> __________________________________________________
> Do You Yahoo!?
> Tired of spam?  Yahoo! Mail has the best spam
> protection around 
> http://mail.yahoo.com > 
> 
> 
> //ALL SPD STUFF IS IN PROGRESS
> 
> #include <device/pci_ids.h>
> 
> void spd_dram_presence( void ){}
> void spd_dram_size( void ) {}
> void spd_dram_other( void ) {}
> //unsigned char spd_trp( void ) { 
> //	return 0;
> //}
> 
> 
> 
> unsigned char spd_dram_timings( void ) { 	// set
> dram bus speed based off CPU's FSB and SPD byte 9
> 
> 	
> 	print_info("sdram_spd_dram_timings..........\r\n");
> 
> 	device_t ctl = (device_t) 0;
> 	unsigned char c;
> 	unsigned char col,row; 
> 	unsigned char freq;
> 	unsigned char reg;
> 	unsigned char i;
> 
> 	// find CPU controlller - the CN400 device 0
> function 3 so use virtual device function 2
> 	pci_write_config8(0,0x4f,1); // but first allow it
> to be seen using multiple function enable
> 
> 	ctl = pci_locate_device(
> 		PCI_ID(
> PCI_VENDOR_ID_VIA,PCI_DEVICE_ID_VIA_CN400_2 ),
> 		0 );
> 	c=pci_read_config8(ctl,0x54); 
> 
> 	print_debug_hex8(c);
> 	print_debug(" is CPU FSB\r\n");
> 
> 	unsigned char fsb;
> 	fsb =  c >> 5; 
> 
> 	print_debug_hex8(fsb);
> 	print_debug(" = fsb \r\n");
> 	
> 	c=smbus_read_byte(SMBUS_MEM_DEVICE_START, 9); //
> one DIMM
> 	print_debug_hex8(c);
> 	print_debug(" returned as spd data for register
> 9\r\n");
> 
> 	// decide supported dram speed per PG
> 	unsigned char speed;
> 	if ( c <= 0x50 ) {
> 		speed = 200;
> 	}
> 	else if ( c <= 0x60 ) {
> 		speed = 166;
> 	}
> 	else if ( c <= 0x75 ) {
> 		speed = 133;
> 	}
> 	else {
> 		speed = 100;
> 	}
> 	print_debug_hex8(speed);
> 	print_debug(" = speed\r\n");  
> 
> 	//revisit
> 	static const unsigned char timing[5][5] = { 
> 
> 			//dram	100	133	166	200	266
> 
> 				{ 0xff,	1,	5,	9,	0xff},   	// row 0 cpu=100
> 				{ 0xff,	0,	1,	5,	0xff}, 	// row 1 cpu=133
> 				{ 0xff,	2,	0,	1,	0xff}, 	// row 2 cpu=166
> 				{ 0xff,	6,	2,	0,	0xff},	// row 3 cpu=200
> 				{ 0xff,	0x0a,	6,	2,	0xff}, 	// row 4 cpu=266
> 
> 						}; // on datasheet there seems to be two
> values for cpu=266 dram=133
> 	// also see p. 30 of PG, looks like 0x0a is right
> 	// note no 100 or 266 dram speeds
> 
> 	// important - revise fsb and speed for correct
> indexing into table
> 	if ( fsb == 0 ) 	row=0;  // 100 
> 	else if ( fsb == 1 ) 	row=1;  // 133
> 	else if ( fsb == 3 ) 	row=2;  // 166 note value 
> 	else if ( fsb == 2 ) 	row=3;  // 200 note value 
> 	else if ( fsb == 4 ) 	row=4;  // 266
> 	else {
> 		print_debug(" bad fsb setting\r\n");
> 		return -1;
> 	}
> 	if ( speed == 100 ) 		col=0;  // 100 
> 	else if ( speed == 133 ) 	col=1;  // 133
> 	else if ( speed == 166 ) 	col=2;  // 166
> 	else if ( speed == 200 ) 	col=3;  // 200
> 	else if ( speed == 266 ) 	col=4;  // 266
> 	else {
> 		print_debug(" bad speed setting\r\n");
> 		return -2;
> 	}
> 
> 	freq=timing[row][col];
> 	print_debug_hex8(freq);  // row / col
> 	print_debug("  = timing table value \r\n");  
> 	
> 	if ( freq==0xff ) {
> 		print_debug(" bad timing value\r\n");
> 		return -3;
> 	}
> 
> 	ctl = pci_locate_device(
> 		PCI_ID(
> PCI_VENDOR_ID_VIA,PCI_DEVICE_ID_VIA_CN400_3 ),
> 		0 );
> 	c=pci_read_config8(ctl,0x68); 
> 	print_debug_hex8(c);
> 	print_debug(" = r68\r\n");
> 	c &=0xf0;
> 	c|=freq;
> 	pci_write_config8(ctl,0x68,c); 
> 	print_debug_hex8(c);
> 	print_debug(" = r68 written \r\n");
> 
> 	/***************** BANK INTERLEAVE ***************/
> 
> 	c=smbus_read_byte(SMBUS_MEM_DEVICE_START, 17); 
> 	if ( ! (c==2 || c==4 ) ) {
> 		print_info("unexpected reg 17 value\r\n");
> 		return -4;
> 	}
> 		
> 	reg=pci_read_config8(ctl,0x69);
> 	reg &=0x3f;
> 	reg |= (c<<6);
> 	pci_write_config8(ctl,0x69,reg);
> 	print_debug_hex8(reg);
> 	print_debug(" = r69 written \r\n");
> 
> 
>
/*********************************************************/
> 
> 
> 
> 	print_info("sdram_set_spd_registers done\r\n");
> 	return 0;
> }	
> 
> 
> 
> 
> unsigned char spd_trp( void ) { 
> 	
> 
> // GRAB A BOGUS NON-CONST VALUE FOR COMPILATION TEST
> ONLY!!!
> 	unsigned char
> speed=smbus_read_byte(SMBUS_MEM_DEVICE_START, 9); 
> 	
> 	print_info("spd_trp.........\r\n");
> 
> 	device_t ctl = (device_t) 0;
> 	
> 	
> 	unsigned char div=0;
> 	unsigned char i;
> 	unsigned char total,subtotal;
> 	unsigned char c;
> 
> 	// find CPU controlller - the CN400 device 0
> function 3 so use virtual device function 2
> 	pci_write_config8(0,0x4f,1); // but first allow it
> to be seen using multiple function enable
> 
> 	ctl = pci_locate_device(
> 		PCI_ID(
> PCI_VENDOR_ID_VIA,PCI_DEVICE_ID_VIA_CN400_3 ),
> 		0 );
> 
> 	/*************** tRP
> **********************************/
> 
> 	if ( speed == 100 ) 		div=0x28;  // divisor
> 	else if ( speed == 133 ) 	div=0x1e;  
> 	else if ( speed == 166 ) 	div=0x18;  
> 	else if ( speed == 200 ) 	div=0x14;  
> 	else {
> 
> //		print_debug(" bad tRP speed setting\r\n");
> 		return -5;
> 	}
> 
> 
> 
> 
=== message truncated ===> -- 
> LinuxBIOS mailing list
> LinuxBIOS at openbios.org
> http://www.openbios.org/mailman/listinfo/linuxbios

__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: spd.c
Type: text/x-csrc
Size: 5623 bytes
Desc: 2307221761-spd.c
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20050927/d5d3264e/attachment.c>


More information about the coreboot mailing list