Messages in this thread | | | Date | Tue, 18 Nov 2014 17:47:02 +0100 | From | Michal Simek <> | Subject | Re: [PATCH 2/2] net: can: comparison of unsigned variable |
| |
On 11/18/2014 02:47 PM, Sudip Mukherjee wrote: > err was of the type u32. it was being compared with < 0, and being > an unsigned variable the comparison would have been always false. > > moreover, err was getting the return value from set_reset_mode() > and xcan_set_bittiming(), and both are returning int. > > Signed-off-by: Sudip Mukherjee <sudip@vectorindia.org> > --- > drivers/net/can/xilinx_can.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/can/xilinx_can.c b/drivers/net/can/xilinx_can.c > index 72fe96f..67c2dd4 100644 > --- a/drivers/net/can/xilinx_can.c > +++ b/drivers/net/can/xilinx_can.c > @@ -300,7 +300,8 @@ static int xcan_set_bittiming(struct net_device *ndev) > static int xcan_chip_start(struct net_device *ndev) > { > struct xcan_priv *priv = netdev_priv(ndev); > - u32 err, reg_msr, reg_sr_mask; > + u32 reg_msr, reg_sr_mask; > + int err; > unsigned long timeout; > > /* Check if it is in reset mode */ >
Reviewed-by: Michal Simek <michal.simek@xilinx.com>
Thanks, Michal
| |