From: David Miller
> From: Santosh Shilimkar <[email protected]>
> Date: Thu, 25 Sep 2014 13:48:36 -0400
>
> > +static inline int gbe_phy_link_status(struct gbe_slave *slave)
> > +{
> > + if (!slave->phy)
> > + return 1;
> > +
> > + if (slave->phy->link)
> > + return 1;
> > +
> > + return 0;
> > +}
>
> Please use 'bool' as the return type and return 'true' or 'false'.
That function body could also be just:
return !slave->phy && slave->phy->link;
which might be more readable if directly coded.
I also wonder if slave>phy can actually be NULL?
If it can be under unusual circumstances it might be worth
assigning the address of a dummy 'phy' structure so that the
tests can all be removed.
David
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
