Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-03 Thread Jani Nikula
On Thu, 29 Nov 2018, Jani Nikula  wrote:
> v11 of [1], incorporating DSI PLL work [2] from Vandita as well as PLL
> mapping and gating patches [3] from me and [4] from Imre.
>
> It made sense to squash some patches in [1] and [2] together, I've tried
> to set authorship and co-developed-by tags fairly.
>
> The series is also available in icl-dsi-2018-11-29 branch of my fdo git
> repo [5].

Pushed the series to dinq except for the three HACK patches at the
end. They'll still need to be addressed one way or another.

Thanks everyone for your contributions in writing the patches,
reviewing, testing, etc. It's been a long ride!

BR,
Jani.



>
>
> BR,
> Jani.
>
>
> [1] https://patchwork.freedesktop.org/series/51011/
> [2] https://patchwork.freedesktop.org/series/51373/
> [3] 
> http://patchwork.freedesktop.org/patch/msgid/20181129115715.9152-1-jani.nik...@intel.com
> [4] 
> http://patchwork.freedesktop.org/patch/msgid/20181127163606.28841-1-imre.d...@intel.com
> [5] https://cgit.freedesktop.org/~jani/drm/
>
>
> Imre Deak (1):
>   drm/i915/icl: Sanitize DDI port clock gating for DSI ports
>
> Jani Nikula (4):
>   drm/i915/icl: push pll to port mapping/unmapping to ddi encoder hooks
>   drm/i915/icl: add dummy DSI GPIO element execution function
>   drm/i915/icl: add pll mapping for DSI
>   HACK: drm/i915/bios: ignore VBT not overflowing the mailbox
>
> Madhav Chauhan (16):
>   drm/i915/icl: Calculate DPLL params for DSI
>   drm/i915/icl: Allocate DSI encoder/connector
>   drm/i915/icl: Fill DSI ports info
>   drm/i915/icl: Allocate DSI hosts and imlement host transfer
>   drm/i915/icl: Get HW state for DSI encoder
>   drm/i915/icl: Add DSI encoder compute config hook
>   drm/i915/icl: Configure DSI Dual link mode
>   drm/i915/icl: Consider DSI for getting transcoder state
>   drm/i915/icl: Get pipe timings for DSI
>   drm/i915/icl: Define missing bitfield for shortplug reg
>   drm/i915/icl: Define Panel power ctrl register
>   drm/i915/icl: Define display GPIO pins for DSI
>   drm/i915/icl: Gate clocks for DSI
>   drm/i915/icl: Ungate DSI clocks
>   HACK: drm/i915/icl: Add changes to program DSI panel GPIOs
>   HACK: drm/i915/icl: Configure backlight functions for DSI
>
> Vandita Kulkarni (2):
>   drm/i915/icl: Use the same pll functions for dsi
>   drm/i915/icl: Add get config functionality for DSI
>
>  drivers/gpu/drm/i915/i915_reg.h   |  12 +
>  drivers/gpu/drm/i915/icl_dsi.c| 492 
> +-
>  drivers/gpu/drm/i915/intel_bios.c |   1 -
>  drivers/gpu/drm/i915/intel_ddi.c  | 153 ++-
>  drivers/gpu/drm/i915/intel_display.c  |  44 +--
>  drivers/gpu/drm/i915/intel_dpll_mgr.c |   3 +-
>  drivers/gpu/drm/i915/intel_drv.h  |   8 +-
>  drivers/gpu/drm/i915/intel_dsi.h  |   5 +
>  drivers/gpu/drm/i915/intel_dsi_vbt.c  |  58 +++-
>  drivers/gpu/drm/i915/intel_panel.c|   3 +-
>  10 files changed, 674 insertions(+), 105 deletions(-)

-- 
Jani Nikula, Intel Open Source Graphics Center
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-03 Thread Chauhan, Madhav
> -Original Message-
> From: Nikula, Jani
> Sent: Monday, December 3, 2018 7:39 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Chauhan, Madhav
> ; Kulkarni, Vandita
> ; Lisovskiy, Stanislav
> ; Deak, Imre 
> Subject: Re: [PATCH v11 00/23] drm/i915/icl: dsi enabling
> 
> On Thu, 29 Nov 2018, Jani Nikula  wrote:
> > v11 of [1], incorporating DSI PLL work [2] from Vandita as well as PLL
> > mapping and gating patches [3] from me and [4] from Imre.
> >
> > It made sense to squash some patches in [1] and [2] together, I've
> > tried to set authorship and co-developed-by tags fairly.
> >
> > The series is also available in icl-dsi-2018-11-29 branch of my fdo
> > git repo [5].
> 
> Pushed the series to dinq except for the three HACK patches at the end.
> They'll still need to be addressed one way or another.
> 
> Thanks everyone for your contributions in writing the patches, reviewing,
> testing, etc. It's been a long ride!

Thanks a lot to you as well Jani N for your continuous support during 
design/development/review/merge
Of this new/big feature :)

Regards,
Madhav

