[PATCH 1/3] drm/bridge: ps8640: Return an error for incorrect attach flags

2020-06-16 Thread Enric Balletbo i Serra
Bridge drivers that implement the new model only shall return an error from their attach() handler when the DRM_BRIDGE_ATTACH_NO_CONNECTOR flag is not set. So make sure we return an error because only the new drm_bridge model is supported. Signed-off-by: Enric Balletbo i Serra --- drivers/gpu/d

[PATCH 3/3] drm/bridge: ps8640: Rework power state handling

2020-06-16 Thread Enric Balletbo i Serra
The get_edid() callback can be triggered anytime by an ioctl, i.e drm_mode_getconnector (ioctl) -> drm_helper_probe_single_connector_modes -> drm_bridge_connector_get_modes -> ps8640_bridge_get_edid Actually if the bridge pre_enable() function was not called before get_edid

[PATCH] drm/kselftest: fix spellint typo in test-drm_mm.c

2020-06-16 Thread Masanari Iida
This patch fix a spelling typo in test-drm_mm.c Signed-off-by: Masanari Iida --- drivers/gpu/drm/selftests/test-drm_mm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/selftests/test-drm_mm.c b/drivers/gpu/drm/selftests/test-drm_mm.c index 9aabe82dcd3a..3160

[RESEND PATCH v4 4/7] drm/mediatek: mtk_dsi: Convert to bridge driver

2020-06-16 Thread Enric Balletbo i Serra
Convert mtk_dsi to a bridge driver with built-in encoder support for compatibility with existing component drivers. Signed-off-by: Enric Balletbo i Serra Acked-by: Sam Ravnborg Reviewed-by: Chun-Kuang Hu --- Changes in v4: - Remove double call to drm_encoder_init(). (Chun-Kuang Hu) - Cleanup t

Re: [PATCH v5 0/8] drm: rcar-du: Add Color Management Module (CMM)

2020-06-16 Thread Eugeniu Rosca
Hi Jacopo, On Fri, Jun 12, 2020 at 05:12:09PM +0200, Jacopo Mondi wrote: > On Mon, Jun 08, 2020 at 11:44:32AM +0200, Eugeniu Rosca wrote: > > FWIW, I seem to hit pre-existing issues in vanilla rcar-du, > > while unplugging HDMI cable during a cyclic suspend-resume: > > > > HW: H3 ES2.0 Salvator-X

Re: [PATCH 13/29] dt: fix broken links due to txt->yaml renames

2020-06-16 Thread Thomas Bogendoerfer
On Mon, Jun 15, 2020 at 08:46:52AM +0200, Mauro Carvalho Chehab wrote: > There are some new broken doc links due to yaml renames > at DT. Developers should really run: > > ./scripts/documentation-file-ref-check > > in order to solve those issues while submitting patches. > This tool can eve

Re: [PATCH v6 0/9] Enable ili9341 and l3gd20 on stm32f429-disco

2020-06-16 Thread Alexandre Torgue
Hi Dillon, On 5/27/20 9:27 AM, dillon.min...@gmail.com wrote: From: dillon min ... dillon min (9): ARM: dts: stm32: Add dma config for spi5 ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board ARM: dts: stm32: enable ltdc binding with ili9341, gyro l3gd20 on s

[RESEND PATCH v4 6/7] drm/mediatek: mtk_dsi: Use the drm_panel_bridge API

2020-06-16 Thread Enric Balletbo i Serra
Replace the manual panel handling code by a drm_panel_bridge. This simplifies the driver and allows all components in the display pipeline to be treated as bridges, paving the way to generic connector handling. Signed-off-by: Enric Balletbo i Serra Reviewed-by: Laurent Pinchart Acked-by: Sam Rav

Re: [Freedreno] [PATCH v6 0/5] Add support for DisplayPort driver on

2020-06-16 Thread Jeffrey Hugo
On Mon, Jun 15, 2020 at 4:51 PM wrote: > > On 2020-06-12 16:26, Stephen Boyd wrote: > > Thanks for reviews Stephen. > > > Quoting Tanmay Shah (2020-06-11 18:50:25) > >> These patches add support for Display-Port driver on SnapDragon > >> hardware. It adds > >> DP driver and DP PLL driver files alo

[PATCH v2] drm/mediatek: remove unnecessary conversion to bool

2020-06-16 Thread Bernard Zhao
In function mtk_dsi_clk_hs_state, remove unnecessary conversion to bool return, this change is to make the code a bit readable. Signed-off-by: Bernard Zhao --- Changes since V1: *optimize to make the code a bit more clear. Link for V1: *https://lore.kernel.org/patchwork/patch/1255327/ --- drive

Re:Re: [PATCH] drm/mediatek: remove unnecessary conversion to bool

2020-06-16 Thread Bernard
发件人:Chun-Kuang Hu 发送日期:2020-06-15 22:00:52 收件人:Joe Perches 抄送人:Bernard Zhao ,Chun-Kuang Hu ,Philipp Zabel ,David Airlie ,Daniel Vetter ,Matthias Brugger ,DRI Development ,Linux ARM ,"moderated list:ARM/Mediatek SoC support" ,linux-kernel ,opensource.ker...@vivo.com 主题:Re: [PATCH] drm/med

Re: [PATCH v6 2/9] ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board

2020-06-16 Thread Alexandre Torgue
Hi Dillon On 5/27/20 9:27 AM, dillon.min...@gmail.com wrote: From: dillon min This patch adds the pin configuration for ltdc and spi5 controller on stm32f429-disco board. Signed-off-by: dillon min --- arch/arm/boot/dts/stm32f4-pinctrl.dtsi | 67 ++ 1 file c

[RESEND PATCH v4 5/7] drm/mediatek: mtk_dsi: Use simple encoder

