All of lore.kernel.org
 help / color / mirror / Atom feed
* [Intel-gfx] [RFC PATCH 0/3] i915 writeback private framework
@ 2022-03-16  9:37 Suraj Kandpal
  2022-03-16  9:37 ` [Intel-gfx] [RFC PATCH 1/3] drm/i915: Creating writeback pipeline to bypass drm_writeback framework Suraj Kandpal
                   ` (10 more replies)
  0 siblings, 11 replies; 26+ messages in thread
From: Suraj Kandpal @ 2022-03-16  9:37 UTC (permalink / raw)
  To: intel-gfx; +Cc: jani.nikula

A patch series was floated in the drm mailing list which aimed to change
the drm_connector and drm_encoder fields to pointer in the
drm_connector_writeback structure, this received a huge pushback from
the community but since i915 expects each connector present in the
drm_device list to be a intel_connector but drm_writeback framework
forces us to use a drm_connector which is not embedded in
intel_connector the current drm_writeback framework becomes very
unfeasible to us as it would mean a lot of checks at a lot of places
to take into account the above issue. One of the solutions is to
make our own writeback pipeline bypassing one provided by drm
which is what these patches do.

Suraj Kandpal (3):
  drm/i915: Creating writeback pipeline to bypass drm_writeback
    framework
  drm/i915: Define WD trancoder for i915
  drm/i915: Enabling WD Transcoder

 drivers/gpu/drm/i915/Makefile                 |   2 +
 drivers/gpu/drm/i915/display/intel_acpi.c     |   1 +
 drivers/gpu/drm/i915/display/intel_display.c  |  85 +-
 drivers/gpu/drm/i915/display/intel_display.h  |  15 +
 .../drm/i915/display/intel_display_types.h    |  21 +
 drivers/gpu/drm/i915/display/intel_dpll.c     |   3 +
 drivers/gpu/drm/i915/display/intel_opregion.c |   3 +
 .../gpu/drm/i915/display/intel_wb_connector.c | 296 ++++++
 .../gpu/drm/i915/display/intel_wb_connector.h |  99 ++
 drivers/gpu/drm/i915/display/intel_wd.c       | 996 ++++++++++++++++++
 drivers/gpu/drm/i915/display/intel_wd.h       |  82 ++
 drivers/gpu/drm/i915/i915_drv.h               |   5 +
 drivers/gpu/drm/i915/i915_irq.c               |   8 +-
 drivers/gpu/drm/i915/i915_pci.c               |   7 +-
 drivers/gpu/drm/i915/i915_reg.h               | 139 +++
 15 files changed, 1759 insertions(+), 3 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/display/intel_wb_connector.c
 create mode 100644 drivers/gpu/drm/i915/display/intel_wb_connector.h
 create mode 100644 drivers/gpu/drm/i915/display/intel_wd.c
 create mode 100644 drivers/gpu/drm/i915/display/intel_wd.h

-- 
2.35.1