> 
> BR,
> Jani.
> 
> 
> 
> >
> >
> > BR,
> > Jani.
> >
> >
> > [1] https://patchwork.freedesktop.org/series/51011/
> > [2] https://patchwork.freedesktop.org/series/51373/
> > [3]
> > http://patchwork.freedesktop.org/patch/msgid/20181129115715.9152-1-
> jan
> > i.nik...@intel.com [4]
> > http://patchwork.freedesktop.org/patch/msgid/20181127163606.28841-1-
> im
> > re.d...@intel.com [5] https://cgit.freedesktop.org/~jani/drm/
> >
> >
> > Imre Deak (1):
> >   drm/i915/icl: Sanitize DDI port clock gating for DSI ports
> >
> > Jani Nikula (4):
> >   drm/i915/icl: push pll to port mapping/unmapping to ddi encoder hooks
> >   drm/i915/icl: add dummy DSI GPIO element execution function
> >   drm/i915/icl: add pll mapping for DSI
> >   HACK: drm/i915/bios: ignore VBT not overflowing the mailbox
> >
> > Madhav Chauhan (16):
> >   drm/i915/icl: Calculate DPLL params for DSI
> >   drm/i915/icl: Allocate DSI encoder/connector
> >   drm/i915/icl: Fill DSI ports info
> >   drm/i915/icl: Allocate DSI hosts and imlement host transfer
> >   drm/i915/icl: Get HW state for DSI encoder
> >   drm/i915/icl: Add DSI encoder compute config hook
> >   drm/i915/icl: Configure DSI Dual link mode
> >   drm/i915/icl: Consider DSI for getting transcoder state
> >   drm/i915/icl: Get pipe timings for DSI
> >   drm/i915/icl: Define missing bitfield for shortplug reg
> >   drm/i915/icl: Define Panel power ctrl register
> >   drm/i915/icl: Define display GPIO pins for DSI
> >   drm/i915/icl: Gate clocks for DSI
> >   drm/i915/icl: Ungate DSI clocks
> >   HACK: drm/i915/icl: Add changes to program DSI panel GPIOs
> >   HACK: drm/i915/icl: Configure backlight functions for DSI
> >
> > Vandita Kulkarni (2):
> >   drm/i915/icl: Use the same pll functions for dsi
> >   drm/i915/icl: Add get config functionality for DSI
> >
> >  drivers/gpu/drm/i915/i915_reg.h   |  12 +
> >  drivers/gpu/drm/i915/icl_dsi.c| 492
> +-
> >  drivers/gpu/drm/i915/intel_bios.c |   1 -
> >  drivers/gpu/drm/i915/intel_ddi.c  | 153 ++-
> >  drivers/gpu/drm/i915/intel_display.c  |  44 +--
> >  drivers/gpu/drm/i915/intel_dpll_mgr.c |   3 +-
> >  drivers/gpu/drm/i915/intel_drv.h  |   8 +-
> >  drivers/gpu/drm/i915/intel_dsi.h  |   5 +
> >  drivers/gpu/drm/i915/intel_dsi_vbt.c  |  58 +++-
> >  drivers/gpu/drm/i915/intel_panel.c|   3 +-
> >  10 files changed, 674 insertions(+), 105 deletions(-)
> 
> --
> Jani Nikula, Intel Open Source Graphics Center
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx


Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-04 Thread Lisovskiy, Stanislav
Hi,

Currently ICL DSI panel seems to work fine, however I still face mainly two 
issues,
which probably need to be addressed:

1) There is still pipe_config mismatch assertion:

[   13.119965] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
[   13.119989] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_htotal (expected 1586, found 793)
[   13.120015] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
[   13.120038] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
[   13.120061] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
[   13.120083] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
[   13.120113] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
[   13.120139] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
[   13.120169] [drm:pipe_config_err [i915]] *ERROR* mismatch in output_format 
(expected 0, found 1)
[   13.120188] [drm:pipe_config_err [i915]] *ERROR* mismatch in pixel_rate 
(expected 245700, found 122850)
[   13.120207] [drm:pipe_config_err [i915]] *ERROR* mismatch in pipe_bpp 
(expected 24, found 0)
[   13.120225] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_clock (expected 245700, found 122850)

Also whenever I run some suspend test case(for example I use  
igt@kms_vblank@pipe-a-ts-continuation-suspend
I start to get it everytime machine wakes up, DSI panel get blank and doesn't 
recover from that.

2) During reboot, there are sometimes flood of "The master control interrupt 
lied (DE PIPE)!" messages visible.
Looks like GEN8_DE_PIPE_IIR(pipe) which reads as 0 toggles this. Looks like 
also if I add a few retries it reads
correctly.

Could this be also because I'm still using old BIOS, which I've got initially 
from Vandita?

Best Regards,

Lisovskiy Stanislav

Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo


From: Chauhan, Madhav
Sent: Tuesday, December 04, 2018 9:40 AM
To: Nikula, Jani; intel-gfx@lists.freedesktop.org
Cc: ville.syrj...@linux.intel.com; Kulkarni, Vandita; Lisovskiy, Stanislav; 
Deak, Imre
Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling

