No point in explicitly setting something to zero right after we explicitly checked that it is zero. Fix this.
Reported-by: Alan Cox <[email protected]> Signed-off-by: Alexander Shishkin <[email protected]> Reviewed-by: Laurent Fert <[email protected]> --- drivers/hwtracing/stm/core.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/hwtracing/stm/core.c b/drivers/hwtracing/stm/core.c index de80d45d8d..18f176eac0 100644 --- a/drivers/hwtracing/stm/core.c +++ b/drivers/hwtracing/stm/core.c @@ -546,8 +546,6 @@ static int stm_char_policy_set_ioctl(struct stm_file *stmf, void __user *arg) if (ret) goto err_free; - ret = 0; - if (stm->data->link) ret = stm->data->link(stm->data, stmf->output.master, stmf->output.channel); -- 2.8.0.rc3
