On 3/28/19 7:17 AM, Dan Carpenter wrote: > The "result" variable isn't initialized on this error path. > > Fixes: 55e331cf7ebe ("drivers: add support for the TI VLYNQ bus") > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> Acked-by: Florian Fainelli <f.fainelli@xxxxxxxxx> Andrew can you pick this up the same you have you done that before? > --- > drivers/vlynq/vlynq.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/vlynq/vlynq.c b/drivers/vlynq/vlynq.c > index c0227f9418eb..44d3485d1108 100644 > --- a/drivers/vlynq/vlynq.c > +++ b/drivers/vlynq/vlynq.c > @@ -727,7 +727,8 @@ static int vlynq_probe(struct platform_device *pdev) > dev->local_irq = dev->irq_end - dev->irq_start; > dev->remote_irq = dev->local_irq - 1; > > - if (device_register(&dev->dev)) > + result = device_register(&dev->dev); > + if (result) > goto fail_register; > platform_set_drvdata(pdev, dev); > > -- Florian