[PATCH][next] drm/amd/display: fix incorrect null check on pointer

2019-04-26 Thread Colin King
From: Colin Ian King Currently an allocation is being made but the allocation failure check is being performed on another pointer. Fix this by checking the correct pointer. Also use the normal kernel idiom for null pointer checks. Addresses-Coverity: ("Resource leak") Fixes: 43e3ac8389ef

Re: [v3 1/3] dt-bindings: display: mediatek: update dpi supported chips

2019-04-26 Thread Rob Herring
On Tue, 16 Apr 2019 13:52:40 +0800, Jitao Shi wrote: > Add decriptions about supported chips, including MT2701 & MT8173 & > mt8183 > > Signed-off-by: Jitao Shi > --- > .../devicetree/bindings/display/mediatek/mediatek,dpi.txt| 1 + > 1 file changed, 1 insertion(+) > Reviewed-by: Rob

[PATCH AUTOSEL 4.19 36/53] drm/sun4i: tcon top: Fix NULL/invalid pointer dereference in sun8i_tcon_top_un/bind

2019-04-26 Thread Sasha Levin
From: Ondrej Jirman [ Upstream commit 1a07a94b47b1f528f39c3e6187b5eaf02efe44ea ] There are two problems here: 1. Not all clk_data->hws[] need to be initialized, depending on various configured quirks. This leads to NULL ptr deref in clk_hw_unregister_gate() in sun8i_tcon_top_unbind() 2.

[PATCH AUTOSEL 4.19 30/53] drm/amd/display: fix cursor black issue

2019-04-26 Thread Sasha Levin
From: tiancyin [ Upstream commit c1cefe115d1cdc460014483319d440b2f0d07c68 ] [Why] the member sdr_white_level of struct dc_cursor_attributes was not initialized, then the random value result that dcn10_set_cursor_sdr_white_level() set error hw_scale value 0x20D9(normal value is 0x3c00), this

[PATCH AUTOSEL 4.19 29/53] drm/amdkfd: Add picasso pci id

2019-04-26 Thread Sasha Levin
From: Alex Deucher [ Upstream commit e7ad88553aa1d48e950ca9a4934d246c1bee4be4 ] Picasso is a new raven variant. Reviewed-by: Kent Russell Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin --- drivers/gpu/drm/amd/amdkfd/kfd_device.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH AUTOSEL 4.19 18/53] drm/omap: hdmi4_cec: Fix CEC clock handling for PM

2019-04-26 Thread Sasha Levin
From: Tony Lindgren [ Upstream commit 36a1da15b5df493241b0011d2185fdd724ac1ed1 ] If CONFIG_OMAP4_DSS_HDMI_CEC is enabled in .config, deeper SoC idle states are blocked because the CEC clock gets always enabled on init. Let's fix the issue by moving the CEC clock handling to happen later in

[PATCH AUTOSEL 5.0 59/79] drm/mediatek: no change parent rate in round_rate() for MT2701 hdmi phy

2019-04-26 Thread Sasha Levin
From: Wangyan Wang [ Upstream commit 9ee76098a1b8ae21cccac641b735ee4d3a77bccf ] This is the third step to make MT2701 HDMI stable. We should not change the rate of parent for hdmi phy when doing round_rate for this clock. The parent clock of hdmi phy must be the same as it. We change it when

[PATCH AUTOSEL 5.0 54/79] drm/mediatek: fix possible object reference leak

2019-04-26 Thread Sasha Levin
From: Wen Yang [ Upstream commit 2ae2c3316fb77dcf64275d011596b60104c45426 ] The call to of_parse_phandle returns a node pointer with refcount incremented thus it must be explicitly decremented after the last usage. Detected by coccinelle with the following warnings:

[PATCH AUTOSEL 5.0 57/79] drm/mediatek: remove flag CLK_SET_RATE_PARENT for MT2701 hdmi phy

2019-04-26 Thread Sasha Levin
From: Wangyan Wang [ Upstream commit 827abdd024207146822f66ba3ba74867135866b9 ] This is the first step to make MT2701 hdmi stable. The parent rate of hdmi phy had set by DPI driver. We should not set or change the parent rate of MT2701 hdmi phy, as a result we should remove the flags of

[PATCH AUTOSEL 4.19 41/53] drm/mediatek: fix possible object reference leak

2019-04-26 Thread Sasha Levin
From: Wen Yang [ Upstream commit 2ae2c3316fb77dcf64275d011596b60104c45426 ] The call to of_parse_phandle returns a node pointer with refcount incremented thus it must be explicitly decremented after the last usage. Detected by coccinelle with the following warnings:

[PATCH AUTOSEL 4.19 23/53] drm/mediatek: Fix an error code in mtk_hdmi_dt_parse_pdata()

2019-04-26 Thread Sasha Levin
From: Dan Carpenter [ Upstream commit 2d85978341e6a32e7443d9f28639da254d53f400 ] We don't want to overwrite "ret", it already holds the correct error code. The "regmap" variable might be a valid pointer as this point. Fixes: 8f83f26891e1 ("drm/mediatek: Add HDMI support") Signed-off-by: Dan

[PATCH 3/3] drm/amd/display: Compensate for pre-DCE12 BTR-VRR hw limitations. (v3)

2019-04-26 Thread Mario Kleiner
Pre-DCE12 needs special treatment for BTR / low framerate compensation for more stable behaviour: According to comments in the code and some testing on DCE-8 and DCE-11, DCE-11 and earlier only apply VTOTAL_MIN/MAX programming with a lag of one frame, so the special BTR hw programming for

Re: [PATCH 3/3] drm/amd/display: Compensate for pre-DCE12 BTR-VRR hw limitations. (v2)

2019-04-26 Thread Mario Kleiner
On Fri, Apr 26, 2019 at 7:12 PM Kazlauskas, Nicholas wrote: > > On 4/26/19 6:50 AM, Mario Kleiner wrote: > > Pre-DCE12 needs special treatment for BTR / low framerate > > compensation for more stable behaviour: > > > > According to comments in the code and some testing on DCE-8 > > and DCE-11,

[PATCH AUTOSEL 5.0 47/79] drm/sun4i: tcon top: Fix NULL/invalid pointer dereference in sun8i_tcon_top_un/bind

2019-04-26 Thread Sasha Levin
From: Ondrej Jirman [ Upstream commit 1a07a94b47b1f528f39c3e6187b5eaf02efe44ea ] There are two problems here: 1. Not all clk_data->hws[] need to be initialized, depending on various configured quirks. This leads to NULL ptr deref in clk_hw_unregister_gate() in sun8i_tcon_top_unbind() 2.

[PATCH AUTOSEL 5.0 38/79] drm/amdkfd: Add picasso pci id

2019-04-26 Thread Sasha Levin
From: Alex Deucher [ Upstream commit e7ad88553aa1d48e950ca9a4934d246c1bee4be4 ] Picasso is a new raven variant. Reviewed-by: Kent Russell Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin --- drivers/gpu/drm/amd/amdkfd/kfd_device.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH AUTOSEL 5.0 55/79] drm/mediatek: fix the rate and divder of hdmi phy for MT2701

2019-04-26 Thread Sasha Levin
From: Wangyan Wang [ Upstream commit 0c24613cda163dedfa229afc8eff6072e57fac8d ] Due to a clerical error,there is one zero less for 1280. Fix it for 12800 Fixes: 0fc721b2968e ("drm/mediatek: add hdmi driver for MT2701 and MT7623") Signed-off-by: Wangyan Wang Signed-off-by: CK Hu

[PATCH AUTOSEL 5.0 58/79] drm/mediatek: using new factor for tvdpll for MT2701 hdmi phy

2019-04-26 Thread Sasha Levin
From: Wangyan Wang [ Upstream commit 8eeb3946feeb00486ac0909e2309da87db8988a5 ] This is the second step to make MT2701 HDMI stable. The factor depends on the divider of DPI in MT2701, therefore, we should fix this factor to the right and new one. Test: search ok Signed-off-by: Wangyan Wang

[PATCH AUTOSEL 5.0 40/79] drm/amdgpu: amdgpu_device_recover_vram always failed if only one node in shadow_list

2019-04-26 Thread Sasha Levin
From: wentalou [ Upstream commit 1712fb1a2f6829150032ac76eb0e39b82a549cfb ] amdgpu_bo_restore_shadow would assign zero to r if succeeded. r would remain zero if there is only one node in shadow_list. current code would always return failure when r <= 0. restart the timeout for each wait was a

[PATCH AUTOSEL 5.0 41/79] drm/amd/display: fix cursor black issue

2019-04-26 Thread Sasha Levin
From: tiancyin [ Upstream commit c1cefe115d1cdc460014483319d440b2f0d07c68 ] [Why] the member sdr_white_level of struct dc_cursor_attributes was not initialized, then the random value result that dcn10_set_cursor_sdr_white_level() set error hw_scale value 0x20D9(normal value is 0x3c00), this

[PATCH AUTOSEL 5.0 39/79] drm/amdgpu: Adjust IB test timeout for XGMI configuration

2019-04-26 Thread Sasha Levin
From: shaoyunl [ Upstream commit d4162c61e253177936fcfe6c29f7b224d9a1efb8 ] On XGMI configuration the ib test may take longer to finish Signed-off-by: shaoyunl Reviewed-by: Christian König Signed-off-by: Alex Deucher Signed-off-by: Sasha Levin --- drivers/gpu/drm/amd/amdgpu/amdgpu_ib.c |

[PATCH AUTOSEL 5.0 23/79] drm/omap: hdmi4_cec: Fix CEC clock handling for PM

2019-04-26 Thread Sasha Levin
From: Tony Lindgren [ Upstream commit 36a1da15b5df493241b0011d2185fdd724ac1ed1 ] If CONFIG_OMAP4_DSS_HDMI_CEC is enabled in .config, deeper SoC idle states are blocked because the CEC clock gets always enabled on init. Let's fix the issue by moving the CEC clock handling to happen later in

[PATCH AUTOSEL 5.0 29/79] drm/mediatek: Fix an error code in mtk_hdmi_dt_parse_pdata()

2019-04-26 Thread Sasha Levin
From: Dan Carpenter [ Upstream commit 2d85978341e6a32e7443d9f28639da254d53f400 ] We don't want to overwrite "ret", it already holds the correct error code. The "regmap" variable might be a valid pointer as this point. Fixes: 8f83f26891e1 ("drm/mediatek: Add HDMI support") Signed-off-by: Dan

[PATCH AUTOSEL 5.0 56/79] drm/mediatek: make implementation of recalc_rate() for MT2701 hdmi phy

2019-04-26 Thread Sasha Levin
From: Wangyan Wang [ Upstream commit 321b628e6f5a3af999f75eadd373adbcb8b4cb1f ] Recalculate the rate of this clock, by querying hardware to make implementation of recalc_rate() to match the definition. Signed-off-by: Wangyan Wang Signed-off-by: CK Hu Signed-off-by: Sasha Levin ---

[PATCH 2/3] drm/amd/display: Enter VRR BTR earlier.

2019-04-26 Thread Mario Kleiner
Use a 2 msecs switching headroom not only for slightly delayed exiting of BTR mode, but now also for entering it a bit before the max frame duration is exceeded. With slowly changing time delay between successive flips or with a bit of jitter in arrival of flips, this adapts vblank early and

VRR BTR patches revision 3

2019-04-26 Thread Mario Kleiner
Same as rev 2, except for patch 3/3 v3 to apply Nicholas latest feedback into account. Retested on DCN1 and DCE8. thanks, -mario ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

[PATCH 1/3] drm/amd/display: Fix and simplify apply_below_the_range()

2019-04-26 Thread Mario Kleiner
The comparison of inserted_frame_duration_in_us against a duration calculated from max_refresh_in_uhz is both wrong in its math and not needed, as the min_duration_in_us value is already cached in in_out_vrr for reuse. No need to recalculate it wrongly at each invocation. Signed-off-by: Mario

[radeon-alex:amd-19.10 2196/2588] drivers/gpu//drm/rcar-du/rcar_du_crtc.c:861:20: error: initialization from incompatible pointer type

2019-04-26 Thread kbuild test robot
tree: git://people.freedesktop.org/~agd5f/linux.git amd-19.10 head: 5243d576a3bba97121d5671260d67de785d150b7 commit: 8bbc49ebffd58f39911ec28a8d6f2f9611391824 [2196/2588] drm/crc: Cleanup crtc_crc_open function config: arm-allmodconfig (attached as .config) compiler: arm-linux-gnueabi-gcc

Re: [PATCH] android: libdrm_platform: add liblog shared dependency

2019-04-26 Thread Mauro Rossi
Here is patch using include which should suffice for the purpose. Cheers Mauro On Mon, Apr 22, 2019 at 11:46 AM Mauro Rossi wrote: > > Hi Emil, Chih-Wei, > > what about the series of latest three patches we have in oreo-x86 branch? > > [oreo-x86 branch] >

Re: [PATCHv2 02/22] drm/bridge: tc358767: reset voltage-swing & pre-emphasis

2019-04-26 Thread Laurent Pinchart
Hi Tomi, On Fri, Apr 26, 2019 at 05:14:17PM +0300, Tomi Valkeinen wrote: > On 20/04/2019 23:30, Laurent Pinchart wrote: > > On Tue, Mar 26, 2019 at 12:31:26PM +0200, Tomi Valkeinen wrote: > >> We need to reset DPCD voltage-swing & pre-emphasis before starting the > >> link training, as otherwise

[PATCH AUTOSEL 4.14 23/32] drm/mediatek: fix possible object reference leak

2019-04-26 Thread Sasha Levin
From: Wen Yang [ Upstream commit 2ae2c3316fb77dcf64275d011596b60104c45426 ] The call to of_parse_phandle returns a node pointer with refcount incremented thus it must be explicitly decremented after the last usage. Detected by coccinelle with the following warnings:

[PATCH AUTOSEL 4.14 12/32] drm/mediatek: Fix an error code in mtk_hdmi_dt_parse_pdata()

2019-04-26 Thread Sasha Levin
From: Dan Carpenter [ Upstream commit 2d85978341e6a32e7443d9f28639da254d53f400 ] We don't want to overwrite "ret", it already holds the correct error code. The "regmap" variable might be a valid pointer as this point. Fixes: 8f83f26891e1 ("drm/mediatek: Add HDMI support") Signed-off-by: Dan

[PATCH AUTOSEL 4.9 12/16] drm/mediatek: fix possible object reference leak

2019-04-26 Thread Sasha Levin
From: Wen Yang [ Upstream commit 2ae2c3316fb77dcf64275d011596b60104c45426 ] The call to of_parse_phandle returns a node pointer with refcount incremented thus it must be explicitly decremented after the last usage. Detected by coccinelle with the following warnings:

[PATCH AUTOSEL 4.9 07/16] drm/mediatek: Fix an error code in mtk_hdmi_dt_parse_pdata()

2019-04-26 Thread Sasha Levin
From: Dan Carpenter [ Upstream commit 2d85978341e6a32e7443d9f28639da254d53f400 ] We don't want to overwrite "ret", it already holds the correct error code. The "regmap" variable might be a valid pointer as this point. Fixes: 8f83f26891e1 ("drm/mediatek: Add HDMI support") Signed-off-by: Dan

Re: [PATCH] drm: support gpu aliases defined in DT data

2019-04-26 Thread Tomi Valkeinen
On 17/04/2019 20:42, Emil Velikov wrote: > Have you looked at the libdrm drmDevice2 (yes use the second version) API? > It provides various information about the different devices, yet if > it's missing anything do send us a patch ;-) No, I didn't notice that. At least with a quick look, looks

[PATCH 06/12] dma-buf: add optional invalidate_mappings callback v5

2019-04-26 Thread Christian König
Each importer can now provide an invalidate_mappings callback. This allows the exporter to provide the mappings without the need to pin the backing store. v2: don't try to invalidate mappings when the callback is NULL, lock the reservation obj while using the attachments, add helper to

[PATCH 04/12] dma-buf: lock the reservation object during (un)map_dma_buf v4

2019-04-26 Thread Christian König
This way we always call the the map/unmap callbacks with the reservation object held if pin/unpin is also implemented. For static dma-buf exporters we still have the fallback of using cached sgt. v2: reordered v3: rebased on sgt caching v4: use the cached sgt when possible v5: cleanup further

[PATCH 07/12] drm: remove prime sg_table caching

2019-04-26 Thread Christian König
That is now done by the DMA-buf helpers instead. Signed-off-by: Christian König --- drivers/gpu/drm/drm_prime.c | 76 - 1 file changed, 16 insertions(+), 60 deletions(-) diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c index

[PATCH 02/12] dma-buf: add explicit buffer pinning v2

2019-04-26 Thread Christian König
Add optional explicit pinning callbacks instead of implicitly assume the exporter pins the buffer when a mapping is created. v2: move in patchset and pin the dma-buf in the old mapping code paths. Signed-off-by: Christian König --- drivers/dma-buf/dma-buf.c | 49

[PATCH 11/12] drm/amdgpu: add independent DMA-buf import v4

2019-04-26 Thread Christian König
Instead of relying on the DRM functions just implement our own import functions. This prepares support for taking care of unpinned DMA-buf. v2: enable for all exporters, not just amdgpu, fix invalidation handling, lock reservation object while setting callback v3: change to new dma_buf attach

[PATCH 05/12] dma-buf: add dma_buf_(un)map_attachment_locked variants v4

2019-04-26 Thread Christian König
Add function variants which can be called with the reservation lock already held. v2: reordered, add lockdep asserts, fix kerneldoc v3: rebased on sgt caching v4: reorder once more Signed-off-by: Christian König --- drivers/dma-buf/dma-buf.c | 83 +--

[PATCH 09/12] drm/ttm: use the parent resv for ghost objects

2019-04-26 Thread Christian König
This way we can even pipeline imported BO evictions. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo_util.c | 18 +- 1 file changed, 1 insertion(+), 17 deletions(-) diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c b/drivers/gpu/drm/ttm/ttm_bo_util.c index

[PATCH 01/12] dma-buf: add struct dma_buf_attach_info

2019-04-26 Thread Christian König
Add a structure for the parameters of dma_buf_attach, this makes it much easier to add new parameters later on. Signed-off-by: Christian König --- drivers/dma-buf/dma-buf.c | 13 +++-- drivers/gpu/drm/armada/armada_gem.c | 6 +-

[PATCH 10/12] drm/amdgpu: add independent DMA-buf export v3

2019-04-26 Thread Christian König
The caching of SGT's is actually quite harmful and should probably removed altogether when all drivers are audited. Start by providing a separate DMA-buf export implementation in amdgpu. This is also a prerequisite of unpinned DMA-buf handling. v2: fix unintended recursion, remove debugging

[PATCH 03/12] dma-buf: start caching of sg_table objects

2019-04-26 Thread Christian König
To allow a smooth transition from pinning buffer objects to dynamic invalidation we first start to cache the sg_table for an attachment unless the driver has implemented the explicite pin/unpin callbacks. Signed-off-by: Christian König --- drivers/dma-buf/dma-buf.c | 24

[PATCH 12/12] drm/amdgpu: add DMA-buf invalidation callback v2

2019-04-26 Thread Christian König
Allow for invalidation of imported DMA-bufs. v2: add dma_buf_pin/dma_buf_unpin support Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 6 ++ drivers/gpu/drm/amd/amdgpu/amdgpu_prime.c | 24 ++ 2 files changed, 30 insertions(+) diff

[PATCH 08/12] drm/ttm: remove the backing store if no placement is given

2019-04-26 Thread Christian König
Pipeline removal of the BOs backing store when no placement is given during validation. Signed-off-by: Christian König --- drivers/gpu/drm/ttm/ttm_bo.c | 12 1 file changed, 12 insertions(+) diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c index

[PATCH] drm/cma-helper: Fix drm_gem_cma_free_object()

2019-04-26 Thread Noralf Trønnes
The logic for freeing an imported buffer with a virtual address is broken. It will free the buffer instead of unmapping the dma buf. Fix by reversing the if ladder and first check if the buffer is imported. Fixes: b9068cde51ee ("drm/cma-helper: Add DRM_GEM_CMA_VMAP_DRIVER_OPS") Cc:

Re: [PATCHv2 02/22] drm/bridge: tc358767: reset voltage-swing & pre-emphasis

2019-04-26 Thread Tomi Valkeinen
On 20/04/2019 23:30, Laurent Pinchart wrote: > Hi Tomi, > > Thank you for the patch. > > On Tue, Mar 26, 2019 at 12:31:26PM +0200, Tomi Valkeinen wrote: >> We need to reset DPCD voltage-swing & pre-emphasis before starting the >> link training, as otherwise tc358767 will use the previous values

Re: [PATCH] drm/cma-helper: Fix drm_gem_cma_free_object()

2019-04-26 Thread Daniel Vetter
On Fri, Apr 26, 2019 at 2:48 PM Noralf Trønnes wrote: > > The logic for freeing an imported buffer with a virtual address is > broken. It will free the buffer instead of unmapping the dma buf. > Fix by reversing the if ladder and first check if the buffer is imported. > > Fixes: b9068cde51ee

[Bug 201273] Fatal error during GPU init amdgpu RX560

2019-04-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=201273 --- Comment #44 from quirin.blae...@freenet.de --- (In reply to Alex Deucher from comment #43) > Does booting with any of the following options on the kernel command line in > grub help? > amd_iommu=off > idle=nomwait > iommu=pt > pci=noats > Can

Re: [PATCH 2/2] drm/stm: ltdc: return appropriate error code during probe

2019-04-26 Thread Philippe CORNU
Hi Fabien, and thank you for your patch, Acked-by: Philippe Cornu Philippe :-) On 4/24/19 4:03 PM, Fabien Dessenne wrote: > During probe, return the "clk_get" error value instead of -ENODEV. > > Signed-off-by: Fabien Dessenne > --- > drivers/gpu/drm/stm/ltdc.c | 5 +++-- > 1 file changed,

Re: [PATCH 1/2] drm/stm: ltdc: manage the get_irq probe defer case

2019-04-26 Thread Philippe CORNU
Hi Fabien, and thank you for your patch, Acked-by: Philippe Cornu Philippe :-) On 4/24/19 4:03 PM, Fabien Dessenne wrote: > Manage the -EPROBE_DEFER error case for the ltdc IRQ. > > Signed-off-by: Fabien Dessenne > --- > drivers/gpu/drm/stm/ltdc.c | 3 +++ > 1 file changed, 3

Re: [PATCH v2 6/9] iommu/amd: use helper pci_dev_id

2019-04-26 Thread Joerg Roedel
On Wed, Apr 24, 2019 at 09:15:25PM +0200, Heiner Kallweit wrote: > Use new helper pci_dev_id() to simplify the code. > > Signed-off-by: Heiner Kallweit Reviewed-by: Joerg Roedel > --- > drivers/iommu/amd_iommu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git

Re: [PATCH v2 7/9] iommu/vt-d: use helper pci_dev_id

2019-04-26 Thread Joerg Roedel
On Wed, Apr 24, 2019 at 09:16:10PM +0200, Heiner Kallweit wrote: > Use new helper pci_dev_id() to simplify the code. > > Signed-off-by: Heiner Kallweit Reviewed-by: Joerg Roedel > --- > drivers/iommu/intel-iommu.c | 2 +- > drivers/iommu/intel_irq_remapping.c | 2 +- > 2 files

Re: [PATCH] staging: xgifb: delete the driver

2019-04-26 Thread Daniel Vetter
On Fri, Apr 26, 2019 at 2:42 AM Aaro Koskinen wrote: > On Fri, Jan 18, 2019 at 12:20:28PM +0100, Daniel Vetter wrote: > > On Fri, Jan 18, 2019 at 11:08:28AM +0100, Greg Kroah-Hartman wrote: > > > There has not been any real work done on cleaning this driver up and > > > getting it out of the

Re: [PATCH v5 6/6] drm/amdgpu: Avoid HW reset if guilty job already signaled.

2019-04-26 Thread Grodzovsky, Andrey
Ping (mostly David and Monk). Andrey On 4/24/19 3:09 AM, Christian König wrote: Am 24.04.19 um 05:02 schrieb Zhou, David(ChunMing): >> -drm_sched_stop(>sched, >base); >> - >> /* after all hw jobs are reset, hw fence is meaningless, so >> force_completion */ >>

Re: [PATCHv2 01/22] drm/bridge: tc358767: fix tc_aux_get_status error handling

2019-04-26 Thread Tomi Valkeinen
On 20/04/2019 23:14, Laurent Pinchart wrote: > Hi Tomi, > > Thank you for the patch. > > On Tue, Mar 26, 2019 at 12:31:25PM +0200, Tomi Valkeinen wrote: >> tc_aux_get_status() does not report AUX_TIMEOUT correctly, as it only >> checks the AUX_TIMEOUT if aux is still busy. Fix this by always

Re: [PATCH] Revert "drm/qxl: drop prime import/export callbacks"

2019-04-26 Thread Daniel Vetter
On Fri, Apr 26, 2019 at 7:33 AM Gerd Hoffmann wrote: > > This reverts commit f4c34b1e2a37d5676180901fa6ff188bcb6371f8. > > Simliar to commit a0cecc23cfcb Revert "drm/virtio: drop prime > import/export callbacks". We have to do the same with qxl, > for the same reasons (it breaks DRI3). > > Drop

[Bug 203439] amdgpu: [REG 4.20 -> 5.0] Brightness minimum level is too high

2019-04-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203439 --- Comment #1 from Błażej Szczygieł (spa...@wp.pl) --- *** Bug 203427 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching the assignee of the bug.

[Bug 203439] New: amdgpu: [REG 4.20 -> 5.0] Brightness minimum level is too high

2019-04-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203439 Bug ID: 203439 Summary: amdgpu: [REG 4.20 -> 5.0] Brightness minimum level is too high Product: Drivers Version: 2.5 Kernel Version: 5.0 Hardware: All

[Bug 203439] amdgpu: [REG 4.20 -> 5.0] Brightness minimum level is too high

2019-04-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203439 --- Comment #3 from Błażej Szczygieł (spa...@wp.pl) --- Thank you for the reply! In bright room you can quickly disable the backlight (set it to 0) using the Fn+F1 keys. You can save power and the display content doesn't disturb you, but you can

Re: [PATCH v4 6/9] dt-bindings: display: hdmi-connector: Add DDC power supply

2019-04-26 Thread Rob Herring
On Sat, Apr 13, 2019 at 06:54:15PM +0200, meg...@megous.com wrote: > From: Ondrej Jirman > > Some Allwinner SoC using boards (Orange Pi 3 for example) need to enable > on-board voltage shifting logic for the DDC bus to be usable. Use > ddc-supply on the hdmi-connector to model this. > > Add

[Bug 203439] amdgpu: [REG 4.20 -> 5.0] Brightness minimum level is too high

2019-04-26 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=203439 Nicholas Kazlauskas (nicholas.kazlaus...@amd.com) changed: What|Removed |Added CC|

Re: [PATCH 3/3] drm/amd/display: Compensate for pre-DCE12 BTR-VRR hw limitations. (v2)

2019-04-26 Thread Kazlauskas, Nicholas
On 4/26/19 6:50 AM, Mario Kleiner wrote: > Pre-DCE12 needs special treatment for BTR / low framerate > compensation for more stable behaviour: > > According to comments in the code and some testing on DCE-8 > and DCE-11, DCE-11 and earlier only apply VTOTAL_MIN/MAX > programming with a lag of one

Re: [git pull] drm fixes for 5.1-rc7

2019-04-26 Thread pr-tracker-bot
The pull request you sent on Fri, 26 Apr 2019 10:46:30 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-fixes-2019-04-26 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/4e40f0f35be86ed5d8724d52bc38af5dbab06470 Thank you! -- Deet-doot-dot, I am a bot.

Re: [PATCH v4 6/9] dt-bindings: display: hdmi-connector: Add DDC power supply

2019-04-26 Thread Rob Herring
On Fri, Apr 26, 2019 at 2:20 PM Ondřej Jirman wrote: > > On Fri, Apr 26, 2019 at 01:23:37PM -0500, Rob Herring wrote: > > On Sat, Apr 13, 2019 at 06:54:15PM +0200, meg...@megous.com wrote: > > > From: Ondrej Jirman > > > > > > Some Allwinner SoC using boards (Orange Pi 3 for example) need to

Re: [PATCH v4 1/2] dt-bindings: Add doc for the Ingenic JZ47xx LCD controller driver

2019-04-26 Thread Rob Herring
On Fri, 26 Apr 2019 02:53:07 +0200, Paul Cercueil wrote: > Add documentation for the devicetree bindings of the LCD controller present in > the JZ47xx family of SoCs from Ingenic. > > Signed-off-by: Paul Cercueil > Tested-by: Artur Rojek > --- > > Notes: > v2: Remove ingenic,panel

[Bug 110521] Make skips easily parseable by machines

2019-04-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110521 Bug ID: 110521 Summary: Make skips easily parseable by machines Product: DRI Version: XOrg git Hardware: Other OS: All Status: NEW Severity: normal

[Bug 110514] [CI][BAT] igt@* - skip - Test requirement: is_i915_device(fd) && has_known_intel_chipset(fd), Skip

2019-04-26 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=110514 --- Comment #6 from Mika Kahola --- It seems that we are missing device info for CML. Hence, we are failing on these is_i915_device() and has_known_intel_chipset() tests. -- You are receiving this mail because: You are the assignee for the

[patch V3 22/29] tracing: Make ftrace_trace_userstack() static and conditional

2019-04-26 Thread Thomas Gleixner
It's only used in trace.c and there is absolutely no point in compiling it in when user space stack traces are not supported. Signed-off-by: Thomas Gleixner Reviewed-by: Steven Rostedt --- kernel/trace/trace.c | 14 -- kernel/trace/trace.h |8 2 files changed, 8

[patch V3 16/29] drm: Simplify stacktrace handling

2019-04-26 Thread Thomas Gleixner
Replace the indirection through struct stack_trace by using the storage array based interfaces. The original code in all printing functions is really wrong. It allocates a storage array on stack which is unused because depot_fetch_stack() does not store anything in it. It overwrites the entries

[PATCH v3 1/6] drm: atmel-hlcdc: add config option for clock selection

2019-04-26 Thread Claudiu.Beznea
From: Claudiu Beznea SAM9x60 LCD Controller has no option to select clock source as previous controllers have. To be able to use the same driver even for this LCD controller add a config option to know if controller supports this. Signed-off-by: Claudiu Beznea Reviewed-by: Sam Ravnborg ---

[patch V3 01/29] tracing: Cleanup stack trace code

2019-04-26 Thread Thomas Gleixner
- Remove the extra array member of stack_dump_trace[] along with the ARRAY_SIZE - 1 initialization for struct stack_trace :: max_entries. Both are historical leftovers of no value. The stack tracer never exceeds the array and there is no extra storage requirement either. - Make variables

[patch V3 15/29] dm persistent data: Simplify stack trace handling

2019-04-26 Thread Thomas Gleixner
Replace the indirection through struct stack_trace with an invocation of the storage array based interface. This results in less storage space and indirection. Signed-off-by: Thomas Gleixner Cc: dm-de...@redhat.com Cc: Mike Snitzer Cc: Alasdair Kergon ---

[PATCH v3 4/6] drm: atmel-hlcdc: enable sys_clk during initalization.

2019-04-26 Thread Claudiu.Beznea
From: Sandeep Sheriker Mallikarjun For SAM9X60 SoC, sys_clk is through lcd_gclk clock source and this needs to be enabled before enabling lcd_clk. Signed-off-by: Sandeep Sheriker Mallikarjun [claudiu.bez...@microchip.com: add fixed_clksrc checks] Signed-off-by: Claudiu Beznea ---

[patch V3 21/29] tracing: Use percpu stack trace buffer more intelligently

2019-04-26 Thread Thomas Gleixner
The per cpu stack trace buffer usage pattern is odd at best. The buffer has place for 512 stack trace entries on 64-bit and 1024 on 32-bit. When interrupts or exceptions nest after the per cpu buffer was acquired the stacktrace length is hardcoded to 8 entries. 512/1024 stack trace entries in

[RESEND][PATCH v3 1/6] drm: atmel-hlcdc: add config option for clock selection

2019-04-26 Thread Claudiu.Beznea
From: Claudiu Beznea SAM9x60 LCD Controller has no option to select clock source as previous controllers have. To be able to use the same driver even for this LCD controller add a config option to know if controller supports this. Signed-off-by: Claudiu Beznea Reviewed-by: Sam Ravnborg ---

[RESEND][PATCH v3 3/6] pwm: atmel-hlcdc: add compatible for SAM9X60 HLCDC's PWM

2019-04-26 Thread Claudiu.Beznea
From: Claudiu Beznea Add compatible string for SAM9X60 HLCDC's PWM. Signed-off-by: Claudiu Beznea Acked-by: Thierry Reding --- drivers/pwm/pwm-atmel-hlcdc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/pwm/pwm-atmel-hlcdc.c b/drivers/pwm/pwm-atmel-hlcdc.c index

[patch V3 08/29] mm/kmemleak: Simplify stacktrace handling

2019-04-26 Thread Thomas Gleixner
Replace the indirection through struct stack_trace by using the storage array based interfaces. Signed-off-by: Thomas Gleixner Acked-by: Catalin Marinas Cc: linux...@kvack.org --- mm/kmemleak.c | 24 +++- 1 file changed, 3 insertions(+), 21 deletions(-) ---

[patch V3 12/29] dma/debug: Simplify stracktrace retrieval

2019-04-26 Thread Thomas Gleixner
Replace the indirection through struct stack_trace with an invocation of the storage array based interface. Signed-off-by: Thomas Gleixner Reviewed-by: Christoph Hellwig Cc: io...@lists.linux-foundation.org Cc: Robin Murphy Cc: Marek Szyprowski --- kernel/dma/debug.c | 14 ++ 1

[PATCH v3 3/6] pwm: atmel-hlcdc: add compatible for SAM9X60 HLCDC's PWM

2019-04-26 Thread Claudiu.Beznea
From: Claudiu Beznea Add compatible string for SAM9X60 HLCDC's PWM. Signed-off-by: Claudiu Beznea Acked-by: Thierry Reding --- drivers/pwm/pwm-atmel-hlcdc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/pwm/pwm-atmel-hlcdc.c b/drivers/pwm/pwm-atmel-hlcdc.c index

[RESEND][PATCH v3 2/6] drm: atmel-hlcdc: avoid initializing cfg with zero

2019-04-26 Thread Claudiu.Beznea
From: Claudiu Beznea Remove cfg initialization with zero and read state with drm_crtc_state_to_atmel_hlcdc_crtc_state() so that cfg to be initialized with state's output_mode. Signed-off-by: Claudiu Beznea Reviewed-by: Sam Ravnborg --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c | 6

Re: [PATCH v2 4/9] powerpc/powernv/npu: use helper pci_dev_id

2019-04-26 Thread Alexey Kardashevskiy
On 25/04/2019 05:14, Heiner Kallweit wrote: > Use new helper pci_dev_id() to simplify the code. > > Signed-off-by: Heiner Kallweit Reviewed-by: Alexey Kardashevskiy > --- > arch/powerpc/platforms/powernv/npu-dma.c | 14 ++ > 1 file changed, 6 insertions(+), 8 deletions(-) >

[patch V3 24/29] tracing: Remove the last struct stack_trace usage

2019-04-26 Thread Thomas Gleixner
Simplify the stack retrieval code by using the storage array based interface. Signed-off-by: Thomas Gleixner Reviewed-by: Steven Rostedt (VMware) --- kernel/trace/trace_stack.c | 37 - 1 file changed, 16 insertions(+), 21 deletions(-) ---

[patch V3 04/29] backtrace-test: Simplify stack trace handling

2019-04-26 Thread Thomas Gleixner
Replace the indirection through struct stack_trace by using the storage array based interfaces. Signed-off-by: Thomas Gleixner --- kernel/backtracetest.c | 11 +++ 1 file changed, 3 insertions(+), 8 deletions(-) --- a/kernel/backtracetest.c +++ b/kernel/backtracetest.c @@ -48,19

[RESEND][PATCH v3 0/6] add LCD support for SAM9X60

2019-04-26 Thread Claudiu.Beznea
From: Claudiu Beznea Hi, These patches adds support for SAM9X60's LCD controller. First patches add option to specify if controller clock source is fixed. Second patch avoid a variable initialization in atmel_hlcdc_crtc_mode_set_nofb(). The 3rd add compatibles in pwm-atmel-hlcdc driver. The

[PATCH v3 1/3] dt-bindings: display: Add Sharp LS020B1DD01D panel documentation

2019-04-26 Thread Paul Cercueil
The LS020B1DD01D is a 2.0" 240x160 16-bit TFT LCD panel. Signed-off-by: Paul Cercueil Reviewed-by: Rob Herring --- Notes: v2: New patch v3: Add Rob's Reviewed-by .../bindings/display/panel/sharp,ls020b1dd01d.txt| 12 1 file changed, 12 insertions(+) create mode

[PATCH v3 2/3] drm: Add bus flag for Sharp-specific signals

2019-04-26 Thread Paul Cercueil
Add the DRM_BUS_FLAG_SHARP_SIGNALS to the drm_bus_flags enum. This flags can be used when the display must be driven with the Sharp-specific signals SPL, CLS, REV, PS. Signed-off-by: Paul Cercueil --- Notes: v3: New patch include/drm/drm_connector.h | 3 +++ 1 file changed, 3

[patch V3 17/29] lockdep: Remove unused trace argument from print_circular_bug()

2019-04-26 Thread Thomas Gleixner
Signed-off-by: Thomas Gleixner --- kernel/locking/lockdep.c |9 - 1 file changed, 4 insertions(+), 5 deletions(-) --- a/kernel/locking/lockdep.c +++ b/kernel/locking/lockdep.c @@ -1522,10 +1522,9 @@ static inline int class_equal(struct loc } static noinline int

[PATCH v3 6/6] drm/atmel-hclcdc: revert shift by 8

2019-04-26 Thread Claudiu.Beznea
From: Claudiu Beznea Revert shift by 8 of state->base.alpha. This introduced regresion on planes. Fixes: 7f73c10b256b ("drm/atmel-hclcdc: Convert to the new generic alpha property") Cc: Maxime Ripard Signed-off-by: Claudiu Beznea --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 2 +- 1

[patch V3 02/29] stacktrace: Provide helpers for common stack trace operations

2019-04-26 Thread Thomas Gleixner
All operations with stack traces are based on struct stack_trace. That's a horrible construct as the struct is a kitchen sink for input and output. Quite some usage sites embed it into their own data structures which creates weird indirections. There is absolutely no point in doing so. For all

[patch V3 29/29] x86/stacktrace: Use common infrastructure

2019-04-26 Thread Thomas Gleixner
Replace the stack_trace_save*() functions with the new arch_stack_walk() interfaces. Signed-off-by: Thomas Gleixner Cc: linux-a...@vger.kernel.org --- arch/x86/Kconfig |1 arch/x86/kernel/stacktrace.c | 116 +++ 2 files changed, 20

[patch V3 25/29] livepatch: Simplify stack trace retrieval

2019-04-26 Thread Thomas Gleixner
Replace the indirection through struct stack_trace by using the storage array based interfaces. Signed-off-by: Thomas Gleixner Acked-by: Miroslav Benes --- kernel/livepatch/transition.c | 22 +- 1 file changed, 9 insertions(+), 13 deletions(-) ---

[patch V3 00/29] stacktrace: Consolidate stack trace usage

2019-04-26 Thread Thomas Gleixner
This is an update to V2 which can be found here: https://lkml.kernel.org/r/20190418084119.056416...@linutronix.de Changes vs. V2: - Fixed the kernel-doc issue pointed out by Mike - Removed the '-1' oddity from the tracer - Restricted the tracer nesting to 4 - Restored the lockdep

[patch V3 26/29] stacktrace: Remove obsolete functions

2019-04-26 Thread Thomas Gleixner
No more users of the struct stack_trace based interfaces. Remove them. Remove the macro stubs for !CONFIG_STACKTRACE as well as they are pointless because the storage on the call sites is conditional on CONFIG_STACKTRACE already. No point to be 'smart'. Signed-off-by: Thomas Gleixner ---

Re: [PATCH] drm/komeda: Use memset to initialize config_id

2019-04-26 Thread Nick Desaulniers
On Wed, Apr 24, 2019 at 11:27 PM Nathan Chancellor wrote: > > Clang warns: > > drivers/gpu/drm/arm/display/komeda/komeda_dev.c:76:38: warning: suggest > braces around initialization of subobject [-Wmissing-braces] > union komeda_config_id config_id = {0,}; >

[PATCH v4 1/2] dt-bindings: Add doc for the Ingenic JZ47xx LCD controller driver

2019-04-26 Thread Paul Cercueil
Add documentation for the devicetree bindings of the LCD controller present in the JZ47xx family of SoCs from Ingenic. Signed-off-by: Paul Cercueil Tested-by: Artur Rojek --- Notes: v2: Remove ingenic,panel property. v3: - Rename compatible strings from ingenic,jz47XX-drm to

[patch V3 27/29] lib/stackdepot: Remove obsolete functions

2019-04-26 Thread Thomas Gleixner
No more users of the struct stack_trace based interfaces. Signed-off-by: Thomas Gleixner Acked-by: Alexander Potapenko --- include/linux/stackdepot.h |4 lib/stackdepot.c | 20 2 files changed, 24 deletions(-) --- a/include/linux/stackdepot.h +++

[patch V3 23/29] tracing: Simplify stack trace retrieval

2019-04-26 Thread Thomas Gleixner
Replace the indirection through struct stack_trace by using the storage array based interfaces. Signed-off-by: Thomas Gleixner Reviewed-by: Steven Rostedt (VMware) --- kernel/trace/trace.c | 40 +--- 1 file changed, 13 insertions(+), 27 deletions(-) ---

[patch V3 11/29] fault-inject: Simplify stacktrace retrieval

2019-04-26 Thread Thomas Gleixner
Replace the indirection through struct stack_trace with an invocation of the storage array based interface. Signed-off-by: Thomas Gleixner Cc: Akinobu Mita --- lib/fault-inject.c | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) --- a/lib/fault-inject.c +++

  1   2   >