Re: [RFC] new uapi policy for drm

2019-10-16 Thread Stéphane Marchesin
On Wed, Oct 16, 2019 at 10:28 AM Matt Roper wrote: > > On Tue, Oct 15, 2019 at 04:16:00AM +1000, Dave Airlie wrote: > > I've kicked this around in my head over the past few weeks but wanted > > to get some feedback on whether it's a good idea or what impact it > > might have that I haven't conside

Re: [PATCH] drm/cirrus: rewrite and modernize driver.

2019-04-03 Thread Stéphane Marchesin
On Wed, Apr 3, 2019 at 7:58 PM David Airlie wrote: > On Wed, Apr 3, 2019 at 5:23 PM Gerd Hoffmann wrote: > > > > Time to kill some bad sample code people are copying from ;) > > > > This is a complete rewrite of the cirrus driver. The cirrus_mode_set() > > function is pretty much the only funct

Re: [PATCH 0/6] Add uAPI to support ICL VME hardware for new media-driver

2019-02-04 Thread Stéphane Marchesin
entious topics. Mostly > positive meh seems plenty good enough feedback I think. FWIW I think the API changes are fine. Sure it feels a bit odd at first, but there's no better alternative that I can see either. Acked-by: Stéphane Marchesin > -Daniel > > > > > Regards

Re: Expose more EDID fields to userspace

2019-01-17 Thread Stéphane Marchesin
On Mon, Jan 7, 2019, 09:07 Brian Starkey On Mon, Jan 07, 2019 at 07:57:54AM -0800, Keith Packard wrote: > > Daniel Vetter writes: > > > > > Best to pull in some other compositor people and get them to agree. > From a > > > kernel pov I'm fine with whatever userspace preferes. > > > > Hrm. It woul

Re: [Intel-gfx] [PATCH v3 0/3] CRTC background color

2018-12-27 Thread Stéphane Marchesin
On Thu, Dec 27, 2018 at 4:45 PM Matt Roper wrote: > > On Thu, Dec 27, 2018 at 04:22:28PM -0800, Stéphane Marchesin wrote: > > On Thu, Dec 27, 2018 at 3:49 PM Li, Wei C wrote: > > > > > > Matt, > > > > > > Is that possible for you to

Re: [Intel-gfx] [PATCH v3 0/3] CRTC background color

2018-12-27 Thread Stéphane Marchesin
using > the TEST=drm-tests/atomictest -t crtc_background_color on a Google Pixelbook > with KBL graphics. > > Thanks, > Wei > > -Original Message- > From: Stéphane Marchesin [mailto:marc...@chromium.org] > Sent: Thursday, December 27, 2018 3:27 PM > To: Roper, Matthew

Re: [Intel-gfx] [PATCH v3 0/3] CRTC background color

2018-12-27 Thread Stéphane Marchesin
Hey, Is there anything missing on the Chrome side to move forward with this series? Stéphane On Thu, Nov 15, 2018 at 2:14 PM Matt Roper wrote: > > Third version of the series previously posted here: > https://lists.freedesktop.org/archives/intel-gfx/2018-November/181777.html > > This version

Re: [PATCH v8 12/35] drm/i915: Implement the HDCP2.2 support for DP

2018-11-27 Thread Stéphane Marchesin
Hi, Just a drive-by comment, but did you check that this fails gracefully on platforms which don't enable the ME? For example Chrome OS :) Stéphane On Tue, Nov 27, 2018 at 2:48 AM Ramalingam C wrote: > > Implements the DP adaptation specific HDCP2.2 functions. > > These functions perform the DP

Re: [PATCH] drm: udl: Destroy framebuffer only if it was initialized

2018-04-25 Thread Stéphane Marchesin
On Tue, Apr 24, 2018 at 6:04 AM, Daniel Vetter wrote: > On Fri, Apr 20, 2018 at 09:55:32AM -0400, Sean Paul wrote: >> On Fri, Apr 20, 2018 at 01:50:01PM +0200, Emil Lundmark wrote: >> > This fixes a NULL pointer dereference that can happen if the UDL >> > driver is unloaded before the framebuffer

Re: [PATCH v3 6/6] arm64: dts: rockchip: Specify override mode for kevin panel

2018-04-24 Thread Stéphane Marchesin
>>> Changes in v2: >>> - Wrap the timing in display-timings node to match binding (Rob/Thierry) >>> Changes in v3: >>> - Unwrap the timing from display-timings and rename panel-timing (Rob) >>> >>> Cc: Doug Anderson >>> Cc: Eric Anholt

Re: RFC for a render API to support adaptive sync and VRR

2018-04-12 Thread Stéphane Marchesin
On Tue, Apr 10, 2018 at 12:37 AM, Michel Dänzer wrote: > On 2018-04-10 08:45 AM, Christian König wrote: >> Am 09.04.2018 um 23:45 schrieb Manasi Navare: >>> Thanks for initiating the discussion. Find my comments below: >>> On Mon, Apr 09, 2018 at 04:00:21PM -0400, Harry Wentland wrote: On 201

Re: [PATCH] virgl: fix caps ioctl size check.

2018-02-12 Thread Stéphane Marchesin
it can work it out from there. > > Userspace will probably still need a fallback as old > kernel are out there. > > Cc: > Signed-off-by: Dave Airlie Reviewed-by: Stéphane Marchesin Thanks! > --- > drivers/gpu/drm/virtio/virtgpu_ioctl.c | 2 +- > 1 file changed, 1

Re: XDC 2017 : Call for paper

2017-08-18 Thread Stéphane Marchesin
plan for the catering, so if you are > attending please add your name as early as possible. > > I am looking forward to seeing you there. If you have any > inquiries/questions, please send them to Stéphane Marchesin (please also > CC: board at foundation.x.org). Hi all, For logistic

