[PATCH] drm: Add reference counting to drm_atomic_state

2016-08-23 Thread Chris Wilson
drm_atomic_state has a complicated single owner model that tracks the single reference from allocation through to destruction on another thread - or perhaps on a local error path. We can simplify this tracking by using reference counting (at a cost of a few more atomics). This is even more

[v14 PATCH 5/5] drm/rockchip: cdn-dp: add cdn DP support for rk3399

2016-08-23 Thread Chris Zhong
Add support for cdn DP controller which is embedded in the rk3399 SoCs. The DP is compliant with DisplayPort Specification, Version 1.3, This IP is compatible with the rockchip type-c PHY IP. There is a uCPU in DP controller, it need a firmware to work, please put the firmware file to

[v14 PATCH 0/5] Rockchip Type-C and DisplayPort driver

2016-08-23 Thread Chris Zhong
Hi all This series patch is for rockchip Type-C phy and DisplayPort controller driver. The USB Type-C PHY is designed to support the USB3 and DP applications. The PHY basically has two main components: USB3 and DisplyPort. USB3 operates in SuperSpeed mode and the DP can operate at RBR, HBR and

[drm-intel:topic/drm-misc 3/3] htmldocs: drivers/gpu/drm/drm_fb_helper.c:1069: warning: No description found for parameter 'suspend'

2016-08-23 Thread kbuild test robot
Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation -- next part -- A non-text attachment was scrubbed... Name: .config.gz Type: application/octet-stream Size: 6422 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160823/78f63634/attachment.obj>

[PATCH 1/1] drm/vmwgfx: avoid possible NULL pointer dereference

2016-08-23 Thread kbuild test robot
Hi Heinrich, [auto build test WARNING on drm/drm-next] [also build test WARNING on v4.8-rc3 next-20160823] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenie

[PATCH 2/2] drm: simple_kms_helper: add support for bridges

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 06:44:18PM +0200, Andrea Merello wrote: > On Tue, Aug 23, 2016 at 5:54 PM, Daniel Vetter wrote: > > > On Tue, Aug 23, 2016 at 05:39:36PM +0200, Andrea Merello wrote: > > > On Tue, Aug 23, 2016 at 5:20 PM, Daniel Vetter wrote: > > > > > > > On Tue, Aug 23, 2016 at

[PATCH 0/4] ARM: dts: exynos: Enable HDMI in Arndale Octa board

2016-08-23 Thread Krzysztof Kozlowski
On Tue, Aug 23, 2016 at 11:13 AM, Milo Kim wrote: > This patch-set enables HDMI in Arndale Octa board and fixes HPD DT property. > It also includes code refactoring on ddc and phy. > > Milo Kim (4): > ARM: dts: exynos: Enable HDMI for Arndale Octa board > ARM: dts: exynos: Use 'hpd-gpios'

[PULL] topic/drm-misc

2016-08-23 Thread Daniel Vetter
Hi Dave, A few bigger things: - start of splitting drm_crtc.c into more manageable and better documneted chunks - DRM_DEV_* logging (Sean) But besides those it's really all one-off misc patches all over this time around, trying to summarize them further would just end up in copying the

[drm-intel:for-linux-next 14/16] htmldocs: drivers/gpu/drm/i915/intel_dp.c:5063: warning: No description found for parameter 'dev_priv'

2016-08-23 Thread kbuild test robot
uct drm_i915_private *dev_priv = to_i915(dev); c395578e Vandana Kannan 2015-01-22 5191 :: The code at line 5063 was first introduced by commit :: 439d7ac0879f9fd4c56f212e03477f358133c56c drm/i915: Add support for DRRS to switch RR :: TO: Pradeep Bhat :: CC: Daniel Vetter --- 0-DAY kernel test infrastructureOpen Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation -- next part -- A non-text attachment was scrubbed... Name: .config.gz Type: application/octet-stream Size: 6422 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160823/8020c425/attachment-0001.obj>

[drm-intel:for-linux-next 1/1] include/linux/io-mapping.h:126:2: error: implicit declaration of function 'pgprot_writecombine'

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 9:16 PM, Chris Wilson wrote: > On Wed, Aug 24, 2016 at 02:22:29AM +0800, kbuild test robot wrote: >> tree: git://anongit.freedesktop.org/drm-intel for-linux-next >> head: ac96b5566926af83463ddcf4655856033c092f26 >> commit: ac96b5566926af83463ddcf4655856033c092f26

[PATCH v4 5/5] drm: simpledrm: honour remove_conflicting_framebuffers()

