All of lore.kernel.org
 help / color / mirror / Atom feed
From: Manasi Navare <manasi.d.navare@intel.com>
To: Jani Nikula <jani.nikula@linux.intel.com>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH 0/6] Remaining patches to enable Transcoder Port Sync for tiled displays
Date: Thu, 12 Sep 2019 11:35:00 -0700	[thread overview]
Message-ID: <20190912183500.GA3257@intel.com> (raw)
In-Reply-To: <87y2yut3r0.fsf@intel.com>

On Thu, Sep 12, 2019 at 09:42:11AM +0300, Jani Nikula wrote:
> On Wed, 11 Sep 2019, Manasi Navare <manasi.d.navare@intel.com> wrote:
> > On Wed, Sep 11, 2019 at 12:08:00PM +0300, Jani Nikula wrote:
> >> On Tue, 10 Sep 2019, Manasi Navare <manasi.d.navare@intel.com> wrote:
> >> > On Tue, Sep 10, 2019 at 11:07:30AM -0700, Manasi Navare wrote:
> >> >> On Tue, Sep 10, 2019 at 12:29:19PM +0300, Jani Nikula wrote:
> >> >> > On Sun, 08 Sep 2019, Manasi Navare <manasi.d.navare@intel.com> wrote:
> >> >> > > This patch series addresses all review comments and now the enable and
> >> >> > > disable paths follow the method of obtaining slave states from master
> >> >> > > and updating master-slaves in correct order during master modeset.
> >> >> > 
> >> >> > Main high level question: what does it take to enable this on gen9+?
> >> >> 
> >> >> As per the Bspec project platforms, the first platform that supports this is
> >> >> ICL
> >> >
> >> > Hi Jani,
> >> >
> >> > Apparently the Bspec caused some confusion, after double checking with the
> >> > HW teams here, this feature is enabled starting BDW for the SST connectors
> >> > and only the MST support is new to Gen 11+.
> >> >
> >> > So i guess I can change the patches to add support starting BDW and
> >> > this should also fix the 5K tiled display issue that Ankit has been
> >> > working on
> >> 
> >> Thanks! I don't mind getting the patches merged for ICL+ at first, and
> >> updating to cover older gens in follow-up.
> >
> > So you are suggesting that i keep the patches as is and have them get
> > merged first and then i could change the GEN checks to add support for
> > BDW+?
> 
> I'm saying it's up to you. Whatever makes most sense to you. If you're
> at the brink of getting the ICL patches merged, there's not much point
> in rehashing and delaying that, is there?

Yes I agree, thanks a lot Jani for your inputs/suggestions. I will push for the
reviews on these for ICL+ and get these merged first.

Manasi

> 
> BR,
> Jani.
> 
> 
> >
> > Regards
> > Manasi
> >
> >> 
> >> BR,
> >> Jani.
> >> 
> >> 
> >> >
> >> > Manasi
> >> >
> >> >> 
> >> >> Regards
> >> >> Manasi
> >> >> 
> >> >> > 
> >> >> > BR,
> >> >> > Jani.
> >> >> > 
> >> >> > 
> >> >> > -- 
> >> >> > Jani Nikula, Intel Open Source Graphics Center
> >> >> _______________________________________________
> >> >> Intel-gfx mailing list
> >> >> Intel-gfx@lists.freedesktop.org
> >> >> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
> >> 
> >> -- 
> >> Jani Nikula, Intel Open Source Graphics Center
> 
> -- 
> Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

      reply	other threads:[~2019-09-12 18:33 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-09  3:43 [PATCH 0/6] Remaining patches to enable Transcoder Port Sync for tiled displays Manasi Navare
2019-09-09  3:43 ` [PATCH 1/6] drm/i915/display/icl: Save Master transcoder in slave's crtc_state for Transcoder Port Sync Manasi Navare
2019-09-17 14:51   ` Maarten Lankhorst
2019-09-09  3:43 ` [PATCH 2/6] drm/i915/display/icl: Enable TRANSCODER PORT SYNC for tiled displays across separate ports Manasi Navare
2019-09-17 14:52   ` Maarten Lankhorst
2019-09-17 14:52   ` Maarten Lankhorst
2019-09-17 16:55     ` Manasi Navare
2019-09-09  3:43 ` [PATCH 3/6] drm/i915/display/icl: HW state readout for transcoder port sync config Manasi Navare
2019-09-17 14:53   ` Maarten Lankhorst
2019-09-09  3:43 ` [PATCH 4/6] drm/i915/display/icl: Enable master-slaves in trans port sync Manasi Navare
2019-09-09 15:52   ` [PATCH v2 " Manasi Navare
2019-09-17 15:01     ` Maarten Lankhorst
2019-09-09  3:43 ` [PATCH 5/6] drm/i915/display/icl: Disable transcoder port sync as part of crtc_disable() sequence Manasi Navare
2019-09-17 15:04   ` Maarten Lankhorst
2019-09-17 16:37     ` Manasi Navare
2019-09-18  8:51       ` Maarten Lankhorst
2019-09-18 15:16         ` Manasi Navare
2019-09-09  3:43 ` [PATCH 6/6] drm/i915/display/icl: In port sync mode disable slaves first then master Manasi Navare
2019-09-09 15:52   ` [PATCH v2 " Manasi Navare
2019-09-17 15:05     ` Maarten Lankhorst
2019-09-09  4:10 ` ✗ Fi.CI.CHECKPATCH: warning for Remaining patches to enable Transcoder Port Sync for tiled displays Patchwork
2019-09-09  4:11 ` ✗ Fi.CI.SPARSE: " Patchwork
2019-09-09 19:44 ` ✗ Fi.CI.SPARSE: warning for Remaining patches to enable Transcoder Port Sync for tiled displays (rev3) Patchwork
2019-09-09 20:06 ` ✗ Fi.CI.BAT: failure " Patchwork
2019-09-10  9:29 ` [PATCH 0/6] Remaining patches to enable Transcoder Port Sync for tiled displays Jani Nikula
2019-09-10 18:07   ` Manasi Navare
2019-09-10 21:19     ` Manasi Navare
2019-09-11  9:08       ` Jani Nikula
2019-09-11 16:27         ` Manasi Navare
2019-09-12  6:42           ` Jani Nikula
2019-09-12 18:35             ` Manasi Navare [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190912183500.GA3257@intel.com \
    --to=manasi.d.navare@intel.com \
    --cc=intel-gfx@lists.freedesktop.org \
    --cc=jani.nikula@linux.intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.