>-----Original Message----- >From: [email protected] >[mailto:[email protected]] On Behalf Of ext >Kanigeri, Hari >Sent: 17 April, 2009 17:21 >To: Gupta, Ramesh; Palande Ameya (Nokia-D/Helsinki); >[email protected] >Subject: RE: [PATCH] [DSPBRIDGE] Various compile warning fixes > >Ameya, > >- if (!(chnlId >= 0) || !(chnlId < CHNL_MAXCHANNELS)) { >+ if (chnlId >= CHNL_MAXCHANNELS) { > >-- How about declaring chnlId as s32 so that the checks can be >extended to negative values too ?
The field in question is declared in arch/arm/plat-omap/include/dspbridge/_chnl_sm.h in line 120 as u32, so checking for negative value makes no sense unless you modify struct SHM too. --MiKu -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