2016-08-23 Thread Noralf Trønnes
Den 23.08.2016 20:01, skrev Daniel Vetter: > On Tue, Aug 23, 2016 at 7:52 PM, Noralf Trønnes > wrote: +static int sdrm_fbdev_event_notify(struct notifier_block *self, + unsigned long action, void *data) +{ + struct sdrm_device *sdrm;

[PATCH] drm/fb-helper: Make docs for fb_set_suspend wrappers consistent

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 08:53:05PM +0200, Noralf Trønnes wrote: > > Den 23.08.2016 17:27, skrev Daniel Vetter: > > I figured I might as well go ocd and make them booleans and rename the > > locked version too. > > > > Reported-by: kbuild test robot > > Cc: Noralf Trønnes > > Fixes:

[Bug 97362] Low performance after suspend on RX 480

2016-08-23 Thread bugzilla-dae...@freedesktop.org
are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160823/49e3b12f/attachment.html>

[PATCH] drm/fb-helper: Make docs for fb_set_suspend wrappers consistent

2016-08-23 Thread Noralf Trønnes
Den 23.08.2016 17:27, skrev Daniel Vetter: > I figured I might as well go ocd and make them booleans and rename the > locked version too. > > Reported-by: kbuild test robot > Cc: Noralf Trønnes > Fixes: cfe63423d9be ("drm/fb-helper: Add > drm_fb_helper_set_suspend_unlocked()") >

[drm-intel:for-linux-next 1/1] include/linux/io-mapping.h:126:2: error: implicit declaration of function 'pgprot_writecombine'

2016-08-23 Thread Chris Wilson
On Wed, Aug 24, 2016 at 02:22:29AM +0800, kbuild test robot wrote: > tree: git://anongit.freedesktop.org/drm-intel for-linux-next > head: ac96b5566926af83463ddcf4655856033c092f26 > commit: ac96b5566926af83463ddcf4655856033c092f26 [1/1] io-mapping.h: > s/PAGE_KERNEL_IO/PAGE_KERNEL/ > config:

[PATCH v4 5/5] drm: simpledrm: honour remove_conflicting_framebuffers()

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 7:52 PM, Noralf Trønnes wrote: >>> +static int sdrm_fbdev_event_notify(struct notifier_block *self, >>> + unsigned long action, void *data) >>> +{ >>> + struct sdrm_device *sdrm; >>> + struct fb_event *event = data; >>> +

[PATCH v4 5/5] drm: simpledrm: honour remove_conflicting_framebuffers()

2016-08-23 Thread Noralf Trønnes
Den 23.08.2016 14:41, skrev Daniel Vetter: > On Mon, Aug 22, 2016 at 10:25:25PM +0200, Noralf Trønnes wrote: >> There is currently no non-fbdev mechanism in place to kick out >> simpledrm when the real hw-driver is probed. As a stop gap until >> that is in place, honour

[PATCH 0/3] drm/tilcdc: Some fixes for LCDC rev1

2016-08-23 Thread Jyri Sarha
Thanks a lot! This is very helpful as I do not have LCDC rev1 HW my self, but only am335x based boards. On 08/23/16 15:56, Karl Beldan wrote: > Hi, > > I found some missing bits for rev1 of the LCDC and here are some of the > changes I am using to use the DRM driver on an LCDCK (which has a

[PATCH] drm/nouveau/tegra: Modify error handling

2016-08-23 Thread Alexandre Courbot
On Mon, Aug 15, 2016 at 6:54 AM, Amitoj Kaur Chawla wrote: > iommu_domain_alloc returns NULL on error so replace an incorrect > IS_ERR check with a NULL check. > > The Coccinelle semantic patch used to find this issue is as follows: > @@ > expression e; > statement S; > @@ > > *e =

[PATCH v2 2/2] Documentation/sphinx: link dma-buf rsts

2016-08-23 Thread Sumit Semwal
Hi Jon, On 23 August 2016 at 18:38, Jonathan Corbet wrote: > On Tue, 23 Aug 2016 08:01:35 +0200 > Daniel Vetter wrote: > >> I'm also not too sure about whether dma-buf really should be it's own >> subdirectory. It's plucked from the device-drivers.tmpl, I think an >> overall device-drivers/ for

[PATCH 2/2] drm: simple_kms_helper: add support for bridges

2016-08-23 Thread Andrea Merello
of functions and #defines just to make > little one-line bits of code a notch more readable. > > And yes the symmetry is nice too ;-) > OK, then I'll add it :) Andrea > -Daniel > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch > -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160823/4b952b0d/attachment.html>

linux-next: build failure after merge of the drm-intel tree

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 11:56:14AM +1000, Stephen Rothwell wrote: > Hi all, > > After merging the drm-intel tree, today's linux-next build (arm > multi_v7_defconfig) failed like this: > > In file included from drivers/gpio/gpiolib-of.c:19:0: > include/linux/io-mapping.h:115:31: fatal error:

[PATCH v2] io-mapping.h: s/PAGE_KERNEL_IO/PAGE_KERNEL/

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 04:50:24PM +0100, Chris Wilson wrote: > PAGE_KERNEL_IO is an x86-ism. Though it is used to define the pgprot_t > used for the iomapped region, it itself is just PAGE_KERNEL. On all > other arches, PAGE_KERNEL_IO is undefined so in a general header we must > refrain from

[PATCH] gpu: ipu-v3: fix a possible NULL dereference

2016-08-23 Thread Corentin LABBE
On 23/08/2016 17:24, Sean Paul wrote: > On Tue, Aug 16, 2016 at 9:33 AM, LABBE Corentin > wrote: >> of_match_device could return NULL, and so cause a NULL pointer >> dereference later. >> >> For fixing this problem, we use of_device_get_match_data(), this will >> simplify the code a little by

[PATCH 4/4] gpu: drm: exynos_hdmi: Use consolidated function on binding PHY DT property

2016-08-23 Thread Milo Kim
Handle legacy and raw 'phy' parsing in single function. And it also removes goto condition. Cc: Inki Dae Cc: Joonyoung Shim Cc: Seung-Woo Kim Cc: Rob Herring Cc: devicetree at vger.kernel.org Cc: linux-arm-kernel at lists.infradead.org Cc: linux-samsung-soc at

[PATCH 3/4] gpu: drm: exynos_hdmi: Use consolidated function on binding DDC DT property

2016-08-23 Thread Milo Kim
Handle legacy and raw 'ddc' parsing in single function. And it also removes goto condition. Cc: Inki Dae Cc: Joonyoung Shim Cc: Seung-Woo Kim Cc: Rob Herring Cc: devicetree at vger.kernel.org Cc: linux-arm-kernel at lists.infradead.org Cc: linux-samsung-soc at

[PATCH 2/4] ARM: dts: exynos: Use 'hpd-gpios' instead of 'hpd-gpio'

2016-08-23 Thread Milo Kim
This patch enables getting a HPD GPIO descriptor quickly. The exynos-hdmi driver uses "hpd" for HDMI hot plug detection. static int hdmi_resources_init(struct hdmi_context *hdata) { ... hdata->hpd_gpio = devm_gpiod_get(dev, "hpd", GPIOD_IN);

[PATCH 1/4] ARM: dts: exynos: Enable HDMI for Arndale Octa board

2016-08-23 Thread Milo Kim
* Support HDMI display data channel I2C #2 is assigned for the HDMI DDC. It enables the EDID access. * GPIO for HDMI hot plug detect GPX3_7 is used. The HPD awareness is done when the GPIO is active high and single ended. * Enable HDMI block in Exynos5420 HDMI PLL consumes 1.0V LDO6

[PATCH 0/4] ARM: dts: exynos: Enable HDMI in Arndale Octa board

2016-08-23 Thread Milo Kim
This patch-set enables HDMI in Arndale Octa board and fixes HPD DT property. It also includes code refactoring on ddc and phy. Milo Kim (4): ARM: dts: exynos: Enable HDMI for Arndale Octa board ARM: dts: exynos: Use 'hpd-gpios' instead of 'hpd-gpio' gpu: drm: exynos_hdmi: Use consolidated

[PATCH 2/2] drm: simple_kms_helper: add support for bridges

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 05:39:36PM +0200, Andrea Merello wrote: > On Tue, Aug 23, 2016 at 5:20 PM, Daniel Vetter wrote: > > > On Tue, Aug 23, 2016 at 04:08:04PM +0200, Andrea Merello wrote: > > > Introduce drm_simple_display_pipe_attach_bridge() in order > > > to make it possible to use drm

[PATCH 2/2] drm: simple_kms_helper: add support for bridges

2016-08-23 Thread Andrea Merello
struct drm_device *dev, > > struct drm_simple_display_pipe *pipe, > > const struct drm_simple_display_pipe_funcs *funcs, > > -- > > 2.7.4 > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch > -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160823/2e28e30d/attachment-0001.html>

[PATCH] drm/fb-helper: Make docs for fb_set_suspend wrappers consistent

2016-08-23 Thread Daniel Vetter
I figured I might as well go ocd and make them booleans and rename the locked version too. Reported-by: kbuild test robot Cc: Noralf Trønnes Fixes: cfe63423d9be ("drm/fb-helper: Add drm_fb_helper_set_suspend_unlocked()") Signed-off-by: Daniel Vetter --- drivers/gpu/drm/drm_fb_helper.c | 8

[PATCH] drm/bridge: introduce bridge detaching mechanism

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 04:10:04PM +0200, Andrea Merello wrote: > Up to now, once a bridge has been attached to a DRM device, it cannot > be undone. > > In particular you couldn't rmmod/insmod a DRM driver that uses a bridge, > because the bridge would remain bound to the first (dead) driver

[PATCH 2/2] drm: simple_kms_helper: add support for bridges

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 04:08:04PM +0200, Andrea Merello wrote: > Introduce drm_simple_display_pipe_attach_bridge() in order > to make it possible to use drm encoders with the simple display > pipes managed by simple_kms_helpers > > Suggested-by: Daniel Vetter > Signed-off-by: Andrea Merello >

[PATCH 1/2] drm: simple_kms_helper: make connector optional at init time

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 04:05:57PM +0200, Andrea Merello wrote: > drm_simple_display_pipe_init() pretendes to attach a connector > to the display pipe. > > In case a drm bridge has to be used, then it's the bridge that > takes care of connectors. > > This patch makes the connector parameter

[PATCH v2 2/2] Documentation/sphinx: link dma-buf rsts

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 08:16:33AM -0600, Jonathan Corbet wrote: > On Tue, 23 Aug 2016 15:28:55 +0200 > Daniel Vetter wrote: > > > I think the more interesting story is, what's your plan with all the > > other driver related subsystem? Especially the ones which already have > > full directories

[PATCH v2] io-mapping.h: s/PAGE_KERNEL_IO/PAGE_KERNEL/

2016-08-23 Thread Chris Wilson
PAGE_KERNEL_IO is an x86-ism. Though it is used to define the pgprot_t used for the iomapped region, it itself is just PAGE_KERNEL. On all other arches, PAGE_KERNEL_IO is undefined so in a general header we must refrain from using it. v2: include pgtable for pgprot_combine() Reported-by: Stephen

[PATCH] drm/bridge: introduce bridge detaching mechanism

2016-08-23 Thread Andrea Merello
Up to now, once a bridge has been attached to a DRM device, it cannot be undone. In particular you couldn't rmmod/insmod a DRM driver that uses a bridge, because the bridge would remain bound to the first (dead) driver instance. This patch fixes this by introducing drm_encoder_detach() and a

[PATCH 2/2] drm: simple_kms_helper: add support for bridges

2016-08-23 Thread Andrea Merello
Introduce drm_simple_display_pipe_attach_bridge() in order to make it possible to use drm encoders with the simple display pipes managed by simple_kms_helpers Suggested-by: Daniel Vetter Signed-off-by: Andrea Merello Cc: Noralf Trønnes Cc: Daniel Vetter Cc: David Airlie diff --git

[PATCH 1/2] drm: simple_kms_helper: make connector optional at init time

2016-08-23 Thread Andrea Merello
drm_simple_display_pipe_init() pretendes to attach a connector to the display pipe. In case a drm bridge has to be used, then it's the bridge that takes care of connectors. This patch makes the connector parameter optional for drm_simple_display_pipe_init(), so that a drm bridge could handle

[PATCH v2 2/2] Documentation/sphinx: link dma-buf rsts

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 3:08 PM, Jonathan Corbet wrote: > On Tue, 23 Aug 2016 08:01:35 +0200 > Daniel Vetter wrote: > >> I'm also not too sure about whether dma-buf really should be it's own >> subdirectory. It's plucked from the device-drivers.tmpl, I think an >> overall device-drivers/ for all

[PATCH 2/4] ARM: dts: exynos: Use 'hpd-gpios' instead of 'hpd-gpio'

2016-08-23 Thread Rob Herring
On Tue, Aug 23, 2016 at 06:13:37PM +0900, Milo Kim wrote: > This patch enables getting a HPD GPIO descriptor quickly. > The exynos-hdmi driver uses "hpd" for HDMI hot plug detection. > > static int hdmi_resources_init(struct hdmi_context *hdata) > { > ... >

[PATCH] drm/doc: Add a few words on validation with IGT

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 11:15:31AM +0200, Tomeu Vizoso wrote: > Also provide some pointers for building IGT as some kernel hackers might > not be that familiar with building stuff on Linux distros. > > Signed-off-by: Tomeu Vizoso > Cc: Daniel Vetter > --- > Documentation/gpu/drm-uapi.rst | 37

[PATCH] drm/doc: Document uapi requirements in DRM

2016-08-23 Thread Daniel Vetter
Everyone knows them, except all the new folks joining from the ARM side haven't lived through all the pain of the past years and are entirely surprised when I raise this. Definitely time to document this. Last time this was a big discussion was about 6 years ago, when qcom tried to land a kernel

[Bug 91960] [i915] kernel warning hsw_unclaimed_reg_debug intel_uncore.c:619

2016-08-23 Thread bugzilla-dae...@freedesktop.org
hments/20160823/f7e40dea/attachment.html>

[Bug 91960] [i915] kernel warning hsw_unclaimed_reg_debug intel_uncore.c:619

2016-08-23 Thread bugzilla-dae...@freedesktop.org
org/archives/dri-devel/attachments/20160823/28449294/attachment-0001.html>

i865, drm_modeset_lock_all: BUG: unable to handle kernel NULL pointer dereference at 00000104

2016-08-23 Thread Meelis Roos
> > > Looks like an incorrect call to drm_encoder_cleanup() from the error > > > path. If we hit the error path we have never called drm_encoder_init. > > > Please try: > > > > > > diff --git a/drivers/gpu/drm/i915/intel_dvo.c > > > b/drivers/gpu/drm/i915/intel_dvo.c > > > index

[PATCH v4 5/5] drm: simpledrm: honour remove_conflicting_framebuffers()

2016-08-23 Thread Daniel Vetter
On Mon, Aug 22, 2016 at 10:25:25PM +0200, Noralf Trønnes wrote: > There is currently no non-fbdev mechanism in place to kick out > simpledrm when the real hw-driver is probed. As a stop gap until > that is in place, honour remove_conflicting_framebuffers() and > delete the simple-framebuffer

i865, drm_modeset_lock_all: BUG: unable to handle kernel NULL pointer dereference at 00000104

2016-08-23 Thread Meelis Roos
> Looks like an incorrect call to drm_encoder_cleanup() from the error > path. If we hit the error path we have never called drm_encoder_init. > Please try: > > diff --git a/drivers/gpu/drm/i915/intel_dvo.c > b/drivers/gpu/drm/i915/intel_dvo.c > index 47bdf9dad0d3..b9e5a63a7c9e 100644 > ---

[PATCH v7 6/8] Documentation: bindings: add dt documentation for rk3399 dmc

2016-08-23 Thread Chanwoo Choi
Hi Lin, I reply the on v6 patch[1]. If you have another opinion, please let me know. If my suggestion is not reasonable, we need to discuss it. [1] https://lkml.org/lkml/2016/8/23/28 Best Regards, Chanwoo Choi On 2016년 08월 22일 12:36, Lin Huang wrote: > This patch adds the documentation

[PATCH v7 4/8] Documentation: bindings: add dt documentation for dfi controller

2016-08-23 Thread Chanwoo Choi
Hi Lin, On 2016년 08월 22일 12:36, Lin Huang wrote: > This patch adds the documentation for rockchip dfi devfreq-event driver. > > Signed-off-by: Lin Huang > --- > Changes in v7: > -None > > Changes in v6: > -None > > Changes in v5: > -None > > Changes in v4: > -None > > Changes in v3: >

[Bug 95306] Random Blank(black) screens on "Carrizo"

2016-08-23 Thread bugzilla-dae...@freedesktop.org
bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160823/9c265afe/attachment.html>

[Bug 97371] AMDGPU/Iceland amdgpu: failed testing IB on ring 9/10

2016-08-23 Thread bugzilla-dae...@freedesktop.org
... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160823/4f55d11d/attachment.html>

[PATCH v2] drm/fb-helper: Add drm_fb_helper_set_suspend_unlocked()

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 01:54:06PM +0200, Noralf Trønnes wrote: > This adds a function that also takes the console lock before calling > fb_set_suspend() in contrast to drm_fb_helper_set_suspend() which is > a plain wrapper around fb_set_suspend(). > Resume is run asynchronously using a worker if

[PATCH v6 6/8] Documentation: bindings: add dt documentation for rk3399 dmc

2016-08-23 Thread Chanwoo Choi
Hi Lin, On 2016년 08월 22일 07:16, hl wrote: > Hi Chanwoo Choi, > > On 2016年08月17日 12:50, Chanwoo Choi wrote: >> Hi Lin, >> >> On 2016년 08월 17일 07:36, Lin Huang wrote: >>> This patch adds the documentation for rockchip rk3399 dmc driver. >>> >>> Signed-off-by: Lin Huang >>> ---

[PATCH v2] drm/fb-helper: Add drm_fb_helper_set_suspend_unlocked()

2016-08-23 Thread Noralf Trønnes
This adds a function that also takes the console lock before calling fb_set_suspend() in contrast to drm_fb_helper_set_suspend() which is a plain wrapper around fb_set_suspend(). Resume is run asynchronously using a worker if the console lock is already taken. This is modelled after the i915

[PATCH v2 2/2] Documentation/sphinx: link dma-buf rsts

2016-08-23 Thread Mauro Carvalho Chehab
Em Tue, 23 Aug 2016 08:16:33 -0600 Jonathan Corbet escreveu: > On Tue, 23 Aug 2016 15:28:55 +0200 > Daniel Vetter wrote: > > > I think the more interesting story is, what's your plan with all the > > other driver related subsystem? Especially the ones which already have > > full directories of

[PATCH] drm: Don't allow legacy cursor updates to stall others or be stalled

2016-08-23 Thread Daniel Vetter
On Tue, Aug 23, 2016 at 12:26:44PM +0100, Chris Wilson wrote: > On Tue, Aug 23, 2016 at 11:48:34AM +0100, Chris Wilson wrote: > > Legacy cursor updates are entirely asynchronous with respect to all > > other users of the atomic pipeline. They neither wait for any > > outstanding flips, nor do they

[RESEND PATCH v2 1/3] drm: dw_hdmi: use of_get_i2c_adapter_by_node interface

2016-08-23 Thread Vladimir Zapolskiy
Hello Russell, On 08/18/2016 05:32 PM, Russell King - ARM Linux wrote: > On Tue, Aug 16, 2016 at 11:26:43PM +0300, Vladimir Zapolskiy wrote: >> This change is needed to properly lock I2C bus driver, which serves >> DDC. >> >> The change fixes an overflow over zero of I2C bus driver user counter:

i865, drm_modeset_lock_all: BUG: unable to handle kernel NULL pointer dereference at 00000104

2016-08-23 Thread Meelis Roos
This on a P4 PC with 82865G chipset and onboard Intel graphics. 4.7.0 worked fine, current 4.8 git shows NULL pointer dereference as shown below at the end of dmesg. [0.00] Linux version 4.8.0-rc3-00013-gef0e1ea (mroos at vanalynx) (gcc version 5.4.0 20160609 (Debian 5.4.0-6) ) #34 Tue

[PATCH 3/3] drm/tilcdc: Advertise the DRM_FORMATs according to the IP revision

2016-08-23 Thread Karl Beldan
ATM the driver unconditionally advertises support for some 24bpp and 32bpp formats while version 1 of the IP only supports up to 16bpp. Signed-off-by: Karl Beldan --- drivers/gpu/drm/tilcdc/tilcdc_plane.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git

[PATCH 2/3] drm/tilcdc: Enable EOF interrupts for v1 LCDC

2016-08-23 Thread Karl Beldan
This got accidentally dropped in the fixed commit and is required for the driver to properly work on the rev1 IP, such as found on the LCDK. Fixes: 2b2080d7e9ae ("drm/tilcdc: Get rid of complex ping-pong mechanism") Signed-off-by: Karl Beldan --- drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 2 ++ 1

[PATCH 1/3] drm/tilcdc: Adjust the FB_CEILING address

2016-08-23 Thread Karl Beldan
The LCDC seems to expect its framebuffer ceiling address pointer to be an inclusive bound. The IP rev2 seems to cope with that but rev1 (as found on the LCDK) don't. Also note that this is what the framebuffer code does in da8xx-fb.c. Since, as the TRM puts it, "The 2 LSBs are hardwired to 00b",

[PATCH 0/3] drm/tilcdc: Some fixes for LCDC rev1

2016-08-23 Thread Karl Beldan
Hi, I found some missing bits for rev1 of the LCDC and here are some of the changes I am using to use the DRM driver on an LCDCK (which has a rev1). 1/3 seems required by rev1 of the IP and without it my the LCDC on my LCDK keeps can't sync, 2/3 is required by the driver logic, while 3/3 is less

[PATCH] Re: Mullins support in xf86-video-amdgpu

2016-08-23 Thread Michel Dänzer
On 23/08/16 11:57 AM, Reid Hekman wrote: > On 08/22/2016 08:27 PM, Michel Dänzer wrote: >> On 23/08/16 10:18 AM, Reid Hekman wrote: >>> >>> I was encouraged by a commit I saw today to add Sea Islands PCI ids to >>> xf86-video-amdgpu. However I did not see MULLINS included. Are there >>> other

[PATCH] drm: Don't allow legacy cursor updates to stall others or be stalled

2016-08-23 Thread Chris Wilson
On Tue, Aug 23, 2016 at 11:48:34AM +0100, Chris Wilson wrote: > Legacy cursor updates are entirely asynchronous with respect to all > other users of the atomic pipeline. They neither wait for any > outstanding flips, nor do they cause subsequent flips to be delayed. The > only ordering we do

[GIT PULL] drm/rockchip: Support rk3399/PSR + fixes

2016-08-23 Thread Sean Paul
Hi Dave, This pull request contains the following rockchip drm changes: - Introduce support for rk3399 vop/crtc - Add PSR framework to the rockchip driver - Implement PSR in the rockchip analogix edp driver - Fix panel on/off in analogix to avoid damaging panels - Some miscellaneous

[PATCH v13 4/7] drm/i915/skl: Update plane watermarks atomically during plane updates

2016-08-23 Thread Maarten Lankhorst
Op 22-08-16 om 18:50 schreef Lyude: > Thanks to Ville for suggesting this as a potential solution to pipe > underruns on Skylake. > > On Skylake all of the registers for configuring planes, including the > registers for configuring their watermarks, are double buffered. New > values written to

[PATCH 16/18] drm/etnaviv: handle MMU exception in IRQ handler

2016-08-23 Thread Lucas Stach
Hi Christian, Am Dienstag, den 23.08.2016, 11:58 +0200 schrieb Christian Gmeiner: > 2016-08-22 13:01 GMT+02:00 Lucas Stach : > > Bit 30 of the interrupt status signals an MMU exception. Handle this > > condition properly and dump some useful registers. > > > > Signed-off-by: Lucas Stach > > ---

[PATCH 16/18] drm/etnaviv: handle MMU exception in IRQ handler

2016-08-23 Thread Christian Gmeiner
2016-08-22 13:01 GMT+02:00 Lucas Stach : > Bit 30 of the interrupt status signals an MMU exception. Handle this > condition properly and dump some useful registers. > > Signed-off-by: Lucas Stach > --- > drivers/gpu/drm/etnaviv/etnaviv_gpu.c | 12 >

[PATCH] drm: Don't allow legacy cursor updates to stall others or be stalled

2016-08-23 Thread Chris Wilson
On Tue, Aug 23, 2016 at 11:48:34AM +0100, Chris Wilson wrote: > Legacy cursor updates are entirely asynchronous with respect to all > other users of the atomic pipeline. They neither wait for any > outstanding flips, nor do they cause subsequent flips to be delayed. The > only ordering we do

linux-next: build failure after merge of the drm-intel tree

2016-08-23 Thread Stephen Rothwell
Hi all, After merging the drm-intel tree, today's linux-next build (arm multi_v7_defconfig) failed like this: In file included from drivers/gpio/gpiolib-of.c:19:0: include/linux/io-mapping.h:115:31: fatal error: asm/pgtable_types.h: No such file or directory In file included from

[PATCH] drm: Don't allow legacy cursor updates to stall others or be stalled

2016-08-23 Thread Chris Wilson
Legacy cursor updates are entirely asynchronous with respect to all other users of the atomic pipeline. They neither wait for any outstanding flips, nor do they cause subsequent flips to be delayed. The only ordering we do require is given by making the legacy cursor update nonblocking (so the

[PATCH v2] drm/rockchip: Don't continue trying to enable crtc on failure

2016-08-23 Thread Sean Paul
On Wed, Aug 17, 2016 at 3:28 AM, Yakir Yang wrote: > Sean, > > On 08/16/2016 07:12 AM, Sean Paul wrote: >> >> If vop_enable fails, don't continue on, it causes system hangs. >> >> Signed-off-by: Sean Paul > > > Also meet this problem on my Rk3399 Kevin board. VOP just failed to get the >

[PATCH] drm/rockchip: Delete unnecessary assignment for the field "owner"

2016-08-23 Thread Sean Paul
On Tue, Aug 16, 2016 at 8:43 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 16 Aug 2016 14:25:35 +0200 > > The field "owner" is set by the core. > Thus delete an unneeded initialisation. > > Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci > Signed-off-by:

[PATCH] drm/bridge: dw-hdmi: Delete unnecessary assignment for the field "owner"

2016-08-23 Thread Sean Paul
On Tue, Aug 16, 2016 at 7:58 AM, SF Markus Elfring wrote: > From: Markus Elfring > Date: Tue, 16 Aug 2016 13:52:19 +0200 > > The field "owner" is set by the core. > Thus delete an unneeded initialisation. > > Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci > Signed-off-by:

[PATCH] gpu: ipu-v3: fix a possible NULL dereference

2016-08-23 Thread Sean Paul
On Tue, Aug 16, 2016 at 9:33 AM, LABBE Corentin wrote: > of_match_device could return NULL, and so cause a NULL pointer > dereference later. > > For fixing this problem, we use of_device_get_match_data(), this will > simplify the code a little by using a standard function for > getting the match

[PATCH v2 2/2] Documentation/sphinx: link dma-buf rsts

2016-08-23 Thread Markus Heiser
Am 23.08.2016 um 08:01 schrieb Daniel Vetter : > On Mon, Aug 22, 2016 at 12:49:30PM -0300, Mauro Carvalho Chehab wrote: >> Em Mon, 22 Aug 2016 20:41:45 +0530 >> Sumit Semwal escreveu: >> >>> Include dma-buf sphinx documentation into top level index. >>> >>> Signed-off-by: Sumit Semwal >>>

[PATCH] drm/doc: Add a few words on validation with IGT

2016-08-23 Thread Tomeu Vizoso
Also provide some pointers for building IGT as some kernel hackers might not be that familiar with building stuff on Linux distros. Signed-off-by: Tomeu Vizoso Cc: Daniel Vetter --- Documentation/gpu/drm-uapi.rst | 37 + 1 file changed, 37 insertions(+)

i865, drm_modeset_lock_all: BUG: unable to handle kernel NULL pointer dereference at 00000104

2016-08-23 Thread Chris Wilson
On Tue, Aug 23, 2016 at 02:35:03PM +0300, Meelis Roos wrote: > > Looks like an incorrect call to drm_encoder_cleanup() from the error > > path. If we hit the error path we have never called drm_encoder_init. > > Please try: > > > > diff --git a/drivers/gpu/drm/i915/intel_dvo.c > >

[RFC 0/2] New feature: Framebuffer processors

2016-08-23 Thread Daniel Stone
Hi, On 22 August 2016 at 16:23, Rob Clark wrote: > I guess a lot comes down to 'how long before hw designers bolt a CP to > the thing'.. at that point, I think you especially don't want a > per-blit kernel interface. Regardless of whether or not we want it, we already _have_ it, in the form of

Mullins support in xf86-video-amdgpu

2016-08-23 Thread Michel Dänzer
On 23/08/16 10:18 AM, Reid Hekman wrote: > > I was encouraged by a commit I saw today to add Sea Islands PCI ids to > xf86-video-amdgpu. However I did not see MULLINS included. Are there > other showstoppers preventing inclusion? I assume the commit you're referring to was the one adding the

[PATCH] drm/fsl-dcu: Fix endian issue when using clk_register_divider

2016-08-23 Thread Meng Yi
> >>fsl_dev->pix_clk = clk_register_divider(dev, pix_clk_name, > >>pix_clk_in_name, 0, base + DCU_DIV_RATIO, > >> - 0, 8, CLK_DIVIDER_ROUND_CLOSEST, NULL); > >> + 24, 8, CLK_DIVIDER_ROUND_CLOSEST, NULL); > > > > Tested-by: Meng Yi > > > >

[PATCH] drm-misc: arm64 now also must be compile-tested before pushing

2016-08-23 Thread Daniel Vetter
hisilicon unfortunately requires this since it's a warn-fest on 32bit builds. Cc: seanpaul at chromium.org Cc: sumit.semwal at linaro.org Cc: sumit.semwal at linaro.org Cc: architt at codeaurora.org Signed-off-by: Daniel Vetter --- drm-misc.rst | 4 ++-- 1 file changed, 2 insertions(+), 2

[RESEND PATCH v13 7/7] drm/i915/skl: Update DDB values atomically with wms/plane attrs

2016-08-23 Thread Lyude
Now that we can hook into update_crtcs and control the order in which we update CRTCs at each modeset, we can finish the final step of fixing Skylake's watermark handling by performing DDB updates at the same time as plane updates and watermark updates. The first major change in this patch is

[RESEND PATCH v13 6/7] drm/i915: Move CRTC updating in atomic_commit into it's own hook

2016-08-23 Thread Lyude
Since we have to write ddb allocations at the same time as we do other plane updates, we're going to need to be able to control the order in which we execute modesets on each pipe. The easiest way to do this is to just factor this section of intel_atomic_commit_tail() (intel_atomic_commit() for

i865, drm_modeset_lock_all: BUG: unable to handle kernel NULL pointer dereference at 00000104

2016-08-23 Thread Chris Wilson
On Tue, Aug 23, 2016 at 12:58:43PM +0300, Meelis Roos wrote: > This on a P4 PC with 82865G chipset and onboard Intel graphics. 4.7.0 > worked fine, current 4.8 git shows NULL pointer dereference as shown > below at the end of dmesg. > > [ 10.066261] BUG: unable to handle kernel NULL pointer

[PATCH RFC 4/5] drm/bridge: add dw-hdmi cec driver using Hans Verkil's CEC code

2016-08-23 Thread Hans Verkuil
On 08/23/16 09:59, Russell King - ARM Linux wrote: > On Tue, Aug 23, 2016 at 09:21:17AM +0200, Hans Verkuil wrote: >> Hi Russell, >> >> On 08/12/2016 04:15 PM, Russell King wrote: >>> Add a CEC driver for the dw-hdmi hardware using Hans Verkil's CEC >>> implementation. >>> >>> Signed-off-by:

[PATCH RFC 4/5] drm/bridge: add dw-hdmi cec driver using Hans Verkil's CEC code

2016-08-23 Thread Hans Verkuil
Hi Russell, On 08/12/16 16:15, Russell King wrote: > + ret = devm_request_threaded_irq(>dev, cec->irq, > + dw_hdmi_cec_hardirq, > + dw_hdmi_cec_thread, IRQF_SHARED, > + DEV_NAME,

[PATCH] drm-misc: Rename drm-misc defconfigs to end with _defconfig

2016-08-23 Thread Sean Paul
Instead of -defconfig, to be consistent with in-tree naming convention Cc: daniel.vetter at intel.com Cc: sumit.semwal at linaro.org Cc: architt at codeaurora.org Signed-off-by: Sean Paul --- drm-misc-arm64-defconfig => drm-misc-arm64_defconfig | 0 drm-misc-arm-defconfig =>

[urgent] drm/fb-helper: Fix the dummy remove_conflicting_framebuffers

2016-08-23 Thread Daniel Vetter
On Mon, Aug 22, 2016 at 09:44:52PM +0100, Chris Wilson wrote: > We always need to remove conflicting framebuffers if any other fb driver > is enabled, and not just if we are setting up an fbdev ourselves. > > Unfortunately remove_conflicting_framebuffers() was incorrectly stubbed > out if !fbdev

[PATCH RFC 4/5] drm/bridge: add dw-hdmi cec driver using Hans Verkil's CEC code

2016-08-23 Thread Hans Verkuil
Hi Russell, On 08/12/2016 04:15 PM, Russell King wrote: > Add a CEC driver for the dw-hdmi hardware using Hans Verkil's CEC > implementation. > > Signed-off-by: Russell King > --- > drivers/gpu/drm/bridge/Kconfig| 7 + >

[PATCH v3 1/5] drm/rockchip: sort registers define by chip's number

2016-08-23 Thread Sean Paul
On Mon, Aug 22, 2016 at 8:40 PM, Mark yao wrote: > On 2016年08月23日 04:30, Sean Paul wrote: >> >> On Thu, Aug 18, 2016 at 6:02 AM, Mark yao wrote: >>> >>> On 2016年08月18日 17:11, Daniel Vetter wrote: On Thu, Aug 18, 2016 at 05:08:14PM +0800, Mark yao wrote: >> >> Hi

[PATCH RFC 4/5] drm/bridge: add dw-hdmi cec driver using Hans Verkil's CEC code

2016-08-23 Thread Russell King - ARM Linux
On Tue, Aug 23, 2016 at 09:21:17AM +0200, Hans Verkuil wrote: > Hi Russell, > > On 08/12/2016 04:15 PM, Russell King wrote: > > Add a CEC driver for the dw-hdmi hardware using Hans Verkil's CEC > > implementation. > > > > Signed-off-by: Russell King > > --- > >

[PATCH v3 1/5] drm/rockchip: sort registers define by chip's number

2016-08-23 Thread Mark yao
On 2016年08月23日 04:30, Sean Paul wrote: > On Thu, Aug 18, 2016 at 6:02 AM, Mark yao wrote: >> On 2016年08月18日 17:11, Daniel Vetter wrote: >>> On Thu, Aug 18, 2016 at 05:08:14PM +0800, Mark yao wrote: > Hi Sean > > Thanks for send v3 patch for rk3399 vop support. > >

[PATCH] drm/simple-helpers: Always add planes to the state update

2016-08-23 Thread Daniel Vetter
Our update function is hooked to the single plane, which might not get called for crtc-only updates. Which is surprising, so fix this by always adding the plane. While at it document how the event should be sent out better in the kerneldoc. Cc: Noralf Trønnes Cc: andrea.merello at gmail.com

[PATCH v4 3/5] drm: add SimpleDRM driver

2016-08-23 Thread Daniel Vetter
On Mon, Aug 22, 2016 at 10:25:23PM +0200, Noralf Trønnes wrote: > The SimpleDRM driver binds to simple-framebuffer devices and provides a > DRM/KMS API. It provides only a single CRTC+encoder+connector combination > plus one initial mode. > > Userspace can create dumb-buffers which can be blit

[PATCH v2 2/2] Documentation/sphinx: link dma-buf rsts

2016-08-23 Thread Jonathan Corbet
On Tue, 23 Aug 2016 15:28:55 +0200 Daniel Vetter wrote: > I think the more interesting story is, what's your plan with all the > other driver related subsystem? Especially the ones which already have > full directories of their own, like e.g. Documentation/gpio/. I think > those should be really

[PATCH v4 4/5] drm: simpledrm: add fbdev fallback support

2016-08-23 Thread Daniel Vetter
On Mon, Aug 22, 2016 at 10:25:24PM +0200, Noralf Trønnes wrote: > Create a simple fbdev device during SimpleDRM setup so legacy user-space > and fbcon can use it. > > Original work by David Herrmann. > > Cc: dh.herrmann at gmail.com > Signed-off-by: Noralf Trønnes > --- > > Changes from

[PATCH v4 2/5] drm/fb-helper: Add drm_fb_helper_set_suspend_lock()

2016-08-23 Thread Daniel Vetter
On Mon, Aug 22, 2016 at 10:25:22PM +0200, Noralf Trønnes wrote: > This adds a function that also takes the console lock before calling > fb_set_suspend() in contrast to drm_fb_helper_set_suspend() which is > a plain wrapper around fb_set_suspend(). > Resume is run asynchronously using a worker if

  1   2   >