On Wed, Dec 16, 2020 at 12:03:37PM +0200, Jani Nikula wrote:
> On Fri, 11 Dec 2020, Dave Airlie wrote:
> > From: Dave Airlie
> >
> > There may be more crtc code that can be pulled out, but this
> > is a good start.
> >
> > RFC: maybe call the new file something different
>
> I checked this is ju
== Series Details ==
Series: drm/i915: Fix mismatch between misplaced vma check and vma insert (rev2)
URL : https://patchwork.freedesktop.org/series/84975/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9490 -> Patchwork_19154
===
If PCON has capability to convert RGB->YUV colorspace and also
to 444->420 downsampling then for any YUV420 only mode, we can
let the PCON do all the conversion.
v2: As suggested by Uma Shankar, considered case for colorspace
BT709 and BT2020, and default to BT609. Also appended dir
'display' in c
From: Dave Airlie
This just cleans these up a bit.
Signed-off-by: Dave Airlie
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_gen9_plane.c | 4 ++--
drivers/gpu/drm/i915/display/intel_sprite.c | 7 +++
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/dr
From: Dave Airlie
Daniel suggested this should move here.
Signed-off-by: Dave Airlie
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_crtc.c | 230
drivers/gpu/drm/i915/display/intel_sprite.c | 228 ---
2 files changed, 230 insertions(+)
From: Dave Airlie
Daniel asked for this, but it's a bit messy and I'm not sure
how best to clean it up yet.
Signed-off-by: Dave Airlie
[Jani: also moved fdi buf trans to intel_fdi.c.]
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_crt.c | 1 +
drivers/gpu/drm/i915/display
From: Dave Airlie
This just refactors out the fdi code to a separate file.
Signed-off-by: Dave Airlie
[Jani: cleaned up intel_fdi.h a bit.]
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/Makefile | 1 +
drivers/gpu/drm/i915/display/intel_display.c | 685 +--
From: Dave Airlie
This pulls a large chunk of the pll calculation code out of
intel_display.c to a new file.
One function makes sense to be an inline, otherwise this
is pretty much a straight copy cover. Also all the
remaining hooks for g45 and older end up the same now.
Signed-off-by: Dave Air
From: Dave Airlie
There may be more crtc code that can be pulled out, but this
is a good start.
Signed-off-by: Dave Airlie
[Jani: cleaned up intel_crtc.h a bit.]
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/Makefile| 1 +
drivers/gpu/drm/i915/display/intel_crtc.c|
From: Dave Airlie
This file is a monster, let's start simple, the cursor plane code
seems pretty standalone, and splits out easily enough.
Reviewed-by: Ville Syrjälä
Signed-off-by: Dave Airlie
[Jani: cleaned up intel_cursor.h a bit.]
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/Makefi
From: Dave Airlie
There is no need for a comma use here.
Reviewed-by: Ville Syrjälä
Signed-off-by: Dave Airlie
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/display/intel_
From: Dave Airlie
This will be used for some refactoring in other files, so move it
first.
Signed-off-by: Dave Airlie
Reviewed-by: Ville Syrjälä
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display.c | 6 --
drivers/gpu/drm/i915/display/intel_display_types.h |
From: Dave Airlie
This function is going to be used in a later change, so clean it
up first before moving it.
Reviewed-by: Ville Syrjälä
Signed-off-by: Dave Airlie
Signed-off-by: Jani Nikula
---
drivers/gpu/drm/i915/display/intel_display.c | 78 +--
.../drm/i915/display/inte
This is Dave's intel_display.c refactoring series [1] with some minor
issues addressed, to get CI results among other things.
I've reviewed and fixed any remaining minor issues I spotted in patches
1-8. I'm aware there may be things to improve, but I think this is
overall forward progress and clea
On Wed, Dec 16, 2020 at 01:46:58AM -0500, Andres Calderon Jaramillo wrote:
> On Tue, Dec 15, 2020 at 3:13 PM Ville Syrjälä
> wrote:
> >
> > On Tue, Dec 15, 2020 at 03:06:30PM -0500, Andres Calderon Jaramillo wrote:
> > > On Tue, Dec 15, 2020 at 1:01 PM Ville Syrjälä
> > > wrote:
> > > >
> > > > O
== Series Details ==
Series: drm/i915: Fix mismatch between misplaced vma check and vma insert (rev2)
URL : https://patchwork.freedesktop.org/series/84975/
State : warning
== Summary ==
$ dim checkpatch origin/drm-tip
efabc7d5411c drm/i915: Fix mismatch between misplaced vma check and vma inse
On Fri, 11 Dec 2020, Dave Airlie wrote:
> From: Dave Airlie
>
> Daniel asked for this, but it's a bit messy and I'm not sure
> how best to clean it up yet.
Cleaned it up some locally by moving both hsw and bdw buf trans tables
to intel_fdi.c, and adding a function to get them.
Surely this could
On Tue, Dec 15, 2020 at 3:13 PM Ville Syrjälä
wrote:
>
> On Tue, Dec 15, 2020 at 03:06:30PM -0500, Andres Calderon Jaramillo wrote:
> > On Tue, Dec 15, 2020 at 1:01 PM Ville Syrjälä
> > wrote:
> > >
> > > On Mon, Dec 14, 2020 at 10:57:03PM +, Shankar, Uma wrote:
> > > >
> > > >
> > > > >
On Fri, 11 Dec 2020, Dave Airlie wrote:
> From: Dave Airlie
>
> This just refactors out the fdi code to a separate file.
LGTM, save for another header fix that I made locally. ;)
See CONFIG_DRM_I915_WERROR=y for what we do in CI to keep stuff clean.
BR,
Jani.
>
> Signed-off-by: Dave Airlie
>
On Fri, 11 Dec 2020, Dave Airlie wrote:
> From: Dave Airlie
>
> This pulls a large chunk of the pll calculation code out of
> intel_display.c to a new file.
>
> One function makse sense to be an inline, otherwise this
> is pretty much a straight copy cover. also all the
> remaining hooks for g45
On Tue, 2020-12-15 at 05:14 -0800, Souza, Jose wrote:
> On Tue, 2020-12-15 at 13:02 +, Mun, Gwan-gyeong wrote:
> > On Mon, 2020-12-14 at 09:49 -0800, José Roberto de Souza wrote:
> > > Now using plane damage clips property to calcualte the damaged
> > > area.
> > > Selective fetch only supports
On Tue, Dec 15, 2020 at 03:18:49PM +0100, Christian König wrote:
> Am 14.12.20 um 17:01 schrieb Daniel Vetter:
> > On Mon, Dec 14, 2020 at 11:33:10AM +0100, Christian König wrote:
> > > Am 11.12.20 um 16:58 schrieb Daniel Vetter:
> > > > Also try to clarify a bit when dma_buf_begin/end_cpu_access s
On Fri, 11 Dec 2020, Dave Airlie wrote:
> From: Dave Airlie
>
> This file is a monster, let's start simple, the cursor plane code
> seems pretty standalone, and splits out easily enough.
I cleaned up intel_cursor.h a bit locally, but the code movement checks
out.
BR,
Jani.
>
> Reviewed-by: Vil
On Fri, 11 Dec 2020, Dave Airlie wrote:
> From: Dave Airlie
>
> There may be more crtc code that can be pulled out, but this
> is a good start.
>
> RFC: maybe call the new file something different
I checked this is just code movement. I did clean up intel_crtc.h
locally a bit though. (I'll proba
On Wed, Dec 16, 2020 at 2:29 AM Laurent Pinchart
wrote:
>
> Hi Daniel,
>
> Thank you for the patch.
>
> On Fri, Oct 23, 2020 at 02:21:25PM +0200, Daniel Vetter wrote:
> > Ends right after drm_atomic_helper_commit_hw_done(), absolutely
> > nothing fancy going on here.
> >
> > Signed-off-by: Daniel
In order to support the PSR state of each transcoder, it adds
i915_psr_status to sub-directory of each transcoder.
v2: Change using of Symbolic permissions 'S_IRUGO' to using of octal
permissions '0444'
v5: Addressed JJani Nikula's review comments
- Remove checking of Gen12 for i915_psr_statu
It is a preliminary work for supporting multiple EDP PSR and
DP PanelReplay. And it refactors singleton PSR to Multi Transcoder
supportable PSR.
And this moves and renames the i915_psr structure of drm_i915_private's to
intel_dp's intel_psr structure.
It also causes changes in PSR interrupt handlin
Quoting Matthew Auld (2020-12-16 09:31:41)
> On Tue, 15 Dec 2020 at 21:07, Chris Wilson wrote:
> >
> > With full-ppgtt, userspacew has complete control over their GTT. Verify
> > that we can place an object at the very beginning and the very end of
> > our GTT.
> >
> > Signed-off-by: Chris Wilson
On Tue, 15 Dec 2020 at 21:07, Chris Wilson wrote:
>
> With full-ppgtt, userspacew has complete control over their GTT. Verify
> that we can place an object at the very beginning and the very end of
> our GTT.
>
> Signed-off-by: Chris Wilson
> ---
> tests/i915/gem_softpin.c | 45 +
When inserting a VMA, we restrict the placement to the low 4G unless the
caller opts into using the full range. This was done to allow usersapce
the opportunity to transition slowly from a 32b address space, and to
avoid breaking inherent 32b assumptions of some commands.
However, for insert we li
== Series Details ==
Series: Add support for DP-HDMI2.1 PCON (rev7)
URL : https://patchwork.freedesktop.org/series/82098/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_9489_full -> Patchwork_19153_full
Summary
---
**
On 15/12/2020 21:49, Umesh Nerlige Ramappa wrote:
Variations in OA formats in the different gens has led to creation of
several sparse arrays to store the formats.
Move oa formats into a single array and add the supported range of
platforms for the oa formats.
Signed-off-by: Umesh Nerlige Ram
On Wed, 16 Dec 2020 at 08:43, Chris Wilson wrote:
>
> Quoting Tang, CQ (2020-12-16 00:51:21)
> >
> >
> > > -Original Message-
> > > From: Chris Wilson
> > > Sent: Tuesday, December 15, 2020 2:02 PM
> > > To: Tang, CQ ; intel-gfx@lists.freedesktop.org
> > > Cc: stable@
> > > Subject: Re:
Quoting Tang, CQ (2020-12-16 00:51:21)
>
>
> > -Original Message-
> > From: Chris Wilson
> > Sent: Tuesday, December 15, 2020 2:02 PM
> > To: Tang, CQ ; intel-gfx@lists.freedesktop.org
> > Cc: stable@
> > Subject: Re: [Intel-gfx] [PATCH] drm/i915: Fix mismatch between misplaced
> > vma
On 15/12/2020 23:49, Umesh Nerlige Ramappa wrote:
Variations in OA formats in the different gens has led to creation of
several sparse arrays to store the formats.
Move oa formats into a single array and add the supported range of
platforms for the oa formats.
Signed-off-by: Umesh Nerlige Ramap
101 - 135 of 135 matches
Mail list logo