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

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@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.

Dear Richard,
in message FD2AC9A020DDD51194710008C7089B20053D4CA6@dlee17.itg.ti.com you wrote:
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.
Just run MAKEALL ...
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...
Just change: label: into: label: ;
(i. e. add a null statement).
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.
There are no general rules. As soon as there is something that is either usable or otherwise useful to others, I recommend to add it.
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...
So submit the patches now.
Best regards,
Wolfgang Denk
participants (2)
-
Wolfgang Denk
-
Woodruff, Richard