On 14/06/2023 13:43, Andi Shyti wrote: > Hi Wang, > >> diff --git a/drivers/i2c/muxes/i2c-mux-gpio.c b/drivers/i2c/muxes/i2c-mux-gpio.c >> index 5d5cbe013..90966de7a 100644 >> --- a/drivers/i2c/muxes/i2c-mux-gpio.c >> +++ b/drivers/i2c/muxes/i2c-mux-gpio.c >> @@ -106,6 +106,7 @@ static int i2c_mux_gpio_probe_fw(struct gpiomux *mux, >> } else if (is_acpi_node(child)) { >> rc = acpi_get_local_address(ACPI_HANDLE_FWNODE(child), values + i); >> if (rc) >> + fwnode_handle_put(child); >> return dev_err_probe(dev, rc, "Cannot get address\n"); >> } > > please, read it again and tell me what's wrong :) No, we can't tell what's wrong because: > The contents of this message and any attachments may contain > confidential and/or privileged information :) Wang, Just run the sparse and smatch before posting patches. Best regards, Krzysztof