[U-Boot-Users] warnings, boards, RE: [PATCH]: Enable device ...

[Woodruff, Richard]

Hello Wolfgang,

Yes, I fixed all but one type of warning in my tree this weekend.  I was
somewhat embarrassed that I didn't fix the two simple ones in the patch I
sent.  I shall be more careful.

One type of warning does still exist in my build and likely others which
seems to deal with complaints about an error exit goto label: 

	flash.c: In function `flash_print_info':
	flash.c:172: warning: deprecated use of label at end of compound
statement
	smc91111.c: In function `smc_phy_configure':
	smc91111.c:1353: warning: deprecated use of label at end of compound
statement

Is there a clean way to make this warning go away while keeping the code
savings by using the goto in these cases?  I've not tried very hard to make
it go away...

Also, I was wondering when its appropriate to attempt to add a board into
the main tree.  I have a working port to a new processor type, but the board
I'm working on is only a stepping stone to a more main stream board.
Putting code for this board into the main tree will result in a port which
won't be maintained, but does have some value for others doing trying to do
the same....at the moment I plan to hold off and submit the other board when
it enters production....but I don't really want to have to redo work if
someone else enters a port in the mean time...

Best Regards,

Richard W.






-----Original Message-----
From: Wolfgang Denk [mailto:wd at denx.de] 
Sent: Sunday, June 22, 2003 11:37 AM
To: Woodruff, Richard
Cc: U-Boot Users
Subject: Re: [PATCH]: Enable device and console for ARM (III). 

* Make sure to run the  MAKEALL  script,  and  resolve  ALL  compiler
  warnings introduced by your patch.