Re: [Intel-gfx] [PATCH v11 01/23] drm/i915/icl: push pll to port mapping/unmapping to ddi encoder hooks

2018-11-30 Thread Madhav Chauhan
ICL DSI encoder needs to do the clock gating and ungating slightly differently, and this allows its own handling in a clean fashion. Looks good to me, Reviewed-by: Madhav Chauhan Regards, Madhav Cc: Madhav Chauhan Cc: Paulo Zanoni Cc: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers

Re: [Intel-gfx] [PATCH v11 17/23] drm/i915/icl: add dummy DSI GPIO element execution function

2018-11-30 Thread Madhav Chauhan
On 11/29/2018 7:42 PM, Jani Nikula wrote: Add dummy debug logging GPIO element execution function for ICL. Looks fine to me. Reviewed-by: Madhav Chauhan Regards, Madhav Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_dsi_vbt.c | 10 +- 1 file changed, 9 insertions

Re: [Intel-gfx] [PATCH v11 20/23] drm/i915/icl: add pll mapping for DSI

2018-11-30 Thread Madhav Chauhan
On 11/29/2018 7:42 PM, Jani Nikula wrote: Add encoder specific pll mapping for DSI. The differences with the DDI version are big enough to warrant a separate function. Cc: Madhav Chauhan Cc: Vandita Kulkarni Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/icl_dsi.c | 24

Re: [Intel-gfx] [PATCH v11 23/23] HACK: drm/i915/bios: ignore VBT not overflowing the mailbox

2018-11-30 Thread Madhav Chauhan
On 11/29/2018 7:42 PM, Jani Nikula wrote: Some machines seem to have a broken opregion where the VBT overflows the mailbox. Ignore this until properly fixed. Right, otherwise DSI modeset doesn't progress further. Acked-by: Madhav Chauhan Regards, Madhav Signed-off-by: Jani N

Re: [Intel-gfx] [PATCH v10 00/15] drm/i915/icl: dsi enabling

2018-11-30 Thread Madhav Chauhan
BR, Jani. [1] https://patchwork.freedesktop.org/series/51011/ Jani Nikula (1): drm/i915/icl: add dummy DSI GPIO element execution function Madhav Chauhan (14): drm/i915/icl: Allocate DSI encoder/connector drm/i915/icl: Fill DSI ports info drm/i915/icl: Allocate DSI hosts and

Re: [Intel-gfx] [PATCH v8 02/38] drm/i915/dsi: move connector mode functions to common file

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan Move DSI connector functions to intel_dsi.c and make them available to both legacy and ICL DSI. v2 by Jani: - Move the functions to intel_dsi.c - Don't reuse intel_dsi_connector_destroy() Patch 1 & 2 v2 changes,

Re: [Intel-gfx] [PATCH v8 03/38] drm/i915/icl: Set max return packet size for DSI panel

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch programs maximum size of the payload transmitted from peripheral back to the host processor using short packet as a part of panel programming. v2: Rebase v3 by Jani: - Add FIXME note. Looks OK to me. Regards

Re: [Intel-gfx] [PATCH v8 05/38] drm/i915/icl: Wait for header/payload credits release

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan Driver needs payload/header credits for sending any command and data over DSI link. These credits are released once command or data sent to link. This patch adds functions to wait for releasing of payload and header credits. As

Re: [Intel-gfx] [PATCH v8 08/38] drm/i915/icl: Disable DSI transcoders

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch disables transcoders by writing to TRANS_CONF registers for each DSI ports. v2 by Jani: - Wait for pipeconf active to go low Thanks for catching this, it has to be low. Regards, Madhav Signed-off-by: Madhav

Re: [Intel-gfx] [PATCH v8 15/38] drm/i915/icl: Define DSI timeout registers

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch defines DSI_HTX_TO, DSI_LRX_H_TO, DSI_PWAIT_TO and DSI_TA_TO registers for DSI transcoders '0' and '1'. They are used for contention recovery on DPHY. v2: Define SHIFT for bitfields. v3 by Jan

Re: [Intel-gfx] [PATCH v8 16/38] drm/i915/icl: Program HS_TX_TIMEOUT/LP_RX_TIMEOUT/TA_TIMEOUT registers

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan Program the timeout values (in escape clock) for HS TX, LP RX and TA timeout. HX TX: Ensure that host does not continuously transmit in the HS state. If this timer expires, then host will gracefully end its HS transmission and

Re: [Intel-gfx] [PATCH v8 17/38] drm/i915/icl: Find DSI presence for ICL

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch detects DSI presence for ICL platform by reading VBT. DSI detection is done while initializing DSI using newly added function intel_gen11_dsi_init. v2 by Jani: - Preserve old behavour of intel_bios_is_dsi_present

Re: [Intel-gfx] [PATCH v8 18/38] drm/i915/icl: Allocate DSI encoder/connector

2018-10-31 Thread Madhav Chauhan
connector. v2 by Jani: - Drop GEN11 prefix from encoder name - Drop extra parenthesis - Drop extra local variable - Squash encoder power domain here Looks good to me. Regards, Madhav Signed-off-by: Madhav Chauhan Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/icl_dsi.c | 96

Re: [Intel-gfx] [PATCH v8 19/38] drm/i915/icl: Allocate hosts for DSI ports

