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

Hi, Arnd: I've made a mistake that I've tried to build these patches on v4.9-rc1, but I does not set CONFIG_DRM_MEDIATEK=y, therefore I didn't find out these build fails. Now I fix the config problem, and I think I should build these patches on latest kernel version even though patch's owner test

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

freedesktop.org/archives/dri-devel/attachments/20161118/924cb5f4/attachment-0001.gz>

[PATCH RFC 01/12] drm/i2c: tda998x: avoid race in tda998x_encoder_mode_set()

On Tue, Nov 08, 2016 at 12:25:00PM +, Russell King wrote: > As priv->audio_params can now be changed at run time, we need to be more > careful about how we deal with a mode set. We must take the audio lock > while checking if there's a valid audio configuration. > > However, it's slightly

[git pull] drm fixes for rc6 (brown paper bag edition)

Hi, So some i915 fixes came in which I thought they might so I'm sending those along with two reverts for two patches to the mediatek driver that didn't seem to build so well, I've fixed up my -fixes ARM build and .config so I could see it, but yes brown paper bag time. Apologies, Dave. The

[PATCH v3] PCI: create revision file in sysfs

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 the GPUs* will be awaken, > causing unwanted delays and increased power

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

On Fri, Nov 18, 2016 at 3:02 AM, Maxime Ripard wrote: > On Wed, Nov 16, 2016 at 05:37:31PM +0800, Chen-Yu Tsai wrote: >> The sun4i DRM driver counts the number of endpoints it found and >> registers the whole DRM pipeline if any endpoints are found. >> >> However, if the TCON and its child

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

On Fri, Nov 18, 2016 at 3:05 AM, Maxime Ripard wrote: > On Wed, Nov 16, 2016 at 05:37:32PM +0800, Chen-Yu Tsai wrote: >> If we attempt to read/write the TCON registers before the bus clock >> is enabled, those accesses get ignored. >> >> In practice this almost never occurs because U-boot had

[PATCH v9 09/10] drm/mediatek: update DSI sub driver flow for sending commands to panel

Hi YT, Sorry for the very late review. My biggest problem with this patch is it describes itself as adding support for a new use case "DSI -> panel", but makes many changes to the existing working flow "DSI -> bridge -> panel". If these changes are really needed, or improve the existing flow,

[PATCH v5] drm/mediatek: fixed the calc method of data rate per lane

Hi CK, On Thu, Nov 17, 2016 at 1:36 PM, CK Hu wrote: > Hi, Jitao: > > > On Wed, 2016-11-16 at 11:20 +0800, Jitao Shi wrote: >> Tune dsi frame rate by pixel clock, dsi add some extra signal (i.e. >> Tlpx, Ths-prepare, Ths-zero, Ths-trail,Ths-exit) when enter and exit LP >> mode, those signals

[PATCH v9 02/10] drm/mediatek: add *driver_data for different hardware settings

Hi YT, I don't see a reason to handle device_data in such a generic way at the generic mtk_ddp_comp layer. The device data is very component specific, so just define different structs for different comp types, ie: struct mtk_disp_ovl_driver_data { unsigned int reg_ovl_addr; unsigned int

[PATCH v5] drm/mediatek: fixed the calc method of data rate per lane

Hi, Daniel: On Fri, 2016-11-18 at 11:22 +0800, Daniel Kurtz wrote: > Hi CK, > > On Thu, Nov 17, 2016 at 1:36 PM, CK Hu wrote: > > Hi, Jitao: > > > > > > On Wed, 2016-11-16 at 11:20 +0800, Jitao Shi wrote: > >> Tune dsi frame rate by pixel clock, dsi add some extra signal (i.e. > >> Tlpx,

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

/commits/Dhinakaran-Pandiyan/Track-available-link-bandwidth-for-DP-MST/20161118-101200 base: git://people.freedesktop.org/~airlied/linux.git drm-next reproduce: make htmldocs; make DOCBOOKS='' pdfdocs All warnings (new ones prefixed by >>): make[3]: warning: jobserver unavailable: usi

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

On Thu, Nov 17, 2016 at 06:03:48PM -0800, Dhinakaran Pandiyan wrote: > static int drm_dp_init_vcpi(struct drm_dp_mst_topology_mgr *mgr, > struct drm_dp_vcpi *vcpi, int pbn) > { > - int num_slots; > + int req_slots; > int ret; > > - num_slots =

[PATCH v2] Revert "dma-buf/sync-file: Avoid enable fence signaling if poll(.timeout=0)"

From: Gustavo Padovan This reverts commit ecebca79f6976ddaddfd054d699272515869ea28. Do not enable fence callback on poll() when using fence_array causes the fence_array to not signal. For now we will revert the change and enable signaling everytime time poll is

[Intel-gfx] [PATCH] drm: Define drm_mm_for_each_node_in_range()

On to, 2016-11-17 at 14:49 +, Chris Wilson wrote: > Some clients would like to iterate over every node within a certain > range. Make a nice little macro for them to hide the mixing of the > rbtree search and linear walk. > > v2: Blurb > > Signed-off-by: Chris Wilson > Cc: Daniel Vetter >

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

On Thu, Nov 17, 2016 at 06:03:46PM -0800, Dhinakaran Pandiyan wrote: > 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

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

On Fri, Nov 18, 2016 at 06:57:01AM +, Chris Wilson wrote: > On Thu, Nov 17, 2016 at 06:03:48PM -0800, Dhinakaran Pandiyan wrote: > > static int drm_dp_init_vcpi(struct drm_dp_mst_topology_mgr *mgr, > > struct drm_dp_vcpi *vcpi, int pbn) > > { > > - int num_slots; >

[PATCH] drm: document standard connector properties

On Thu, Nov 17, 2016 at 10:36:12PM -0800, Manasi Navare wrote: > 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

[PATCH] drm/vgem: Allow root to inject hanging fences onto dmabufs

On Wed, Jul 20, 2016 at 08:39:43PM +0100, Chris Wilson wrote: > When performing driver testing, one factor we want to test is how we > handle a foreign fence that is never signaled. We can wait on that fence > indefinitely, in which case the driver appears hung, or we can take some > remedial

[PATCH v2] Revert "dma-buf/sync-file: Avoid enable fence signaling if poll(.timeout=0)"

On Fri, Nov 18, 2016 at 05:26:43PM +0900, Gustavo Padovan wrote: > From: Gustavo Padovan > > This reverts commit ecebca79f6976ddaddfd054d699272515869ea28. > > Do not enable fence callback on poll() when using fence_array causes the > fence_array to not signal. > > For now we will revert the

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

On Fri, Nov 18, 2016 at 9:43 AM, Daniel Vetter wrote: > On Thu, Nov 17, 2016 at 06:03:46PM -0800, Dhinakaran Pandiyan wrote: >> 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

[PATCH v2] Revert "dma-buf/sync-file: Avoid enable fence signaling if poll(.timeout=0)"

On Fri, Nov 18, 2016 at 08:56:11AM +, Chris Wilson wrote: > On Fri, Nov 18, 2016 at 05:26:43PM +0900, Gustavo Padovan wrote: > > From: Gustavo Padovan > > > > This reverts commit ecebca79f6976ddaddfd054d699272515869ea28. > > > > Do not enable fence callback on poll() when using fence_array

[Intel-gfx] [PATCH] drm/vgem: Allow root to inject hanging fences onto dmabufs

On Fri, Nov 18, 2016 at 08:49:37AM +, Chris Wilson wrote: > On Wed, Jul 20, 2016 at 08:39:43PM +0100, Chris Wilson wrote: > > When performing driver testing, one factor we want to test is how we > > handle a foreign fence that is never signaled. We can wait on that fence > > indefinitely, in

[PATCH v3] PCI: create revision file in sysfs

On Thu, Nov 17, 2016 at 08:40:10PM -0600, Bjorn Helgaas wrote: > 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

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

On Thu, Nov 17, 2016 at 02:20:59PM -0600, Bjorn Helgaas wrote: > On Thu, Nov 17, 2016 at 07:59:21PM +0100, Daniel Vetter wrote: > > 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

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

oks like in game before and after. -- 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/20161118/7a21eae4/attachment.html>

[Intel-gfx] [PATCH] drm/vgem: Allow root to inject hanging fences onto dmabufs

On Fri, Nov 18, 2016 at 10:40:02AM +0100, Daniel Vetter wrote: > On Fri, Nov 18, 2016 at 08:49:37AM +, Chris Wilson wrote: > > On Wed, Jul 20, 2016 at 08:39:43PM +0100, Chris Wilson wrote: > > > When performing driver testing, one factor we want to test is how we > > > handle a foreign fence

[PATCH v3] PCI: create revision file in sysfs

On Fri, Nov 18, 2016 at 10:42:07AM +0100, Daniel Vetter wrote: > On Thu, Nov 17, 2016 at 08:40:10PM -0600, Bjorn Helgaas wrote: > > 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

[Bug 188091] New: Resume with two monitors, second monitor is not resumed until VT switch

https://bugzilla.kernel.org/show_bug.cgi?id=188091 Bug ID: 188091 Summary: Resume with two monitors, second monitor is not resumed until VT switch Product: Drivers Version: 2.5 Kernel Version: 4.9 rc5 Hardware: All

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

2016-11-17 21:06 GMT+01:00 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

v4.9-rc3: graphical artefacts in X

e.asc Type: application/pgp-signature Size: 181 bytes Desc: Digital signature URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161118/33ead4bf/attachment.sig>

[Intel-gfx] v4.9-rc3: graphical artefacts in X

On Fri, Nov 18, 2016 at 12:02:56PM +0100, Pavel Machek wrote: > Hi! > > With v4.9, if I maximize "nowcast -x" application, I get broken > display (as if someone split the window into rectangles and shuffled > them a bit). Switching virtual desktops either fixes it or breaks it, > depending in how

[Bug 98738] AMD E450| HD 6320 - crashing playback with kodi, maybe vdpau problem, mesa, r600

.org/archives/dri-devel/attachments/20161118/416891fe/attachment.html>

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

On Fri, 18 Nov 2016, Manasi Navare wrote: > 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: >

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

On Fri, 18 Nov 2016, Manasi Navare wrote: > 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 >

[PATCH v2 1/7] drm/tilcdc: Enable sync lost error and recovery handling for rev 1 LCDC

2016-11-16 13:41 GMT+01:00 Jyri Sarha : > Revision 1 LCDC support also sync lost errors and can benefit from > sync lost recovery routine. > Hi Jyri, I think I found the issue with this patch. Please see below. > Signed-off-by: Jyri Sarha > --- > drivers/gpu/drm/tilcdc/tilcdc_crtc.c | 39 >

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

Op 18-11-16 om 08:13 schreef 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:

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

On Fri, Nov 18, 2016 at 02:50:52PM +0100, Maarten Lankhorst wrote: > Op 18-11-16 om 08:13 schreef 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

[PATCH] vgaarb: Downgrade userspace-triggerable messages to debug

To avoid userspace-triggerable dmesg spam, downgrade messages in the sysfs write parsing code to debug level. Signed-off-by: Bjorn Helgaas --- drivers/gpu/vga/vgaarb.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/vga/vgaarb.c

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

Op 18-11-16 om 15:11 schreef Ville Syrjälä: > On Fri, Nov 18, 2016 at 02:50:52PM +0100, Maarten Lankhorst wrote: >> Op 18-11-16 om 08:13 schreef Manasi Navare: >>> CRTC state connector_changed needs to be set to true >>> if connector link status property has changed. This will tell the >>>

[PATCH v3] PCI: create revision file in sysfs

On Fri, Nov 18, 2016 at 4:48 AM, Daniel Vetter wrote: > On Fri, Nov 18, 2016 at 10:42:07AM +0100, Daniel Vetter wrote: >> On Thu, Nov 17, 2016 at 08:40:10PM -0600, Bjorn Helgaas wrote: >> > On Fri, Nov 18, 2016 at 10:42:20AM +0900, Michel Dänzer wrote: >> > > On 18/11/16 08:48 AM, Bjorn Helgaas

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

On Fri, Nov 18, 2016 at 04:11:47PM +0200, Ville Syrjälä wrote: > On Fri, Nov 18, 2016 at 02:50:52PM +0100, Maarten Lankhorst wrote: > > Op 18-11-16 om 08:13 schreef Manasi Navare: > > > CRTC state connector_changed needs to be set to true > > > if connector link status property has changed. This

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

On Fri, Nov 18, 2016 at 03:31:48PM +0200, Jani Nikula wrote: > On Fri, 18 Nov 2016, Manasi Navare wrote: > > 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 > >

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

On Fri, Nov 18, 2016 at 03:18:06PM +0100, Maarten Lankhorst wrote: > Op 18-11-16 om 15:11 schreef Ville Syrjälä: > > On Fri, Nov 18, 2016 at 02:50:52PM +0100, Maarten Lankhorst wrote: > >> Op 18-11-16 om 08:13 schreef Manasi Navare: > >>> CRTC state connector_changed needs to be set to true >

[PATCH] vgaarb: Downgrade userspace-triggerable messages to debug

On Fri, Nov 18, 2016 at 08:11:58AM -0600, Bjorn Helgaas wrote: > To avoid userspace-triggerable dmesg spam, downgrade messages in the sysfs > write parsing code to debug level. > > Signed-off-by: Bjorn Helgaas Applied to drm-misc, thanks. -Daniel > --- > drivers/gpu/vga/vgaarb.c | 10

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

attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161118/378ebb78/attachment.html>

[PATCH v2 7/7] drm/tilcdc: Load palette at the end of mode_set_nofb()

2016-11-16 13:41 GMT+01:00 Jyri Sarha : > Load palette at the end of mode_set_nofb() and only if the palette has > not been loaded since last runtime resume. Moving the palette loading > to mode_set_nofb() saves us from storing and restoring of LCDC dma > addresses that were just recently updated.

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

are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161118/54ce97a8/attachment.html>

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

On Fri, Nov 18, 2016 at 05:28:54PM +0200, Ville Syrjälä wrote: > On Fri, Nov 18, 2016 at 03:18:06PM +0100, Maarten Lankhorst wrote: > > Op 18-11-16 om 15:11 schreef Ville Syrjälä: > > > On Fri, Nov 18, 2016 at 02:50:52PM +0100, Maarten Lankhorst wrote: > > >> Op 18-11-16 om 08:13 schreef

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

On Fri, Nov 18, 2016 at 03:22:49PM +0200, Jani Nikula wrote: > On Fri, 18 Nov 2016, Manasi Navare wrote: > > 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

[PATCH v2 7/7] drm/tilcdc: Load palette at the end of mode_set_nofb()

2016-11-18 16:34 GMT+01:00 Bartosz Golaszewski : > 2016-11-16 13:41 GMT+01:00 Jyri Sarha : >> Load palette at the end of mode_set_nofb() and only if the palette has >> not been loaded since last runtime resume. Moving the palette loading >> to mode_set_nofb() saves us from storing and restoring of

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

On Fri, Nov 18, 2016 at 04:35:25PM +0100, Daniel Vetter wrote: > On Fri, Nov 18, 2016 at 05:28:54PM +0200, Ville Syrjälä wrote: > > On Fri, Nov 18, 2016 at 03:18:06PM +0100, Maarten Lankhorst wrote: > > > Op 18-11-16 om 15:11 schreef Ville Syrjälä: > > > > On Fri, Nov 18, 2016 at 02:50:52PM

[PATCHv3 3/4] drm/tilcdc: Use unload to handle initialization failures

2016-11-02 16:57 GMT+01:00 Jyri Sarha : > Use unload to handle initialization failures instead of complex goto > label mess. To do this the initialization sequence needed slight > reordering and some unload functions needed to become conditional. > > Signed-off-by: Jyri Sarha > --- I'm not sure

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

On Fri, Nov 18, 2016 at 06:21:21PM +0200, Ville Syrjälä wrote: > On Fri, Nov 18, 2016 at 04:35:25PM +0100, Daniel Vetter wrote: > > On Fri, Nov 18, 2016 at 05:28:54PM +0200, Ville Syrjälä wrote: > > > On Fri, Nov 18, 2016 at 03:18:06PM +0100, Maarten Lankhorst wrote: > > > > Op 18-11-16 om

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

are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161118/5e6ef835/attachment.html>

[Bug 185681] amdgpu: powerplay initialization failed

https://bugzilla.kernel.org/show_bug.cgi?id=185681 René Linder changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

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

On Fri, Nov 18, 2016 at 04:35:25PM +0100, Daniel Vetter wrote: > On Fri, Nov 18, 2016 at 05:28:54PM +0200, Ville Syrjälä wrote: > > On Fri, Nov 18, 2016 at 03:18:06PM +0100, Maarten Lankhorst wrote: > > > Op 18-11-16 om 15:11 schreef Ville Syrjälä: > > > > On Fri, Nov 18, 2016 at 02:50:52PM

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

-- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161118/e944d18d/attachment.html>

[PATCH v2 00/37] drm: Deduplicate fb format information (v2)

From: Ville Syrjälä Second installment of my effort to remove the duplicated depth/bpp/pixel_format from drm_framebuffer and just use struct drm_format_info instead. I tried to address all of the review feedback, and collect up all the r-bs I already got. Thanks

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

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

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

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ä Reviewed-by: Alex

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

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ä Reviewed-by: Alex Deucher --- drivers/gpu/drm/radeon/radeon_legacy_crtc.c | 5 ++--- 1 file

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

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ä ---

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

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

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

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

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

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

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

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

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

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

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

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

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

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( +

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

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ä ---

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

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

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

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().

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

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.

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

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ä ---

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

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().

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

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. v2: Don't populate the fb->format pointer in drm_framebuffer_init(). instead we'll treat a NULL format as an

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

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:

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

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

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

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 ++

[PATCH 22/37] drm: Reject fbs w/o format info in drm_framebuffer_init()

From: Ville Syrjälä Any framebuffer that doesn't have proper format information when drm_framebuffer_init() is called is a bug. Let's warn and return an error to avoid oopsing the kernel later due to dereferencing the NULL fb->format pointer. Cc: Laurent

[PATCH 24/37] drm/i915: Eliminate the ugly 'fb?:' constructs from the ilk/skl wm code

From: Ville Syrjälä Don't access plane_state->fb until we know the plane to be visible. It it's visible, it will have an fb, and thus we don't have to consider the NULL fb case. Makes the code look nicer. Cc: intel-gfx at lists.freedesktop.org Signed-off-by:

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

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ä Reviewed-by: Laurent Pinchart --- drivers/gpu/drm/drm_fb_cma_helper.c | 5

[PATCH 23/37] drm: Replace drm_format_num_planes() with fb->format->num_planes

From: Ville Syrjälä Replace drm_format_num_planes(fb->pixel_format) with just fb->format->num_planes. Avoids the expensive format info lookup. @@ struct drm_framebuffer *a; struct drm_framebuffer b; @@ ( - drm_format_num_planes(a->pixel_format) +

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

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

[PATCH 29/37] drm: Add drm_framebuffer_plane_{width,height}()

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

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

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

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

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.

[PATCH 34/37] drm: Replace 'format->format' comparisons to just 'format' comparisons

From: Ville Syrjälä Rather than compare the format u32s of two format infos, we can direclty compare the format info pointers themselves. Noramlly all the ->format pointers all point to somwehere in the big array, so this is a valid way to test for equality.

[PATCH 31/37] drm: Nuke fb->depth

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; ... } @@

[PATCH 25/37] drm: Replace drm_format_plane_cpp() with fb->format->cpp[]

From: Ville Syrjälä Replace drm_format_plane_cpp(fb->pixel_format) with just fb->format->cpp[]. Avoids the expensive format info lookup. @@ struct drm_framebuffer *a; struct drm_framebuffer b; expression E; @@ ( - drm_format_plane_cpp(a->pixel_format, E) +

[PATCH 37/37] drm/i915: Implement .get_format_info() hook for CCS

From: Ville Syrjälä By providing our own format information for the CCS formats, we should be able to make framebuffer_check() do the right thing for the CCS surface as well. Note that we'll return the same format info for both Y and Yf tiled format as that's

[PATCH v2 33/37] drm: Nuke fb->pixel_format

From: Ville Syrjälä Replace uses of fb->pixel_format 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. @@

[PATCH 36/37] drm: Add mode_config .get_format_info() hook

From: Ville Syrjälä Allow drivers to return a custom drm_format_info structure for special fb layouts. We'll use this for the compression control surface in i915. Cc: Ben Widawsky Cc: intel-gfx at lists.freedesktop.org Signed-off-by: Ville Syrjälä ---

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

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ä ---

[PATCH 35/37] drm: Eliminate the useless "non-RGB fb" debug message

From: Ville Syrjälä No point in spamming the log whenever a non-RGB fb is being constructed. And since there's nothing to do anymore that fb->bits_per_pixel and fb->depth are gone, we can just kill off this entire piece of code. Cc: Laurent Pinchart

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

dri-devel/attachments/20161118/aeb0b04a/attachment.html>

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

Hi Ville, On Fri, Nov 18, 2016 at 09:52:45PM +0200, Ville Syrjälä wrote: >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

[Bug 98005] VCE dual instance encoding inconsistent since st/va: enable dual instances encode by sync surface

ks -- 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/20161118/2d46c50c/attachment.html>

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

because: You are the assignee for the bug. -- next part -- An HTML attachment was scrubbed... URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20161118/d29664a6/attachment.html>

  1   2   >