[Intel-gfx] [PATCH 3/8] drm/i915: Make the intel_device_info structure kept in dev_priv writable
Damien Lespiau
damien.lespiau at intel.com
Thu Dec 12 16:38:05 CET 2013
On Thu, Dec 12, 2013 at 05:30:14PM +0200, Jani Nikula wrote:
> > -#define INTEL_INFO(dev) (to_i915(dev)->info)
> > +#define INTEL_INFO(dev) ((const struct intel_device_info *)&to_i915(dev)->info)
>
> If that were an inline function you wouldn't have to cast to add const:
>
> static inline const struct intel_device_info *
> intel_info(const struct drm_device *dev)
> {
> return to_i915(dev)->info;
> }
>
> I'm not insisting on this bikeshed, though.
True, I'd happily review a follow-up patch once the series has landed :)
--
Damien
More information about the Intel-gfx
mailing list