RE: [PATCH] [DSPBRIDGE] Various compile warning fixes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>-----Original Message-----
>From: linux-omap-owner@xxxxxxxxxxxxxxx 
>[mailto:linux-omap-owner@xxxxxxxxxxxxxxx] On Behalf Of ext 
>Kanigeri, Hari
>Sent: 17 April, 2009 17:21
>To: Gupta, Ramesh; Palande Ameya (Nokia-D/Helsinki); 
>linux-omap@xxxxxxxxxxxxxxx
>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 majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux