diff options
author | Russell King - ARM Linux <linux@arm.linux.org.uk> | 2011-01-03 22:34:27 +0000 |
---|---|---|
committer | Dan Williams <dan.j.williams@intel.com> | 2011-01-04 19:16:11 -0800 |
commit | ad0a3ad33c96cbba98ba62116771fb836c551e60 (patch) | |
tree | 7d91cc40d91bf63f6bfd0023c114fcc8d549700e /drivers/hwmon | |
parent | 730404ac1c47403af67420705980c99e90bf182f (diff) | |
download | kernel_samsung_aries-ad0a3ad33c96cbba98ba62116771fb836c551e60.zip kernel_samsung_aries-ad0a3ad33c96cbba98ba62116771fb836c551e60.tar.gz kernel_samsung_aries-ad0a3ad33c96cbba98ba62116771fb836c551e60.tar.bz2 |
ARM: PL08x: remove unnecessary NULL and BUG checks
The tasklet always is initialized with a non-NULL data argument. It
is not possible for it to be called with a NULL data argument (unless
something is very wrong in the tasklet code - in which case lots of
stuff will break). Therefore, as plchan can never be NULL, remove
this unnecessary BUG check.
In pl08x_tasklet(), we've already dereferenced plchan->at, so it can't
be NULL here. Remove this unnecessary BUG check.
pl08x_fill_llis_for_desc() and pl08x_free_txd() are always called with
a non-NULL txd argument - either as a consequence of the code paths or
as a result of other checks already in place. We don't need to repeat
the non-NULL check in these functions.
Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
Acked-by: Linus Walleij <linus.walleij@stericsson.com>
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/hwmon')
0 files changed, 0 insertions, 0 deletions