2020-06-16 Thread Enric Balletbo i Serra
The mtk_dsi driver uses an empty implementation for its encoder. Replace the code with the generic simple encoder. Signed-off-by: Enric Balletbo i Serra Reviewed-by: Laurent Pinchart Acked-by: Sam Ravnborg Reviewed-by: Chun-Kuang Hu --- Changes in v4: None Changes in v3: None Changes in v2: N

[RESEND PATCH v4 7/7] drm/mediatek: mtk_dsi: Create connector for bridges

2020-06-16 Thread Enric Balletbo i Serra
Use the drm_bridge_connector helper to create a connector for pipelines that use drm_bridge. This allows splitting connector operations across multiple bridges when necessary, instead of having the last bridge in the chain creating the connector and handling all connector operations internally. Si

[RESEND PATCH v4 0/7] Convert mtk-dsi to drm_bridge API and get EDID for ps8640 bridge

2020-06-16 Thread Enric Balletbo i Serra
(This resend is to fix some trivial conflicts due the merge window) The PS8640 dsi-to-eDP bridge driver is using the panel bridge API, however, not all the components in the chain have been ported to the drm_bridge API. Actually, when a panel is attached the default panel's mode is used, but in so

[PATCH 4/6] video: backlight: tosa_lcd: convert to use i2c_new_client_device()

2020-06-16 Thread Wolfram Sang
Move away from the deprecated API and return the shiny new ERRPTR where useful. Signed-off-by: Wolfram Sang Reviewed-by: Daniel Thompson --- I'd like to push it via I2C for 5.8-rc2. drivers/video/backlight/tosa_lcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/driv

[PATCH 0/6] remove deprecated i2c_new_device API

2020-06-16 Thread Wolfram Sang
I want to remove the above API this cycle, and just a few patches have not made it into 5.8-rc1. They have been reviewed and most had been promised to get into linux-next, but well, things happen. So, I hope it is okay for everyone to collect them like this and push them via I2C for 5.8-rc2. One m

[RESEND PATCH v4 1/7] drm/bridge: ps8640: Get the EDID from eDP control

2020-06-16 Thread Enric Balletbo i Serra
The PS8640 DSI-to-eDP bridge can retrieve the EDID, so implement the .get_edid callback and set the flag to indicate the core to use it. Signed-off-by: Enric Balletbo i Serra Reviewed-by: Laurent Pinchart Acked-by: Sam Ravnborg --- Changes in v4: None Changes in v3: None Changes in v2: None

Re: [PATCH v2] drm/etnaviv: fix ref count leak via pm_runtime_get_sync

2020-06-16 Thread Markus Elfring
… > In case of failure, decrement the ref count before returning. Can it be nicer to use the term “reference count” here? Will the tag “Fixes” become helpful for the commit message? … > +++ b/drivers/gpu/drm/etnaviv/etnaviv_gpu.c … > @@ -1326,6 +1331,7 @@ struct dma_fence *etnaviv_gpu_submit(s

[PATCH 0/3] drm/bridge: ps8640: Make sure all needed is powered to get the EDID

2020-06-16 Thread Enric Balletbo i Serra
The first patch is not really related and can be picked alone. The second and the third are to rework the power handling to get the EDID. Basically, we need to make sure all the needed is powered to be able to get the EDID. Before, we saw that getting the EDID failed as explained in the third patch

Re: [PATCH v6 2/9] ARM: dts: stm32: Add pin map for ltdc & spi5 on stm32f429-disco board

2020-06-16 Thread dillon min
On Mon, Jun 15, 2020 at 5:45 PM Alexandre Torgue wrote: > > Hi Dillon > > On 5/27/20 9:27 AM, dillon.min...@gmail.com wrote: > > From: dillon min > > > > This patch adds the pin configuration for ltdc and spi5 controller > > on stm32f429-disco board. > > > > Signed-off-by: dillon min > > --- > >

[RESEND PATCH v4 2/7] drm/bridge_connector: Set default status connected for eDP connectors

2020-06-16 Thread Enric Balletbo i Serra
In an eDP application, HPD is not required and on most bridge chips useless. If HPD is not used, we need to set initial status as connected, otherwise the connector created by the drm_bridge_connector API remains in an unknown state. Signed-off-by: Enric Balletbo i Serra Reviewed-by: Laurent Pinc

[RESEND PATCH v4 3/7] drm/mediatek: mtk_dsi: Rename bridge to next_bridge

2020-06-16 Thread Enric Balletbo i Serra
This is really a cosmetic change just to make a bit more readable the code after convert the driver to drm_bridge. The bridge variable name will be used by the encoder drm_bridge, and the chained bridge will be named next_bridge. Signed-off-by: Enric Balletbo i Serra Reviewed-by: Laurent Pinchart

[PATCH 1/6] drm: encoder_slave: fix refcouting error for modules

2020-06-16 Thread Wolfram Sang
module_put() balances try_module_get(), not request_module(). Fix the error path to match that. Fixes: 2066facca4c7 ("drm/kms: slave encoder interface.") Signed-off-by: Wolfram Sang Reviewed-by: Emil Velikov --- I'd like to push it via I2C for 5.8-rc2. drivers/gpu/drm/drm_encoder_slave.c | 5

Re: [PATCH 00/17] spelling.txt: /decriptors/descriptors/

2020-06-16 Thread Jason Gunthorpe
On Tue, Jun 09, 2020 at 01:45:53PM +0100, Kieran Bingham wrote: > I wouldn't normally go through spelling fixes, but I caught sight of > this typo twice, and then foolishly grepped the tree for it, and saw how > pervasive it was. > > so here I am ... fixing a typo globally... but with an addition

[PATCH 2/3] drm/bridge: ps8640: Print an error if VDO control fails

2020-06-16 Thread Enric Balletbo i Serra
Print an error message inside ps8640_bridge_vdo_control() function when it fails so we can simplify a bit the callers, they will only need to check the error code. Signed-off-by: Enric Balletbo i Serra --- drivers/gpu/drm/bridge/parade-ps8640.c | 13 ++--- 1 file changed, 6 insertions(+

[PATCH 2/2] drm/amdgpu/debugfs: fix memory leak when amdgpu_virt_enable_access_debugfs failed

2020-06-16 Thread Chen Tao
Fix memory leak in amdgpu_debugfs_gpr_read not freeing data when amdgpu_virt_enable_access_debugfs failed. Fixes: 95a2f917387a2 ("drm/amdgpu: restrict debugfs register accessunder SR-IOV") Signed-off-by: Chen Tao --- drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | 4 +++- 1 file changed, 3 insert

[PATCH 1/2] drm/amdgpu/debugfs: fix memory leak when pm_runtime_get_sync failed

2020-06-16 Thread Chen Tao
Fix memory leak in amdgpu_debugfs_gpr_read not freeing data when pm_runtime_get_sync failed. Fixes: a9ffe2a983383 ("drm/amdgpu/debugfs: properly handle runtime pm") Signed-off-by: Chen Tao --- drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) d

[PATCH] drivers/video/backlight: Use kobj_to_dev() instead

2020-06-16 Thread Wang Qing
Use kobj_to_dev() instead of container_of() Signed-off-by: Wang Qing --- drivers/video/backlight/lm3533_bl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) mode change 100644 => 100755 drivers/video/backlight/lm3533_bl.c diff --git a/drivers/video/backlight/lm3533_bl.c b/drivers/video/

