[drm-intel:drm-intel-next-queued 2/2] drivers/gpu/drm/i915/i915_gem_stolen.c:516:20: warning: unused variable 'ggtt'

2016-11-17 Thread kbuild test robot
- next part -- A non-text attachment was scrubbed... Name: .config.gz Type: application/gzip Size: 25157 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/6f938c34/attachment-0001.gz>

[PATCH 1/2] devicetree/bindings: display: Add bindings for LVDS panels

2016-11-17 Thread Laurent Pinchart
Hi Rob, On Friday 14 Oct 2016 07:40:14 Rob Herring wrote: > On Sun, Oct 9, 2016 at 11:33 AM, Laurent Pinchartwrote: > > On Saturday 08 Oct 2016 20:29:39 Rob Herring wrote: > >> On Tue, Oct 04, 2016 at 07:23:29PM +0300, Laurent Pinchart wrote: > >>> LVDS is a physical layer specification defined in

[PATCH 4/5] drm/i915: Find fallback link rate/lane count

2016-11-17 Thread Manasi Navare
If link training fails, then we need to fallback to lower link rate first and if link training fails at RBR, then fallback to lower lane count. This function finds the next lower link rate/lane count value after link training failure. v4: * Remove the redundant variable link_train_failed v3: * Rem

[PATCH 5/5] drm/i915: Implement Link Rate fallback on Link training failure

2016-11-17 Thread Manasi Navare
If link training at a link rate optimal for a particular mode fails during modeset's atomic commit phase, then we let the modeset complete and then retry. We save the link rate value at which link training failed, update the link status property to "BAD" and use a lower link rate to prune the modes

[Bug 98619] amdgpu 0000:01:00.0: GPU fault detected: 146 0x09d88404 (Shadow of Mordor)

2016-11-17 Thread bugzilla-dae...@freedesktop.org
-- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/6fae0653/attachment.html>

[PATCH 5/5] drm/i915: Implement Link Rate fallback on Link training failure

2016-11-17 Thread Manasi Navare
If link training at a link rate optimal for a particular mode fails during modeset's atomic commit phase, then we let the modeset complete and then retry. We save the link rate value at which link training failed, update the link status property to "BAD" and use a lower link rate to prune the modes

[PATCH 4/5] drm/i915: Find fallback link rate/lane count

2016-11-17 Thread Manasi Navare
If link training fails, then we need to fallback to lower link rate first and if link training fails at RBR, then fallback to lower lane count. This function finds the next lower link rate/lane count value after link training failure. v3: * Remove fallback_link_rate_index variable, just obtain tha

[PATCH 3/5] drm/i915: Update CRTC state if connector link status property changed

2016-11-17 Thread Manasi Navare
CRTC state connector_changed needs to be set to true if connector link status property has changed. This will tell the driver to do a complete modeset due to change in connector property. Acked-by: Harry Wentland Acked-by: Tony Cheng Cc: dri-devel at lists.freedesktop.org Cc: Jani Nikula Cc: Da

[PATCH 2/5] drm: Set DRM connector link status property

2016-11-17 Thread Manasi Navare
In the usual working scenarios, this property is "Good". If something fails during modeset, the DRM driver can set the link status to "Bad", prune the mode list based on the link rate/lane count fallback values and send hotplug uevent so that userspace that is aware of this property can take an ap

[PATCH 1/5] drm: Add a new connector property for link status

2016-11-17 Thread Manasi Navare
At the time userspace does setcrtc, we've already promised the mode would work. The promise is based on the theoretical capabilities of the link, but it's possible we can't reach this in practice. The DP spec describes how the link should be reduced, but we can't reduce the link below the requireme

[PATCH 0/5] Link Training failure handling during modeset

2016-11-17 Thread Manasi Navare
Submitting this series again since it had to be rebased dur to changes in drm that got merged. The idea presented in these patches is to address link training failure in a way that: a) changes the current happy day scenario as little as possible, to avoid regressions, b) can be implemented the sam

[PATCH] drm: document standard connector properties

2016-11-17 Thread Manasi Navare
On Thu, Nov 17, 2016 at 09:56:48AM +0100, Daniel Vetter wrote: > There's a really big pile of additional connector properties, a lot of > them standardized. But they're all for specific outputs (panels, TV, > scaling, ...) so I left them out for now since this is enough for a > start. > > I typed

