[media] dvb: negative value assigned to unsigned int in CDRXD().
Commit Message
From: Santosh Nayak <santoshprasadnayak@gmail.com>
In CDRXD(), Negative number is assigned to unsigned variable
'state->noise_cal.tdCal2.
Members of 'SNoiseCal' should be 'signed short'.
Signed-off-by: Santosh Nayak <santoshprasadnayak@gmail.com>
---
drivers/media/dvb/frontends/drxd_hard.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
Comments
On Wed, Mar 07, 2012 at 09:11:03AM +0530, santosh nayak wrote:
> From: Santosh Nayak <santoshprasadnayak@gmail.com>
>
> In CDRXD(), Negative number is assigned to unsigned variable
> 'state->noise_cal.tdCal2.
>
> Members of 'SNoiseCal' should be 'signed short'.
>
In your changelogs could you please write something about how this
change will affect what the user sees. That way users know if it
fixes something they care about and they will apply your patch.
regards,
dan carpenter
@@ -101,9 +101,9 @@ struct SCfgAgc {
struct SNoiseCal {
int cpOpt;
- u16 cpNexpOfs;
- u16 tdCal2k;
- u16 tdCal8k;
+ short cpNexpOfs;
+ short tdCal2k;
+ short tdCal8k;
};
enum app_env {