On Fri, 19 Aug 2016 13:04:54 +0300 Jani Nikula <[email protected]> wrote:
> On Sat, 06 Aug 2016, Bob Paauwe <[email protected]> wrote: > > On Fri, 5 Aug 2016 15:23:23 -0700 > > "Xiong, James" <[email protected]> wrote: > > > >> Reviewed-by James Xiong <[email protected]> > > > > Merged to gold. Thanks for the review. > > What does this mean? Why do you Cc both internal and external lists? > > BR, > Jani. > Hi Jani, The patch was sent to both the upstream list for review and to our IOTG internal list. Since we're freezing our code for release and this is needed to fully enable MIPI functionality on our platform, it was merged into our release branch. I should have been more careful about where the reply went. It didn't need to go the public list. Sorry for the confusion. Bob > > > > > Bob > >> > >> -----Original Message----- > >> From: [email protected] > >> [mailto:[email protected]] On Behalf Of Paauwe, Bob J > >> Sent: Thursday, August 4, 2016 11:16 AM > >> To: isg-gms <[email protected]>; intel-gfx > >> <[email protected]> > >> Cc: Paauwe, Bob J <[email protected]> > >> Subject: [isg-gms] [PATCH] drm/i915/bxt: Bring MIPI out of reset > >> > >> and power up the DSI regulator when initializing a MIPI display. > >> > >> Signed-off-by: Bob Paauwe <[email protected]> > >> --- > >> drivers/gpu/drm/i915/i915_reg.h | 8 ++++++++ > >> drivers/gpu/drm/i915/intel_dsi.c | 13 +++++++++++++ > >> 2 files changed, 21 insertions(+) > >> > >> diff --git a/drivers/gpu/drm/i915/i915_reg.h > >> b/drivers/gpu/drm/i915/i915_reg.h index 6607aaf..da29d74 100644 > >> --- a/drivers/gpu/drm/i915/i915_reg.h > >> +++ b/drivers/gpu/drm/i915/i915_reg.h > >> @@ -1262,11 +1262,19 @@ enum skl_disp_power_wells { #define > >> CHV_TX_DW14(ch, lane) _TXLANE(ch, lane, 0xb8) > >> #define DPIO_UPAR_SHIFT 30 > >> > >> +/* BXT DSI Regulator registers */ > >> +#define BXT_DSI_CFG _MMIO(0x160020) > >> +#define STRAP_SELECT (1 << 0) > >> + > >> +#define BXT_DSI_TXCNTRL _MMIO(0x160054) > >> +#define HS_IO_CONTROL_SELECT 0x0 > >> + > >> /* BXT PHY registers */ > >> #define _BXT_PHY(phy, a, b) _MMIO_PIPE((phy), (a), (b)) > >> > >> #define BXT_P_CR_GT_DISP_PWRON _MMIO(0x138090) > >> #define GT_DISPLAY_POWER_ON(phy) (1 << (phy)) > >> +#define MIPIO_RST_CTRL (1 << 2) > >> > >> #define _PHY_CTL_FAMILY_EDP 0x64C80 > >> #define _PHY_CTL_FAMILY_DDI 0x64C90 > >> diff --git a/drivers/gpu/drm/i915/intel_dsi.c > >> b/drivers/gpu/drm/i915/intel_dsi.c > >> index b2d2cba..c2aa9e1 100644 > >> --- a/drivers/gpu/drm/i915/intel_dsi.c > >> +++ b/drivers/gpu/drm/i915/intel_dsi.c > >> @@ -549,6 +549,19 @@ static void intel_dsi_pre_enable(struct intel_encoder > >> *encoder) > >> I915_WRITE(DSPCLK_GATE_D, tmp); > >> } > >> > >> + if (IS_BROXTON(dev)) { > >> + /* > >> + * Bring the MIPI IO out of reset and power up > >> + * the DSI regulator. > >> + */ > >> + tmp = I915_READ(BXT_P_CR_GT_DISP_PWRON); > >> + tmp |= MIPIO_RST_CTRL; > >> + I915_WRITE(BXT_P_CR_GT_DISP_PWRON, tmp); > >> + > >> + I915_WRITE(BXT_DSI_CFG, STRAP_SELECT); > >> + I915_WRITE(BXT_DSI_TXCNTRL, HS_IO_CONTROL_SELECT); > >> + } > >> + > >> /* put device in ready state */ > >> intel_dsi_device_ready(encoder); > >> > >> -- > >> 2.7.4 > >> > >> ------------------------------------- > >> [email protected] > >> https://eclists.intel.com/sympa/info/isg-gms > >> Unsubscribe by sending email to [email protected] with subject > >> "Unsubscribe isg-gms" > -- -- Bob Paauwe [email protected] IOTG / PED Software Organization Intel Corp. Folsom, CA (916) 356-6193 _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