Re: [Intel-gfx] [PATCH RESEND] drm/i915: Fix pipe/transcoder enum mismatches

2017-07-13 Thread Stéphane Marchesin
On Thu, Jul 13, 2017 at 10:42 AM, Ville Syrjälä wrote: > On Thu, Jul 13, 2017 at 09:23:11AM -0700, Stéphane Marchesin wrote: >> On Thu, Jul 13, 2017 at 3:13 AM, Ville Syrjälä >> wrote: >> > On Wed, Jul 12, 2017 at 07:28:14PM -0700, Stéphane Marchesin wrote: >> >

Re: [PATCH] drm/udl: Make page_flip asynchronous

2017-07-13 Thread Stéphane Marchesin
On Mon, Jul 10, 2017 at 11:58 PM, Daniel Vetter wrote: > On Fri, Jul 7, 2017 at 7:48 AM, Dawid Kurek > wrote: >> In page_flip vblank is sent with no delay. Driver does not know when the >> actual update is present on the display and has no means for getting >> this information from a device. It

Re: [Intel-gfx] [PATCH RESEND] drm/i915: Fix pipe/transcoder enum mismatches

2017-07-13 Thread Stéphane Marchesin
On Thu, Jul 13, 2017 at 3:13 AM, Ville Syrjälä wrote: > On Wed, Jul 12, 2017 at 07:28:14PM -0700, Stéphane Marchesin wrote: >> On Fri, May 5, 2017 at 10:40 AM, Ville Syrjälä >> wrote: >> > >> > On Fri, May 05, 2017 at 10:26:36AM -0700, Matthias Kaehlcke wrote:

Re: [Intel-gfx] [PATCH RESEND] drm/i915: Fix pipe/transcoder enum mismatches

2017-07-12 Thread Stéphane Marchesin
On Fri, May 5, 2017 at 10:40 AM, Ville Syrjälä wrote: > > On Fri, May 05, 2017 at 10:26:36AM -0700, Matthias Kaehlcke wrote: > > El Thu, Apr 20, 2017 at 02:56:05PM -0700 Matthias Kaehlcke ha dit: > > > > > In several instances the driver passes an 'enum pipe' value to a > > > function expecting an

Re: [PATCH v3 2/2] drm/panel: add innolux,p079zca panel driver

2017-03-23 Thread Stéphane Marchesin
can you add bps here too? On Tue, Mar 21, 2017 at 6:53 PM, Chris Zhong wrote: > Support Innolux P079ZCA 7.85" 768x1024 TFT LCD panel, it is a MIPI DSI > panel. > > Signed-off-by: Chris Zhong > Reviewed-by: Sean Paul > Tested-by: Brian Norris > --- > > Changes in v3: > - printk err after regula

Re: [PATCH 01/41] drm/panel: simple: Change mode for Sharp lq123p1jx31

2017-03-20 Thread Stéphane Marchesin
ckchip-friendly such that we can use the fixed PLLs to > >> generate the pixel clock > >> > >> Cc: Chris Zhong > >> Cc: Stéphane Marchesin > >> Signed-off-by: Sean Paul > >> --- > >> drivers/gpu/drm/panel/panel-simple.c | 7 --

Re: KMS backlight ABI proposition

2017-02-23 Thread Stéphane Marchesin
On Thu, Feb 23, 2017 at 12:40 AM, Jani Nikula wrote: > On Wed, 22 Feb 2017, Stéphane Marchesin wrote: >> On Fri, Feb 17, 2017 at 4:58 AM, Martin Peres >> wrote: >>> If the KMS property exposes a fixed number of steps (say 100), it becomes >>> easy for th

Re: KMS backlight ABI proposition

2017-02-22 Thread Stéphane Marchesin
On Fri, Feb 17, 2017 at 4:58 AM, Martin Peres wrote: > Hey everyone, > > We have been working towards exposing the backlight as a KMS property > instead of relying on the backlight drivers. We have CC:ed the people we > have found to be the more likely to be interested in the discussion but > plea

Unix Device Memory Allocation project

2017-01-04 Thread Stéphane Marchesin
On Tue, Jan 3, 2017 at 3:38 PM, Marek Olšák wrote: > On Thu, Oct 20, 2016 at 8:31 AM, Daniel Vetter wrote: >> On Wed, Oct 19, 2016 at 6:46 PM, Marek Olšák wrote: > We've had per buffer metadata in Radeon since KMS, which I believe first > appeared in 2009. It's 4 bytes large and is u

RFC: fb restore on drm master close

2017-01-04 Thread Stéphane Marchesin
On Wed, Jan 4, 2017 at 12:30 AM, Daniel Vetter wrote: > On Wed, Dec 21, 2016 at 12:13:06PM -0600, vcaputo at pengaru.com wrote: >> Hello list, >> >> I've been playing with an unaccelerated drm program[1] and have been >> annoyed that whenever this program exits the fbcon isn't restored, with >> th

[PATCH v3 2/2] drm/panel: simple: Add support BOE nv101wxmn51

2016-12-15 Thread Stéphane Marchesin
Reviewed-by: Stéphane Marchesin On Tue, Dec 13, 2016 at 7:19 PM, Caesar Wang wrote: > 10.1WXGA is a color active matrix TFT LCD module using amorphous silicon > TFT's as an active switching devices. It can be supported by the > simple-panel driver. > > Read the panel default edid information:

[PATCH v2 2/2] drm/panel: simple: Add support BOE nv101wxmn51