2018-10-31 Thread Madhav Chauhan
structure. v2 by Jani: - indentation Looks fine. Regards, Madhav Signed-off-by: Madhav Chauhan Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/icl_dsi.c | 32 1 file changed, 32 insertions(+) diff --git a/drivers/gpu/drm/i915/icl_dsi.c b/drivers/gpu/drm

Re: [Intel-gfx] [PATCH v8 20/38] drm/i915/icl: Add DSI packet payload/header registers

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch defines payload/header registers for each DSI transcoder used for transmitting DSI packets. v2 by Jani: - Drop full register mask and shift for payload - Use lower case for hex 0x v2 change are fine. Regards

Re: [Intel-gfx] [PATCH v8 21/38] drm/i915/icl: Fetch DSI pkt to be transferred

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch retrieves DSI pkt (from DSI msg) to be sent over DSI link using DRM DSI exported functions. A wrapper function is also added as "DSI host transfer" for sending DSI data/cmd. v2 by Jani: - Use the new credit

Re: [Intel-gfx] [PATCH v8 22/38] drm/i915/icl: Load DSI packet payload to queue

2018-10-31 Thread Madhav Chauhan
v2 changes are fine. Regards, Madhav Signed-off-by: Madhav Chauhan Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/icl_dsi.c | 57 ++ 1 file changed, 57 insertions(+) diff --git a/drivers/gpu/drm/i915/icl_dsi.c b/drivers/gpu/drm/i915/icl_dsi.c

Re: [Intel-gfx] [PATCH v8 24/38] drm/i915/icl: Get HW state for DSI encoder

2018-10-31 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch read out the current hw state for DSI and return true if encoder is active. v2 by Jani: - Squash connector get hw state hook here - Squash encode get hw state fix here Looks fine to me. Regards, Madhav Signed

Re: [Intel-gfx] [PATCH v8 25/38] drm/i915/icl: Add DSI connector functions

2018-11-01 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch assigns connector functions for DSI to DRM connector structure. v2 by Jani: - use common connector destroy hook Looks OK to me. Regards, Madhav Signed-off-by: Madhav Chauhan Signed-off-by: Jani Nikula

Re: [Intel-gfx] [PATCH v8 26/38] drm/i915/icl: Add DSI connector helper functions

2018-11-01 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch registers DSI connectors helper functions with DRM driver. v2 by Jani: - Indentation change Ok. Regards, Madhav Signed-off-by: Madhav Chauhan Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/icl_dsi.c | 7

Re: [Intel-gfx] [PATCH v8 27/38] drm/i915/icl: Add DSI encoder remaining functions

2018-11-01 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch implements compute config and enable function for Gen11 DSI encoder which is required at the time of modeset. Enable function is empty as functionality is implemented inside pre-enable function but still needed otherwise

Re: [Intel-gfx] [PATCH v8 30/38] drm/i915/icl: Configure DSI Dual link mode

2018-11-01 Thread Madhav Chauhan
On 10/30/2018 5:26 PM, Jani Nikula wrote: From: Madhav Chauhan This patch configures DSI video mode dual link by programming DSS_CTL registers. v2: Use new bitfield definitions from Anusha's patch Correct register to be programmed and use max depth buffer value (James) v3 by

Re: [Intel-gfx] [PATCH v8 34/38] drm/i915/icl: Add changes to program DSI panel GPIOs

2018-11-01 Thread Madhav Chauhan
On 10/30/2018 7:31 PM, Ville Syrjälä wrote: On Tue, Oct 30, 2018 at 01:56:40PM +0200, Jani Nikula wrote: From: Madhav Chauhan For ICELAKE DSI, Display Pins are the only GPIOs that need to be programmed. So DSI driver should have its own implementation to toggle these pins based on GPIO info

Re: [Intel-gfx] [PATCH v9 01/15] drm/i915/icl: Allocate DSI encoder/connector

2018-11-01 Thread Madhav Chauhan
On 11/1/2018 9:03 PM, Jani Nikula wrote: From: Madhav Chauhan This patch allocates memory for DSI encoder and connector which will be used for various DSI encoder/connector operations and attaching the same to DRM subsystem. This patch also extracts DSI modes info from VBT and save the desired

Re: [Intel-gfx] [PATCH v9 03/15] drm/i915/icl: Allocate DSI hosts and imlement host transfer

2018-11-01 Thread Madhav Chauhan
f comes in header and payload functionality :). Regards, Madhav Signed-off-by: Madhav Chauhan Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/icl_dsi.c | 147 + 1 file changed, 147 insertions(+) diff --git a/drivers/gpu/drm/i915/icl_dsi.c b/drivers/gp

Re: [Intel-gfx] [PATCH v9 08/15] drm/i915/icl: Consider DSI for getting transcoder state

