On Wed, 20 Dec 2017, Michal Wajdeczko <[email protected]> wrote: > Our main header is huge. Lets try to make some cleanup.
Overall seems fine, I think I agree with Chris' review. I do want Rodrigo's ack wrt upcoming stuff that's bound to conflict with many of the patches here. I don't want to cause unnecessary burden. BR, Jani. > > Cc: Chris Wilson <[email protected]> > Cc: Rodrigo Vivi <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > > Michal Wajdeczko (8): > drm/i915: Move uint_fixed_16_16_t to i915_types.h > drm/i915: Move some utility functions to i915_util.h > drm/i915: Move display related definitions to dedicated header > drm/i915: Move opregion definitions to dedicated intel_opregion.h > drm/i915: Move intel_device_info definitions to its own header > drm/i915: Update intel_device_info_runtime_init() parameter > drm/i915: Add pretty printer for runtime part of intel_device_info > drm/i915: Dump device info at once > > drivers/gpu/drm/i915/i915_drv.c | 15 +- > drivers/gpu/drm/i915/i915_drv.h | 633 > +------------------------------ > drivers/gpu/drm/i915/i915_types.h | 168 ++++++++ > drivers/gpu/drm/i915/i915_utils.h | 15 + > drivers/gpu/drm/i915/intel_device_info.c | 55 ++- > drivers/gpu/drm/i915/intel_device_info.h | 175 +++++++++ > drivers/gpu/drm/i915/intel_display.h | 312 +++++++++++++++ > drivers/gpu/drm/i915/intel_opregion.c | 2 + > drivers/gpu/drm/i915/intel_opregion.h | 99 +++++ > 9 files changed, 817 insertions(+), 657 deletions(-) > create mode 100644 drivers/gpu/drm/i915/i915_types.h > create mode 100644 drivers/gpu/drm/i915/intel_device_info.h > create mode 100644 drivers/gpu/drm/i915/intel_display.h > create mode 100644 drivers/gpu/drm/i915/intel_opregion.h -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