> -Original Message-
> From: Nikula, Jani
> Sent: Monday, December 3, 2018 7:39 PM
> To: intel-gfx@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Chauhan, Madhav
> ; Kulkarni, Vandita
> ; Lisovskiy, Stanislav
> ; Deak, Imre 
> Subject: Re: [PATCH v11 00/23] drm/i915/icl: dsi enabling
>
> On Thu, 29 Nov 2018, Jani Nikula  wrote:
> > v11 of [1], incorporating DSI PLL work [2] from Vandita as well as PLL
> > mapping and gating patches [3] from me and [4] from Imre.
> >
> > It made sense to squash some patches in [1] and [2] together, I've
> > tried to set authorship and co-developed-by tags fairly.
> >
> > The series is also available in icl-dsi-2018-11-29 branch of my fdo
> > git repo [5].
>
> Pushed the series to dinq except for the three HACK patches at the end.
> They'll still need to be addressed one way or another.
>
> Thanks everyone for your contributions in writing the patches, reviewing,
> testing, etc. It's been a long ride!

Thanks a lot to you as well Jani N for your continuous support during 
design/development/review/merge
Of this new/big feature :)

Regards,
Madhav

>
> BR,
> Jani.
>
>
>
> >
> >
> > BR,
> > Jani.
> >
> >
> > [1] https://patchwork.freedesktop.org/series/51011/
> > [2] https://patchwork.freedesktop.org/series/51373/
> > [3]
> > http://patchwork.freedesktop.org/patch/msgid/20181129115715.9152-1-
> jan
> > i.nik...@intel.com [4]
> > http://patchwork.freedesktop.org/patch/msgid/20181127163606.28841-1-
> im
> > re.d...@intel.com [5] https://cgit.freedesktop.org/~jani/drm/
> >
> >
> > Imre Deak (1):
> >   drm/i915/icl: Sanitize DDI port clock gating for DSI ports
> >
> > Jani Nikula (4):
> >   drm/i915/icl: push pll to port mapping/unmapping to ddi encoder hooks
> >   drm/i915/icl: add dummy DSI GPIO element execution function
> >   drm/i915/icl: add pll mapping for DSI
> >   HACK: drm/i915/bios: ignore VBT not overflowing the mailbox
> >
> > Madhav Chauhan (16):
> >   drm/i915/icl: Calculate DPLL params for DSI
> >   drm/i915/icl: Allocate DSI encoder/connector
> >   drm/i915/icl: Fill DSI ports info
> >   drm/i915/icl: Allocate DSI hosts and imlement host transfer
> >   drm/i915/icl: Get HW state for DSI encoder
> >   drm/i915/icl: Add DSI encoder compute config hook
> >   drm/i915/icl: Configure DSI Dual link mode
> >   drm/i915/icl: Consider DSI for getting transcoder state
> >   drm/i915/icl: Get pipe timings for DSI
> >   drm/i915/i

Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-04 Thread Jani Nikula
On Tue, 04 Dec 2018, "Lisovskiy, Stanislav"  
wrote:
> Hi,
>
> Currently ICL DSI panel seems to work fine, however I still face
> mainly two issues, which probably need to be addressed:

Please try with current drm-tip with

commit 0716931a82b4d0e211d2ef66616ad7130107e455
Author: Jani Nikula 
Date:   Tue Dec 4 12:19:26 2018 +0200

drm/i915/icl: fix transcoder state readout

plus the hack patches from the end of the series. It's possible only the
VBT one is required.

BR,
Jani.