2018-11-02 Thread Madhav Chauhan
On 11/1/2018 9:04 PM, Jani Nikula wrote: From: Madhav Chauhan For Gen11 DSI, we use similar registers like for eDP to find if DSI encoder is connected or not to a pipe. This patch refactors existing hsw_get_transcoder_state() to handle this. v2 by Jani: - Add WARN_ON(dsi && edp

Re: [Intel-gfx] [PATCH v9 09/15] drm/i915/icl: Get pipe timings for DSI

2018-11-02 Thread Madhav Chauhan
On 11/1/2018 9:04 PM, Jani Nikula wrote: From: Madhav Chauhan Transcoder timings for Gen11 DSI encoder is available at pipe level unlike in older platform where port specific registers need to be accessed. v2 by Jani: - get timings for (!dsi || icl) instead of (dsi && icl).

Re: [Intel-gfx] [PATCH v10 13/15] drm/i915/icl: add dummy DSI GPIO element execution function

2018-11-02 Thread Madhav Chauhan
On 11/2/2018 5:17 PM, Jani Nikula wrote: Add dummy debug logging GPIO element execution function for ICL. Signed-off-by: Jani Nikula Its kinda split of "drm/i915/icl: Add changes to program DSI panel GPIOs" to pave out the way further. Looks good to me. Reviewed-by: Madhav Chauhan

Re: [Intel-gfx] [PATCH v10 06/15] drm/i915/icl: Add DSI encoder remaining functions

2018-11-02 Thread Madhav Chauhan
On 11/2/2018 5:17 PM, Jani Nikula wrote: From: Madhav Chauhan This patch implements compute config for Gen11 DSI encoder which is required at the time of modeset. v2 by Jani: - drop the enable nop hook - fixed_mode is always true - HAS_GMCH_DISPLAY() is always false v3 by Jani: - set

Re: [Intel-gfx] [PATCH v10 05/15] drm/i915/icl: Get HW state for DSI encoder

2018-11-02 Thread Madhav Chauhan
On 11/2/2018 5:17 PM, Jani Nikula wrote: From: Madhav Chauhan This patch read out the current hw state for DSI and return true if encoder is active. v2 by Jani: - Squash connector get hw state hook here - Squash encode get hw state fix here v3 by Jani: - Add encoder->get_power_doma

Re: [Intel-gfx] [PATCH v10 00/15] drm/i915/icl: dsi enabling

2018-11-02 Thread Madhav Chauhan
On 11/2/2018 7:15 PM, Jani Nikula wrote: On Fri, 02 Nov 2018, Jani Nikula wrote: Next version of [1]. Sorry for the spam, needed to get the authorship straight. Fixed power domains and compute config hook initialization. It looks like DDI intel_ddi_get_hw_state() returns true for ICL DSI as we

[Intel-gfx] [PATCH] drm/i915/glk: limit pixel clock to 99% of cdclk workaround

2017-04-05 Thread Madhav Chauhan
-working scenario in commit message Cc: Ander Conselvan de Oliveira Cc: Ville Syrjälä Signed-off-by: Madhav Chauhan Signed-off-by: Jani Nikula Reviewed-by: Ander Conselvan de Oliveira --- drivers/gpu/drm/i915/intel_cdclk.c | 16 +--- 1 file changed, 13 insertions(+), 3 deletions

[Intel-gfx] [PATCH 1/2] drm/i915/glk: Calculate high/low switch count for GLK

2017-05-08 Thread Madhav Chauhan
screen flicker/shift issue while resuming from S3/S4. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/intel_dsi_vbt.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dsi_vbt.c b/drivers/gpu/drm/i915/intel_dsi_vbt.c index 0dce779..7158c7c

[Intel-gfx] [PATCH 2/2] drm/i915/glk: Enable cold boot for GLK DSI

2017-05-08 Thread Madhav Chauhan
As per BSEPC, if device ready bit is '0' in enable IO sequence then its a cold boot/reset scenario eg: S3/S4 resume. In these conditions we need to program certain registers and prepare port from the middle of DSI enable sequence otherwise feature like S3/S4 doesn't work. Signed

[Intel-gfx] [PATCH 2/2] drm/i915/glk: Enable cold boot for GLK DSI

2017-05-09 Thread Madhav Chauhan
As per BSEPC, if device ready bit is '0' in enable IO sequence then its a cold boot/reset scenario eg: S3/S4 resume. In these conditions we need to program certain registers and prepare port from the middle of DSI enable sequence otherwise feature like S3/S4 doesn't work. Signed

[Intel-gfx] [PATCH 1/2] drm/i915/glk: Calculate high/low switch count for GLK

2017-05-09 Thread Madhav Chauhan
screen flicker/shift issue while resuming from S3/S4. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/intel_dsi_vbt.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_dsi_vbt.c b/drivers/gpu/drm/i915/intel_dsi_vbt.c index 0dce779..7158c7c

Re: [Intel-gfx] [PATCH v7 01/23] drm/i915: make encoder enable and disable hooks optional

2018-10-15 Thread Madhav Chauhan
rs_disable().I think, we need to put the check there as well. With that fix, Reviewed-by: Madhav Chauhan Regards, Madhav } diff --git a/drivers/gpu/drm/i915/vlv_dsi.c b/drivers/gpu/drm/i915/vlv_dsi.c index bafeb2a19b90..dbca30460a6b 100644 --- a/drivers/gpu/drm/i915/vlv_dsi.c +++ b/driver

Re: [Intel-gfx] [PATCH v7 02/23] drm/i915/dsi: refactor bitrate calculations in intel_dsi_vbt_init()

2018-10-16 Thread Madhav Chauhan
On 10/15/2018 7:57 PM, Jani Nikula wrote: Abstract bitrate calculation to a newly resurrected intel_dsi.c file that will contain common code for VLV and ICL DSI. No functional changes. Cc: Madhav Chauhan Cc: Ville Syrjala Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/Makefile

