On Fri, Dec 10, 2021 at 02:21:53AM +0200, Andi Shyti wrote:
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.
With the 'inline' re-added, you can include
Reviewed-by: Matt Roper matthew.d.roper@intel.com
Thanks! Andi