Re: [PATCH 26/38] vt: use newly defined CUR_* macros

2020-06-16 Thread Helge Deller
On 15.06.20 09:48, Jiri Slaby wrote: > We defined macros for all the magic constants in the previous patch. So > let us use the macro in the code now. > > No functional change intended. > > Signed-off-by: Jiri Slaby > Cc: Thomas Winischhofer > Cc: Bartlomiej Zolnierkiewicz > Cc: "James E.J. Bott

[PATCH 2/6] drm: encoder_slave: use new I2C API

2020-06-16 Thread Wolfram Sang
i2c_new_client() is deprecated, use the replacement i2c_new_client_device(). Also, we have a helper to check if a driver is bound. Use it to simplify the code. Note that this changes the errno for a failed device creation from ENOMEM to ENODEV. No callers currently interpret this errno, though, so

Re: [PATCH v2 5/5] drm/tegra: plane: Support 180° rotation

2020-06-16 Thread Dmitry Osipenko
15.06.2020 19:57, Ville Syrjälä пишет: > On Sun, Jun 14, 2020 at 11:01:21PM +0300, Dmitry Osipenko wrote: >> Combining horizontal and vertical reflections gives us 180 degrees of >> rotation. >> >> Signed-off-by: Dmitry Osipenko >> --- >> drivers/gpu/drm/tegra/dc.c | 13 - >> 1 file c

Re: [PATCH 00/17] spelling.txt: /decriptors/descriptors/

2020-06-16 Thread Martin K. Petersen
On Tue, 9 Jun 2020 13:45:53 +0100, Kieran Bingham wrote: > I wouldn't normally go through spelling fixes, but I caught sight of > this typo twice, and then foolishly grepped the tree for it, and saw how > pervasive it was. > > so here I am ... fixing a typo globally... but with an addition in > s

[Bug 208205] New: [amdgpu] System hang / freeze

2020-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208205 Bug ID: 208205 Summary: [amdgpu] System hang / freeze Product: Drivers Version: 2.5 Kernel Version: 5.7.2 Hardware: x86-64 OS: Linux Tree: Mainline

RE: [PATCH v3 07/15] drm/amdgpu: Use PCI_IRQ_MSI_TYPES where appropriate

2020-06-16 Thread Stankiewicz, Piotr
> -Original Message- > From: Alex Deucher > Sent: Tuesday, June 9, 2020 10:24 PM > > On Tue, Jun 9, 2020 at 5:18 AM Piotr Stankiewicz > wrote: > > > > Seeing as there is shorthand available to use when asking for any type > > of interrupt, or any type of message signalled interrupt, leve

[Bug 208205] [amdgpu] System hang / freeze

2020-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208205 Automne von Einzbern (mar...@automne.me) changed: What|Removed |Added URL||https://bugs.ar

Re: [PATCH] drivers/video/backlight: Use kobj_to_dev() instead

2020-06-16 Thread Lee Jones
On Mon, 15 Jun 2020, Wang Qing wrote: > Use kobj_to_dev() instead of container_of() > > Signed-off-by: Wang Qing > --- > drivers/video/backlight/lm3533_bl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > mode change 100644 => 100755 drivers/video/backlight/lm3533_bl.c I've fixed th

[PATCH][next] drm/i915: fix a couple of spelling mistakes in kernel parameter help text

2020-06-16 Thread Colin King
From: Colin Ian King There are a couple of spelling mistakes in kernel parameter help text, namely "helpfull" and "paramters". Fix them. Signed-off-by: Colin Ian King --- drivers/gpu/drm/i915/i915_params.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm

[PATCH] drm/amdgpu: Simplify IRQ vector request logic

2020-06-16 Thread Piotr Stankiewicz
pci_alloc_irq_vectors() will handle fallback from MSI-X to MSI internally, if necessary. So remove checks which determine if we are dealing with MSI or MSI-X and rely on pci_alloc_irq_vectors() to do the right thing. Signed-off-by: Piotr Stankiewicz Reviewed-by: Andy Shevchenko --- drivers/gpu/

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

