[U-Boot-Users] [Patch] commonspartan
13 May
2004
13 May
'04
4:46 p.m.
ChangeLog:
* Patch by Kai-Uwe Bloem, Robert Schwebel, 13 May 2004: When code is compiled without -fsigned-char compiler claims about 'always false due to limited data type'. This patch doesn't change functionality but makes it work on fussy 2.95.3-arm compilers.
Robert
--
Dipl.-Ing. Robert Schwebel | http://www.pengutronix.de
Pengutronix - Linux Solutions for Science and Industry
Handelsregister: Amtsgericht Hildesheim, HRA 2686
Hornemannstraße 12, 31137 Hildesheim, Germany
Phone: +49-5121-28619-0 | Fax: +49-5121-28619-4
13 May
13 May
6:14 p.m.
In message 20040513144612.GR27774@pengutronix.de you wrote:
- Patch by Kai-Uwe Bloem, Robert Schwebel, 13 May 2004: When code is compiled without -fsigned-char compiler claims about 'always false due to limited data type'. This patch doesn't change functionality but makes it work on fussy 2.95.3-arm compilers.
... char val; ...
(*fn->wr) ((val < 0), TRUE, cookie);
(*fn->wr) ((val & 0x80), TRUE, cookie);
Actually I don't like the idea of testing the MSB when I really want to test for <0; maybe the correct way to fix this warning is to make val a "signed char" instead?
What do you think?
Best regards,
Wolfgang Denk
--
Software Engineering: Embedded and Realtime Systems, Embedded Linux
Phone: (+49)-8142-4596-87 Fax: (+49)-8142-4596-88 Email: wd@denx.de
There are three things I always forget. Names, faces - the third I
can't remember. - Italo Svevo
7669
Age (days ago)
7669
Last active (days ago)
1 comments
2 participants
participants (2)
-
Robert Schwebel -
Wolfgang Denk