On Fri, May 7, 2021 at 11:43 PM Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> wrote: > if (IS_ERR(info->rtc_dev)) { > ret = PTR_ERR(info->rtc_dev); > dev_err(&pdev->dev, "Failed to register RTC device: %d\n", ret); Following the recent conversations, I think it might make sense to do dev_err(&pdev->dev, "Failed to register RTC device: %pe\n", info->rtc_dev); Is that right?