>
> 1) There is still pipe_config mismatch assertion:
>
> [   13.119965] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
> [   13.119989] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_htotal (expected 1586, found 793)
> [   13.120015] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
> [   13.120038] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
> [   13.120061] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
> [   13.120083] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
> [   13.120113] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
> [   13.120139] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
> [   13.120169] [drm:pipe_config_err [i915]] *ERROR* mismatch in output_format 
> (expected 0, found 1)
> [   13.120188] [drm:pipe_config_err [i915]] *ERROR* mismatch in pixel_rate 
> (expected 245700, found 122850)
> [   13.120207] [drm:pipe_config_err [i915]] *ERROR* mismatch in pipe_bpp 
> (expected 24, found 0)
> [   13.120225] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_clock (expected 245700, found 122850)
>
> Also whenever I run some suspend test case(for example I use  
> igt@kms_vblank@pipe-a-ts-continuation-suspend
> I start to get it everytime machine wakes up, DSI panel get blank and doesn't 
> recover from that.
>
> 2) During reboot, there are sometimes flood of "The master control interrupt 
> lied (DE PIPE)!" messages visible.
> Looks like GEN8_DE_PIPE_IIR(pipe) which reads as 0 toggles this. Looks like 
> also if I add a few retries it reads
> correctly.
>
> Could this be also because I'm still using old BIOS, which I've got initially 
> from Vandita?
>
> Best Regards,
>
> Lisovskiy Stanislav
>
> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
>
> 
> From: Chauhan, Madhav
> Sent: Tuesday, December 04, 2018 9:40 AM
> To: Nikula, Jani; intel-gfx@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Kulkarni, Vandita; Lisovskiy, Stanislav; 
> Deak, Imre
> Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling
>
>> -Original Message-
>> From: Nikula, Jani
>> Sent: Monday, December 3, 2018 7:39 PM
>> To: intel-gfx@lists.freedesktop.org
>> Cc: ville.syrj...@linux.intel.com; Chauhan, Madhav
>> ; Kulkarni, Vandita
>> ; Lisovskiy, Stanislav
>> ; Deak, Imre 
>> Subject: Re: [PATCH v11 00/23] drm/i915/icl: dsi enabling
>>
>> On Thu, 29 Nov 2018, Jani Nikula  wrote:
>> > v11 of [1], incorporating DSI PLL work [2] from Vandita as well as PLL
>> > mapping and gating patches [3] from me and [4] from Imre.
>> >
>> > It made sense to squash some patches in [1] and [2] together, I've
>> > tried to set authorship and co-developed-by tags fairly.
>> >
>> > The series is also available in icl-dsi-2018-11-29 branch of my fdo
>> > git repo [5].
>>
>> Pushed the series to dinq except for the three HACK patches at the end.
>> They'll still need to be addressed one way or another.
>>
>> Thanks everyone for your contributions in writing the patches, reviewing,
>> testing, etc. It's been a long ride!
>
> Thanks a lot to you as well Jani N for your continuous support during 
> design/development/review/merge
> Of this new/big feature :)
>
> Regards,
> Madhav
>
>>
>> BR,
>> Jani.
>>
>>
>>
>> >
>> >
>> > BR,
>> > Jani.
>> >
>> >
>> > [1] https://patchwork.freedesktop.org/series/51011/
>> > [2] https://patchwork.freedesktop.org/series/51373/
>> > [3]
>> > http://patchwork.freedesktop.org/patch/msgid/20181129115715.9152-1-
>> jan
>> > i.nik...@intel.com [4]
>> > http://patchwork.freedesktop.org/patch/msgid/20181127163606.28841-1-
>> im
>> > re.d...@intel.com [5] https://cgit.freedesktop.org/~jani/drm/
>> >
>> >
>> > Imre Deak (1):
>> >   drm/i915/icl: Sanitize DDI port clock gating for DSI ports
>> >
>> > Jani Nikula (4):
>> >   drm/i915/icl: push pll to port mapping/unmapping to ddi encoder hooks
>> >   drm/i915/icl: add dummy DSI GPIO element execution function
>> >   drm/i915/icl: add pll mapping for DSI
>> >   HACK: drm/i915/bios: ignore VBT not overfl

Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-04 Thread Lisovskiy, Stanislav
Hi Jani,

I've tried previously with branch icl-dsi-2018-12-03 for your github repo. 
I think it has everything except this 4.12.2018  "fix transcoder state readout" 
commit.

I will apply it and try with that now, thanks.

Best Regards,

Lisovskiy Stanislav

Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo


From: Nikula, Jani
Sent: Tuesday, December 04, 2018 7:13 PM
To: Lisovskiy, Stanislav; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
Cc: ville.syrj...@linux.intel.com; Kulkarni, Vandita; Deak, Imre
Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling

On Tue, 04 Dec 2018, "Lisovskiy, Stanislav"  
wrote:
> Hi,
>
> Currently ICL DSI panel seems to work fine, however I still face
> mainly two issues, which probably need to be addressed:

Please try with current drm-tip with

commit 0716931a82b4d0e211d2ef66616ad7130107e455
Author: Jani Nikula 
Date:   Tue Dec 4 12:19:26 2018 +0200

drm/i915/icl: fix transcoder state readout

plus the hack patches from the end of the series. It's possible only the
VBT one is required.

BR,
Jani.