2016-12-12 Thread Stéphane Marchesin
On Wed, Dec 7, 2016 at 11:26 PM, Caesar Wang wrote: > 10.1WXGA is a color active matrix TFT LCD module using amorphous silicon > TFT's as an active switching devices. It can be supported by the > simple-panel driver. > > Read the panel edid information; > > EDID MODE DETAILS > name

[PATCH] modetest: Also print the pixel clock

2016-08-31 Thread Stéphane Marchesin
This can be useful for debugging. xrandr prints it, so why not. --- tests/modetest/modetest.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c index 21d5438..dedd286 100644 --- a/tests/modetest/modetest.c +++ b/tests/mod

[PATCH v2 1/2] dt-bindings: add Starry KR122EA0SRA panel binding

2016-06-10 Thread Stéphane Marchesin
On Fri, Jun 10, 2016 at 3:03 PM, Rob Clark wrote: > On Fri, Jun 10, 2016 at 3:52 PM, Doug Anderson > wrote: >> Rob, >> >> On Fri, Jun 10, 2016 at 11:43 AM, Rob Clark wrote: >>> On Fri, Jun 10, 2016 at 1:02 PM, Douglas Anderson >> chromium.org> wrote: The Starry KR122EA0SRA is a 12.2", 1920

[PATCH v2 4/6] drm/panel: simple: Add support for Samsung LSN122DL01-C01 2560x1600 panel

2016-06-09 Thread Stéphane Marchesin
On Wed, Jun 8, 2016 at 4:52 AM, Yakir Yang wrote: > The Samsung LSN122DL01-C01 is an 12.2" 2560x1600 (WQXGA) TFT-LCD panel > connected using eDP interfaces. > > Signed-off-by: Yakir Yang > --- > Changes in v2: None > > drivers/gpu/drm/panel/panel-simple.c | 25 + > 1 file

[PATCH] drm/rockchip: support prime fd import

2016-02-23 Thread Stéphane Marchesin
On Tue, Feb 23, 2016 at 3:56 PM, Rob Clark wrote: > On Tue, Feb 23, 2016 at 6:29 PM, Emil Velikov > wrote: >> Hi Zach, >> >> On 2 February 2016 at 23:37, Zach Reizner wrote: >>> The prime fd to handle ioctl was not used with rockchip before. Support >>> was added in order to support potential u

[PATCH] drm/rockchip: support prime fd import

2016-02-23 Thread Stéphane Marchesin
Reviewed-by: Stéphane Marchesin On Tue, Feb 2, 2016 at 3:37 PM, Zach Reizner wrote: > The prime fd to handle ioctl was not used with rockchip before. Support > was added in order to support potential uses (e.g. zero-copy video > decode, camera). > > Signed-off-by: Zach Reizner > --- > drivers

[PATCH 2/2] drm: make unplugged flag specific to udl driver

2016-02-10 Thread Stéphane Marchesin
On Wed, Feb 10, 2016 at 1:54 PM, David Herrmann wrote: > Hi > > On Wed, Feb 10, 2016 at 10:46 PM, Stéphane Marchesin > wrote: >> On Wed, Feb 10, 2016 at 1:38 PM, David Herrmann >> wrote: >>> Hi >>> >>> On Wed, Feb 10, 2016 at 9:39 PM, Haixia Shi wrote: > + if (udl_device_is_un

[PATCH 2/2] drm: make unplugged flag specific to udl driver

2016-02-10 Thread Stéphane Marchesin
On Wed, Feb 10, 2016 at 1:38 PM, David Herrmann wrote: > Hi > > On Wed, Feb 10, 2016 at 9:39 PM, Haixia Shi wrote: >> >>> + if (udl_device_is_unplugged(dev) && >>> + nr != DRM_IOCTL_NR(DRM_IOCTL_MODE_SETCRTC) && >>> + nr != DRM_IOCTL_NR(DRM_IOCTL_MODE_RMFB) && >

Direct userspace dma-buf mmap (v7)

2016-02-04 Thread Stéphane Marchesin
On Tue, Dec 22, 2015 at 1:36 PM, Tiago Vignatti wrote: > Hey back, > > Thank you Daniel, Chris, Alex and Thomas for reviewing the last series. I > think I addressed most of the comments now in version 7, including: > - being even more wording in the doc about sync usage. > - pass .write = fals

dumb BOs and prime

2015-12-05 Thread Stéphane Marchesin
On Sat, Dec 5, 2015 at 3:40 PM, Rob Herring wrote: > On Fri, Dec 4, 2015 at 5:48 PM, Greg Hackmann wrote: >> On 12/04/2015 11:23 AM, Rob Herring wrote: >>> >>> On Fri, Dec 4, 2015 at 12:40 PM, Benjamin Gaignard >>> wrote: Hi Rob, I got the same problem today with sti drm/kms

KMS timings (Re: [PATCH 6/8] drm/bochs: phase 3: provide a custom ->atomic_commit implementation)

2015-07-20 Thread Stéphane Marchesin
On Mon, Jul 20, 2015 at 7:21 AM, Daniel Vetter wrote: > On Mon, Jul 20, 2015 at 12:35:48PM +0300, Pekka Paalanen wrote: >> On Mon, 20 Jul 2015 01:58:33 -0700 >> Stéphane Marchesin wrote: >> >> > On Mon, Jul 20, 2015 at 12:46 AM, Pekka Paalanen >> > wrote: >> > > On Sun, 19 Jul 2015 17:20:32 -0

[PATCH 6/8] drm/bochs: phase 3: provide a custom ->atomic_commit implementation

2015-07-20 Thread Stéphane Marchesin
On Mon, Jul 20, 2015 at 12:46 AM, Pekka Paalanen wrote: > On Sun, 19 Jul 2015 17:20:32 -0700 > Stéphane Marchesin wrote: > >> On Thu, Jul 16, 2015 at 11:08 PM, Pekka Paalanen >> wrote: >> > >> > On Thu, 16 Jul 2015 20:20:39 +0800 >> > John Hunter wrote: >> > >> > > From: Zhao Junwang >> > >

[PATCH 6/8] drm/bochs: phase 3: provide a custom ->atomic_commit implementation

2015-07-19 Thread Stéphane Marchesin
On Thu, Jul 16, 2015 at 11:08 PM, Pekka Paalanen wrote: > > On Thu, 16 Jul 2015 20:20:39 +0800 > John Hunter wrote: > > > From: Zhao Junwang > > > > This supports the asynchronous commits, required for page-flipping > > Since it's virtual hw it's ok to commit async stuff right away, we > > never

eDP display control registers in Linux kernel

2015-02-23 Thread Stéphane Marchesin
On Sun, Feb 22, 2015 at 11:59 PM, Jani Nikula wrote: > > Hi Michael - > > Please always cc: the relevant mailing lists; done now. > > On Sun, 22 Feb 2015, Michael Leuchtenburg wrote: >> Hi Jani, >> >> I've been trying to figure out how to control the dynamic backlight control >> on my new Dell XP

[PATCH 3/6] libdrm: fix the Android 64bit build

2015-01-29 Thread Stéphane Marchesin
On Thu, Jan 29, 2015 at 10:47 AM, Emil Velikov wrote: > With 64bit bionic mmap now handles 64bit offset, thus we no longer > need the __mmap2 trick. > > Fix from Chih-Wei Huang, over at the google forums. > > Cc: Chih-Wei Huang > Signed-off-by: Emil Velikov > --- > libdrm.h | 2 +- > 1 file ch

[PATCH v2] allow 32bpp framebuffers for cirrus drm

2014-11-17 Thread Stéphane Marchesin
Dave, any objections to merging this? Stéphane On Wed, Oct 29, 2014 at 11:04 AM, Zach Reizner wrote: > This patch allows framebuffers for cirrus to be created with > 32bpp pixel formats provided that they do not violate certain > restrictions of the cirrus hardware. > > v2: Use pci resource le

[RFC] dpms handling on atomic drivers

2014-11-06 Thread Stéphane Marchesin
On Thu, Nov 6, 2014 at 1:43 AM, Daniel Vetter wrote: > > Hi all, > > After a few atomic irc chats I've shockingly realized that I've completely > ignored dpms handling in my helper series. Oops. > > But there's a few things which are seriously wrong with DPMS, so I've > figured I'll start a discus

[PATCH] allow 32bpp framebuffers for cirrus drm

2014-10-24 Thread Stéphane Marchesin
On Tue, Oct 7, 2014 at 12:49 PM, Zach Reizner wrote: > This patch allows framebuffers for cirrus to be created with > 32bpp pixel formats provided that they do not violate certain > restrictions of the cirrus hardware. > > Signed-off-by: Zach Reizner > Reviewed-by: St?phane Marchesin Dave, Ad

[Intel-gfx] [RFC PATCH 0/3] drm driver for baytrail's vxd392

2014-10-15 Thread Stéphane Marchesin
On Wed, Oct 15, 2014 at 1:13 AM, Thierry Reding wrote: > On Tue, Oct 14, 2014 at 08:50:35AM -0700, Sean V Kelley wrote: > > On Tue, Oct 14, 2014 at 4:53 AM, Thierry Reding > > wrote: > > > On Mon, Oct 13, 2014 at 08:15:00PM +0800, Yao Cheng wrote: > > >> drm/ipvr is a new GEM driver for baytrail

[PATCH V2 4/9] drm/exynos: add exynos_dp_panel driver registration to drm driver

2014-08-21 Thread Stéphane Marchesin
On Thu, Aug 21, 2014 at 12:36 AM, Thierry Reding wrote: > On Tue, Aug 19, 2014 at 09:02:39PM -0700, St?phane Marchesin wrote: >> On Tue, Apr 22, 2014 at 8:26 AM, Thierry Reding >> wrote: >> > On Tue, Apr 22, 2014 at 08:33:23PM +0530, Ajay kumar wrote: >> >> Hi Thierry, >> >> >> >> On Tue, Apr 22,

[PATCH V2 4/9] drm/exynos: add exynos_dp_panel driver registration to drm driver

2014-08-19 Thread Stéphane Marchesin
On Tue, Apr 22, 2014 at 8:26 AM, Thierry Reding wrote: > On Tue, Apr 22, 2014 at 08:33:23PM +0530, Ajay kumar wrote: >> Hi Thierry, >> >> On Tue, Apr 22, 2014 at 2:03 PM, Thierry Reding >> wrote: >> > On Tue, Apr 22, 2014 at 04:09:13AM +0530, Ajay Kumar wrote: >> >> Register exynos_dp_panel befor

[PATCH 2/2] drm/udl: Implement page_flip ioctl

2014-07-02 Thread Stéphane Marchesin
This is a very crude page_flip implementation for UDL. There are ways to make it better (make it asynchronous, make it do actual vsynced flips...) but that's for another patch. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/udl/udl_modeset.c | 21 + 1 file changed, 21

[PATCH 1/2] drm/udl: add vblank support

2014-07-02 Thread Stéphane Marchesin
This is needed to be able to send page flip completion events. Also while I'm at it, fix the error paths on init. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/udl/udl_main.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/drivers/gpu/drm/udl/udl_main.c b/drivers/gpu

[PATCH] drm: reduce default drm vblank off delay to 50ms

2014-07-02 Thread Stéphane Marchesin
On Tue, Oct 30, 2012 at 12:20 PM, Daniel Vetter wrote: > On Tue, Oct 30, 2012 at 8:09 PM, Jesse Barnes > wrote: >> People keep whining about this, but no one seems to send a patch. This >> *ought* to be safe now that we've dealt with the hw races in Mario's >> updated code, and fixed the bugs w

[PATCH v2 3/4] drm/tegra: Add SET/GET_FLAGS IOCTLs

2014-06-27 Thread Stéphane Marchesin
On Fri, Jun 27, 2014 at 1:28 AM, Thierry Reding wrote: > From: Thierry Reding > > The DRM_TEGRA_GEM_SET_FLAGS IOCTL can be used to set the flags of a > buffer object after it has been allocated or imported. Flags associated > with a buffer object can be queried using the DRM_TEGRA_GEM_GET_FLAGS >

[PATCH v2 2/4] drm/tegra: Add SET/GET_TILING IOCTLs

2014-06-27 Thread Stéphane Marchesin
On Fri, Jun 27, 2014 at 1:28 AM, Thierry Reding wrote: > From: Thierry Reding > > Currently the tiling parameters of buffer objects can only be set at > allocation time, and only a single tiled mode is supported. This new > DRM_TEGRA_GEM_SET_TILING IOCTL allows more modes to be set and also > all

[Nouveau] [PATCH v2 2/3] drm/ttm: introduce dma cache sync helpers

2014-06-24 Thread Stéphane Marchesin
On Tue, Jun 24, 2014 at 6:25 AM, Lucas Stach wrote: > Am Dienstag, den 24.06.2014, 14:27 +0200 schrieb Maarten Lankhorst: >> op 24-06-14 14:23, Alexandre Courbot schreef: >> > On Tue, Jun 24, 2014 at 7:55 PM, Alexandre Courbot > > nvidia.com> wrote: >> >> On 06/24/2014 07:33 PM, Alexandre Courbot

[PATCH 3/3] drm/tegra: sor - Use bpc value from drm_panel

2014-06-19 Thread Stéphane Marchesin
This change uses the bpc value coming from drm_panel to remove one more hardcoded value. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/tegra/sor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c index 925f955..

[PATCH 2/3] drm/panel: panel-simple - Add support for bpc

2014-06-19 Thread Stéphane Marchesin
bpc is provided by the EDID normally, but if we're using drm_panel, we need to store it somewhere. So we add a drm_panel entry for it. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/panel/panel-simple.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/gpu/drm/panel/pan

[PATCH 1/3] drm/tegra: sor - Add register debugging support

2014-06-19 Thread Stéphane Marchesin
This adds a debugfs entry to print the register state. This can be pretty useful when debugging eDP link issues. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/tegra/sor.c | 159 +++- 1 file changed, 158 insertions(+), 1 deletion(-) diff --git a/dr

[PATCH] drm/tegra: sor - Add register debugging support

2014-06-19 Thread Stéphane Marchesin
This adds a debugfs entry to print the register state. This can be fairly useful when debugging eDP link issues. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/tegra/sor.c | 163 +++- 1 file changed, 162 insertions(+), 1 deletion(-) diff --git a/dr

[RFC PATCH 1/4] memory: tegra124-emc: Add EMC driver

2014-06-18 Thread Stéphane Marchesin
On Wed, Jun 18, 2014 at 3:00 PM, Thierry Reding wrote: > On Wed, Jun 18, 2014 at 07:23:47PM +0200, Tomeu Vizoso wrote: >> On 06/17/2014 06:15 PM, Stephen Warren wrote: >> >On 06/17/2014 06:16 AM, Tomeu Vizoso wrote: >> >>On 06/16/2014 10:02 PM, Stephen Warren wrote: >> >>>On 06/16/2014 07:35 AM, T

[PATCH 2/4] drm/tegra: Add SET/GET_TILING IOCTLs

2014-06-11 Thread Stéphane Marchesin
On Tue, Jun 10, 2014 at 4:04 AM, Thierry Reding wrote: > From: Thierry Reding > > Currently the tiling parameters of buffer objects can only be set at > allocation time, and only a single tiled mode is supported. This new > DRM_TEGRA_GEM_SET_TILING IOCTL allows more modes to be set and also > all

[PATCH 3/4] drm/tegra: Add SET/GET_FLAGS IOCTLs

2014-06-11 Thread Stéphane Marchesin
On Tue, Jun 10, 2014 at 4:04 AM, Thierry Reding wrote: > From: Thierry Reding > > The DRM_TEGRA_GEM_SET_FLAGS IOCTL can be used to set the flags of a > buffer object after it has been allocated or imported. Flags associated > with a buffer object can be queried using the DRM_TEGRA_GEM_GET_FLAGS >

[PATCH 4/4] drm/tegra: Allow non-authenticated processes to create buffer objects

2014-06-11 Thread Stéphane Marchesin
On Tue, Jun 10, 2014 at 4:04 AM, Thierry Reding wrote: > From: Thierry Reding > > This matches what other drivers do for equivalent IOCTLs. > > Signed-off-by: Thierry Reding Reviewed-by: St?phane Marchesin > --- > drivers/gpu/drm/tegra/drm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deleti

[Intel-gfx] [PATCH 5/5] drm/i915: enable fastboot by default

2014-06-10 Thread Stéphane Marchesin
On Tue, Jun 10, 2014 at 10:31 AM, Jesse Barnes wrote: > On Tue, 10 Jun 2014 16:07:44 +0200 > Daniel Vetter wrote: > >> On Thu, Jun 05, 2014 at 11:24:31AM -0700, Jesse Barnes wrote: >> > Let them eat mincemeat pie. >> > >> > Signed-off-by: Jesse Barnes >> > --- >> > drivers/gpu/drm/i915/i915_pa

[Nouveau] [RFC] drm/nouveau: disable caching for VRAM BOs on ARM

2014-05-26 Thread Stéphane Marchesin
On Mon, May 26, 2014 at 7:42 PM, Alexandre Courbot wrote: > On Tue, May 27, 2014 at 10:07 AM, St?phane Marchesin > wrote: >> On Mon, May 26, 2014 at 5:02 PM, Alexandre Courbot >> wrote: >>> On Mon, May 26, 2014 at 6:21 PM, Lucas Stach >>> wrote: Am Montag, den 26.05.2014, 09:45 +0300 sch

[Nouveau] [RFC] drm/nouveau: disable caching for VRAM BOs on ARM

2014-05-26 Thread Stéphane Marchesin
On Mon, May 26, 2014 at 5:02 PM, Alexandre Courbot wrote: > On Mon, May 26, 2014 at 6:21 PM, Lucas Stach > wrote: >> Am Montag, den 26.05.2014, 09:45 +0300 schrieb Terje Bergstr?m: >>> On 23.05.2014 17:40, Alex Courbot wrote: >>> > On 05/23/2014 06:59 PM, Lucas Stach wrote: >>> > So after checki

[PATCH 2/3] drm/tegra: Support setting the EMC clock

2014-05-26 Thread Stéphane Marchesin
On Mon, May 26, 2014 at 2:35 AM, Thierry Reding wrote: > On Mon, May 26, 2014 at 11:28:42AM +0200, St?phane Marchesin wrote: > > > > > > > > On Mon, May 26, 2014 at 2:07 AM, Lucas Stach > wrote: > > > Am Freitag, den 23.05.2014, 18:58 -0700 schrieb St?phane Marc

[PATCH 2/3] drm/tegra: Support setting the EMC clock

2014-05-26 Thread Stéphane Marchesin
On Mon, May 26, 2014 at 2:07 AM, Lucas Stach wrote: > Am Freitag, den 23.05.2014, 18:58 -0700 schrieb St?phane Marchesin: > > The current code doesn't enable the EMC clock, without which the > > display cannot function, so let's enable this clock. We also need a > > bit of code to pick the right

[PATCH] drm/panel: add support for AUO B133XTN01 panel

2014-05-23 Thread Stéphane Marchesin
On Thu, May 22, 2014 at 4:02 AM, Thierry Reding wrote: > On Tue, May 20, 2014 at 04:36:48PM -0700, St?phane Marchesin wrote: > > This panel is used by my tegra board and supported by the simple-panel > > driver. > > One thing I like to have is some indication in the commit message about > which bo

[PATCH] drm/panel: add support for AUO B133XTN01 panel

2014-05-23 Thread Stéphane Marchesin
This panel is used by nyan-big and supported by the simple-panel driver. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/panel/panel-simple.c | 25 + 1 file changed, 25 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/panel-s

[PATCH 3/3] drm/tegra: Don't hardcode link parameters

2014-05-23 Thread Stéphane Marchesin
The currently hardcoded link parameters don't work on my eDP display, however the android driver has code to compute the parameters. This is a port of this code to the drm driver which allows the drm driver to compute the link parameters correctly. Note that we still hardcode the bpp, this will be

[PATCH 2/3] drm/tegra: Support setting the EMC clock

2014-05-23 Thread Stéphane Marchesin
The current code doesn't enable the EMC clock, without which the display cannot function, so let's enable this clock. We also need a bit of code to pick the right frequency for the EMC clock depending on the current video mode settings. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/tegra

[PATCH 1/3] drm/tegra: Align FB pitch

2014-05-23 Thread Stéphane Marchesin
The display controller doesn't handle non-256 byte aligned buffer pitches. If you give it an unaligned buffer, it will stop after the first line and will report underflows in the debug registers (DC_WINBUF_UFLOW_STATUS and friends). So let's make sure that all our framebuffer pitches are 256-byte a

[PATCH 3/3] drm/tegra: Fix sequencing

2014-05-22 Thread Stéphane Marchesin
I suspect this is a copy/paste error. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/tegra/sor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c index 43709ee..23fb9b1 100644 --- a/drivers/gpu/drm/tegra/sor.c ++

[PATCH 2/3] drm/tegra: Fix copy/paste error

2014-05-22 Thread Stéphane Marchesin
The comment above mentions link A/B but this isn't what the code does, so let's fix that. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/tegra/sor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/tegra/sor.c b/drivers/gpu/drm/tegra/sor.c index 75b8690.

[PATCH 1/3] drm/tegra: Remove pixel clock rounding

2014-05-22 Thread Stéphane Marchesin
The code currently rounds up the clock to the next MHZ, which is rounding up a 69.5MHz clock to 70MHz on my machine. This in turn prevents the display from syncing. Removing this rounding fixes eDP for me. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/tegra/sor.c | 7 ++- 1 file chan

[PATCH] drm/panel: add support for AUO B133XTN01 panel

2014-05-20 Thread Stéphane Marchesin
This panel is used by my tegra board and supported by the simple-panel driver. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/panel/panel-simple.c | 25 + 1 file changed, 25 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers/gpu/drm/panel/p

[PATCH] drm/exynos: add phy settings for RB resolutions

2014-03-20 Thread Stéphane Marchesin
On Wed, Mar 12, 2014 at 10:28 PM, Shirish S wrote: > This patch adds support for the below mentioned > pixel clocks in Exynos5250. > Without them, following display modes won't > be supported: > > 71 MHz - 1280x800 at 60Hz RB > 73.25 MHz - 800x600 at 120Hz RB > 88.75 MHz - 1440x900 a

[PATCH] drm/edid: Remove part of the HDTV hack

2014-03-10 Thread Stéphane Marchesin
The part which turns 1360x765 into 1366x768 breaks this mode on a Toshiba TV here. 1360x765 works fine, though. Reviewed-by: Adam Jackson Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/drm_edid.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/gpu/drm/drm_ed

[PATCH] drm: Set the plane's crtc before calling disable_plane.

2014-03-03 Thread Stéphane Marchesin
On Mon, Mar 3, 2014 at 2:06 PM, Rob Clark wrote: > On Mon, Mar 3, 2014 at 4:45 PM, Jesse Barnes > wrote: >> On Mon, 3 Mar 2014 13:38:36 -0800 >> St?phane Marchesin wrote: >> >>> Some drivers like exynos need the crtc to be able to disable the plane, >>> so set it before calling disable_plane.

[PATCH] drm: Set the plane's crtc before calling disable_plane.

2014-03-03 Thread Stéphane Marchesin
Some drivers like exynos need the crtc to be able to disable the plane, so set it before calling disable_plane. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/drm_crtc.c | 21 +++-- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/drivers/gpu/drm/drm_crtc.c

[Intel-gfx] [PATCH 0/6] Intel Color Manager Framework

2014-02-23 Thread Stéphane Marchesin
On Fri, Feb 21, 2014 at 7:49 PM, Sharma, Shashank wrote: >On Thu, Feb 20, 2014 at 5:11 AM, Ville Syrj?l? < > ville.syrjala at linux.intel.com> wrote: > > On Thu, Feb 20, 2014 at 06:07:21PM +0530, Shashank Sharma wrote: > >> Color manager is a new framework in i915 driver, which provides > >>

[Intel-gfx] [PATCH 0/6] Intel Color Manager Framework

2014-02-21 Thread Stéphane Marchesin
On Thu, Feb 20, 2014 at 5:11 AM, Ville Syrj?l? < ville.syrjala at linux.intel.com> wrote: > On Thu, Feb 20, 2014 at 06:07:21PM +0530, Shashank Sharma wrote: > > Color manager is a new framework in i915 driver, which provides > > a unified interface for various color correction methods supported >

[PATCH] drm: Avoid NULL master_priv access in i915 kernel driver

2014-02-12 Thread Stéphane Marchesin
From: Stuart Abercrombie In several places, including the interrupt handler, the i915 driver assumes it can deref. dev->primary->master->driver_priv if dev->primary->master is non-NULL. This wasn't true if drm_open_helper was midway through, so rearrange the initialization order. v2: Address th

[RFC 00/16] drm/nouveau: initial support for GK20A (Tegra K1)

2014-02-02 Thread Stéphane Marchesin
On Sun, Feb 2, 2014 at 6:44 PM, Alexandre Courbot wrote: > On 02/03/2014 04:10 AM, Ilia Mirkin wrote: > >> Hi Alexandre, >> >> On Fri, Jan 31, 2014 at 10:16 PM, Alexandre Courbot >> wrote: >> >>> I guess my email address might surprise some of you, so let me >>> anticipate some >>> questions you

[PATCH v2 12/26] drm/exynos: Split manager/display/subdrv

2013-10-29 Thread Stéphane Marchesin
On Tue, Oct 29, 2013 at 1:50 PM, Tomasz Figa wrote: > Hi Sean, > > On Tuesday 29 of October 2013 16:36:47 Sean Paul wrote: > > On Mon, Oct 28, 2013 at 7:13 PM, Tomasz Figa > wrote: > > > Hi, > > > > > > On Wednesday 23 of October 2013 12:09:06 Sean Paul wrote: > > >> On Wed, Oct 23, 2013 at 11:5

[PATCH v2 12/26] drm/exynos: Split manager/display/subdrv

2013-10-22 Thread Stéphane Marchesin
On Tue, Oct 22, 2013 at 9:15 PM, Inki Dae wrote: > 2013/10/23 St?phane Marchesin : > > > > > > > > On Tue, Oct 22, 2013 at 8:38 PM, Inki Dae wrote: > >> > >> 2013/10/23 St?phane Marchesin : > >> > > >> > > >> > > >> > On Tue, Oct 22, 2013 at 7:28 PM, Inki Dae > wrote: > >> >> > >> >> 2013/10/22

[PATCH v2 12/26] drm/exynos: Split manager/display/subdrv

2013-10-22 Thread Stéphane Marchesin
On Tue, Oct 22, 2013 at 8:38 PM, Inki Dae wrote: > 2013/10/23 St?phane Marchesin : > > > > > > > > On Tue, Oct 22, 2013 at 7:28 PM, Inki Dae wrote: > >> > >> 2013/10/22 Sean Paul : > >> > On Tue, Oct 22, 2013 at 1:30 AM, Inki Dae > wrote: > >> >> > >> >> > >> >>> -Original Message- > >>

[PATCH v2 12/26] drm/exynos: Split manager/display/subdrv

2013-10-22 Thread Stéphane Marchesin
On Tue, Oct 22, 2013 at 7:28 PM, Inki Dae wrote: > 2013/10/22 Sean Paul : > > On Tue, Oct 22, 2013 at 1:30 AM, Inki Dae wrote: > >> > >> > >>> -Original Message- > >>> From: Sean Paul [mailto:seanpaul at chromium.org] > >>> Sent: Tuesday, October 22, 2013 6:18 AM > >>> To: Inki Dae > >>>

[PATCH 1/2] drm: Remove drm_mode_validate_clocks

2013-08-14 Thread Stéphane Marchesin
This function is unused. Signed-off-by: Stéphane Marchesin Reviewed-by: David Herrmann --- drivers/gpu/drm/drm_modes.c | 37 - include/drm/drm_crtc.h | 3 --- 2 files changed, 40 deletions(-) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm

[PATCH 1/2] drm: Remove drm_mode_validate_clocks

2013-08-13 Thread Stéphane Marchesin
This function is unused. Signed-off-by: St?phane Marchesin Reviewed-by: David Herrmann --- drivers/gpu/drm/drm_modes.c | 37 - include/drm/drm_crtc.h | 3 --- 2 files changed, 40 deletions(-) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/d

Re: [PATCH] drm: Add drm_bridge

2013-08-08 Thread Stéphane Marchesin
On Thu, Aug 8, 2013 at 5:36 PM, Daniel Vetter wrote: > On Thu, Aug 8, 2013 at 11:03 PM, Sean Paul wrote: >> This patch adds the notion of a drm_bridge. A bridge is a chained >> device which hangs off an encoder. The drm driver using the bridge >> should provide the association between encoder and

[PATCH] drm: Add drm_bridge

2013-08-08 Thread Stéphane Marchesin
On Thu, Aug 8, 2013 at 5:36 PM, Daniel Vetter wrote: > On Thu, Aug 8, 2013 at 11:03 PM, Sean Paul wrote: >> This patch adds the notion of a drm_bridge. A bridge is a chained >> device which hangs off an encoder. The drm driver using the bridge >> should provide the association between encoder and

[PATCH 3/3] drm: Remove drm_mode_validate_clocks

2013-07-30 Thread Stéphane Marchesin
Signed-off-by: Stéphane Marchesin --- drivers/gpu/drm/drm_modes.c | 37 - include/drm/drm_crtc.h | 3 --- 2 files changed, 40 deletions(-) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index a6729bf..504a602 100644 --- a/drivers

[PATCH 2/3] drm/i915: Remove useless define

2013-07-30 Thread Stéphane Marchesin
Signed-off-by: Stéphane Marchesin --- drivers/gpu/drm/i915/intel_display.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 5fb3058..37b33c9 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu

[PATCH 1/3] drm/gma500: Remove useless define

2013-07-30 Thread Stéphane Marchesin
Signed-off-by: Stéphane Marchesin --- drivers/gpu/drm/gma500/cdv_intel_display.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/gma500/cdv_intel_display.c b/drivers/gpu/drm/gma500/cdv_intel_display.c index 82430ad..d691a3a 100644 --- a/drivers/gpu/drm/gma500

[PATCH 3/3] drm: Remove drm_mode_validate_clocks

2013-07-30 Thread Stéphane Marchesin
Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/drm_modes.c | 37 - include/drm/drm_crtc.h | 3 --- 2 files changed, 40 deletions(-) diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index a6729bf..504a602 100644 --- a/drivers/

[PATCH 2/3] drm/i915: Remove useless define

2013-07-30 Thread Stéphane Marchesin
Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/i915/intel_display.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 5fb3058..37b33c9 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm

[PATCH 1/3] drm/gma500: Remove useless define

2013-07-30 Thread Stéphane Marchesin
Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/gma500/cdv_intel_display.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/gma500/cdv_intel_display.c b/drivers/gpu/drm/gma500/cdv_intel_display.c index 82430ad..d691a3a 100644 --- a/drivers/gpu/drm/gma500/cdv_intel_displ

Re: Best practice device tree design for display subsystems/DRM

2013-07-02 Thread Stéphane Marchesin
On Tue, Jul 2, 2013 at 3:02 PM, Dave Airlie wrote: > On Wed, Jul 3, 2013 at 7:50 AM, Sascha Hauer wrote: >> On Tue, Jul 02, 2013 at 09:25:48PM +0100, Russell King wrote: >>> On Tue, Jul 02, 2013 at 09:57:32PM +0200, Sebastian Hesselbarth wrote: >>> > I am against a super node which contains lcd a

Best practice device tree design for display subsystems/DRM

2013-07-02 Thread Stéphane Marchesin
On Tue, Jul 2, 2013 at 3:02 PM, Dave Airlie wrote: > On Wed, Jul 3, 2013 at 7:50 AM, Sascha Hauer > wrote: >> On Tue, Jul 02, 2013 at 09:25:48PM +0100, Russell King wrote: >>> On Tue, Jul 02, 2013 at 09:57:32PM +0200, Sebastian Hesselbarth wrote: >>> > I am against a super node which contains lc

[PATCH] drm: Initialize list in event->base.link

2013-06-15 Thread Stéphane Marchesin
The structure was kzalloced, so prev == next == NULL by default which is invalid. Signed-off-by: Stéphane Marchesin --- drivers/gpu/drm/drm_irq.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index 8bcce78..143a311 100644 --- a/drivers

[PATCH] drm: Initialize list in event->base.link

2013-06-14 Thread Stéphane Marchesin
The structure was kzalloced, so prev == next == NULL by default which is invalid. Signed-off-by: St?phane Marchesin --- drivers/gpu/drm/drm_irq.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/drm_irq.c b/drivers/gpu/drm/drm_irq.c index 8bcce78..143a311 100644 --- a/drivers/

Re: [pull] drm-intel-fixes

2013-05-22 Thread Stéphane Marchesin
On Wed, May 22, 2013 at 8:51 AM, Daniel Vetter wrote: > On Wed, May 22, 2013 at 5:25 PM, Stéphane Marchesin > wrote: >> On Wed, May 22, 2013 at 12:13 AM, Daniel Vetter wrote: >>> On Wed, May 22, 2013 at 3:24 AM, Stéphane Marchesin >>> wrote: >>>> On Mo

  1   2   >