Hi Matt, > > -static inline struct intel_pxp *i915_dev_to_pxp(struct device *i915_kdev) > > +static struct intel_pxp *i915_dev_to_pxp(struct device *i915_kdev) > > Was dropping the inline here intentional? It doesn't seem like there's > any reason to drop it, and if it was intentional the whitespace isn't > quite right. No, it wasn't intentional and it's strange that checkpatch didn't catch it. I will resend this one. Thanks! Andi