2020-06-16 Thread Joonas Lahtinen
Quoting Stephen Rothwell (2020-06-16 02:39:12) > Hi all, > > After merging the drm-intel-fixes tree, today's linux-next build (x86_64 > allmodconfig) failed like this: > > In file included from drivers/gpu/drm/i915/gt/intel_lrc.c:5972: > drivers/gpu/drm/i915/gt/selftest_lrc.c: In function > 'liv

Re: [RFC PATCH 0/4] DirectX on Linux

2020-06-16 Thread Pavel Machek
Hi! > > The driver creates the /dev/dxg device, which can be opened by user mode > > application and handles their ioctls. The IOCTL interface to the driver > > is defined in dxgkmthk.h (Dxgkrnl Graphics Port Driver ioctl > > definitions). The interface matches the D3DKMT interface on Windows. > >

Re: [EXTERNAL] Re: [RFC PATCH 0/4] DirectX on Linux

2020-06-16 Thread Pavel Machek
Hi! > Thanks for the discussion. I may not be able to immediately answer all of > your questions, but I'll do my best . > Could you do something with your email settings? Because this is not how you should use email on lkml. "[EXTERNAL]" in the subject, top-posting, unwrapped lines... Tha

Re: [RFC PATCH 0/4] DirectX on Linux

2020-06-16 Thread Pavel Machek
> > Having said that, I hit one stumbling block: > > "Further, at this time there are no presentation integration. " > > > > If we upstream this driver as-is into some hyperv specific place, and > > you decide to add presentation integration this is more than likely > > going to mean you will want

Re: [PATCH] drm/shmem-helper: Fix obj->filp derefence

2020-06-16 Thread Thomas Zimmermann
Hi Daniel Am 15.06.20 um 17:10 schrieb Daniel Vetter: > I broke that in my refactoring: > > commit 7d2cd72a9aa3df3604cafd169a2d4a525afb68ca > Author: Daniel Vetter > Date: Fri May 29 16:05:42 2020 +0200 > > drm/shmem-helpers: Simplify dma-buf importing > > Reported-by: Thomas Zimmermann

[PATCH] drm/i915/display: fix missing null check on allocated dsb object

2020-06-16 Thread Colin King
From: Colin Ian King Currently there is no null check for a failed memory allocation on the dsb object and without this a null pointer dereference error can occur. Fix this by adding a null check. Note: added a drm_err message in keeping with the error message style in the function. Addresses-C

[PATCH] drm/shmem-helper: Only dma-buf imports are private obj

2020-06-16 Thread Daniel Vetter
I broke that in my refactoring: commit 7d2cd72a9aa3df3604cafd169a2d4a525afb68ca Author: Daniel Vetter Date: Fri May 29 16:05:42 2020 +0200 drm/shmem-helpers: Simplify dma-buf importing I'm not entirely sure of the history here, but I suspect that in one of the rebases or when applying the

Re: [PATCH] drm/i915/display: fix missing null check on allocated dsb object

2020-06-16 Thread Colin Ian King
On 16/06/2020 12:54, Dan Carpenter wrote: > On Tue, Jun 16, 2020 at 12:42:21PM +0100, Colin King wrote: >> From: Colin Ian King >> >> Currently there is no null check for a failed memory allocation >> on the dsb object and without this a null pointer dereference >> error can occur. Fix this by add

Re: [PATCH] drm/i915/display: fix missing null check on allocated dsb object

2020-06-16 Thread Dan Carpenter
On Tue, Jun 16, 2020 at 12:42:21PM +0100, Colin King wrote: > From: Colin Ian King > > Currently there is no null check for a failed memory allocation > on the dsb object and without this a null pointer dereference > error can occur. Fix this by adding a null check. > > Note: added a drm_err mes

Re: [PATCH 57/59] drm/ast: Use managed pci functions

2020-06-16 Thread Daniel Vetter
On Thu, Jun 11, 2020 at 02:04:03PM +0200, Thomas Zimmermann wrote: > Hi > > Am 15.04.20 um 09:40 schrieb Daniel Vetter: > > Allows us to remove a bit of cleanup code. > > > > Signed-off-by: Daniel Vetter > > Cc: Dave Airlie > > Cc: Thomas Zimmermann > > Cc: Gerd Hoffmann > > Cc: Daniel Vetter

Re: [PATCH] drm/shmem-helper: Only dma-buf imports are private obj

2020-06-16 Thread Thomas Zimmermann
Hi Am 16.06.20 um 13:47 schrieb Daniel Vetter: > I broke that in my refactoring: > > commit 7d2cd72a9aa3df3604cafd169a2d4a525afb68ca > Author: Daniel Vetter > Date: Fri May 29 16:05:42 2020 +0200 > > drm/shmem-helpers: Simplify dma-buf importing > > I'm not entirely sure of the history h

Re: [Linaro-mm-sig] [PATCH 04/18] dma-fence: prime lockdep annotations

2020-06-16 Thread Daniel Vetter
Hi Jason, Somehow this got stuck somewhere in the mail queues, only popped up just now ... On Thu, Jun 11, 2020 at 11:15:15AM -0300, Jason Gunthorpe wrote: > On Thu, Jun 11, 2020 at 10:34:30AM +0200, Daniel Vetter wrote: > > > I still have my doubts about allowing fence waiting from within shrink

Re: [PATCH] drm/bridge: fix reference count leaks due to pm_runtime_get_sync()

