[Intel-gfx] [PATCH 0/4] drm/i915: GMCH refactoring

Jani Nikula jani.nikula at intel.com
Wed Jan 25 11:57:12 UTC 2023


On Tue, 17 Jan 2023, Rodrigo Vivi <rodrigo.vivi at intel.com> wrote:
> On Tue, Jan 17, 2023 at 02:33:05PM +0200, Jani Nikula wrote:
>> Let's see if this sticks.
>
> Suddenly all of these 'bridge_dev' makes some sense to me.
>
> I believe we could enjoy the refactor here and add some documentation
> about this... at least the name Graphics Memory Controller Hub name...
>
> I also wonder if we should make this a child device now that it is
> really organized...
>
> But anyway, everything can be a follow up, the current refactor
> is already a very good clean up step, so for the series:
>
>
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi at intel.com>

Thanks for the reviews, pushed to drm-intel-next with Ville's IRC ack.

BR,
Jani.

>
>
>> 
>> Jani Nikula (4):
>>   drm/i915: add gmch substruct to struct drm_i915_private
>>   drm/i915/gmch: split out soc/intel_gmch
>>   drm/i915/gmch: mass rename dev_priv to i915
>>   drm/i915/gmch: move VGA set state to GMCH code
>> 
>>  drivers/gpu/drm/i915/Makefile             |   1 +
>>  drivers/gpu/drm/i915/display/intel_vga.c  |  32 +---
>>  drivers/gpu/drm/i915/gt/intel_ggtt_gmch.c |   2 +-
>>  drivers/gpu/drm/i915/i915_driver.c        | 145 +-----------------
>>  drivers/gpu/drm/i915/i915_drv.h           |  10 +-
>>  drivers/gpu/drm/i915/soc/intel_gmch.c     | 171 ++++++++++++++++++++++
>>  drivers/gpu/drm/i915/soc/intel_gmch.h     |  18 +++
>>  7 files changed, 204 insertions(+), 175 deletions(-)
>>  create mode 100644 drivers/gpu/drm/i915/soc/intel_gmch.c
>>  create mode 100644 drivers/gpu/drm/i915/soc/intel_gmch.h
>> 
>> -- 
>> 2.34.1
>> 

-- 
Jani Nikula, Intel Open Source Graphics Center


More information about the Intel-gfx mailing list