Skip to content

drm/i915: gmbus pin/port cleanup and bxt enabling

Jani Nikula intel-gfx 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 5 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5drm/i915: rename GMBUS_PORT_* macros as GMBUS_PIN_*drivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/i915_reg.h drivers/gpu/drm/i915/intel_bios.c +5 more+23 -230 replies
2/5drm/i915: refer to pin instead of port in the intel_i2c.c interfacesdrivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/intel_bios.c drivers/gpu/drm/i915/intel_dvo.c +3 more+12 -120 replies
3/5drm/i915: index gmbus tables using the pin pair numberdrivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/i915_reg.h drivers/gpu/drm/i915/intel_i2c.c+40 -303 replies
4/5drm/i915: base gmbus pin validity check on the gmbus pin map arraydrivers/gpu/drm/i915/i915_drv.h drivers/gpu/drm/i915/intel_bios.c drivers/gpu/drm/i915/intel_dvo.c +3 more+17 -121 reply
5/5drm/i915: add bxt gmbus supportdrivers/gpu/drm/i915/i915_reg.h drivers/gpu/drm/i915/intel_i2c.c+26 -67 replies

Reviews trailers per patch

2 of 5 patches have a Reviewed-by or Acked-by.

PatchVille SyrjäläIntel Graphics QA PRTS (Patch Regression Test System Contact:Imre Deak
1/5
2/5
3/5R-b R-b
4/5
5/5T-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.

Read the v1 thread

Cover letter [PATCH 0/5] drm/i915: gmbus pin/port cleanup and bxt enabling

Hi all, Imre in particular -

This series spun out of reviewing the bxt gmbus enabling patches [1],
[2] and [3] that were getting, at least for my poor brain, hard to follow.

The first four are platform independent cleanup and prep work that could be merged before any of the bxt enabling series. It's all pretty straightforward too.

The last patch enables gmbus on bxt.

Together this series replaces the three referenced patches.

As a follow-up, it's fairly easy to e.g. add the correct names for the pins for CHV, and drop legacy pins (ssc, lvds) for hsw+ that we still register adapters for! I just kept this series small for starters.

BR,
Jani.

[1] http://mid.gmane.org/[email protected]
[2] http://mid.gmane.org/[email protected]
[3] http://mid.gmane.org/[email protected]

A.Sunil Kamath (1):
drm/i915: add bxt gmbus support

Jani Nikula (4):
drm/i915: rename GMBUS_PORT_* macros as GMBUS_PIN_* drm/i915: refer to pin instead of port in the intel_i2c.c interfaces drm/i915: index gmbus tables using the pin pair number drm/i915: base gmbus pin validity check on the gmbus pin map array

 drivers/gpu/drm/i915/i915_drv.h   | 13 ++----
 drivers/gpu/drm/i915/i915_reg.h   | 23 ++++++----
 drivers/gpu/drm/i915/intel_bios.c |  4 +-
 drivers/gpu/drm/i915/intel_crt.c  |  2 +-
 drivers/gpu/drm/i915/intel_dvo.c  | 10 ++--
 drivers/gpu/drm/i915/intel_hdmi.c |  8 ++--
 drivers/gpu/drm/i915/intel_i2c.c  | 96 ++++++++++++++++++++++++++-------------
 drivers/gpu/drm/i915/intel_lvds.c |  4 +-
 drivers/gpu/drm/i915/intel_sdvo.c |  5 +-
 9 files changed, 100 insertions(+), 65 deletions(-)
Signature · 6 lines
2.1.4

_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx