]> Git Repo - linux.git/commitdiff
sc16is7xx: Defer probe if device read fails
authorAnnaliese McDermond <[email protected]>
Mon, 29 Mar 2021 20:10:49 +0000 (20:10 +0000)
committerGreg Kroah-Hartman <[email protected]>
Mon, 5 Apr 2021 10:07:32 +0000 (12:07 +0200)
A test was added to the probe function to ensure the device was
actually connected and working before successfully completing a
probe.  If the device was actually there, but the I2C bus was not
ready yet for whatever reason, the probe fails permanently.

Change the probe so that we defer the probe on a regmap read
failure so that we try the probe again when the dependent drivers
are potentially loaded.  This should not affect the case where the
device truly isn't present because the probe will never successfully
complete.

Fixes: 2aa916e67db3 ("sc16is7xx: Read the LSR register for basic device presence check")
Cc: [email protected]
Signed-off-by: Annaliese McDermond <[email protected]>
Link: https://lore.kernel.org/r/010101787f9c3fd8-c1815c00-2d6b-4c85-a96a-a13e68597fda-000000@us-west-2.amazonses.com
Signed-off-by: Greg Kroah-Hartman <[email protected]>
drivers/tty/serial/sc16is7xx.c

index f86ec2d2635b7c76121914c12b40d564711742da..9adb8362578c5d8c54a5c1a34061e5688eaf727d 100644 (file)
@@ -1196,7 +1196,7 @@ static int sc16is7xx_probe(struct device *dev,
        ret = regmap_read(regmap,
                          SC16IS7XX_LSR_REG << SC16IS7XX_REG_SHIFT, &val);
        if (ret < 0)
-               return ret;
+               return -EPROBE_DEFER;
 
        /* Alloc port structure */
        s = devm_kzalloc(dev, struct_size(s, p, devtype->nr_uart), GFP_KERNEL);
This page took 0.060817 seconds and 4 git commands to generate.