Re: [Intel-gfx] [PATCH v7 03/23] drm/i915/dsi: abstract dphy parameter init

2018-10-16 Thread Madhav Chauhan
On 10/15/2018 7:57 PM, Jani Nikula wrote: intel_dsi_vbt_init() has grown too unwieldy, and it's about to be modified due to ICL DSI. Abstract out the VLV specific dphy param init. No functional changes. Intentionally no stylistic changes during code movement. Cc: Madhav Chauhan Cc:

Re: [Intel-gfx] [PATCH v7 04/23] drm/i915/dsi: abstract intel_dsi_tlpx_ns()

2018-10-16 Thread Madhav Chauhan
On 10/15/2018 7:57 PM, Jani Nikula wrote: Will be needed in the future. No functional changes. Agree, will be needing this while setting up DSI protocol timeouts for ICL. Cc: Madhav Chauhan Cc: Ville Syrjala Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/intel_dsi.c | 13

Re: [Intel-gfx] [PATCH v7 05/23] drm/i915/icl: Make common DSI functions available

2018-10-16 Thread Madhav Chauhan
On 10/15/2018 7:57 PM, Jani Nikula wrote: From: Madhav Chauhan This patch moves couple of legacy DSI functions to header and common DSI files so that they can be re-used by Gen11 DSI. No functional change. v2 by Jani: - Move intel_dsi_msleep() to intel_dsi_vbt.c This will be used by icl

Re: [Intel-gfx] [PATCH v7 02/23] drm/i915/dsi: refactor bitrate calculations in intel_dsi_vbt_init()

2018-10-16 Thread Madhav Chauhan
On 10/16/2018 6:14 PM, Jani Nikula wrote: On Tue, 16 Oct 2018, Madhav Chauhan wrote: On 10/15/2018 7:57 PM, Jani Nikula wrote: Abstract bitrate calculation to a newly resurrected intel_dsi.c file that will contain common code for VLV and ICL DSI. No functional changes. Cc: Madhav Chauhan

Re: [Intel-gfx] [PATCH v7 05/23] drm/i915/icl: Make common DSI functions available

2018-10-16 Thread Madhav Chauhan
On 10/16/2018 6:09 PM, Jani Nikula wrote: On Tue, 16 Oct 2018, Madhav Chauhan wrote: On 10/15/2018 7:57 PM, Jani Nikula wrote: From: Madhav Chauhan This patch moves couple of legacy DSI functions to header and common DSI files so that they can be re-used by Gen11 DSI. No functional change

Re: [Intel-gfx] [PATCH v7 04/23] drm/i915/dsi: abstract intel_dsi_tlpx_ns()

2018-10-20 Thread Madhav Chauhan
On 10/16/2018 6:36 PM, Jani Nikula wrote: On Tue, 16 Oct 2018, Madhav Chauhan wrote: On 10/15/2018 7:57 PM, Jani Nikula wrote: Will be needed in the future. No functional changes. Agree, will be needing this while setting up DSI protocol timeouts for ICL. Cc: Madhav Chauhan Cc: Ville

Re: [Intel-gfx] [PATCH v7 06/23] drm/i915/icl: Program DSI clock and data lane timing params

2018-10-20 Thread Madhav Chauhan
On 10/15/2018 7:57 PM, Jani Nikula wrote: From: Madhav Chauhan This patch programs D-PHY timing parameters for the clock and data lane (in escape clocks) of DSI controller (DSI port 0 and 1). These programmed timings would be used by DSI Controller to calculate link transition latencies of the

Re: [Intel-gfx] [PATCH v7 07/23] drm/i915/icl: Program TA_TIMING_PARAM registers

