On Mon, 15 Mar 2021 12:02:30 -0700 Tony Nguyen wrote: > +static __maybe_unused int e1000e_pm_prepare(struct device *dev) > +{ > + return pm_runtime_suspended(dev) && > + pm_suspend_via_firmware();
nit: I don't think you need to mark functions called by __maybe_unused as __maybe_unused, do you? The series LGTM although I don't know much about PM.