[Bug 98664] Fragment shader while loop causes geometry corruption

2016-11-17 Thread bugzilla-dae...@freedesktop.org
s.freedesktop.org/archives/dri-devel/attachments/20161117/9ad0d2fd/attachment.html>

[Intel-gfx] [PATCH 3/3] drm/dp/mst: Track available time slots in DP Multi-Stream Transport Packet

2016-11-17 Thread Manasi Navare
On Thu, Nov 17, 2016 at 06:03:48PM -0800, Dhinakaran Pandiyan wrote: > The avail_slots member in struct drm_dp_mst_topology_mgr does not really > track the available time slots in a MTP(Multi-Stream Transport Packet). It > is assigned an initial value when the topology manager is setup but not > up

[PATCH v3 2/3] drm/bridge: Add ti-tfp410 DVI transmitter driver

2016-11-17 Thread Jyri Sarha
On 11/17/16 19:20, Laurent Pinchart wrote: >> > Oops, forgot to handle the return values. This is how I fixed it: >> > static int __init tfp410_module_init(void) >> > { >> >int ret; >> > >> >ret = i2c_add_driver(&tfp410_i2c_driver); >> >if (ret) >> >return ret; >> > >> >

[Bug 98761] [regression][radeonsi][polaris]"radeonsi: set IF_THRESHOLD to 3" breaks Witcher2's ground

2016-11-17 Thread bugzilla-dae...@freedesktop.org
. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/2c88cfa9/attachment.html>

[Bug 98761] [regression][radeonsi][polaris]"radeonsi: set IF_THRESHOLD to 3" breaks Witcher2's ground

2016-11-17 Thread bugzilla-dae...@freedesktop.org
bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/6b9a5b84/attachment.html>

[PATCH] ARM: dts: am335x-boneblack: Add blue-and-red-wiring -property to LCDC node

2016-11-17 Thread Jyri Sarha
Add blue-and-red-wiring -property to LCDC node. Also adds comments on how to get support 24 bit RGB mode. After this patch am335x-boneblack support RGB565, BGR888, and XBGR color formats. See details in Documentation/devicetree/bindings/display/tilcdc/tilcdc.txt. The BBB has straight color wir

[PATCH v2 0/7] drm/tilcdc: LCDC Revision 1 related fixes

2016-11-17 Thread Jyri Sarha
On 11/17/16 13:31, Bartosz Golaszewski wrote: > 2016-11-16 19:00 GMT+01:00 Jyri Sarha : >> On 11/16/16 17:18, Bartosz Golaszewski wrote: >>> 2016-11-16 13:40 GMT+01:00 Jyri Sarha : Changes since first version of the series: - Move tilcdc_regs.h changes from "drm/tilcdc: Enable palett

[Bug 98664] Fragment shader while loop causes geometry corruption

2016-11-17 Thread bugzilla-dae...@freedesktop.org
An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/4b16e039/attachment-0001.html>

[PATCH] amdkfd: fix spelling mistake in kfd_ioctl_dbg_unrgesiter

2016-11-17 Thread Oded Gabbay
On Sat, Nov 12, 2016 at 7:33 PM, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistake, rename kfd_ioctl_dbg_unrgesiter > to kfd_ioctl_dbg_unregister > > Signed-off-by: Colin Ian King > --- > drivers/gpu/drm/amd/amdkfd/kfd_chardev.c | 6 +++--- > 1 file changed, 3 inserti

[PATCH v4 2/2] ARM: dts: sun6i: hummingbird-a31: Enable display output through VGA bridge

2016-11-17 Thread Maxime Ripard
: 801 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/14fb64a4/attachment.sig>

[PATCH 14/32] drm/vmwgfx: Populate fb->pixel_format

2016-11-17 Thread Ville Syrjälä
On Thu, Nov 17, 2016 at 07:52:23PM +0200, Laurent Pinchart wrote: > Hi Ville, > > Thank you for the patch. > > On Thursday 17 Nov 2016 18:14:13 ville.syrjala at linux.intel.com wrote: > > From: Ville Syrjälä > > > > Stuff something semi-reasonable into fb->pixel_format. I had to guess > > as

[GIT PULL] hisilicon hibmc DRM driver for 4.10

2016-11-17 Thread Rongrong Zou
Hi David, This pull request contains 7 patches adding a new drm driver for hisilicon hibmc SoC. Thanks, Rongrong. The following changes since commit b7c0e47d98249c2ddf21ea197b651093c6aaee00: Merge tag 'drm-vc4-next-2016-11-16' of https://github.com/anholt/linux into drm-next (2016-11-17 09:

[PATCH v2 31/32] drm: Nuke fb->bits_per_pixel

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Replace uses of fb->bits_per_pixel with fb->format->cpp[0]*8. Less duplicated information is a good thing. Note that I didn't put parens around the cpp*8 in the below cocci script, on account of not wanting spurious parens all over the place. Instead I did the unsafe way, a

DRM: urgent v4.9-rc6 build regression: master build: 2 failures 1 warnings (v4.9-rc5-213-g961b708)

2016-11-17 Thread Dave Airlie
t, but -fixes I don't always do it, since who would send broken -fixes, Arnd could you send a git pull with the two reverts, with my Acked-by on it? I won't be in a place to do it for 8-9hrs. Dave. > > Arnd > ___ > dri-devel mailing list > dri-devel at lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/10429d4e/attachment.html>

[PATCH 31/32] drm: Nuke fb->bits_per_pixel

2016-11-17 Thread Laurent Pinchart
Hi Ville, On Thursday 17 Nov 2016 20:35:46 Ville Syrjälä wrote: > On Thu, Nov 17, 2016 at 08:14:16PM +0200, Laurent Pinchart wrote: > > On Thursday 17 Nov 2016 18:14:30 ville.syrjala at linux.intel.com wrote: > >> From: Ville Syrjälä > >> > >> Replace uses of fb->bits_per_pixel with fb->form

[PATCH v3] PCI: create revision file in sysfs

2016-11-17 Thread Bjorn Helgaas
On Fri, Nov 18, 2016 at 10:42:20AM +0900, Michel Dänzer wrote: > On 18/11/16 08:48 AM, Bjorn Helgaas wrote: > > > > Popping the stack all the way back to Emil's Nov 8 message: > > > > When using the Mesa drivers alongside firefox [1] (since Mesa 13.0), > > glxinfo (Mesa 10.0) and others, all

[PATCH 32/32] drm: Nuke fb->pixel_format

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:31 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > Replace uses of fb->bits_per_pixel with fb->format->format. I assume you mean fb->pixel_format. > Less duplicated information is a good thing. > > Note that

[PATCH 31/32] drm: Nuke fb->bits_per_pixel

2016-11-17 Thread Ville Syrjälä
On Thu, Nov 17, 2016 at 08:14:16PM +0200, Laurent Pinchart wrote: > Hi Ville, > > Thank you for the patch. > > On Thursday 17 Nov 2016 18:14:30 ville.syrjala at linux.intel.com wrote: > > From: Ville Syrjälä > > > > Replace uses of fb->bits_per_pixel with fb->format->cpp[0]*8. > > Less duplic

[GIT PULL] hisilicon hibmc DRM driver for 4.10

2016-11-17 Thread Rongrong Zou
Hi David, This pull request contains 7 patches adding a new drm driver for hisilicon hibmc SoC. Thanks, Rongrong. The following changes since commit b7c0e47d98249c2ddf21ea197b651093c6aaee00: Merge tag 'drm-vc4-next-2016-11-16' of https://github.com/anholt/linux into drm-next (2016-11-17 09:

[PATCH 19/32] drm: Store a pointer to drm_format_info under drm_framebuffer

2016-11-17 Thread Laurent Pinchart
Hi Ville, On Thursday 17 Nov 2016 20:13:42 Ville Syrjälä wrote: > On Thu, Nov 17, 2016 at 08:06:10PM +0200, Ville Syrjälä wrote: > > On Thu, Nov 17, 2016 at 07:57:27PM +0200, Laurent Pinchart wrote: > >> On Thursday 17 Nov 2016 18:14:18 ville.syrjala at linux.intel.com wrote: > >>> From: Ville

[PATCH 31/32] drm: Nuke fb->bits_per_pixel

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:30 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > Replace uses of fb->bits_per_pixel with fb->format->cpp[0]*8. > Less duplicated information is a good thing. > > Note that I didn't put parens around the cpp*

[PATCH 19/32] drm: Store a pointer to drm_format_info under drm_framebuffer

2016-11-17 Thread Ville Syrjälä
On Thu, Nov 17, 2016 at 08:06:10PM +0200, Ville Syrjälä wrote: > On Thu, Nov 17, 2016 at 07:57:27PM +0200, Laurent Pinchart wrote: > > Hi Ville, > > > > Thank you for the patch. > > > > On Thursday 17 Nov 2016 18:14:18 ville.syrjala at linux.intel.com wrote: > > > From: Ville Syrjälä > > >

[PATCH 19/32] drm: Store a pointer to drm_format_info under drm_framebuffer

2016-11-17 Thread Ville Syrjälä
On Thu, Nov 17, 2016 at 07:57:27PM +0200, Laurent Pinchart wrote: > Hi Ville, > > Thank you for the patch. > > On Thursday 17 Nov 2016 18:14:18 ville.syrjala at linux.intel.com wrote: > > From: Ville Syrjälä > > > > To avoid having to look up the format information struct every time, > > let'

[PATCH] drm/sun4i: tcon: Initialize regmap after enabling bus clocks

2016-11-17 Thread Maxime Ripard
bedded Linux and Kernel engineering http://free-electrons.com -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/ac4950fb/attachment-0001.sig>

[PATCH 30/32] drm: Nuke fb->depth

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:29 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > Replace uses of fb->depth with fb->format->depth. Less duplicate > information is a good thing. > > @@ > struct drm_framebuffer *fb; > expression E1, E2; > @@

[PATCH] drm/sun4i: Only count TCON endpoints as valid outputs

2016-11-17 Thread Maxime Ripard
nux and Kernel engineering http://free-electrons.com -- next part -- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 801 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/9e378540/attachment.sig>

[PATCH v2] vgaarb: Use dev_printk() when possible

2016-11-17 Thread Daniel Vetter
On Thu, Nov 17, 2016 at 11:47:58AM -0600, Bjorn Helgaas wrote: > Use dev_printk() when possible. This makes messages more consistent with > other device-related messages and, in some cases, adds useful information. > This changes messages like this: > > vgaarb: failed to allocate pci device >

[PATCH 23/32] drm/fb_cma_helper: Replace drm_format_info() with fb->format

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:22 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > Get the format information via the neat fb->format pointer rather than > doing a linear search over all the format info structures. > > Signed-off-by: Ville S

[PATCH 22/32] drm/atomic: Replace drm_format_num_planes() with fb->format->num_planes

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:21 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > Avoid the format info lookup and just use the pointer stored under > drm_framebuffer. > > Signed-off-by: Ville Syrjälä > --- > drivers/gpu/drm/drm_atomic.

[PATCH 19/32] drm: Store a pointer to drm_format_info under drm_framebuffer

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:18 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > To avoid having to look up the format information struct every time, > let's just store a pointer to it under drm_framebuffer. > > Signed-off-by: Ville Syrjä

[PATCH 14/32] drm/vmwgfx: Populate fb->pixel_format

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:13 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > Stuff something semi-reasonable into fb->pixel_format. I had to guess > as to which formats we should pick. Did I guess correctly? > > We can't quite use drm_

[PATCH 18/32] drm: Populate fb->dev from drm_helper_mode_fill_fb_struct()

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:17 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > Populating fb->dev before drm_framebuffer_init() allows us to use > fb->dev already while validating the framebuffer. Let's have > drm_helper_mode_fill_fb_stru

[PATCH 13/32] drm: Pass 'dev' to drm_helper_mode_fill_fb_struct()

2016-11-17 Thread Laurent Pinchart
Hi Ville, Thank you for the patch. On Thursday 17 Nov 2016 18:14:12 ville.syrjala at linux.intel.com wrote: > From: Ville Syrjälä > > Pass the drm_device to drm_helper_mode_fill_fb_struct() so that we can > populate fb->dev early. Will make it easier to use the fb before we > register it. >

[PATCH v3 2/3] drm/bridge: Add ti-tfp410 DVI transmitter driver

2016-11-17 Thread Laurent Pinchart
Hi Jyri, On Thursday 17 Nov 2016 15:39:26 Jyri Sarha wrote: > On 11/17/16 15:28, Jyri Sarha wrote: > > Add very basic ti-ftp410 DVI transmitter driver. The only feature > > separating this from a completely dummy bridge is the EDID read > > support trough DDC I2C. Even that functionality should be

[Bug 90320] Lenovo ThinkPad E455 (Kaveri A10-7300): Blank built-in screen with radeon kms driver

2016-11-17 Thread bugzilla-dae...@freedesktop.org
ubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/9e6174d9/attachment.html>

[Bug 90320] Lenovo ThinkPad E455 (Kaveri A10-7300): Blank built-in screen with radeon kms driver

2016-11-17 Thread bugzilla-dae...@freedesktop.org
-- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/3a30a576/attachment.html>

[Bug 90320] Lenovo ThinkPad E455 (Kaveri A10-7300): Blank built-in screen with radeon kms driver

2016-11-17 Thread bugzilla-dae...@freedesktop.org
nee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/afca74e0/attachment.html>

[Bug 90320] Lenovo ThinkPad E455 (Kaveri A10-7300): Blank built-in screen with radeon kms driver

2016-11-17 Thread bugzilla-dae...@freedesktop.org
attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/9b375596/attachment.html>

[Bug 90320] Lenovo ThinkPad E455 (Kaveri A10-7300): Blank built-in screen with radeon kms driver

2016-11-17 Thread bugzilla-dae...@freedesktop.org
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/20161117/d20f5ebe/attachment.html>

[Bug 90320] Lenovo ThinkPad E455 (Kaveri A10-7300): Blank built-in screen with radeon kms driver

2016-11-17 Thread bugzilla-dae...@freedesktop.org
- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/3bcbedaf/attachment.html>

[PATCH 32/32] drm: Nuke fb->pixel_format

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Replace uses of fb->bits_per_pixel with fb->format->format. Less duplicated information is a good thing. Note that coccinelle failed to eliminate the "/* fourcc format */" comment from drm_framebuffer.h, so I had to do that part manually. @@ struct drm_framebuffer *fb; exp

[PATCH 31/32] drm: Nuke fb->bits_per_pixel

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Replace uses of fb->bits_per_pixel with fb->format->cpp[0]*8. Less duplicated information is a good thing. Note that I didn't put parens around the cpp*8 in the below cocci script, on account of not wanting spurious parens all over the place. Instead I did the unsafe way, a

[PATCH 30/32] drm: Nuke fb->depth

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Replace uses of fb->depth with fb->format->depth. Less duplicate information is a good thing. @@ struct drm_framebuffer *fb; expression E1, E2; @@ drm_helper_mode_fill_fb_struct(...) { ... - fb->depth = E1; ... } @@ struct vmw_framebuffer_surface *v

[PATCH 29/32] drm/i915: Use drm_framebuffer_plane_{width, height}() where possible

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Replace drm_format_plane_{width,height}() usage with drm_framebuffer_plane_{width,height}() to avoid the lookup of the format info. Cc: intel-gfx at lists.freedesktop.org Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 5 ++--- 1 file changed, 2

[PATCH 28/32] drm: Add drm_framebuffer_plane_{width,height}()

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add variants of drm_format_plane_{width,height}() that take an entire fb object instead of just the format. These should be more efficent as they can just look up the format info from the fb->format pointer rather than having to look it up (using a linear search based on the

[PATCH 27/32] drm/i915: Replace drm_format_num_planes() with fb->format->num_planes

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä No need to look up the format info to get the number of planes, just use fb->format->num_planes directly. Cc: intel-gfx at lists.freedesktop.org Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/intel_display.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 26/32] drm/i915: Replace drm_format_plane_cpp() with fb->format->cpp[]

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Avoid the fb format information lookups when we already have the fb->format pointer at hand. @@ struct drm_framebuffer *fb; expression E; @@ - drm_format_plane_cpp(fb->pixel_format, E) + fb->format->cpp[E] @@ struct drm_framebuffer fb; expression E; @@ - drm_format_plane_c

[PATCH 25/32] drm/i915: Store a pointer to the pixel format info for fbc

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Rather than store the pixel format and look up the format info as needed, let's just store a pointer to the format info directly and speed up our lookups. Cc: Paulo Zanoni Cc: intel-gfx at lists.freedesktop.org Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/i91

[PATCH 24/32] drm/nouveau: Use fb->format rather than drm_format_info()

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Let's use the pointer to the format information cached under drm_framebuffer rather than look it up manually. Cc: Ben Skeggs Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/nouveau/nv50_display.c | 12 +--- 1 file changed, 5 insertions(+), 7 deletions(-) diff

[PATCH 23/32] drm/fb_cma_helper: Replace drm_format_info() with fb->format

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Get the format information via the neat fb->format pointer rather than doing a linear search over all the format info structures. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_fb_cma_helper.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a

[PATCH 22/32] drm/atomic: Replace drm_format_num_planes() with fb->format->num_planes

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Avoid the format info lookup and just use the pointer stored under drm_framebuffer. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_atomic.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomi

[PATCH 21/32] drm/i915: Populate fb->format early for inherited fbs

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä MAke sure the framebuffer format info is available as early as possible for fbs we inherit from the BIOS. This will allow us to use the fb as if it was fully formed before we register it. Cc: intel-gfx at lists.freedesktop.org Signed-off-by: Ville Syrjälä --- drivers/gp

[PATCH 20/32] drm/vmwgfx: Populate fb->format correctly

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Populate the fb->format to point at the correct format information structure. Cc: linux-graphics-maintainer at vmware.com Cc: Sinclair Yeh Cc: Thomas Hellstrom Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 2 ++ 1 file changed, 2 insertions(+)

[PATCH 19/32] drm: Store a pointer to drm_format_info under drm_framebuffer

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä To avoid having to look up the format information struct every time, let's just store a pointer to it under drm_framebuffer. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_framebuffer.c| 4 +++- drivers/gpu/drm/drm_modeset_helper.c | 1 + include/drm/drm_frame

[PATCH 18/32] drm: Populate fb->dev from drm_helper_mode_fill_fb_struct()

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Populating fb->dev before drm_framebuffer_init() allows us to use fb->dev already while validating the framebuffer. Let's have drm_helper_mode_fill_fb_struct() do that for us. Also make drm_framebuffer_init() warn us if a different device pointer is passed to it than was pa

[PATCH 17/32] drm/i915: Set fb->dev early on for inherited fbs

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä We want the fbs inherited from the BIOS to be more or less fully working prior to actually registering them. This will allow us to just pass the fb to various helper function instead of having to pass all the different parameters separately. Cc: intel-gfx at lists.freedeskt

[PATCH 16/32] drm/virtio: Call drm_helper_mode_fill_fb_struct() before drm_framebuffer_init()

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä We want framebuffers to be mostly useable already before drm_framebuffer_init() get called, and so we will start demanding that all the interesting format/size/etc. information be filled in before drm_framebuffer_init(). drm_helper_mode_fill_fb_struct() will do that for us,

[PATCH 15/32] drm/qxl: Call drm_helper_mode_fill_fb_struct() before drm_framebuffer_init()

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä We want framebuffers to be mostly useable already before drm_framebuffer_init() is called, and so we will start demanding that all the interesting format/size/etc. information be filled in before drm_framebuffer_init(). drm_helper_mode_fill_fb_struct() will do that for us, s

[PATCH 14/32] drm/vmwgfx: Populate fb->pixel_format

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Stuff something semi-reasonable into fb->pixel_format. I had to guess as to which formats we should pick. Did I guess correctly? We can't quite use drm_mode_legacy_fb_format() due to the ARGB1555 vs. XRGB155 mess. However use of 'A' formats should imply per-pixel alpha blen

[PATCH 13/32] drm: Pass 'dev' to drm_helper_mode_fill_fb_struct()

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Pass the drm_device to drm_helper_mode_fill_fb_struct() so that we can populate fb->dev early. Will make it easier to use the fb before we register it. @@ identifier fb, mode_cmd; @@ void drm_helper_mode_fill_fb_struct( +struct drm_devic

[PATCH 12/32] drm/vmwgfx: Populate fb->dev before drm_framebuffer_init()

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä drm_framebuffer_init() will start to check that fb->dev is already populated, so let's to that manually since vmwgfx isn't using drm_helper_mode_fill_fb_struct(). Cc: linux-graphics-maintainer at vmware.com Cc: Sinclair Yeh Cc: Thomas Hellstrom Signed-off-by: Ville Syrjä

[PATCH 11/32] drm/nouveau: Add local 'fb' variables

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my poor coccinelle skills later. Cc: Ben Skeggs Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/nouveau/dispnv04/crtc.c | 5 +++-- drivers/gpu/drm/nou

[PATCH 10/32] drm/nouveau: Fix crtc->primary->fb vs. drm_fb fail

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä So it looks like the code is trying to pick between the passed in fb and crtc->primary->fb based on that funky 'bool atomic'. But later it will mix uses of both drm_fb (which was picked by the aforementioned logic) and crtc->primary->fb. So looks like a bug to me. Let's make

[PATCH 09/32] drm/arm: Add local 'fb' variables

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my ppor coccinelle skills later. In some places the local variable was already there, just not used consistently. Cc: Liviu Dudau Cc: Brian Starkey Cc: M

[PATCH 08/32] drm/arcpgu: Add local 'fb' variables

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my ppor coccinelle skills later. Cc: Alexey Brodkin Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/arc/arcpgu_crtc.c | 3 ++- 1 file changed, 2 inser

[PATCH 07/32] drm/cirrus: Add some local 'fb' variables

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my poor coccinelle skills later. Cc: Dave Airlie Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/cirrus/cirrus_mode.c | 9 + 1 file changed, 5

[PATCH 06/32] drm/gma500: Add some local 'fb' variables

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my poor coccinelle skills later. Cc: Patrik Jakobsson Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/gma500/gma_display.c | 13 +++--

[PATCH 05/32] drm/ast: Add local 'fb' variables

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my poor coccinelle skills later. Cc: Dave Airlie Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/ast/ast_mode.c | 15 ++- 1 file changed,

[PATCH 04/32] drm/mgag200: Add local 'fb' variable

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my poor coccinelle skills later. Cc: Dave Airlie Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/mgag200/mgag200_mode.c | 17 + 1 file

[PATCH 03/32] drm/radeon: Use DIV_ROUND_UP()

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Use DIV_ROUND_UP() instead of hand rolling it. Just a drive-by change. Cc: Alex Deucher Cc: "Christian König" Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/radeon/radeon_legacy_crtc.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers

[PATCH 02/32] drm/radeon: Add local 'fb' variables

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my poor coccinelle skills later. Cc: Alex Deucher Cc: "Christian König" Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/radeon/r100.c

[PATCH 01/32] drm/i915: Add local 'fb' variables

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä Add a local 'fb' variable to a few places to get rid of the 'crtc->primary->fb' stuff. Looks neater and helps me with my poor coccinelle skills later. While at it switch over to using the pixel format rather than depth+bpp. Cc: intel-gfx at lists.freedesktop.org Signed-off

[PATCH 00/32] drm: Deduplicate fb format information

2016-11-17 Thread ville.syrj...@linux.intel.com
From: Ville Syrjälä This series aims to remove the duplicated format information stored under drm_framebuffer (depth,bits_per_pixel,pixel_format), and instead we just use the drm_format_info structure. And we store a pointer to the approriate drm_format_info under drm_framebuffer so that we don

[PATCH 3/3] drm/dp/mst: Track available time slots in DP Multi-Stream Transport Packet

2016-11-17 Thread Dhinakaran Pandiyan
The avail_slots member in struct drm_dp_mst_topology_mgr does not really track the available time slots in a MTP(Multi-Stream Transport Packet). It is assigned an initial value when the topology manager is setup but not updated after that. So, let's use avail_slots to store the number of unallocat

[PATCH 2/3] drm/dp/mst: Calculate total link bandwidth instead of hardcoding it

2016-11-17 Thread Dhinakaran Pandiyan
The total or the nominal link bandwidth, which we save in terms of PBN, is a factor of link rate and lane count. But, currently we hardcode it to 2560 PBN. This results in incorrect computation of total slots. E.g, 2 lane HBR2 configuration and 4k at 60Hz, 24bpp mode nominal link bw = 1080 MBps

[PATCH 1/3] drm/i915/dp: Fail DP MST config when there are not enough vcpi slots

2016-11-17 Thread Dhinakaran Pandiyan
drm_dp_find_vcpi_slots() returns an error when there is not enough available bandwidth on a link to support a mode. This error should make compute_config() to fail. Not returning false could end up in a modeset which will not work. Signed-off-by: Dhinakaran Pandiyan --- drivers/gpu/drm/i915/inte

[PATCH 0/3] Track available link bandwidth for DP MST

2016-11-17 Thread Dhinakaran Pandiyan
The number of available time slots in a MTP should be updated when a new vcpi is added or an existing one is removed. Keeping this updated will be useful to reject modes for which there is not enough link bandwidth early. Dhinakaran Pandiyan (3): drm/i915/dp: Fail DP MST config when there are no

[Bug 90320] Lenovo ThinkPad E455 (Kaveri A10-7300): Blank built-in screen with radeon kms driver

2016-11-17 Thread bugzilla-dae...@freedesktop.org
nt was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/2fab31f0/attachment.html>

[PATCH v3] PCI: create revision file in sysfs

2016-11-17 Thread Bjorn Helgaas
On Thu, Nov 17, 2016 at 03:44:02PM +0100, Lukas Wunner wrote: > On Wed, Nov 16, 2016 at 02:58:11PM -0600, Bjorn Helgaas wrote: > > On Mon, Nov 14, 2016 at 07:40:03PM +0100, Daniel Vetter wrote: > > > On Fri, Nov 11, 2016 at 02:37:23PM +, Emil Velikov wrote: > > > > From: Emil Velikov > > > >

[PATCH] ARM: dts: am335x-boneblack: Add blue-and-red-wiring -property to LCDC node

2016-11-17 Thread Tony Lindgren
* Jyri Sarha [161117 12:14]: > Add blue-and-red-wiring -property to LCDC node. Also adds comments on > how to get support 24 bit RGB mode. After this patch am335x-boneblack > support RGB565, BGR888, and XBGR color formats. See details in > Documentation/devicetree/bindings/display/tilcdc/tilcd

[Bug 98664] Fragment shader while loop causes geometry corruption

2016-11-17 Thread bugzilla-dae...@freedesktop.org
he bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/98302d39/attachment.html>

[Bug 98664] Fragment shader while loop causes geometry corruption

2016-11-17 Thread bugzilla-dae...@freedesktop.org
strip decomposition or not.) -- You 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/20161117/b6110722/attachment-0001.html>

AMDGPU DAL branch

2016-11-17 Thread Sandeep
ves/dri-devel/attachments/20161117/5517f226/attachment.html>

[Bug 98761] [regression][radeonsi][polaris]"radeonsi: set IF_THRESHOLD to 3" breaks Wither2's ground

2016-11-17 Thread bugzilla-dae...@freedesktop.org
the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161117/62ce829a/attachment.html>

[PATCH] drm/i2c/tda998x: fix spelling mistake

2016-11-17 Thread Russell King - ARM Linux
On Mon, Nov 14, 2016 at 10:46:43PM +, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistake "configutation" to "configuration" > in dev_err message > > Signed-off-by: Colin Ian King Applied, thanks. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patche

[PATCH 1/2] drm/i2c: tda998x: allow interrupt to be shared

2016-11-17 Thread Russell King - ARM Linux
On Thu, Nov 17, 2016 at 12:24:57PM +, Brian Starkey wrote: > I tested these two and the power-down one on mali-dp and hdlcd. The > output, hotplugging and EDID continued to work; so you can have my > tested-by and reviewed-by for all 3. Thanks! -- RMK's Patch system: http://www.armlinux.org.

[Bug 98761] [regression][radeonsi][polaris]"radeonsi: set IF_THRESHOLD to 3" breaks Wither2's ground

2016-11-17 Thread bugzilla-dae...@freedesktop.org
Rs Reviewed-by: Nicolai Hähnle :04 04 2c773601e2956578e8da4a6031cb02837255b953 7d5f7c3faebe4a11631e70bde258d78fa7357008 M src -- You are receiving this mail because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesk

DRM: urgent v4.9-rc6 build regression: master build: 2 failures 1 warnings (v4.9-rc5-213-g961b708)

2016-11-17 Thread Arnd Bergmann
On Thursday, November 17, 2016 8:50:05 PM CET Dave Airlie wrote: > > Arnd could you send a git pull with the two reverts, with my Acked-by on > it? I won't be in a place to do it for 8-9hrs. I don't think it's that urgent, as long as we make sure it's fixed in the next -rc. I've sent out the reve

  1   2   >