>
> 1) There is still pipe_config mismatch assertion:
>
> [   13.119965] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
> [   13.119989] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_htotal (expected 1586, found 793)
> [   13.120015] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
> [   13.120038] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
> [   13.120061] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
> [   13.120083] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
> [   13.120113] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
> [   13.120139] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
> [   13.120169] [drm:pipe_config_err [i915]] *ERROR* mismatch in output_format 
> (expected 0, found 1)
> [   13.120188] [drm:pipe_config_err [i915]] *ERROR* mismatch in pixel_rate 
> (expected 245700, found 122850)
> [   13.120207] [drm:pipe_config_err [i915]] *ERROR* mismatch in pipe_bpp 
> (expected 24, found 0)
> [   13.120225] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_clock (expected 245700, found 122850)
>
> Also whenever I run some suspend test case(for example I use  
> igt@kms_vblank@pipe-a-ts-continuation-suspend
> I start to get it everytime machine wakes up, DSI panel get blank and doesn't 
> recover from that.
>
> 2) During reboot, there are sometimes flood of "The master control interrupt 
> lied (DE PIPE)!" messages visible.
> Looks like GEN8_DE_PIPE_IIR(pipe) which reads as 0 toggles this. Looks like 
> also if I add a few retries it reads
> correctly.
>
> Could this be also because I'm still using old BIOS, which I've got initially 
> from Vandita?
>
> Best Regards,
>
> Lisovskiy Stanislav
>
> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
>
> 
> From: Chauhan, Madhav
> Sent: Tuesday, December 04, 2018 9:40 AM
> To: Nikula, Jani; intel-gfx@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Kulkarni, Vandita; Lisovskiy, Stanislav; 
> Deak, Imre
> Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling
>
>> -Original Message-
>> From: Nikula, Jani
>> Sent: Monday, December 3, 2018 7:39 PM
>> To: intel-gfx@lists.freedesktop.org
>> Cc: ville.syrj...@linux.intel.com; Chauhan, Madhav
>> ; Kulkarni, Vandita
>> ; Lisovskiy, Stanislav
>> ; Deak, Imre 
>> Subject: Re: [PATCH v11 00/23] drm/i915/icl: dsi enabling
>>
>> On Thu, 29 Nov 2018, Jani Nikula  wrote:
>> > v11 of [1], incorporating DSI PLL work [2] from Vandita as well as PLL
>> > mapping and gating patches [3] from me and [4] from Imre.
>> >
>> > It made sense to squash some patches in [1] and [2] together, I've
>> > tried to set authorship and co-developed-by tags fairly.
>> >
>> > The series is also available in icl-dsi-2018-11-29 branch of my fdo
>> > git repo [5].
>>
>> Pushed the series to dinq except for the three HACK patches at the end.
>> They'll still need to be addressed one way or another.
>>
>> Thanks everyone for your contributions in writing the patches, reviewing,
>> testing, etc. It's been a long ride!
>
> Thanks a lot to you as well Jani N for your continuous support during 
> design/development/review/merge
> Of this new/big feature :)
>
> Regards,
> Madhav
>
>>
>> BR,
>> Jani.
>>
>>
>>
>> >
>> >
>> > BR,
>> > Jani.
>> >
>> >
>> > [1] https://patchwork.freedesktop.org/series/51011/
>> > [2] https://patchwork.freedesktop.org/

Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-05 Thread Lisovskiy, Stanislav
I still see this pipe config mismatch(with icl-dsi-2018-12-03(4.20.0-rc5) + 
"fix transcoder state readout" commit applied):

[   12.773332] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
[   12.773425] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_htotal (expected 1586, found 793)
[   12.773510] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
[   12.773588] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
[   12.773663] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
[   12.773735] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
[   12.773813] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
[   12.773897] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
[   12.773976] [drm:pipe_config_err [i915]] *ERROR* mismatch in output_format 
(expected 0, found 1)
[   12.774039] [drm:pipe_config_err [i915]] *ERROR* mismatch in pixel_rate 
(expected 245700, found 122850)
[   12.774099] [drm:pipe_config_err [i915]] *ERROR* mismatch in pipe_bpp 
(expected 24, found 0)
[   12.774157] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
base.adjusted_mode.crtc_clock (expected 245700, found 122850)

To me it looks different from 
https://bugs.freedesktop.org/show_bug.cgi?id=108928 bug.

Also there are still "The master control interrupt lied (DE PIPE)!" messages( 
Pipe IIR register is read as 0, while
master_ctl has a correspondent flag set) - however with this one I can at least 
cope by adding a few retries in the interrupt handler as a workaround. Then the 
flooding stops. Not sure if this is a proper fix though.

