[Intel-gfx] [PATCH v3 0/7] drm/i915: Baytrail MIPI DSI support Updated

Shobhit Kumar shobhit.kumar at intel.com
Tue Dec 10 07:44:53 CET 2013


Hi All -
These patches enhance the current support for MIPI DSI for Baytrail. They
continue on the sub-encoder design and adds few more dev_ops to handle
sequence correctly. Major changes are -

1. DSI Clock calculation based on pixel clock
2. Add new dev_ops for better sequencing the enable/disable path
3. Parameterized the hardcoded DSI parameters. These also forms building
   block for the generic MIPI driver to come in future based on enhancements
   in VBT. All these parameters are initialized or computed in the sub-encoder
   driver. Some of them might look unneccesary for now.

I am also aware of the drm_bridge support now comming in and will in future
migrate from sub-encoder design to drm_bridge.

This DSI sequence has been validated with couple of test panels and is working now.
Still no sub-encoder driver is included and this support will be mostly be disabled
untill a panel sub-encoder driver is added. Proper detection or VBT is still pending.

v2: Mostly changes from review comments from Jani Nikula and Ville Syrjala
    - Split the parameters into new patch
    - Split the dsi_clk computation and m-n-p modification in separate patches
    - The DSI sequence refactoring has been splitted into multiple patches and also
      few code changes are not needed after reworking/relooking at them and have been
      removed
    - Backlight enabling has been removed as that depends on platform PMIC driver which
      is not yet there in upstream kernel. Will be added later.
    - Other general code cleanup as suggested
    - drm/i915: Use FLISDSI interface for band gap reset - has no changes and is included
      for completeness of the patch set

v3: Next round of changes resulting from Jani's review
    - Removed the I915_WRITE_BITS as it was found that we need not do read/modify/write and
      can just write the value
    - Moved the sub-encoder hook in dsi_pre_enable and dsi_post_enable
    - Minor coding tidbits are fixed

Regards
Shobhit

Shobhit Kumar (7):
  drm/i915: Add more dev ops for MIPI sub encoder
  drm/i915: Use FLISDSI interface for band gap reset
  drm/i915: Compute dsi_clk from pixel clock
  drm/i915: Try harder to get best m,n,p values with minimal error
  drm/i915: Reorganize the DSI enable/disable sequence
  drm/i915: Remove redundant DSI PLL enabling
  drm/i915: Parametrize the dphy and other spec specific parameters

 drivers/gpu/drm/i915/i915_drv.h       |   2 +
 drivers/gpu/drm/i915/i915_reg.h       |   1 +
 drivers/gpu/drm/i915/intel_dsi.c      | 188 ++++++++++++++++++----------------
 drivers/gpu/drm/i915/intel_dsi.h      |  21 ++++
 drivers/gpu/drm/i915/intel_dsi_pll.c  | 119 +++++++++------------
 drivers/gpu/drm/i915/intel_sideband.c |  14 +++
 6 files changed, 191 insertions(+), 154 deletions(-)

-- 
1.8.3.2




More information about the Intel-gfx mailing list