2018-10-20 Thread Madhav Chauhan
On 10/15/2018 7:57 PM, Jani Nikula wrote: From: Madhav Chauhan This patch programs D-PHY timing parameters for the bus turn around flow(in escape clocks) only if dsi link frequency <=800 MHz using DPHY_TA_TIMING_PARAM and its identical register DSI_TA_TIMING_PARAM (inside DSI Controller wit

Re: [Intel-gfx] [PATCH v7 10/23] drm/i915/icl: Define TRANS_DSI_FUNC_CONF register

2018-10-20 Thread Madhav Chauhan
On 10/15/2018 7:57 PM, Jani Nikula wrote: From: Madhav Chauhan This patch defines transcoder function configuration registers and its bitfields for both DSI ports. Used while programming/enabling DSI transcoder. v2: Changes (Jani N) - Define _SHIFT and _MASK for bitfields - Define

Re: [Intel-gfx] [PATCH v7 11/23] drm/i915/icl: Configure DSI transcoders

2018-10-20 Thread Madhav Chauhan
On 10/15/2018 7:57 PM, Jani Nikula wrote: From: Madhav Chauhan This patch programs DSI operation mode, pixel format, BGR info, link calibration etc for the DSI transcoder. This patch also extract BGR info of the DSI panel from VBT and save it inside struct intel_dsi which used for configuring

Re: [Intel-gfx] [PATCH v7 12/23] drm/i915/icl: Define TRANS_DDI_FUNC_CTL DSI registers

2018-10-22 Thread Madhav Chauhan
On 10/15/2018 7:58 PM, Jani Nikula wrote: From: Madhav Chauhan This patch defines TRANS_DDI_FUNC_CTL and TRANS_DDI_FUNC_CTL2 registers and their bitfields for DSI. These registers are used for enabling port sync mode, input pipe select, data lane width configuration etc. v2: Changes

Re: [Intel-gfx] [PATCH v7 13/23] drm/i915/icl: Program TRANS_DDI_FUNC_CTL registers

2018-10-22 Thread Madhav Chauhan
On 10/15/2018 7:58 PM, Jani Nikula wrote: From: Madhav Chauhan This patch select input PIPE for DSI, data lanes width, enable port sync mode and wait for DSI link to become ready. v2 by Jani: - Use MISSING_CASE with fallthrough instead of DRM_ERROR - minor stylistic changes v2 changes

Re: [Intel-gfx] [PATCH v7 14/23] drm/i915/icl: Define DSI transcoder timing registers

2018-10-22 Thread Madhav Chauhan
On 10/15/2018 7:58 PM, Jani Nikula wrote: From: Madhav Chauhan This patch defines registers and bitfields used for programming DSI transcoder's horizontal and vertical timings. v2: Remove TRANS_TIMING_SHIFT definition v3 by Jani: - Group macros by transcoder Separation of DSI0 and

Re: [Intel-gfx] [PATCH v7 15/23] drm/i915/icl: Configure DSI transcoder timings

2018-10-22 Thread Madhav Chauhan
On 10/15/2018 7:58 PM, Jani Nikula wrote: From: Madhav Chauhan As part of DSI enable sequence, transcoder timings (horizontal & vertical) need to be set so that transcoder will generate the stream output as per those timings. This patch set required transcoder timings as per BSPEC. v2: Re

Re: [Intel-gfx] [PATCH v7 16/23] drm/i915/icl: Define TRANS_CONF register for DSI

2018-10-22 Thread Madhav Chauhan
On 10/15/2018 7:58 PM, Jani Nikula wrote: From: Madhav Chauhan This patch defines TRANS_CONF registers for DSI ports 0 and 1. Bitfields of these registers used for enabling and reading the current state of transcoder. v2: Add blank line before comment v3 by Jani: - Move DSI specific

Re: [Intel-gfx] [PATCH v7 17/23] drm/i915/icl: Enable DSI transcoders

2018-10-22 Thread Madhav Chauhan
On 10/15/2018 7:58 PM, Jani Nikula wrote: From: Madhav Chauhan This patch enables DSI transcoders by writing to TRANS_CONF registers and wait for its state to be enabled. v2 by Jani: - Rebase Patch *17-23* changes like rebase, alignment, comments style changes looks good to me. Regards

[Intel-gfx] [PATCH v2 07/20] drm/i915/icl: Define AUX lane registers for Port A/B

2018-07-03 Thread Madhav Chauhan
This patch defines AUX lane registers for PORT_PCS_DW1, PORT_TX_DW2, PORT_TX_DW4, PORT_TX_DW5 used during dsi enabling. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 22 ++ 1 file changed, 22 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b

[Intel-gfx] [PATCH v2 02/20] drm/i915/icl: Program DSI Escape clock Divider

2018-07-03 Thread Madhav Chauhan
Escape Clock is used for LP communication across the DSI Link. To achieve the constant frequency of the escape clock from the variable DPLL frequency output, a variable divider(M) is needed. This patch programs the same. v2: (Jani N) Don't end line with "(". Signed-off-by:

[Intel-gfx] [PATCH v2 04/20] drm/i915/icl: Enable DSI IO power

2018-07-03 Thread Madhav Chauhan
This patch configures mode of operation for DSI and enable DDI IO power by configuring power well. v2: Use for_each_dsi_port() for power get (Jani N) Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/intel_dsi_new.c | 23 +++ 1 file changed, 23 insertions(+) diff

[Intel-gfx] [PATCH v2 13/20] drm/i915/icl: Define data/clock lanes dphy timing registers

2018-07-03 Thread Madhav Chauhan
This patch defines DSI_CLK_TIMING_PARAM, DPHY_CLK_TIMING_PARAM, DSI_DATA_TIMING_PARAM, DPHY_DATA_TIMING_PARAM register used in dphy programming. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 40 1 file changed, 40 insertions

[Intel-gfx] [PATCH v2 09/20] drm/i915/icl: DSI vswing programming sequence

2018-07-03 Thread Madhav Chauhan
This patch setup voltage swing before enabling combo PHY DDI (shared with DSI). Note that DSI voltage swing programming is for high speed data buffers. HW automatically handles the voltage swing for the low power data buffers. v2: Rebase Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915

[Intel-gfx] [PATCH v2 01/20] drm/i915/icl: Define register for DSI PLL

2018-07-03 Thread Madhav Chauhan
This patch adds the new registers and corresponding bit definitions which will be used for programming/enable DSI PLL. v2: Review comments from Jani N - Fix spaces while defining ICL_ESC_CLK_DIV_MASK - Define shift and mask for bitfields. Signed-off-by: Madhav Chauhan Reviewed-by: Jani

[Intel-gfx] [PATCH v2 08/20] drm/i915/icl: Configure lane sequencing of combo phy transmitter

2018-07-03 Thread Madhav Chauhan
This patch set the loadgen select and latency optimization for aux and transmit lanes of combo phy transmitters. It will be used for MIPI DSI HS operations. v2: Rebase Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/intel_dsi_new.c | 38 1 file

[Intel-gfx] [PATCH v2 05/20] drm/i915/icl: Define PORT_CL_DW_10 register

2018-07-03 Thread Madhav Chauhan
- Define PWR_DOWN_LN* macros shifted in place Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 20 1 file changed, 20 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h index dfd603c..3fa8f02 100644 --- a/drive

[Intel-gfx] [PATCH v2 15/20] drm/i915/icl: Define TA_TIMING_PARAM registers

2018-07-03 Thread Madhav Chauhan
This patch defines DSI_TA_TIMING_PARAM and DPHY_TA_TIMING_PARAM registers used in dphy programming. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915

[Intel-gfx] [PATCH v2 10/20] drm/i915/icl: Enable DDI Buffer

2018-07-03 Thread Madhav Chauhan
This patch enables DDI buffer by writing to DDI_BUF_CTL register and wait for DDI status to be *not idle* for a port. v2: Rebase Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/intel_dsi_new.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/drivers/gpu/drm

[Intel-gfx] [PATCH v2 03/20] drm/i915/icl: Define DSI mode ctl register

2018-07-03 Thread Madhav Chauhan
This patch defines DSI IO mode control register and it's bits used while enabling IO power for DSI. Signed-off-by: Madhav Chauhan Reviewed-by: Jani Nikula --- drivers/gpu/drm/i915/i915_reg.h | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/dr

[Intel-gfx] [PATCH v2 00/20] ICELAKE DSI DRIVER

2018-07-03 Thread Madhav Chauhan
and rebase for some other few patches. Madhav Chauhan (20): drm/i915/icl: Define register for DSI PLL drm/i915/icl: Program DSI Escape clock Divider drm/i915/icl: Define DSI mode ctl register drm/i915/icl: Enable DSI IO power drm/i915/icl: Define PORT_CL_DW_10 register drm/i915/icl

[Intel-gfx] [PATCH v2 14/20] drm/i915/icl: Program DSI clock and data lane timing params

2018-07-03 Thread Madhav Chauhan
This patch programs D-PHY timing parameters for the clock and data lane (in escape clocks) of DSI controller (DSI port 0 and 1). These programmed timings would be used by DSI Controller to calculate link transition latencies of the data and clock lanes. Signed-off-by: Madhav Chauhan --- drivers

[Intel-gfx] [PATCH v2 11/20] drm/i915/icl: Define T_INIT_MASTER registers

2018-07-03 Thread Madhav Chauhan
This patch defines DSI_T_INIT_MASTER register for DSI ports 0/1 which will be used in dphy programming. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h

[Intel-gfx] [PATCH v2 19/20] drm/i915/icl: Define TRANS_DSI_FUNC_CONF register

2018-07-03 Thread Madhav Chauhan
This patch defines transcoder function configuration registers and its bitfields for both DSI ports. Used while programming/enabling DSI transcoder. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 47 + 1 file changed, 47 insertions

[Intel-gfx] [PATCH v2 12/20] drm/i915/icl: Program T_INIT_MASTER registers

2018-07-03 Thread Madhav Chauhan
This patch programs the time (in escape clocks) to drive the link in the initialization (i.e. LP-11) state. v2: Rebase Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/intel_dsi_new.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/drivers/gpu/drm/i915

[Intel-gfx] [PATCH v2 06/20] drm/i915/icl: Power down unused DSI lanes

2018-07-03 Thread Madhav Chauhan
To save power, unused lanes should be powered down using the bitfield of PORT_CL_DW10. v2: Review comments from Jani N - Put default label next to case 4 - Include the shifts in the macros Signed-off-by: Madhav Chauhan Reviewed-by: Jani Nikula --- drivers/gpu/drm/i915/intel_dsi_new.c

[Intel-gfx] [PATCH v2 20/20] drm/i915/icl: Configure DSI transcoders

2018-07-03 Thread Madhav Chauhan
This patch programs DSI operation mode, pixel format, BGR info, link calibration etc for the DSI transcoder. This patch also extract BGR info of the DSI panel from VBT and save it inside struct intel_dsi which used for configuring DSI transcoder. v2: Rebase Signed-off-by: Madhav Chauhan

[Intel-gfx] [PATCH v2 17/20] drm/i915/icl: Get DSI transcoder for a given port

2018-07-03 Thread Madhav Chauhan
This patch adds a helper function to retrieve DSI transcoder for a given DSI port using newly defined enum names for DSI transcoders. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/intel_display.h | 6 -- drivers/gpu/drm/i915/intel_dsi_new.c | 9 + 2 files changed, 13

[Intel-gfx] [PATCH v2 16/20] drm/i915/icl: Program TA_TIMING_PARAM registers

2018-07-03 Thread Madhav Chauhan
This patch programs D-PHY timing parameters for the bus turn around flow(in escape clocks) only if dsi link frequency <=800 MHz using DPHY_TA_TIMING_PARAM and its identical register DSI_TA_TIMING_PARAM (inside DSI Controller within the Display Core). Signed-off-by: Madhav Chauhan --- driv

[Intel-gfx] [PATCH v2 18/20] drm/i915/icl: Add macros for MMIO of DSI transcoder registers

2018-07-03 Thread Madhav Chauhan
This patch adds _MMIO_DSI and _DSI_TRANS macros for accessing DSI transcoder registers. Credits-to: Jani N Cc: Jani Nikula Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu

[Intel-gfx] [PATCH v3 00/20] ICELAKE DSI DRIVER

2018-07-05 Thread Madhav Chauhan
and rebase for some other few patches. v3: Renamed intel_dsi_new.c to gen11_dsi.c as per discussion with Jani, Daniel, Ville. Also addressed review comments for couple of patches. Madhav Chauhan (20): drm/i915/icl: Define register for DSI PLL drm/i915/icl: Program DSI Escape clock Divider

[Intel-gfx] [PATCH v3 03/20] drm/i915/icl: Define DSI mode ctl register

2018-07-05 Thread Madhav Chauhan
This patch defines DSI IO mode control register and it's bits used while enabling IO power for DSI. Signed-off-by: Madhav Chauhan Reviewed-by: Jani Nikula --- drivers/gpu/drm/i915/i915_reg.h | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/dr

[Intel-gfx] [PATCH v3 01/20] drm/i915/icl: Define register for DSI PLL

2018-07-05 Thread Madhav Chauhan
This patch adds the new registers and corresponding bit definitions which will be used for programming/enable DSI PLL. v2: Review comments from Jani N - Fix spaces while defining ICL_ESC_CLK_DIV_MASK - Define shift and mask for bitfields. Signed-off-by: Madhav Chauhan Reviewed-by: Jani

[Intel-gfx] [PATCH v3 04/20] drm/i915/icl: Enable DSI IO power

2018-07-05 Thread Madhav Chauhan
This patch configures mode of operation for DSI and enable DDI IO power by configuring power well. v2: Use for_each_dsi_port() for power get (Jani N) Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/gen11_dsi.c | 23 +++ 1 file changed, 23 insertions(+) diff --git a

[Intel-gfx] [PATCH v3 05/20] drm/i915/icl: Define PORT_CL_DW_10 register

2018-07-05 Thread Madhav Chauhan
- Define PWR_DOWN_LN* macros shifted in place v3: Correct PWR_DOWN_LN_MASK value (Jani N) Signed-off-by: Madhav Chauhan Reviewed-by: Jani Nikula --- drivers/gpu/drm/i915/i915_reg.h | 20 1 file changed, 20 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drive

[Intel-gfx] [PATCH v3 02/20] drm/i915/icl: Program DSI Escape clock Divider

2018-07-05 Thread Madhav Chauhan
Escape Clock is used for LP communication across the DSI Link. To achieve the constant frequency of the escape clock from the variable DPLL frequency output, a variable divider(M) is needed. This patch programs the same. v2: (Jani N) Don't end line with "(". Signed-off-by:

[Intel-gfx] [PATCH v3 08/20] drm/i915/icl: Configure lane sequencing of combo phy transmitter

2018-07-05 Thread Madhav Chauhan
This patch set the loadgen select and latency optimization for aux and transmit lanes of combo phy transmitters. It will be used for MIPI DSI HS operations. v2: Rebase Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/gen11_dsi.c | 38 ++ 1 file changed

[Intel-gfx] [PATCH v3 09/20] drm/i915/icl: DSI vswing programming sequence

2018-07-05 Thread Madhav Chauhan
This patch setup voltage swing before enabling combo PHY DDI (shared with DSI). Note that DSI voltage swing programming is for high speed data buffers. HW automatically handles the voltage swing for the low power data buffers. v2: Rebase Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915

[Intel-gfx] [PATCH v3 07/20] drm/i915/icl: Define AUX lane registers for Port A/B

2018-07-05 Thread Madhav Chauhan
This patch defines AUX lane registers for PORT_PCS_DW1, PORT_TX_DW2, PORT_TX_DW4, PORT_TX_DW5 used during dsi enabling. v2: Review comments from Jani N: - Define _ICL_PORT_PCS_DW1_AUX_A for consistency - Three spaces for bitfield definition. Signed-off-by: Madhav Chauhan Reviewed-by

[Intel-gfx] [PATCH v3 10/20] drm/i915/icl: Enable DDI Buffer

2018-07-05 Thread Madhav Chauhan
This patch enables DDI buffer by writing to DDI_BUF_CTL register and wait for DDI status to be *not idle* for a port. v2: Rebase Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/gen11_dsi.c | 22 ++ 1 file changed, 22 insertions(+) diff --git a/drivers/gpu/drm/i915

[Intel-gfx] [PATCH v3 06/20] drm/i915/icl: Power down unused DSI lanes

2018-07-05 Thread Madhav Chauhan
To save power, unused lanes should be powered down using the bitfield of PORT_CL_DW10. v2: Review comments from Jani N - Put default label next to case 4 - Include the shifts in the macros Signed-off-by: Madhav Chauhan Reviewed-by: Jani Nikula --- drivers/gpu/drm/i915/gen11_dsi.c | 40

[Intel-gfx] [PATCH v3 12/20] drm/i915/icl: Program T_INIT_MASTER registers

2018-07-05 Thread Madhav Chauhan
This patch programs the time (in escape clocks) to drive the link in the initialization (i.e. LP-11) state. v2: Rebase Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/gen11_dsi.c | 19 +++ 1 file changed, 19 insertions(+) diff --git a/drivers/gpu/drm/i915/gen11_dsi.c b

[Intel-gfx] [PATCH v3 11/20] drm/i915/icl: Define T_INIT_MASTER registers

2018-07-05 Thread Madhav Chauhan
This patch defines DSI_T_INIT_MASTER register for DSI ports 0/1 which will be used in dphy programming. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h

[Intel-gfx] [PATCH v3 13/20] drm/i915/icl: Define data/clock lanes dphy timing registers

2018-07-05 Thread Madhav Chauhan
This patch defines DSI_CLK_TIMING_PARAM, DPHY_CLK_TIMING_PARAM, DSI_DATA_TIMING_PARAM, DPHY_DATA_TIMING_PARAM register used in dphy programming. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 40 1 file changed, 40 insertions

[Intel-gfx] [PATCH v3 15/20] drm/i915/icl: Define TA_TIMING_PARAM registers

2018-07-05 Thread Madhav Chauhan
This patch defines DSI_TA_TIMING_PARAM and DPHY_TA_TIMING_PARAM registers used in dphy programming. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 14 ++ 1 file changed, 14 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915

[Intel-gfx] [PATCH v3 18/20] drm/i915/icl: Add macros for MMIO of DSI transcoder registers

2018-07-05 Thread Madhav Chauhan
This patch adds _MMIO_DSI and _DSI_TRANS macros for accessing DSI transcoder registers. Credits-to: Jani N Cc: Jani Nikula Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu

[Intel-gfx] [PATCH v3 14/20] drm/i915/icl: Program DSI clock and data lane timing params

2018-07-05 Thread Madhav Chauhan
This patch programs D-PHY timing parameters for the clock and data lane (in escape clocks) of DSI controller (DSI port 0 and 1). These programmed timings would be used by DSI Controller to calculate link transition latencies of the data and clock lanes. Signed-off-by: Madhav Chauhan --- drivers

[Intel-gfx] [PATCH v3 16/20] drm/i915/icl: Program TA_TIMING_PARAM registers

2018-07-05 Thread Madhav Chauhan
This patch programs D-PHY timing parameters for the bus turn around flow(in escape clocks) only if dsi link frequency <=800 MHz using DPHY_TA_TIMING_PARAM and its identical register DSI_TA_TIMING_PARAM (inside DSI Controller within the Display Core). Signed-off-by: Madhav Chauhan --- driv

[Intel-gfx] [PATCH v3 17/20] drm/i915/icl: Get DSI transcoder for a given port

2018-07-05 Thread Madhav Chauhan
This patch adds a helper function to retrieve DSI transcoder for a given DSI port using newly defined enum names for DSI transcoders. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/gen11_dsi.c | 9 + drivers/gpu/drm/i915/intel_display.h | 6 -- 2 files changed, 13

[Intel-gfx] [PATCH v3 19/20] drm/i915/icl: Define TRANS_DSI_FUNC_CONF register

2018-07-05 Thread Madhav Chauhan
This patch defines transcoder function configuration registers and its bitfields for both DSI ports. Used while programming/enabling DSI transcoder. Signed-off-by: Madhav Chauhan --- drivers/gpu/drm/i915/i915_reg.h | 47 + 1 file changed, 47 insertions

[Intel-gfx] [PATCH v3 20/20] drm/i915/icl: Configure DSI transcoders

2018-07-05 Thread Madhav Chauhan
This patch programs DSI operation mode, pixel format, BGR info, link calibration etc for the DSI transcoder. This patch also extract BGR info of the DSI panel from VBT and save it inside struct intel_dsi which used for configuring DSI transcoder. v2: Rebase Signed-off-by: Madhav Chauhan

[Intel-gfx] [PATCH v4 00/20] ICELAKE DSI DRIVER

2018-07-05 Thread Madhav Chauhan
and rebase for some other few patches. v3: Renamed intel_dsi_new.c to gen11_dsi.c as per discussion with Jani, Daniel, Ville. Also addressed review comments for couple of patches. v4: Rename gen11_dsi.c to icl_dsi.c (Ville). No functional changes. Madhav Chauhan (20): drm/i915/icl: Define

[Intel-gfx] [PATCH v4 01/20] drm/i915/icl: Define register for DSI PLL

2018-07-05 Thread Madhav Chauhan
This patch adds the new registers and corresponding bit definitions which will be used for programming/enable DSI PLL. v2: Review comments from Jani N - Fix spaces while defining ICL_ESC_CLK_DIV_MASK - Define shift and mask for bitfields. Signed-off-by: Madhav Chauhan Reviewed-by: Jani

[Intel-gfx] [PATCH v4 02/20] drm/i915/icl: Program DSI Escape clock Divider

2018-07-05 Thread Madhav Chauhan
Escape Clock is used for LP communication across the DSI Link. To achieve the constant frequency of the escape clock from the variable DPLL frequency output, a variable divider(M) is needed. This patch programs the same. v2: (Jani N) Don't end line with "(". Signed-off-by:

[Intel-gfx] [PATCH v4 03/20] drm/i915/icl: Define DSI mode ctl register

2018-07-05 Thread Madhav Chauhan
This patch defines DSI IO mode control register and it's bits used while enabling IO power for DSI. Signed-off-by: Madhav Chauhan Reviewed-by: Jani Nikula --- drivers/gpu/drm/i915/i915_reg.h | 8 1 file changed, 8 insertions(+) diff --git a/drivers/gpu/drm/i915/i915_reg.h b/dr

  1   2   3   >