Hi Cai, On Thu, Sep 16, 2021 at 5:32 PM Cai Huoqing <caihuoqing@xxxxxxxxx> wrote: > if (IS_ERR(ts->reset_gpio)) { > error = PTR_ERR(ts->reset_gpio); > - dev_err(dev, "Failed to request reset gpio, error %d\n", error); > + dev_err_probe(dev, error, "Failed to request reset gpio\n"); > return ERR_PTR(error); In this case you're supposed to do return dev_err_probe(dev, error, "Failed to request reset gpio\n"); Yours, Linus Walleij