On 07/06/2020 13:06:09-0400, liambeguin@xxxxxxxxx wrote: > static int pcf2127_nvmem_read(void *priv, unsigned int offset, > @@ -434,6 +546,8 @@ static int pcf2127_probe(struct device *dev, struct regmap *regmap, > > pcf2127->rtc->ops = &pcf2127_rtc_ops; > > + device_init_wakeup(dev, true); > + This can't be done unconditionally, You need to have been able to request an interrupt or the wakeup-source property needs to be present. The interrupt handler is also missing from the patch. -- Alexandre Belloni, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
