gt; the default) let's unexport it.
>
> Signed-off-by: Daniel Vetter
> Cc: Gustavo Padovan
> Cc: Maarten Lankhorst
> Cc: Sean Paul
> Cc: David Airlie
> Cc: VMware Graphics
> Cc: Sinclair Yeh
> Cc: Thomas Hellstrom
> Cc: Archit Taneja
> Cc: Neil Armstrong
Hi John,
On Friday, 24 August 2018 00:12:46 EEST John Stultz wrote:
> On Thu, Aug 23, 2018 at 1:49 PM, Laurent Pinchart wrote:
> > On Thursday, 23 August 2018 20:48:40 EEST John Stultz wrote:
> >> On Thu, Aug 23, 2018 at 1:09 AM, Daniel Vetter wrote:
> >>> On T
Hi John,
On Thursday, 23 August 2018 20:48:40 EEST John Stultz wrote:
> On Thu, Aug 23, 2018 at 1:09 AM, Daniel Vetter wrote:
> > On Thu, Aug 23, 2018 at 10:46:15AM +0300, Laurent Pinchart wrote:
> >> On Thursday, 23 August 2018 07:14:08 EEST John Stultz wrote:
> >>&
should still be ok, as only
> 1920x1080 is actually displayed at one time.
>
> So it seems like we might need to multiply the max_height by the
> overalloc factor when we are checking it in
> drm_internal_framebuffer_create?
>
> Does that approach sound sane, or would folks p
-crtc
> - init sources-list only for gen3
> Changes Since V2:
> - Adopt to driver style
> - Address other review comments from Laurent Pinchart
I'm pretty sure this has changed since v4 as well.
> Signed-off-by: Mahesh Kumar
> Cc: dri-de...@lists.freedesktop.org
> Cc: Lauren
-crtc
> - init sources-list only for gen3
>
> Signed-off-by: Mahesh Kumar
> Cc: Laurent Pinchart
> ---
> drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 96 ++-
> drivers/gpu/drm/rcar-du/rcar_du_crtc.h | 3 ++
> 2 files changed, 98 insertions(+), 1 de
r optimize the code
>
> Signed-off-by: Mahesh Kumar
> Cc: dri-de...@lists.freedesktop.org
> Cc: Laurent Pinchart
> Reviewed-by: Maarten Lankhorst
> ---
> drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 71 +--
> 1 file changed, 51 insertions(+), 20
Hi Mahesh,
Thank you for the patch.
On Thursday, 12 July 2018 11:36:35 EEST Mahesh Kumar wrote:
> This patch implements get_crc_sources callback, which returns list of
> all the crc sources supported by driver in current platform.
>
> Signed-off-by: Mahesh Kumar
> Cc: L
y allocation.
> Now set_crc_source takes only two variable inputs, values_cnt we
> already gets as part of verify_crc_source.
>
> Changes since V1:
> - refactor code to use single spin lock
>
> Signed-off-by: Mahesh Kumar
> Cc: dri-de...@lists.freedesktop.org
> Cc: L
-by: Mahesh Kumar
> Cc: dri-de...@lists.freedesktop.org
> Cc: Laurent Pinchart
> Reviewed-by: Maarten Lankhorst
> ---
> drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 66 ---
> 1 file changed, 53 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/gpu
-core
> Changes Since V2:
> - update commit message
> - update callback documentation
> - print one source name per line
>
> Signed-off-by: Mahesh Kumar
> Cc: dri-de...@lists.freedesktop.org
> Cc: Laurent Pinchart
> Reviewed-by: Maarten Lankhorst
> ---
> drivers
drivers/gpu/drm/rcar-du/rcar_du_kms.h | 1 +
> drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 26 -
> include/drm/drm_crtc.h | 40 ++-
> 14 files changed, 396 insertions(+), 74 deletions(-)
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
g string for source.
>
> Changes since V1:
> - do not yet verify_crc_source during open.
>
> Signed-off-by: Mahesh Kumar
> Cc: dri-de...@lists.freedesktop.org
> Cc: Laurent Pinchart
> Reviewed-by: Maarten Lankhorst
> ---
> drivers/gpu/drm/drm_debugfs_crc.c | 8 +
Hi Mahesh,
On Tuesday, 10 July 2018 15:59:11 EEST Kumar, Mahesh wrote:
> On 7/10/2018 5:07 PM, Laurent Pinchart wrote:
> > On Monday, 2 July 2018 14:07:24 EEST Mahesh Kumar wrote:
> >> This patch implements "verify_crc_source" callback function for
> >> r
Hi Mahesh,
On Tuesday, 10 July 2018 14:54:11 EEST Kumar, Mahesh wrote:
> On 7/10/2018 4:56 PM, Laurent Pinchart wrote:
> > On Monday, 2 July 2018 14:07:20 EEST Mahesh Kumar wrote:
> >> This patch adds a new callback function "verify_crc_source" which will
> &g
Hi Mahesh,
On Tuesday, 10 July 2018 15:01:38 EEST Kumar, Mahesh wrote:
> On 7/10/2018 4:52 PM, Laurent Pinchart wrote:
> > Hi Mahesh,
> > On Monday, 2 July 2018 14:07:21 EEST Mahesh Kumar wrote:
> >> This patch introduce a callback function "get_crc_sources" whi
ort CRC
> + * generation or no prework is required before reading the crc
> + *
> + * RETURNS:
> + *
> + * 0 on success or a negative error code on failure.
> + */
> + int (*pre_crc_read)(struct drm_crtc *crtc);
>
> /**
>* @a
er read call, it can use
> poll/read blocking read() call.
>
> Suggested-by: Ville Syrjälä
> Signed-off-by: Mahesh Kumar
> Cc: dri-de...@lists.freedesktop.org
> Cc: Tomeu Vizoso
Reviewed-by: Laurent Pinchart
> ---
> drivers/gpu/drm/drm_debugfs_crc.c | 16
, NULL, _cnt);
> + crtc->funcs->set_crc_source(crtc, NULL);
>
> spin_lock_irq(>lock);
> crtc_crc_cleanup(crc);
> @@ -370,7 +362,7 @@ int drm_debugfs_crtc_crc_add(struct drm_crtc *crtc)
> {
> struct dentry *crc_ent, *ent;
>
> - if (!crtc->funcs->set_crc_source)
> + if (!crtc->funcs->set_crc_source || !crtc->funcs->verify_crc_source)
> return 0;
>
> crc_ent = debugfs_create_dir("crc", crtc->debugfs_entry);
[snip]
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
tatic const struct drm_crtc_funcs crtc_funcs_gen3 = {
> .enable_vblank = rcar_du_crtc_enable_vblank,
> .disable_vblank = rcar_du_crtc_disable_vblank,
> .set_crc_source = rcar_du_crtc_set_crc_source,
> + .verify_crc_source = rcar_du_crtc_verify_crc_source,
> };
>
> /*
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
* source for CRC and during crc open.
> + *
> + * This callback is optional if the driver does not support any CRC
> + * generation functionality.
> + *
> + * RETURNS:
> + *
> + * 0 on success or a negative error code on failure.
> +
CRC sources list. CRC-core does the verification of sources.
> + *
> + * RETURNS:
> + *
> + * a constant character pointer to the list of all the available CRC
> + * sources
> + */
> + const char *const *(*get_crc_sources)(struct drm_crtc *crtc,
> + size_t *count);
>
> /**
>* @atomic_print_state:
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Hi Dave,
On Thursday, 17 May 2018 08:05:03 EEST Dave Airlie wrote:
> On 17 May 2018 at 14:42, Dave Airlie <airl...@gmail.com> wrote:
> > On 16 May 2018 at 01:37, Laurent Pinchart wrote:
> >> On Tuesday, 15 May 2018 17:24:52 EEST kbuild test robot wrote:
> >>> t
how would you like to handle this ? I can rebase my drm/du/next branch
on top of your tree once you merge drm-misc-next and send a new pull request,
but I'd like to avoid missing the v4.18 merge window. Another option would be
to handle the error in the drm-misc-next merge. If there's an easier option,
-bit format
>
> lib/igt_gt.c | 24 ++--
> lib/igt_pm.c | 2 +-
> tests/kms_addfb_basic.c | 33 ++---
> tests/kms_panel_fitting.c | 1 +
> tests/kms_plane_lowres.c | 1 +
> 5 files chang
rm-core.c| 6 +++---
> > drivers/gpu/drm/tve200/tve200_drv.c| 4 ++--
> > drivers/gpu/drm/udl/udl_drv.c | 2 +-
> > drivers/gpu/drm/vc4/vc4_drv.c | 4 ++--
> > drivers
t replace
> all current use cases of drm_dev_unref() with drm_dev_put and remove
> the function itself.
>
> Coccinelle was used for mass-patching.
>
> Signed-off-by: Vaishali Thakkar <vthakkar1...@gmail.com>
I love seeing deprecated functions go.
Reviewed-by
Hi Daniel,
On Wednesday, 21 March 2018 10:34:33 EET Daniel Vetter wrote:
> On Tue, Mar 20, 2018 at 01:24:09PM +0200, Laurent Pinchart wrote:
> > Hi Ulrich,
> >
> > Thank you for the patch.
> >
> > On Thursday, 15 March 2018 16:45:38 EET Ulrich Hecht wrote:
&
ugh...
That's good news ! (Not that I'm happy that we have bugs, but catching them
shows that igt is useful for us). I hope this will help convincing management
that we should keep contributing to igt going forward.
Ulrich, for the record, I agree with all of Daniel's and Ville's comments
posted
Hi Ulrich,
Thank you for the patch.
On Thursday, 15 March 2018 16:45:37 EET Ulrich Hecht wrote:
> Add is_i915_device() requirement to tests using Intel-specific APIs.
>
> Signed-off-by: Ulrich Hecht <ulrich.hecht+rene...@gmail.com>
Reviewed-by: Laurent Pinchart
t in this test is Intel-
specific ? Can't we replace the Intel generation check with a different
feature check ?
> for_each_pipe_with_valid_output(display, pipe, output) {
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Hi Noralf,
Thank you for the patch.
On Thursday, 4 January 2018 00:21:04 EET Noralf Trønnes wrote:
> Remove a couple of trailing spaces.
>
> Signed-off-by: Noralf Trønnes <nor...@tronnes.org>
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
>
rm/drm_dumb_buffers.h
> @@ -0,0 +1,10 @@
You should add an SPDX header to specify the file license.
> +#ifndef _DRM_DUMB_BUFFERS_H_
> +#define _DRM_DUMB_BUFFERS_H_
> +
> +struct drm_device;
> +struct drm_file;
> +
> +int drm_mode_create_dumb_ioctl(struct drm_device
Hi Daniel,
On Tuesday, 19 December 2017 16:00:36 EET Daniel Vetter wrote:
> On Tue, Dec 19, 2017 at 03:33:57PM +0200, Laurent Pinchart wrote:
> > On Tuesday, 19 December 2017 12:03:55 EET Daniel Vetter wrote:
> > > On Mon, Dec 18, 2017 at 06:13:46PM +0200, Laur
Hi Daniel,
On Tuesday, 19 December 2017 12:03:55 EET Daniel Vetter wrote:
> On Mon, Dec 18, 2017 at 06:13:46PM +0200, Laurent Pinchart wrote:
> > On Thursday, 14 December 2017 22:30:53 EET Daniel Vetter wrote:
> >> DK put some nice docs into the commit introducing driver
gt; Cc: Dhinakaran Pandiyan <dhinakaran.pandi...@intel.com>
> Cc: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
> Cc: Ville Syrjälä <ville.syrj...@linux.intel.com>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Rob Clark <robdcl.
t; > Cc: Harry Wentland <harry.wentl...@amd.com>
> > Cc: Dhinakaran Pandiyan <dhinakaran.pandi...@intel.com>
> > Cc: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
> > Cc: Ville Syrjälä <ville.syrj...@linux.intel.com>
> > Cc: Laurent Pinchar
d to drm-misc-next. Thanks for the reviews.
And I now realize my review comments aren't very useful (apart possibly the
one about adding a comment to the drm_plane_helper_check_update() function,
feel free to submit a patch for that if you think it's useful) as the series
has been merged already :-/ I should really start reading mails in the reverse
chronological order.
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
let's switch over.
>
> Cc: VMware Graphics <linux-graphics-maintai...@vmware.com>
> Cc: Sinclair Yeh <s...@vmware.com>
> Cc: Thomas Hellstrom <thellst...@vmware.com>
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>
Reviewed-by: Laurent Pinchart
for the src coords,
> which have been checked by the core already.
>
> Cc: VMware Graphics <linux-graphics-maintai...@vmware.com>
> Cc: Sinclair Yeh <s...@vmware.com>
> Cc: Thomas Hellstrom <thellst...@vmware.com>
> Signed-off-by: Ville Syrjälä <vill
for the
reason explaining as a reply to that patch).
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> ---
> drivers/gpu/drm/arm/hdlcd_crtc.c| 8 +--
> drivers/gpu/drm/arm/malidp_planes.c | 4 +-
> drivers/gpu/drm/drm_atomi
per_check_update() that this change should be reverted
when the function is dropped, otherwise we'll carry the CRTC state argument
for a long time without realizing it's not needed anymore.
Apart from that,
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-o
other drivers.
I suppose it's a mix of cargo-cult and the fact that adjusted_mode hints that
it contains the timings that should be applied to the hardware. That's why I
use it in rcar-du.
Are drivers allowed to adjust the hdisplay and vdisplay values though ? I
thought unsupported values there ha
guous buffer with multiple entries that
> have consecutive sg_dma_addresses, effectively making it contiguous.
> Allow for that scenario by testing the entries in the sg_table for
> contiguous coverage.
>
> Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-
Hi Noralf,
On Wednesday, 13 September 2017 18:19:22 EEST Noralf Trønnes wrote:
> Den 13.09.2017 07.09, skrev Laurent Pinchart:
> > On Monday, 11 September 2017 17:31:54 EEST Noralf Trønnes wrote:
> >> Hi,
> >>
> >> I want to start out by saying that
rm_fb_helper.h | 9 +
> include/drm/drm_file.h | 2 +
> include/drm/drm_framebuffer.h | 4 +
> 11 files changed, 305 insertions(+), 237 deletions(-)
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Wednesday, 30 August 2017 17:17:36 EEST Daniel Vetter wrote:
> On Wed, Aug 30, 2017 at 05:10:43PM +0300, Laurent Pinchart wrote:
> > Hi Maarten,
> >
> > Thank you for the patch.
> >
> > On Wednesday, 30 August 2017 15:17:51 EEST Maarten Lankhorst wrote
create a single disable commit. (danvet)
> - Fix leak in intel_legacy_cursor_update.
>
> Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
> Testcase: kms_atomic_transition.plane-use-after-nonblocking-unbind*
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard
ent commits can synchronize with previous ones.
> + */
> + struct drm_crtc_commit *commit;
> +
> struct drm_atomic_state *state;
> };
>
> @@ -808,10 +817,16 @@ struct drm_crtc {
>* @commit_list:
>*
>* List of _crtc_
_atomic.h b/include/drm/drm_atomic.h
> index 8a5808eb5628..e76d9f95c191 100644
> --- a/include/drm/drm_atomic.h
> +++ b/include/drm/drm_atomic.h
> @@ -144,7 +144,6 @@ struct __drm_planes_state {
> struct __drm_crtcs_state {
> struct drm_crtc *ptr;
> struct drm_crtc_state *state, *old_state, *new_state;
> - struct drm_crtc_commit *commit;
> s32 __user *out_fence_ptr;
> unsigned last_vblank_count;
> };
> diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
> index 1a642020e306..4cc7c9cdb6ca 100644
> --- a/include/drm/drm_crtc.h
> +++ b/include/drm/drm_crtc.h
> @@ -253,6 +253,15 @@ struct drm_crtc_state {
>*/
> struct drm_pending_vblank_event *event;
>
> + /**
> + * @commit:
> + *
> + * This tracks how the commit for this update proceeds through the
> + * various phases. This is never cleared, except when we destroy the
> + * state, so that subsequent commits can synchronize with previous ones.
> + */
> + struct drm_crtc_commit *commit;
> +
> struct drm_atomic_state *state;
> };
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
On Monday 14 Aug 2017 09:25:47 Daniel Vetter wrote:
> On Sat, Aug 12, 2017 at 12:20 AM, Laurent Pinchart
>
> <laurent.pinch...@ideasonboard.com> wrote:
> > On Tuesday 25 Jul 2017 10:01:16 Daniel Vetter wrote:
> >> diff --git a/drivers/gpu/drm/drm_mode_o
int drm_mode_object_get_properties(struct
> drm_mode_object *obj, bool atomic, continue;
>
> if (*arg_count_props > count) {
> - ret = drm_object_property_get_value(obj, prop, );
> + ret = __drm_object_property_get_value(o
viewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
> Cc: Liviu Dudau <liviu.du...@arm.com>
> Cc: Brian Starkey <brian.star...@arm.com>
> Cc: Mali DP Maintainers <mal...@foss.arm.com
Hi Joe,
On Tuesday 01 Aug 2017 10:24:25 Joe Kniss wrote:
> On Tue, Aug 1, 2017 at 5:09 AM, Laurent Pinchart wrote:
> > On Monday 31 Jul 2017 11:29:13 Joe Kniss wrote:
> >> New getfb2 functionality uses drm_mode_fb_cmd2 struct to be symmetric
> >> with addfb2.
&
index b2c52843bc70..c81c75335cca 100644
> --- a/include/uapi/drm/drm.h
> +++ b/include/uapi/drm/drm.h
> @@ -814,6 +814,8 @@ extern "C" {
> #define DRM_IOCTL_MODE_CREATEPROPBLOBDRM_IOWR(0xBD, struct
> drm_mode_create_blob)
> #define DRM_IOCTL_MODE_DESTROYPROPBLOBDRM_IOWR
v2: don't forget zorder and get_property!
> >>
> >> Cc: Tomi Valkeinen <tomi.valkei...@ti.com
> >> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> >> Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
> >> ---
> >>
Hi Maarten,
On Wednesday 26 Jul 2017 14:53:33 Laurent Pinchart wrote:
> On Wednesday 19 Jul 2017 16:39:16 Maarten Lankhorst wrote:
> > for_each_obj_in_state is about to be removed, so use the correct new
> > iterator macros.
> >
> > Also look at new_plane_state i
n use. They should be the same except when reallocating,
> (in which case this code is skipped) and we should really stop looking
> at obj->state whenever possible.
>
> Changes since v1:
> - Actually compile correctly.
>
> Signed-off-by: Maarten Lankhorst <maarten.lank
onnector.c | 1 -
> drivers/gpu/drm/rcar-du/rcar_du_lvdscon.c | 1 -
For the above,
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
ssume the
> >> sink will accept the infoframe. Also if the EDID contains the HDMI
> >> 2.0 HDMI Forum VSDB we can assume the sink is prepared to receive
> >> the infoframe.
> >> Cc: Archit Taneja <arch...@codeaurora.org>
> >> Cc: Andrzej Hajda <a
y from drm_dev_release(), calls drm_vblank_cleanup(). The
commit message could be a bit more explicit. Apart from that,
Acked-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-by: Daniel Vetter
rror code for failed to
> look up an object throughout drm, so let's use it in the cma helpers,
> too.
Regardless of which is best, it's true that ENOENT is used through the DRM
code, so
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
Someone should however ment
op.org/wiki/CodeOfConduct
> Chris pointed out on irc that the grammar went a bit wrong here. I'll fix
> this to
>
> "As a freedesktop.org project, dri-devel, and the DRM community, follows
> the Contributor Covenant, ..."
s/follows/follow/ ?
Apart from that and the other
side of how to
> make hw work correctly where it's possible is imo already sufficiently
> documented.
>
> v2: Review from Laurent.
>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
Reviewed-by: Lauren
Hi Maarten,
Thank you for the resend.
For the whole series,
Tested-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
On Wednesday 01 Mar 2017 10:21:26 Maarten Lankhorst wrote:
> There are new iterator macros that annotate whether the new or old
> state should be used. Th
conn_state->crtc);
I believe more *_state -> *_(new|old)_state renames could be useful, but that
can be addressed as a subsequent patch, so
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> crtc_stat
r clarity, also rename the
> variables from $obj_state to (old,new)_$obj_state as well.
>
> Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> ---
> drivers/gpu/drm/drm_blend.c | 22
t <maarten.lankho...@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> ---
> include/drm/drm_atomic.h | 108 +++
> 1 file changed, 108 insertions(+)
>
> diff --git a/include/drm/drm_atomic.h b/inc
e dropped.
>
> Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> ---
> drivers/gpu/drm/drm_atomic_helper.c | 7 ---
> drivers/gpu/drm/drm_plane_helper.c | 2 +-
> i
e_state, i) { const struct drm_plane_helper_funcs *funcs;
> + struct drm_plane_state *plane_state;
> +
> + /*
> + * This might be called before swapping when commit is
aborted,
> + * in which case we have to free the new state.
s/free/clean
Hi Daniel,
On Tuesday 14 Feb 2017 21:03:07 Daniel Vetter wrote:
> On Tue, Jan 17, 2017 at 01:55:50AM +0200, Laurent Pinchart wrote:
> > On Monday 16 Jan 2017 10:37:40 Maarten Lankhorst wrote:
> >
> > Could we please get a description ? Apart from that,
>
> Typed
omic.c | 39 ++--
> drivers/gpu/drm/drm_atomic_helper.c | 377 ++--
> drivers/gpu/drm/drm_blend.c | 23 +--
> drivers/gpu/drm/i915/intel_display.c | 13 +-
> include/drm/drm_atomic.h | 180 -
> include/drm/drm_atomic_hel
gpu/drm/i915/intel_display.c | 13 +++---
> include/drm/drm_atomic.h | 81 +++--
> include/drm/drm_atomic_helper.h | 2 +
> 5 files changed, 149 insertions(+), 18 deletions(-)
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Hi Daniel,
On Monday 23 Jan 2017 09:48:54 Daniel Vetter wrote:
> On Thu, Jan 19, 2017 at 12:56:29AM +0200, Laurent Pinchart wrote:
> > On Tuesday 17 Jan 2017 08:41:03 Maarten Lankhorst wrote:
> >> Op 17-01-17 om 00:11 schreef Laurent Pinchart:
> >>> On Monda
rm_dev_init() to note that driver->release()
> allows for arbitrary embedding.
>
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> ---
> driver
tch
that shows usage of the release callback in a driver ?
> Signed-off-by: Chris Wilson <ch...@chris-wilson.co.uk>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Daniel Vetter <daniel.vet...@ffwll.ch>
> ---
> drivers/gpu/drm/drm_drv.c | 56
Hi Chris,
On Thursday 19 Jan 2017 11:20:37 Chris Wilson wrote:
> On Wed, Dec 14, 2016 at 12:04:38AM +0200, Laurent Pinchart wrote:
> > On Thursday 08 Dec 2016 08:18:40 Chris Wilson wrote:
> >> Some state is coupled into the device lifetime outside of the
> >> load/un
Hi Maarten,
On Tuesday 17 Jan 2017 08:41:03 Maarten Lankhorst wrote:
> Op 17-01-17 om 00:11 schreef Laurent Pinchart:
> > On Monday 16 Jan 2017 10:37:38 Maarten Lankhorst wrote:
> >> Add for_each_(old)(new)_(plane,connector,crtc)_in_state iterators to
> >> replace th
Hi Maarten,
On Wednesday 18 Jan 2017 15:49:56 Maarten Lankhorst wrote:
> Op 17-01-17 om 02:01 schreef Laurent Pinchart:
> > On Monday 16 Jan 2017 10:37:41 Maarten Lankhorst wrote:
> >> Signed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
> >>
clude/drm/drm_atomic.h | 180 -
> include/drm/drm_atomic_helper.h | 2 +
> 6 files changed, 438 insertions(+), 196 deletions(-)
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
t if your explanation isn't convincing enough ;-)).
> if (!old_crtc_state->active ||
> !drm_atomic_crtc_needs_modeset(old_conn_state->crtc-
>state))
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Hi Maarten,
Thank you for the patch.
On Monday 16 Jan 2017 10:37:44 Maarten Lankhorst wrote:
Could we please get a commit message ?
Apart from that,
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-by: Maarten Lankhorst <maarten.lankho...@lin
fter state swap, didn't it use the new state before this patch ?
> const struct drm_plane_helper_funcs *plane_funcs;
>
> plane_funcs = plane->helper_private;
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
struct drm_connector *connector)
> +{
> + int index = drm_connector_index(connector);
> +
> + if (index >= state->num_connector)
> + return NULL;
> +
> + return state->connectors[index].new_state;
> +}
> +
> +/**
> * __drm_atomic_get_current_plane_state - get current plane state
> * @state: global atomic state object
> * @plane: plane to grab
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
*funcs;
>
> + /*
> + * This might be called before swapping when commit is
aborted,
> + * in which case we have to free the new state.
Should this be "cleanup the new state" instead of "free the new state" ?
> + */
> + if (plane_state == plane->state)
> + plane_state = new_plane_state;
> +
> funcs = plane->helper_private;
>
> if (funcs->cleanup_fb)
[snip]
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
drm/drm_atomic_helper.c | 377 ++--
> drivers/gpu/drm/drm_blend.c | 23 +--
> drivers/gpu/drm/i915/intel_display.c | 13 +-
> include/drm/drm_atomic.h | 180 -
> include/drm/drm_atomic_helper.h | 2 +
>
Hi Maarten,
Thank you for the patch.
On Monday 16 Jan 2017 10:37:40 Maarten Lankhorst wrote:
Could we please get a description ? Apart from that,
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-by: Maarten Lankhorst <maarten.lankho...@lin
ed-off-by: Maarten Lankhorst <maarten.lankho...@linux.intel.com>
Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> ---
> drivers/gpu/drm/drm_atomic.c | 11 ---
> 1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_at
lanes[__i].ptr, \
> + (old_plane_state) = (__state)->planes[__i].old_state, 1); \
> + (__i)++) \
> + for_each_if (plane)
> +
> +#define for_each_new_plane_in_state(__state, plane, new_
__i].old_state, 1); \
> + (__i)++) \
> + for_each_if (plane)
> +
> +#define for_each_new_plane_in_state(__state, plane, new_plane_state, __i) \
> + for ((__i) = 0;
the
> per-member kerneldoc tends to be long).
>
> Also some minor drive-by polish where it makes sense, I read a lot
> of docs ...
>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: Jani Nikula <jani.nik...@linux.intel.com>
> Cc: Chris Wilson <c
KMS feature, only
> used by the fbdev deferred_io support to implement flushing/upload.
>
> Cc: Noralf Trønnes <nor...@tronnes.org>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Signed-off-by: Daniel Vetter <daniel.vet...@intel.com>
| 26 ++
> drivers/gpu/drm/drmfs.c| 566 ++
> drivers/gpu/drm/i915/i915_guc_submission.c | 33 +-
> include/drm/drm_drv.h | 3 +
> include/drm/drmfs.h | 77
On Fri, Dec 09, 2016 at 12:42:19AM +0200, Laurent Pinchart wrote:
> >>>> On Thursday 08 Dec 2016 16:41:04 Daniel Vetter wrote:
> >>>>> On Thu, Dec 08, 2016 at 02:45:25PM +0100, Maarten Lankhorst wrote:
> >>>>>> Atomic drivers may set propertie
Hi Maarten,
On Tuesday 13 Dec 2016 15:13:54 Maarten Lankhorst wrote:
> Op 09-12-16 om 09:25 schreef Daniel Vetter:
> > On Fri, Dec 09, 2016 at 12:42:19AM +0200, Laurent Pinchart wrote:
> >> On Thursday 08 Dec 2016 16:41:04 Daniel Vetter wrote:
> >>> On Thu, De
d (*release) (struct drm_device *);
> +
> int (*dma_ioctl) (struct drm_device *dev, void *data, struct drm_file
> *file_priv);
> int (*dma_quiescent) (struct drm_device *);
> int (*context_dtor) (struct drm_device *dev, int context);
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
: Chris Wilson <ch...@chris-wilson.co.uk>
This doesn't seem very DRM-specific, is there a reason why we can't use the
ansi_cprng already present in the kernel ?
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.
> >
> > const struct drm_plane_helper_funcs *funcs;
> >
> > - if (!drm_atomic_helper_framebuffer_changed(dev, old_state,
> > plane_state->crtc)) - continue;
> > -
> >
> > funcs = plane->helper_private;
> >
> > if (funcs->cleanup_fb)
--
Regards,
Laurent Pinchart
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Hi Ville,
On Monday 21 Nov 2016 15:31:57 Ville Syrjälä wrote:
> On Mon, Nov 21, 2016 at 03:23:19PM +0200, Laurent Pinchart wrote:
> > On Monday 21 Nov 2016 15:18:23 Ville Syrjälä wrote:
> >> On Sun, Nov 20, 2016 at 10:13:10AM +0200, Laurent Pinchart wrote:
> >>>
Hi Ville,
On Monday 21 Nov 2016 15:18:23 Ville Syrjälä wrote:
> On Sun, Nov 20, 2016 at 10:13:10AM +0200, Laurent Pinchart wrote:
> > On Friday 18 Nov 2016 21:53:12 ville.syrj...@linux.intel.com wrote:
> >> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> >&
101 - 200 of 272 matches
Mail list logo