^ permalink raw reply	[flat|nested] 26+ messages in thread
* [RFC PATCH 0/3] i915 writeback private framework
@ 2022-04-21  5:07 Suraj Kandpal
  2022-04-21  5:07 ` [Intel-gfx] [RFC PATCH 2/3] drm/i915: Define WD trancoder for i915 Suraj Kandpal
  0 siblings, 1 reply; 26+ messages in thread
From: Suraj Kandpal @ 2022-04-21  5:07 UTC (permalink / raw)
  To: intel-gfx, dri-devel; +Cc: jani.nikula, Suraj Kandpal, arun.r.murthy

A patch series was floated in the drm mailing list which aimed to change
the drm_connector and drm_encoder fields to pointer in the
drm_connector_writeback structure, this received a huge pushback from
the community but since i915 expects each connector present in the
drm_device list to be a intel_connector but drm_writeback framework.
[1] https://patchwork.kernel.org/project/dri-devel/patch/20220202081702.22119-1-suraj.kandpal@intel.com/
[2] https://patchwork.kernel.org/project/dri-devel/patch/20220202085429.22261-6-suraj.kandpal@intel.com/
This forces us to use a drm_connector which is not embedded in
intel_connector the current drm_writeback framework becomes very
unfeasible to us as it would mean a lot of checks at a lot of places
to take into account the above issue.Since no one had an issue with
encoder field being changed into a pointer it was decided to break the
connector and encoder pointer changes into two different series.The
encoder field changes is currently being worked upon by Abhinav Kumar
[3]https://patchwork.kernel.org/project/dri-devel/list/?series=633565
In the meantime for i915 to start using the writeback functionality we
came up with a interim solution to own writeback pipeline bypassing one
provided by drm which is what these patches do.
Note: these are temp patches till we figure out how we can either change
drm core writeback to work with our intel_connector structure or find a
different solution which allows us to work with the current
drm_writeback framework

Suraj Kandpal (3):
  drm/i915: Creating writeback pipeline to bypass drm_writeback
    framework
  drm/i915: Define WD trancoder for i915
  drm/i915: Enabling WD Transcoder

 drivers/gpu/drm/i915/Makefile                 |   2 +
 drivers/gpu/drm/i915/display/intel_acpi.c     |   1 +
 drivers/gpu/drm/i915/display/intel_display.c  |  89 +-
 drivers/gpu/drm/i915/display/intel_display.h  |  15 +
 .../drm/i915/display/intel_display_types.h    |  18 +
 drivers/gpu/drm/i915/display/intel_dpll.c     |   3 +
 drivers/gpu/drm/i915/display/intel_opregion.c |   3 +
 .../gpu/drm/i915/display/intel_wb_connector.c | 296 ++++++
 .../gpu/drm/i915/display/intel_wb_connector.h |  99 ++
 drivers/gpu/drm/i915/display/intel_wd.c       | 978 ++++++++++++++++++
 drivers/gpu/drm/i915/display/intel_wd.h       |  82 ++
 drivers/gpu/drm/i915/i915_drv.h               |   5 +
 drivers/gpu/drm/i915/i915_irq.c               |   8 +-
 drivers/gpu/drm/i915/i915_pci.c               |   7 +-
 drivers/gpu/drm/i915/i915_reg.h               | 139 +++
 15 files changed, 1742 insertions(+), 3 deletions(-)
 create mode 100644 drivers/gpu/drm/i915/display/intel_wb_connector.c
 create mode 100644 drivers/gpu/drm/i915/display/intel_wb_connector.h
 create mode 100644 drivers/gpu/drm/i915/display/intel_wd.c
 create mode 100644 drivers/gpu/drm/i915/display/intel_wd.h

-- 
2.35.1


^ permalink raw reply	[flat|nested] 26+ messages in thread

end of thread, other threads:[~2022-04-28  5:53 UTC | newest]

Thread overview: 26+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-16  9:37 [Intel-gfx] [RFC PATCH 0/3] i915 writeback private framework Suraj Kandpal
2022-03-16  9:37 ` [Intel-gfx] [RFC PATCH 1/3] drm/i915: Creating writeback pipeline to bypass drm_writeback framework Suraj Kandpal
2022-03-16 14:25   ` kernel test robot
2022-03-18  4:18   ` [Intel-gfx] [RFC PATCH v2 " Suraj Kandpal
2022-04-06  4:58     ` Kandpal, Suraj
2022-03-16  9:37 ` [Intel-gfx] [RFC PATCH 2/3] drm/i915: Define WD trancoder for i915 Suraj Kandpal
2022-03-18  4:20   ` [Intel-gfx] [RFC PATCH v2 " Suraj Kandpal
2022-03-16  9:37 ` [Intel-gfx] [RFC PATCH 3/3] drm/i915: Enabling WD Transcoder Suraj Kandpal
2022-03-18  4:21   ` [Intel-gfx] [RFC PATCH v2 " Suraj Kandpal
2022-03-25  5:35     ` Kandpal, Suraj
2022-03-16  9:51 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for i915 writeback private framework Patchwork
2022-03-16  9:52 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2022-03-16 10:07 ` [Intel-gfx] ✗ Fi.CI.BAT: failure " Patchwork
2022-03-17  8:39 ` [Intel-gfx] [RFC PATCH v2 0/3] " Suraj Kandpal
2022-03-17  8:39   ` [Intel-gfx] [RFC PATCH v2 1/3] drm/i915: Creating writeback pipeline to bypass drm_writeback framework Suraj Kandpal
2022-03-17  8:39   ` [Intel-gfx] [RFC PATCH v2 2/3] drm/i915: Define WD trancoder for i915 Suraj Kandpal
2022-04-06  5:00     ` Kandpal, Suraj
2022-03-17  8:39   ` [Intel-gfx] [RFC PATCH v2 3/3] drm/i915: Enabling WD Transcoder Suraj Kandpal
2022-04-06  4:58     ` Kandpal, Suraj
2022-04-06  4:56   ` [Intel-gfx] [RFC PATCH v2 0/3] i915 writeback private framework Kandpal, Suraj
2022-03-18  4:48 ` [Intel-gfx] ✗ Fi.CI.CHECKPATCH: warning for i915 writeback private framework (rev4) Patchwork
2022-03-18  4:49 ` [Intel-gfx] ✗ Fi.CI.SPARSE: " Patchwork
2022-03-18  5:20 ` [Intel-gfx] ✓ Fi.CI.BAT: success " Patchwork
2022-03-18  8:29 ` [Intel-gfx] ✓ Fi.CI.IGT: " Patchwork
2022-04-21  5:07 [RFC PATCH 0/3] i915 writeback private framework Suraj Kandpal
2022-04-21  5:07 ` [Intel-gfx] [RFC PATCH 2/3] drm/i915: Define WD trancoder for i915 Suraj Kandpal
2022-04-28  5:53   ` Kandpal, Suraj

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.