Skip to content

Commit 38dfa56

Browse files
Saravana Kannangregkh
authored andcommitted
gpiolib: Clear the gpio_device's fwnode initialized flag before adding
Registering an irqdomain sets the flag for the fwnode. But having the flag set when a device is added is interpreted by fw_devlink to mean the device has already been initialized and will never probe. This prevents fw_devlink from creating device links with the gpio_device as a supplier. So, clear the flag before adding the device. Signed-off-by: Saravana Kannan <[email protected]> Acked-by: Bartosz Golaszewski <[email protected]> Tested-by: Colin Foster <[email protected]> Tested-by: Sudeep Holla <[email protected]> Reviewed-by: Andy Shevchenko <[email protected]> Tested-by: Douglas Anderson <[email protected]> Tested-by: Geert Uytterhoeven <[email protected]> Tested-by: Luca Weiss <[email protected]> # qcom/sm7225-fairphone-fp4 Link: https://lore.kernel.org/r/[email protected] Signed-off-by: Greg Kroah-Hartman <[email protected]>
1 parent 8640166 commit 38dfa56

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

drivers/gpio/gpiolib.c

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -578,6 +578,13 @@ static int gpiochip_setup_dev(struct gpio_device *gdev)
578578
{
579579
int ret;
580580

581+
/*
582+
* If fwnode doesn't belong to another device, it's safe to clear its
583+
* initialized flag.
584+
*/
585+
if (gdev->dev.fwnode && !gdev->dev.fwnode->dev)
586+
fwnode_dev_initialized(gdev->dev.fwnode, false);
587+
581588
ret = gcdev_register(gdev, gpio_devt);
582589
if (ret)
583590
return ret;

0 commit comments

Comments
 (0)