2020-06-16 Thread Daniel Vetter
On Sun, Jun 14, 2020 at 04:46:55PM +0300, Laurent Pinchart wrote: > Hi Aditya, > > (CC'ing Rafael) > > Thank you for the patch. > > On Sat, Jun 13, 2020 at 09:40:05PM -0500, Aditya Pakki wrote: > > On calling pm_runtime_get_sync() the reference count of the device > > is incremented. In case of

Re: [PATCH] drm/shmem-helper: Fix obj->filp derefence

2020-06-16 Thread Thomas Zimmermann
Hi, as discussed on IRC, we still need this patch. Am 15.06.20 um 17:10 schrieb Daniel Vetter: > I broke that in my refactoring: > > commit 7d2cd72a9aa3df3604cafd169a2d4a525afb68ca > Author: Daniel Vetter > Date: Fri May 29 16:05:42 2020 +0200 > > drm/shmem-helpers: Simplify dma-buf impo

Re: [PATCH 0/6] remove deprecated i2c_new_device API

2020-06-16 Thread Daniel Vetter
On Mon, Jun 15, 2020 at 09:58:09AM +0200, Wolfram Sang wrote: > I want to remove the above API this cycle, and just a few patches have > not made it into 5.8-rc1. They have been reviewed and most had been > promised to get into linux-next, but well, things happen. So, I hope it > is okay for everyo

Re: [PATCH] drm/kselftest: fix spellint typo in test-drm_mm.c

2020-06-16 Thread Daniel Vetter
On Mon, Jun 15, 2020 at 09:11:51PM +0900, Masanari Iida wrote: > This patch fix a spelling typo in test-drm_mm.c > > Signed-off-by: Masanari Iida Applied, thanks for your patch. -Daniel > --- > drivers/gpu/drm/selftests/test-drm_mm.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > >

[PATCH v2] drm/tegra: Add zpos property for cursor planes

2020-06-16 Thread Thierry Reding
From: Thierry Reding As of commit 4dc55525b095 ("drm: plane: Verify that no or all planes have a zpos property") a warning is emitted if there's a mix of planes with and without a zpos property. On Tegra, cursor planes are always composited on top of all other planes, which is why they never had

Re: [PATCH] dmabuf: use spinlock to access dmabuf->name

2020-06-16 Thread Sumit Semwal
Hi Daniel, Chris, On Thu, 11 Jun 2020 at 19:10, Charan Teja Kalla wrote: > > There exists a sleep-while-atomic bug while accessing the dmabuf->name > under mutex in the dmabuffs_dname(). This is caused from the SELinux > permissions checks on a process where it tries to validate the inherited > f

Re: [PATCH v2] dma-buf: Move dma_buf_release() from fops to dentry_ops

2020-06-16 Thread Sumit Semwal
Hello, If there are no objections to this, I will plan to merge it soon. Daniel, Chris, Chenbo? On Thu, 11 Jun 2020 at 17:14, Sumit Semwal wrote: > > Charan Teja reported a 'use-after-free' in dmabuffs_dname [1], which > happens if the dma_buf_release() is called while the userspace is > access

Re: [PATCH v2] dma-buf: Move dma_buf_release() from fops to dentry_ops

2020-06-16 Thread Chris Wilson
Quoting Sumit Semwal (2020-06-16 13:42:13) > Hello, > > If there are no objections to this, I will plan to merge it soon. I was going to suggest running it against our CI, but that's unavailable at the moment. There's a particularly nasty BUG_ON() in dma_buf_release that we hit irregularly, that

[PATCH libdrm] intel: sync i915_pciids.h with kernel

2020-06-16 Thread ramadevi . gandi
From: Gandi Ramadevi Add DG1 PCI ID Signed-off-by: Gandi Ramadevi --- intel/i915_pciids.h | 5 + 1 file changed, 5 insertions(+) diff --git a/intel/i915_pciids.h b/intel/i915_pciids.h index 662d8351..724e68a0 100644 --- a/intel/i915_pciids.h +++ b/intel/i915_pciids.h @@ -605,4 +605,9 @@

Re: [PATCH v2] dma-buf: Move dma_buf_release() from fops to dentry_ops

2020-06-16 Thread Sumit Semwal
Hi Chris, On Tue, 16 Jun 2020 at 18:20, Chris Wilson wrote: > > Quoting Sumit Semwal (2020-06-16 13:42:13) > > Hello, > > > > If there are no objections to this, I will plan to merge it soon. > > I was going to suggest running it against our CI, but that's unavailable > at the moment. > > There's

Re: [RFC PATCH 0/4] DirectX on Linux

2020-06-16 Thread Sasha Levin
On Tue, Jun 16, 2020 at 12:51:56PM +0200, Pavel Machek wrote: > Having said that, I hit one stumbling block: > "Further, at this time there are no presentation integration. " > > If we upstream this driver as-is into some hyperv specific place, and > you decide to add presentation integration thi

Re: [RFC PATCH 0/4] DirectX on Linux

2020-06-16 Thread Sasha Levin
On Tue, Jun 16, 2020 at 12:51:13PM +0200, Pavel Machek wrote: Hi! > The driver creates the /dev/dxg device, which can be opened by user mode > application and handles their ioctls. The IOCTL interface to the driver > is defined in dxgkmthk.h (Dxgkrnl Graphics Port Driver ioctl > definitions). T

Re: [drm/mgag200] e44e907dd8: phoronix-test-suite.glmark2.800x600.score -64.9% regression

2020-06-16 Thread Thomas Zimmermann
ld Operating System: Linux 4.9.0-8-amd64 x86_64 Debian > Current Operating System: Linux lkp-nhm-2ep1 > 5.7.0-rc5-01428-ge44e907dd8f937 #1 SMP Tue Jun 2 19:51:38 CST 2020 x86_64 > Kernel command line:  ip=lkp-nhm-2ep1::dhcp > root=/dev/disk/by-id/wwn-0x55cd2e4123123127-part2 > rootf

RE: [PATCH] dmabuf: use spinlock to access dmabuf->name

2020-06-16 Thread Ruhl, Michael J
>-Original Message- >From: dri-devel On Behalf Of >Charan Teja Kalla >Sent: Thursday, June 11, 2020 9:40 AM >To: Sumit Semwal ; open list:DMA BUFFER >SHARING FRAMEWORK ; DRI mailing list de...@lists.freedesktop.org> >Cc: Linaro MM SIG ; >vinme...@codeaurora.org; LKML ; >sta...@vger.kernel.

Re: [drm/mgag200] e44e907dd8: phoronix-test-suite.glmark2.800x600.score -64.9% regression

2020-06-16 Thread Emil Velikov
Hi Rong, Thanks for the prompt reply and information. Can I suggest including the suggested information in future reports. I've included a command for each one, to aid automating things. Namely: Xorg: 1.20.4 (or None) $ which Xorg 2>/dev/null || echo None && Xorg -version |& grep -o "X Server.*"

Re: [Intel-gfx] [PATCH 7/8] drm/mipi-dbi: Remove ->enabled

2020-06-16 Thread Emil Velikov
On Tue, 16 Jun 2020 at 07:50, Daniel Vetter wrote: > > On Mon, Jun 15, 2020 at 11:35 PM Emil Velikov > wrote: > > > > Hi Daniel, > > > > On Fri, 12 Jun 2020 at 17:01, Daniel Vetter wrote: > > > > > > The atomic helpers try really hard to not lose track of things, > > > duplicating enabled track

RE: [PATCH] dmabuf: use spinlock to access dmabuf->name

2020-06-16 Thread Ruhl, Michael J
>-Original Message- >From: dri-devel On Behalf Of >Ruhl, Michael J >Sent: Tuesday, June 16, 2020 9:51 AM >To: Charan Teja Kalla ; Sumit Semwal >; open list:DMA BUFFER SHARING FRAMEWORK >; DRI mailing list de...@lists.freedesktop.org> >Cc: Linaro MM SIG ; >vinme...@codeaurora.org; LKML ; >s

Re: [PATCH 0/6] remove deprecated i2c_new_device API

2020-06-16 Thread Emil Velikov
Hi all, On Tue, 16 Jun 2020 at 13:12, Daniel Vetter wrote: > > On Mon, Jun 15, 2020 at 09:58:09AM +0200, Wolfram Sang wrote: > > I want to remove the above API this cycle, and just a few patches have > > not made it into 5.8-rc1. They have been reviewed and most had been > > promised to get into

[PATCH 6/6] drm/i915/dp_mst: Ensure the DPCD ACT sent flag is cleared before waiting for it

2020-06-16 Thread Imre Deak
Atm, we clear the ACT sent flag in the sink's DPCD before updating the sink's payload table, along clearing the payload table updated flag. The sink is supposed to set this flag once it detects that the source has completed the ACT sequence (after detecting the 4 required ACT MTPH symbols sent by t

Re: [drm/mgag200] e44e907dd8: phoronix-test-suite.glmark2.800x600.score -64.9% regression

2020-06-16 Thread Thomas Zimmermann
Hi Am 15.06.20 um 22:58 schrieb Emil Velikov: > Hi all, > > On Thu, 4 Jun 2020 at 08:11, kernel test robot wrote: >> >> Greeting, >> >> FYI, we noticed a -64.9% regression of >> phoronix-test-suite.glmark2.800x600.score due to commit: >> > On one hand, I'm really happy to see performance testin

[PATCH] drm/mgag200: Don't set in MISC

2020-06-16 Thread Thomas Zimmermann
The original modesetting code set MISC to 0x2d, which is , and With the conversion to atomic modesetting, accidentally got enabled as well. Revert this change and initialize MISC with a constant value of and . The bits are set in mga_crtc_set_plls(), sync flags are set in mgag200_set_mode_reg

[PATCH][next] drm/i915/selftests: Fix inconsistent IS_ERR and PTR_ERR

2020-06-16 Thread Gustavo A. R. Silva
Fix inconsistent IS_ERR and PTR_ERR in live_timeslice_nopreempt(). The proper pointer to be passed as argument to PTR_ERR() is ce. This bug was detected with the help of Coccinelle. Fixes: b72f02d78e4f ("drm/i915/gt: Prevent timeslicing into unpreemptable requests") Signed-off-by: Gustavo A. R.

Re: [PATCH][next] drm/i915/selftests: Fix inconsistent IS_ERR and PTR_ERR

2020-06-16 Thread Chris Wilson
Quoting Gustavo A. R. Silva (2020-06-16 15:54:52) > Fix inconsistent IS_ERR and PTR_ERR in live_timeslice_nopreempt(). > > The proper pointer to be passed as argument to PTR_ERR() is ce. > > This bug was detected with the help of Coccinelle. > > Fixes: b72f02d78e4f ("drm/i915/gt: Prevent timesli

Re: [PATCH] drm/mgag200: Don't set in MISC

2020-06-16 Thread Emil Velikov
Hi Thomas, On Tue, 16 Jun 2020 at 15:26, Thomas Zimmermann wrote: > > The original modesetting code set MISC to 0x2d, which is , > and > > With the conversion to atomic modesetting, accidentally > got enabled as well. Revert this change and initialize MISC with a > constant value of and . The

[Bug 208205] [amdgpu] System hang / freeze

2020-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208205 Alex Deucher (alexdeuc...@gmail.com) changed: What|Removed |Added CC||alexdeuc...@gmail.c

[Bug 208209] New: [amdgpu] driver crash -- enable_link_dp -- RX 570

2020-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208209 Bug ID: 208209 Summary: [amdgpu] driver crash -- enable_link_dp -- RX 570 Product: Drivers Version: 2.5 Kernel Version: 5.3.0-55-generic Hardware: x86-64 OS: Linux

[Bug 208209] [amdgpu] driver crash -- enable_link_dp -- RX 570

2020-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208209 Alex Deucher (alexdeuc...@gmail.com) changed: What|Removed |Added CC||alexdeuc...@gmail.c

Re: [PATCH v7 3/3] drm/i915/dp: Expose connector VRR monitor range via debugfs

2020-06-16 Thread Emil Velikov
On Mon, 15 Jun 2020 at 22:47, Manasi Navare wrote: > > On Mon, Jun 15, 2020 at 10:36:28PM +0100, Emil Velikov wrote: > > Hi Manasi, > > > > On Sat, 13 Jun 2020 at 00:55, Manasi Navare > > wrote: > > > > > > From: Bhanuprakash Modem > > > > > > [Why] > > > It's useful to know the min and max vrr

Re: [PATCH 6/6] drm/i915/dp_mst: Ensure the DPCD ACT sent flag is cleared before waiting for it

2020-06-16 Thread Ville Syrjälä
On Tue, Jun 16, 2020 at 05:18:55PM +0300, Imre Deak wrote: > Atm, we clear the ACT sent flag in the sink's DPCD before updating the > sink's payload table, along clearing the payload table updated flag. > The sink is supposed to set this flag once it detects that the source > has completed the ACT

[Bug 206475] amdgpu under load drop signal to monitor until hard reset

2020-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206475 --- Comment #13 from Marco (rodomar...@protonmail.com) --- The only way I had it "fixed" (it's more of a workaround, but it is working) is to slightly drop the clocks (my GPU has by default a max boost clock of 1430 MHz, I have dropped it to 1340

Re: [PATCH v2 0/5] 180 degrees rotation support for NVIDIA Tegra DRM

2020-06-16 Thread Emil Velikov
On Tue, 16 Jun 2020 at 12:40, Dmitry Osipenko wrote: > > 16.06.2020 01:26, Emil Velikov пишет: > > Hi Dmitry, > > > > On Mon, 15 Jun 2020 at 08:28, Dmitry Osipenko wrote: > >> > >> Hello! > >> > >> This series adds 180° display plane rotation support to the NVIDIA Tegra > >> DRM driver which is n

Re: [PATCH 6/6] drm/i915/dp_mst: Ensure the DPCD ACT sent flag is cleared before waiting for it

2020-06-16 Thread Imre Deak
On Tue, Jun 16, 2020 at 06:45:46PM +0300, Ville Syrjälä wrote: > On Tue, Jun 16, 2020 at 05:18:55PM +0300, Imre Deak wrote: > > Atm, we clear the ACT sent flag in the sink's DPCD before updating the > > sink's payload table, along clearing the payload table updated flag. > > The sink is supposed to

Re: [RFC PATCH 0/4] DirectX on Linux

2020-06-16 Thread Sasha Levin
On Tue, Jun 16, 2020 at 04:41:22PM +0200, Pavel Machek wrote: On Tue 2020-06-16 09:28:19, Sasha Levin wrote: On Tue, Jun 16, 2020 at 12:51:13PM +0200, Pavel Machek wrote: > Hi! > > > > The driver creates the /dev/dxg device, which can be opened by user mode > > > application and handles their io

Re: [PATCH v2] drm/tegra: Add zpos property for cursor planes

2020-06-16 Thread Thierry Reding
On Tue, Jun 16, 2020 at 06:54:35PM +0300, Dmitry Osipenko wrote: > 16.06.2020 15:17, Thierry Reding пишет: > > From: Thierry Reding > > > > As of commit 4dc55525b095 ("drm: plane: Verify that no or all planes > > have a zpos property") a warning is emitted if there's a mix of planes > > with and

Re: [PATCH 4/4] drm: pl111: Update documentation

2020-06-16 Thread Emil Velikov
Hi all, Inspired by Linus and my personal confusion with the report, allow me to put some suggestions. Followed by an illustrative example. On Wed, 10 Jun 2020 at 08:38, kernel test robot wrote: > > Hi Linus, > > I love your patch! Perhaps something to improve: > > [auto build test WARNING on dr

Re: [PATCH 6/6] drm/i915/dp_mst: Ensure the DPCD ACT sent flag is cleared before waiting for it

2020-06-16 Thread Ville Syrjälä
On Tue, Jun 16, 2020 at 06:54:41PM +0300, Imre Deak wrote: > On Tue, Jun 16, 2020 at 06:45:46PM +0300, Ville Syrjälä wrote: > > On Tue, Jun 16, 2020 at 05:18:55PM +0300, Imre Deak wrote: > > > Atm, we clear the ACT sent flag in the sink's DPCD before updating the > > > sink's payload table, along c

Re: [PATCH] drm/panel-simple: fix connector type for LogicPD Type28 Display

2020-06-16 Thread Tomi Valkeinen
On 15/06/2020 17:53, Adam Ford wrote: On Mon, Jun 15, 2020 at 9:46 AM Fabio Estevam wrote: On Mon, Jun 15, 2020 at 10:19 AM Adam Ford wrote: The LogicPD Type28 display used by several Logic PD products has not worked since v5.5. Maybe you could tell which commit exactly and then put a Fix

[Bug 206475] amdgpu under load drop signal to monitor until hard reset

2020-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206475 --- Comment #14 from Andrew Ammerlaan (andrewammerl...@riseup.net) --- I sort of worked around this too. I changed two things: 1) the iGPU is now the primary GPU, and I use DRI_PRIME=1 to offload to the AMD gpu. This has reduced the amount of th

Re: [PATCH 6/6] drm/i915/dp_mst: Ensure the DPCD ACT sent flag is cleared before waiting for it

2020-06-16 Thread Ville Syrjälä
On Tue, Jun 16, 2020 at 07:23:21PM +0300, Ville Syrjälä wrote: > On Tue, Jun 16, 2020 at 06:54:41PM +0300, Imre Deak wrote: > > On Tue, Jun 16, 2020 at 06:45:46PM +0300, Ville Syrjälä wrote: > > > On Tue, Jun 16, 2020 at 05:18:55PM +0300, Imre Deak wrote: > > > > Atm, we clear the ACT sent flag in

Re: [PATCH 6/6] drm/i915/dp_mst: Ensure the DPCD ACT sent flag is cleared before waiting for it

2020-06-16 Thread Imre Deak
On Tue, Jun 16, 2020 at 07:40:47PM +0300, Ville Syrjälä wrote: > On Tue, Jun 16, 2020 at 07:23:21PM +0300, Ville Syrjälä wrote: > > On Tue, Jun 16, 2020 at 06:54:41PM +0300, Imre Deak wrote: > > > On Tue, Jun 16, 2020 at 06:45:46PM +0300, Ville Syrjälä wrote: > > > > On Tue, Jun 16, 2020 at 05:18:5

Re: [PATCH 1/5] drm/omap: Fix suspend resume regression after platform data removal

2020-06-16 Thread Tomi Valkeinen
On 11/06/2020 17:00, Grygorii Strashko wrote: On 09/06/2020 18:26, Tomi Valkeinen wrote: On 09/06/2020 18:19, Tony Lindgren wrote: But there's an extra runtime PM reference (dev.power.usage_count) that seems to come out of nowhere. So when omap_drm_suspend is finished, there's still usage_cou

Re: [PATCH] drm/shmem-helper: Fix obj->filp derefence

2020-06-16 Thread Daniel Vetter
On Tue, Jun 16, 2020 at 02:10:10PM +0200, Thomas Zimmermann wrote: > Hi, > > as discussed on IRC, we still need this patch. > > Am 15.06.20 um 17:10 schrieb Daniel Vetter: > > I broke that in my refactoring: > > > > commit 7d2cd72a9aa3df3604cafd169a2d4a525afb68ca > > Author: Daniel Vetter > > D

Re: [PATCH] drm/shmem-helper: Only dma-buf imports are private obj

2020-06-16 Thread Daniel Vetter
On Tue, Jun 16, 2020 at 02:06:24PM +0200, Thomas Zimmermann wrote: > Hi > > Am 16.06.20 um 13:47 schrieb Daniel Vetter: > > I broke that in my refactoring: > > > > commit 7d2cd72a9aa3df3604cafd169a2d4a525afb68ca > > Author: Daniel Vetter > > Date: Fri May 29 16:05:42 2020 +0200 > > > > dr

Re: [Intel-gfx] [PATCH 7/8] drm/mipi-dbi: Remove ->enabled

2020-06-16 Thread Daniel Vetter
On Tue, Jun 16, 2020 at 3:57 PM Emil Velikov wrote: > > On Tue, 16 Jun 2020 at 07:50, Daniel Vetter wrote: > > > > On Mon, Jun 15, 2020 at 11:35 PM Emil Velikov > > wrote: > > > > > > Hi Daniel, > > > > > > On Fri, 12 Jun 2020 at 17:01, Daniel Vetter > > > wrote: > > > > > > > > The atomic he

Re: [PATCH v2 0/5] 180 degrees rotation support for NVIDIA Tegra DRM

2020-06-16 Thread Laurent Pinchart
On Tue, Jun 16, 2020 at 08:20:57PM +0300, Dmitry Osipenko wrote: > 16.06.2020 18:48, Emil Velikov пишет: > > On Tue, 16 Jun 2020 at 12:40, Dmitry Osipenko wrote: > >> > >> 16.06.2020 01:26, Emil Velikov пишет: > >>> Hi Dmitry, > >>> > >>> On Mon, 15 Jun 2020 at 08:28, Dmitry Osipenko wrote: > >>>

Re: [RFC PATCH 0/4] DSI/DBI and TinyDRM driver

2020-06-16 Thread Emil Velikov
Hi all, Allow me to compare this approach with some work Linus W [1] did a while back, which I've just noticed. Pauls' approach: - Perhaps the shortest one possible Porting an existing DSI panel to DBI is 3 lines of code - compat line in the SPI/DSI bridge, a bus_type and mipi_dsi_maybe_registe

[PATCH v3] drm/tegra: Add zpos property for cursor planes

2020-06-16 Thread Thierry Reding
From: Thierry Reding As of commit 4dc55525b095 ("drm: plane: Verify that no or all planes have a zpos property") a warning is emitted if there's a mix of planes with and without a zpos property. On Tegra, cursor planes are always composited on top of all other planes, which is why they never had

Re: [PATCH v2 0/5] 180 degrees rotation support for NVIDIA Tegra DRM

2020-06-16 Thread Emil Velikov
On Tue, 16 Jun 2020 at 18:20, Dmitry Osipenko wrote: > > 16.06.2020 18:48, Emil Velikov пишет: > > On Tue, 16 Jun 2020 at 12:40, Dmitry Osipenko wrote: > >> > >> 16.06.2020 01:26, Emil Velikov пишет: > >>> Hi Dmitry, > >>> > >>> On Mon, 15 Jun 2020 at 08:28, Dmitry Osipenko wrote: > >

[Bug 208205] [amdgpu] System hang / freeze

2020-06-16 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=208205 --- Comment #2 from Automne von Einzbern (mar...@automne.me) --- Created attachment 289711 --> https://bugzilla.kernel.org/attachment.cgi?id=289711&action=edit Log before a crash -- You are receiving this mail because: You are watching the ass

  1   2   >