I also run kms_draw_crc test with this board(investigating 
https://bugs.freedesktop.org/show_bug.cgi?id=103184),
and sometimes half of the tests fail with the crc mismatch, I think this is 
kind of different thing.


Best Regards,

Lisovskiy Stanislav

Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo


From: Intel-gfx [intel-gfx-boun...@lists.freedesktop.org] on behalf of 
Lisovskiy, Stanislav [stanislav.lisovs...@intel.com]
Sent: Wednesday, December 05, 2018 9:49 AM
To: Nikula, Jani; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

Hi Jani,

I've tried previously with branch icl-dsi-2018-12-03 for your github repo.
I think it has everything except this 4.12.2018  "fix transcoder state readout" 
commit.

I will apply it and try with that now, thanks.

Best Regards,

Lisovskiy Stanislav

Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo


From: Nikula, Jani
Sent: Tuesday, December 04, 2018 7:13 PM
To: Lisovskiy, Stanislav; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
Cc: ville.syrj...@linux.intel.com; Kulkarni, Vandita; Deak, Imre
Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling

On Tue, 04 Dec 2018, "Lisovskiy, Stanislav"  
wrote:
> Hi,
>
> Currently ICL DSI panel seems to work fine, however I still face
> mainly two issues, which probably need to be addressed:

Please try with current drm-tip with

commit 0716931a82b4d0e211d2ef66616ad7130107e455
Author: Jani Nikula 
Date:   Tue Dec 4 12:19:26 2018 +0200

drm/i915/icl: fix transcoder state readout

plus the hack patches from the end of the series. It's possible only the
VBT one is required.

BR,
Jani.

>
> 1) There is still pipe_config mismatch assertion:
>
> [   13.119965] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
> [   13.119989] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_htotal (expected 1586, found 793)
> [   13.120015] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
> [   13.120038] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
> [   13.120061] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
> [   13.120083] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
> [   13.120113] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
> [   13.120139] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_end (expected

Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-05 Thread Jani Nikula
On Wed, 05 Dec 2018, "Lisovskiy, Stanislav"  
wrote:
> I still see this pipe config mismatch(with icl-dsi-2018-12-03(4.20.0-rc5) + 
> "fix transcoder state readout" commit applied):
>
> [   12.773332] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
> [   12.773425] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_htotal (expected 1586, found 793)
> [   12.773510] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
> [   12.773588] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
> [   12.773663] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
> [   12.773735] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
> [   12.773813] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
> [   12.773897] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
> [   12.773976] [drm:pipe_config_err [i915]] *ERROR* mismatch in output_format 
> (expected 0, found 1)
> [   12.774039] [drm:pipe_config_err [i915]] *ERROR* mismatch in pixel_rate 
> (expected 245700, found 122850)
> [   12.774099] [drm:pipe_config_err [i915]] *ERROR* mismatch in pipe_bpp 
> (expected 24, found 0)
> [   12.774157] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_clock (expected 245700, found 122850)
>
> To me it looks different from 
> https://bugs.freedesktop.org/show_bug.cgi?id=108928 bug.

Okay, please file a new bug with the full dmesg. The above is not
enough.

BR,
Jani.

>
> Also there are still "The master control interrupt lied (DE PIPE)!" messages( 
> Pipe IIR register is read as 0, while
> master_ctl has a correspondent flag set) - however with this one I can at 
> least cope by adding a few retries in the interrupt handler as a workaround. 
> Then the flooding stops. Not sure if this is a proper fix though.
>
> I also run kms_draw_crc test with this board(investigating 
> https://bugs.freedesktop.org/show_bug.cgi?id=103184),
> and sometimes half of the tests fail with the crc mismatch, I think this is 
> kind of different thing.
>
>
> Best Regards,
>
> Lisovskiy Stanislav
>
> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
>
> 
> From: Intel-gfx [intel-gfx-boun...@lists.freedesktop.org] on behalf of 
> Lisovskiy, Stanislav [stanislav.lisovs...@intel.com]
> Sent: Wednesday, December 05, 2018 9:49 AM
> To: Nikula, Jani; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling
>
> Hi Jani,
>
> I've tried previously with branch icl-dsi-2018-12-03 for your github repo.
> I think it has everything except this 4.12.2018  "fix transcoder state 
> readout" commit.
>
> I will apply it and try with that now, thanks.
>
> Best Regards,
>
> Lisovskiy Stanislav
>
> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
>
> 
> From: Nikula, Jani
> Sent: Tuesday, December 04, 2018 7:13 PM
> To: Lisovskiy, Stanislav; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Kulkarni, Vandita; Deak, Imre
> Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling
>
> On Tue, 04 Dec 2018, "Lisovskiy, Stanislav"  
> wrote:
>> Hi,
>>
>> Currently ICL DSI panel seems to work fine, however I still face
>> mainly two issues, which probably need to be addressed:
>
> Please try with current drm-tip with
>
> commit 0716931a82b4d0e211d2ef66616ad7130107e455
> Author: Jani Nikula 
> Date:   Tue Dec 4 12:19:26 2018 +0200
>
> drm/i915/icl: fix transcoder state readout
>
> plus the hack patches from the end of the series. It's possible only the
> VBT one is required.
>
> BR,
> Jani.
>
>>
>> 1) There is still pipe_config mismatch assertion:
>>
>> [   13.119965] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
>> base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
>> [   13.119989] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
>> base.adjusted_mode.crtc_htotal (expected 1586, found 793)
>> [   13.120015] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
>> base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
>> [   13

Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-05 Thread Lisovskiy, Stanislav
Ok, I didn't file a bug yet, because I still have suspicion that this could be 
a bios thing.

Vandita, Madhav, did you happen to see same issue?

Best Regards,

Lisovskiy Stanislav

Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo


From: Nikula, Jani
Sent: Wednesday, December 05, 2018 10:35 AM
To: Lisovskiy, Stanislav; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
Cc: Peres, Martin; Saarinen, Jani
Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling

On Wed, 05 Dec 2018, "Lisovskiy, Stanislav"  
wrote:
> I still see this pipe config mismatch(with icl-dsi-2018-12-03(4.20.0-rc5) + 
> "fix transcoder state readout" commit applied):
>
> [   12.773332] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
> [   12.773425] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_htotal (expected 1586, found 793)
> [   12.773510] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
> [   12.773588] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
> [   12.773663] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
> [   12.773735] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
> [   12.773813] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
> [   12.773897] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
> [   12.773976] [drm:pipe_config_err [i915]] *ERROR* mismatch in output_format 
> (expected 0, found 1)
> [   12.774039] [drm:pipe_config_err [i915]] *ERROR* mismatch in pixel_rate 
> (expected 245700, found 122850)
> [   12.774099] [drm:pipe_config_err [i915]] *ERROR* mismatch in pipe_bpp 
> (expected 24, found 0)
> [   12.774157] [drm:pipe_config_err [i915]] *ERROR* mismatch in 
> base.adjusted_mode.crtc_clock (expected 245700, found 122850)
>
> To me it looks different from 
> https://bugs.freedesktop.org/show_bug.cgi?id=108928 bug.

Okay, please file a new bug with the full dmesg. The above is not
enough.

BR,
Jani.

>
> Also there are still "The master control interrupt lied (DE PIPE)!" messages( 
> Pipe IIR register is read as 0, while
> master_ctl has a correspondent flag set) - however with this one I can at 
> least cope by adding a few retries in the interrupt handler as a workaround. 
> Then the flooding stops. Not sure if this is a proper fix though.
>
> I also run kms_draw_crc test with this board(investigating 
> https://bugs.freedesktop.org/show_bug.cgi?id=103184),
> and sometimes half of the tests fail with the crc mismatch, I think this is 
> kind of different thing.
>
>
> Best Regards,
>
> Lisovskiy Stanislav
>
> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
>
> 
> From: Intel-gfx [intel-gfx-boun...@lists.freedesktop.org] on behalf of 
> Lisovskiy, Stanislav [stanislav.lisovs...@intel.com]
> Sent: Wednesday, December 05, 2018 9:49 AM
> To: Nikula, Jani; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling
>
> Hi Jani,
>
> I've tried previously with branch icl-dsi-2018-12-03 for your github repo.
> I think it has everything except this 4.12.2018  "fix transcoder state 
> readout" commit.
>
> I will apply it and try with that now, thanks.
>
> Best Regards,
>
> Lisovskiy Stanislav
>
> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo
>
> 
> From: Nikula, Jani
> Sent: Tuesday, December 04, 2018 7:13 PM
> To: Lisovskiy, Stanislav; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
> Cc: ville.syrj...@linux.intel.com; Kulkarni, Vandita; Deak, Imre
> Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling
>
> On Tue, 04 Dec 2018, "Lisovskiy, Stanislav"  
> wrote:
>> Hi,
>>
>> Currently ICL DSI panel seems to work fine, however I still face
>> mainly two issues, which probably need to be addressed:
>
> Please try with current drm-tip with
>
> commit 0716931a82b4d0e211d2ef66616ad7130107e455
> Author: Jani Nikula 
> Date:   Tue Dec 4 12:19:26 2018 +0200
>
> drm/i915/icl: fix transcoder state readout
>
> plus the hack patches from the end of the series. It's possible only the
> VBT one is required.
>
> BR,
&g

Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-05 Thread Chauhan, Madhav
However I have not seen fdo bug mentioned below, but root cause for these pipe 
config error messages
are due to the fact that for dual link we are programming half the timings but 
while comparing
We use undivided values. 

Moreover some timings are not even gets programmed for DSI like HBLANK/VBLANK 
etc.
So while dumping pipe_config errors we need to consider dual link scenario for 
DSI.

Regards,
Madhav

> -Original Message-
> From: Lisovskiy, Stanislav
> Sent: Wednesday, December 5, 2018 2:18 PM
> To: Nikula, Jani ; Chauhan, Madhav
> ; intel-gfx@lists.freedesktop.org
> Cc: Peres, Martin ; Saarinen, Jani
> ; Kulkarni, Vandita 
> Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling
> 
> Ok, I didn't file a bug yet, because I still have suspicion that this could 
> be a
> bios thing.
> 
> Vandita, Madhav, did you happen to see same issue?
> 
> Best Regards,
> 
> Lisovskiy Stanislav
> 
> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160
> Espoo
> 
> 
> From: Nikula, Jani
> Sent: Wednesday, December 05, 2018 10:35 AM
> To: Lisovskiy, Stanislav; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
> Cc: Peres, Martin; Saarinen, Jani
> Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling
> 
> On Wed, 05 Dec 2018, "Lisovskiy, Stanislav" 
> wrote:
> > I still see this pipe config mismatch(with icl-dsi-2018-12-03(4.20.0-rc5) + 
> > "fix
> transcoder state readout" commit applied):
> >
> > [   12.773332] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
> > [   12.773425] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_htotal (expected 1586, found 793)
> > [   12.773510] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
> > [   12.773588] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
> > [   12.773663] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
> > [   12.773735] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
> > [   12.773813] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
> > [   12.773897] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
> > [   12.773976] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> output_format (expected 0, found 1)
> > [   12.774039] [drm:pipe_config_err [i915]] *ERROR* mismatch in pixel_rate
> (expected 245700, found 122850)
> > [   12.774099] [drm:pipe_config_err [i915]] *ERROR* mismatch in pipe_bpp
> (expected 24, found 0)
> > [   12.774157] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_clock (expected 245700, found 122850)
> >
> > To me it looks different from
> https://bugs.freedesktop.org/show_bug.cgi?id=108928 bug.
> 
> Okay, please file a new bug with the full dmesg. The above is not enough.
> 
> BR,
> Jani.
> 
> >
> > Also there are still "The master control interrupt lied (DE PIPE)!"
> > messages( Pipe IIR register is read as 0, while master_ctl has a
> correspondent flag set) - however with this one I can at least cope by adding
> a few retries in the interrupt handler as a workaround. Then the flooding
> stops. Not sure if this is a proper fix though.
> >
> > I also run kms_draw_crc test with this board(investigating
> > https://bugs.freedesktop.org/show_bug.cgi?id=103184),
> > and sometimes half of the tests fail with the crc mismatch, I think this is
> kind of different thing.
> >
> >
> > Best Regards,
> >
> > Lisovskiy Stanislav
> >
> > Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7,
> > 02160 Espoo
> >
> > 
> > From: Intel-gfx [intel-gfx-boun...@lists.freedesktop.org] on behalf of
> > Lisovskiy, Stanislav [stanislav.lisovs...@intel.com]
> > Sent: Wednesday, December 05, 2018 9:49 AM
> > To: Nikula, Jani; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
> > Subject: Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling
> >
> > Hi Jani,
> >
> > I've tried previously with branch icl-dsi-2018-12-03 for your github repo.
> > I think it has everything except this 4.12.2018  "fix transcoder state
> readout" commit.
> >
> > I will apply it and try

Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling

2018-12-07 Thread Lisovskiy, Stanislav
.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
> > [   12.773976] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> output_format (expected 0, found 1)
> > [   12.774039] [drm:pipe_config_err [i915]] *ERROR* mismatch in pixel_rate
> (expected 245700, found 122850)
> > [   12.774099] [drm:pipe_config_err [i915]] *ERROR* mismatch in pipe_bpp
> (expected 24, found 0)
> > [   12.774157] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_clock (expected 245700, found 122850)
> >
> > To me it looks different from
> https://bugs.freedesktop.org/show_bug.cgi?id=108928 bug.
>
> Okay, please file a new bug with the full dmesg. The above is not enough.
>
> BR,
> Jani.
>
> >
> > Also there are still "The master control interrupt lied (DE PIPE)!"
> > messages( Pipe IIR register is read as 0, while master_ctl has a
> correspondent flag set) - however with this one I can at least cope by adding
> a few retries in the interrupt handler as a workaround. Then the flooding
> stops. Not sure if this is a proper fix though.
> >
> > I also run kms_draw_crc test with this board(investigating
> > https://bugs.freedesktop.org/show_bug.cgi?id=103184),
> > and sometimes half of the tests fail with the crc mismatch, I think this is
> kind of different thing.
> >
> >
> > Best Regards,
> >
> > Lisovskiy Stanislav
> >
> > Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7,
> > 02160 Espoo
> >
> > 
> > From: Intel-gfx [intel-gfx-boun...@lists.freedesktop.org] on behalf of
> > Lisovskiy, Stanislav [stanislav.lisovs...@intel.com]
> > Sent: Wednesday, December 05, 2018 9:49 AM
> > To: Nikula, Jani; Chauhan, Madhav; intel-gfx@lists.freedesktop.org
> > Subject: Re: [Intel-gfx] [PATCH v11 00/23] drm/i915/icl: dsi enabling
> >
> > Hi Jani,
> >
> > I've tried previously with branch icl-dsi-2018-12-03 for your github repo.
> > I think it has everything except this 4.12.2018  "fix transcoder state
> readout" commit.
> >
> > I will apply it and try with that now, thanks.
> >
> > Best Regards,
> >
> > Lisovskiy Stanislav
> >
> > Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7,
> > 02160 Espoo
> >
> > 
> > From: Nikula, Jani
> > Sent: Tuesday, December 04, 2018 7:13 PM
> > To: Lisovskiy, Stanislav; Chauhan, Madhav;
> > intel-gfx@lists.freedesktop.org
> > Cc: ville.syrj...@linux.intel.com; Kulkarni, Vandita; Deak, Imre
> > Subject: RE: [PATCH v11 00/23] drm/i915/icl: dsi enabling
> >
> > On Tue, 04 Dec 2018, "Lisovskiy, Stanislav" 
> wrote:
> >> Hi,
> >>
> >> Currently ICL DSI panel seems to work fine, however I still face
> >> mainly two issues, which probably need to be addressed:
> >
> > Please try with current drm-tip with
> >
> > commit 0716931a82b4d0e211d2ef66616ad7130107e455
> > Author: Jani Nikula 
> > Date:   Tue Dec 4 12:19:26 2018 +0200
> >
> > drm/i915/icl: fix transcoder state readout
> >
> > plus the hack patches from the end of the series. It's possible only
> > the VBT one is required.
> >
> > BR,
> > Jani.
> >
> >>
> >> 1) There is still pipe_config mismatch assertion:
> >>
> >> [   13.119965] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hdisplay (expected 1440, found 720)
> >> [   13.119989] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_htotal (expected 1586, found 793)
> >> [   13.120015] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hblank_start (expected 1440, found 1)
> >> [   13.120038] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hblank_end (expected 1586, found 1)
> >> [   13.120061] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hsync_start (expected 1540, found 770)
> >> [   13.120083] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_hsync_end (expected 1550, found 775)
> >> [   13.120113] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_vblank_start (expected 2560, found 1)
> >> [   13.120139] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> base.adjusted_mode.crtc_vblank_end (expected 2582, found 1)
> >> [   13.120169] [drm:pipe_config_err [i915]] *ERROR* mismatch in
> output_format (expected 0, found 1)
> >> [   13.120188] [d