8 Apr
2010
8 Apr
'10
9:27 a.m.
On Apr 8, 2010, at 1:14 AM, Liu Dave-R63238 wrote:
@@ -369,8 +370,8 @@ static int fsl_diu_enable_panel(struct
fb_info *info)
struct diu_ad *ad = &fsl_diu_fb_ad;
debug("Entered: enable_panel\n");
- if (hw->desc[0] != (unsigned int)ad)
hw->desc[0] = (unsigned int)ad;
- if (in_be32(&hw->desc[0]) != (unsigned int)ad)
out_be32(&hw->desc[0], ad);'ad' should be cast to a u32, not an "unsigned int". You shouldn't compare a sized type with an unsized type.
Grep the include/asm-ppc/types.h, I got typedef unsigned int u32; The u32 is same as "unsigned int".
Kumar, any comments?
Timur is correct, while u32 is typedef to 'unsigned int' its cleaner in the code to use u32 and if we have a 64-bit port of u-boot on PPC in the future it makes things cleaner.
- k