[PATCH v6 5/8] dt-bindings: arm: fsl: add Plymovent M2M board

2020-12-03 Thread Oleksij Rempel
Add Plymovent Group BV M2M iMX6dl based board Signed-off-by: Oleksij Rempel Acked-by: Rob Herring --- Documentation/devicetree/bindings/arm/fsl.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.ya

Re: [PATCH v2 3/5] thermal: devfreq_cooling: add new registration functions with Energy Model

2020-12-03 Thread Lukasz Luba
On 12/2/20 11:49 AM, Ionela Voinescu wrote: On Wednesday 02 Dec 2020 at 11:14:02 (+), Lukasz Luba wrote: Hi Ionela, On 12/2/20 10:24 AM, Ionela Voinescu wrote: Hi Lukasz, On Wednesday 18 Nov 2020 at 12:03:56 (+), Lukasz Luba wrote: [snip] + struct device_node *np = NULL;

[PATCH drm/hisilicon v2 0/3] Add the new api to install irq

2020-12-03 Thread Tian Tao
patch #1 is code refactorings to use devm_drm_dev_alloc. patch #2 add the new api to install irq, patch #3 is hibmc driver uses the newly added api to register interrupts. Changes since v1: The devm_drm_irq_install function returns devm_add_action_or_reset directly without checking that devm_add_a

Re: [PATCH v2 0/5] Thermal devfreq cooling improvements with Energy Model

2020-12-03 Thread Daniel Lezcano
On 18/11/2020 13:03, Lukasz Luba wrote: > Hi all, > > This patch set is a continuation of my previous work, which aimed > to add Energy Model to all devices. This series is a follow up > for the patches which got merged to v5.9-rc1. It aims to change > the thermal devfreq cooling and use the Energ

[PATCH v6 0/8] mainline Plymovent M2M and BAS board

2020-12-03 Thread Oleksij Rempel
changes v5: - do more panel-simple.yaml related cleanups changes v5: - rebase against latest shawngup/for-next - add patch to fix checkpatch warning on PLYM2M dts changes v4: - add PLYBAS board - PLYM2M: add touchscreen node - PLYM2M: add rename led nodes to led-x changes v3: - use old style cop

Re: [PATCH v2 1/5] thermal: devfreq_cooling: change tracing function and arguments

2020-12-03 Thread Ionela Voinescu
On Wednesday 18 Nov 2020 at 12:03:54 (+), Lukasz Luba wrote: > Prepare for deleting the static and dynamic power calculation and clean > the trace function. These two fields are going to be removed in the next > changes. > > Reviewed-by: Steven Rostedt (VMware) # for tracing code > Signed-off

Re: [PATCH rdma-core v3 2/6] verbs: Support dma-buf based memory region

2020-12-03 Thread Yishai Hadas
On 11/27/2020 10:55 PM, Jianxin Xiong wrote: Add new API function and new provider method for registering dma-buf based memory region. Update the man page and bump the API version. Signed-off-by: Jianxin Xiong --- debian/libibverbs1.symbols | 2 ++ libibverbs/CMakeLists.txt| 2 +- l

[PATCH drm/hisilicon 1/3] drm/hisilicon: Code refactoring for hibmc_drm_drv

2020-12-03 Thread Tian Tao
Use the devm_drm_dev_alloc provided by the drm framework to alloc a structure hibmc_drm_private. Signed-off-by: Tian Tao --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 2 +- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 46 +++- drivers/gpu/drm/hisilicon/hibmc/h

[PATCH] drm/hisilicon: Use managed VRAM-helper initialization

2020-12-03 Thread Tian Tao
updated to use drmm_vram_helper_init() Signed-off-by: Tian Tao --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 1 - drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.h | 1 - drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c | 19 +++ 3 files changed, 3 insertions(+), 18 deleti

[PATCH v6 2/8] dt-bindings: display: simple: add EDT compatibles already supported by the driver

2020-12-03 Thread Oleksij Rempel
Some EDT compatibles are already supported by the driver but will fail on checkpatch script. Fix it by syncing dt-bindings documentation with the driver. Signed-off-by: Oleksij Rempel --- .../devicetree/bindings/display/panel/panel-simple.yaml| 3 +++ 1 file changed, 3 insertions(+) dif

Re: [PATCH v2 3/5] thermal: devfreq_cooling: add new registration functions with Energy Model

2020-12-03 Thread Ionela Voinescu
On Wednesday 02 Dec 2020 at 11:14:02 (+), Lukasz Luba wrote: > Hi Ionela, > > On 12/2/20 10:24 AM, Ionela Voinescu wrote: > > Hi Lukasz, > > > > On Wednesday 18 Nov 2020 at 12:03:56 (+), Lukasz Luba wrote: > > [snip] > > > > + struct device_node *np = NULL; > > [snip] > > > > + > > >

[PATCH drm/hisilicon v2 3/3] drm/hisilicon: Use the new api devm_drm_irq_install

2020-12-03 Thread Tian Tao
Use devm_drm_irq_install to register interrupts so that drm_irq_uninstall is not called when hibmc is removed. Signed-off-by: Tian Tao Reviewed-by: Thomas Zimmermann --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drive

[PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread James Park
Attempting to submit patch in response to https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6162#note_712454 This will allow Mesa to port code to Windows more easily. Hide BSD header and drm_handle_t behind _WIN32 check. Change __volatile__ to volatile, which is standard. Signed-off-by

[PATCH v6 3/8] dt-bindings: display: simple: Add Kyocera tcg070wvlq panel

2020-12-03 Thread Oleksij Rempel
So far, this panel seems to be compatible with "lg,lb070wv8", on other hand it is better to set this compatible in the devicetree. So, let's add it for now only to the dt-binding documentation to fix the checkpatch warnings. Signed-off-by: Oleksij Rempel --- .../devicetree/bindings/display/panel

Re: [PATCH RESEND 1/2] dma-fence: allow signaling drivers to set fence timestamp

2020-12-03 Thread veeras
On 2020-11-19 03:58, Daniel Vetter wrote: On Thu, Nov 12, 2020 at 7:27 PM Veera Sundaram Sankaran wrote: Some drivers have hardware capability to get the precise timestamp of certain events based on which the fences are triggered. This allows it to set accurate timestamp factoring out any soft

Re: [PATCH v2 3/5] thermal: devfreq_cooling: add new registration functions with Energy Model

2020-12-03 Thread Ionela Voinescu
Hi Lukasz, On Wednesday 18 Nov 2020 at 12:03:56 (+), Lukasz Luba wrote: > + * Register a devfreq cooling device and attempt to register Energy Model. > The > + * available OPPs must be registered for the device. > + * > + * If @dfc_power is provided, the cooling device is registered with the

[PATCH v2 2/6] dt-bindings: display: imx: Add i.MX8qxp/qm PRG binding

2020-12-03 Thread Liu Ying
This patch adds bindings for i.MX8qxp/qm Display Prefetch Resolve Gasket. Signed-off-by: Liu Ying --- Note that this depends on the 'two cell binding' clock patch set which has already landed in Shawn's i.MX clk/imx git branch. Otherwise, imx8-lpcg.h won't be found. v1->v2: * Use new dt binding

[PATCH v2 0/6] drm/imx: Introduce i.MX8qm/qxp DPU DRM

2020-12-03 Thread Liu Ying
Hi, This patch set introduces i.MX8qm/qxp Display Processing Unit(DPU) DRM support. DPU is comprised of a blit engine for 2D graphics, a display controller and a command sequencer. Outside of DPU, optional prefetch engines can fetch data from memory prior to some DPU fetchunits of blit engine a

Re: [PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread James Park
I can avoid modifying drm.h by doing this to drm_fourcc.h: #ifdef _WIN32 #include typedef uint64_t __u64; #else #include "drm.h" #endif And this to amdgpu_drm.h: #ifdef _WIN32 #include typedef int32_t __s32; typedef uint32_t __u32; typedef uint64_t __u64; #else #include "drm.h" #endif But no

[PATCH v2 6/6] MAINTAINERS: add maintainer for i.MX8qxp DPU DRM driver

2020-12-03 Thread Liu Ying
Add myself as the maintainer of the i.MX8qxp DPU DRM driver. Signed-off-by: Liu Ying --- v1->v2: * No change. MAINTAINERS | 9 + 1 file changed, 9 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 970d9ce..dee4586 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -5834,6 +5834,15

[PATCH v2 3/6] dt-bindings: display: imx: Add i.MX8qxp/qm DPR channel binding

2020-12-03 Thread Liu Ying
This patch adds bindings for i.MX8qxp/qm Display Prefetch Resolve Channel. Signed-off-by: Liu Ying --- Note that this depends on the 'two cell binding' clock patch set which has already landed in Shawn's i.MX clk/imx git branch. Otherwise, imx8-lpcg.h won't be found. v1->v2: * Use new dt bindin

[PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread James Park
This will allow Mesa to port code to Windows more easily. Hide BSD header and drm_handle_t behind _WIN32 check. Change __volatile__ to volatile, which is standard. Signed-off-by: James Park --- include/uapi/drm/drm.h | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git

Re: [PATCH v2 2/5] thermal: devfreq_cooling: get a copy of device status

2020-12-03 Thread Ionela Voinescu
On Wednesday 18 Nov 2020 at 12:03:55 (+), Lukasz Luba wrote: > Devfreq cooling needs to now the correct status of the device in order > to operate. Do not rely on Devfreq last_status which might be a stale data > and get more up-to-date values of the load. > > Devfreq framework can change the

[PATCH v6 1/8] dt-bindings: display: simple: fix alphabetical order for EDT compatibles

2020-12-03 Thread Oleksij Rempel
Reorder it alphabetically and remove one double entry. Signed-off-by: Oleksij Rempel --- .../bindings/display/panel/panel-simple.yaml | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/Documentation/devicetree/bindings/display/panel/panel-simple.yaml b/Docu

[PATCH drm/hisilicon v2 2/3] drm/irq: Add the new api to install irq

2020-12-03 Thread Tian Tao
Add new api devm_drm_irq_install() to register interrupts, no need to call drm_irq_uninstall() when the drm module is removed. Signed-off-by: Tian Tao --- drivers/gpu/drm/drm_irq.c | 32 include/drm/drm_irq.h | 2 +- 2 files changed, 33 insertions(+), 1 dele

Re: [PATCH v2 4/5] thermal: devfreq_cooling: remove old power model and use EM

2020-12-03 Thread Ionela Voinescu
On Wednesday 18 Nov 2020 at 12:03:57 (+), Lukasz Luba wrote: > Remove old power model and use new Energy Model to calculate the power > budget. It drops static + dynamic power calculations and power table > in order to use Energy Model performance domain data. This model > should be easy to use

[PATCH v2] drm/vc4: hdmi: Don't poll for the infoframes status on setup

2020-12-03 Thread Maxime Ripard
The infoframes are sent at a regular interval as a data island packet, so we don't need to wait for them to be sent when we're setting them up. However, we do need to poll when we're enabling since the we can't update the packet RAM until it has been sent. Let's add a boolean flag to tell whether

[PATCH v6 6/8] ARM: dts: add Plymovent M2M board

2020-12-03 Thread Oleksij Rempel
Plymovent M2M is a control interface produced for the Plymovent filter systems. Co-Developed-by: David Jander Signed-off-by: David Jander Signed-off-by: Oleksij Rempel --- arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/imx6dl-plym2m.dts | 446 2 fil

[PATCH drm/hisilicon v2 1/3] drm/hisilicon: Code refactoring for hibmc_drm_drv

2020-12-03 Thread Tian Tao
Use the devm_drm_dev_alloc provided by the drm framework to alloc a structure hibmc_drm_private. Signed-off-by: Tian Tao Acked-by: Thomas Zimmermann --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_de.c | 2 +- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 46 +++- driv

[PATCH v6 4/8] dt-bindings: vendor-prefixes: Add an entry for Plymovent

2020-12-03 Thread Oleksij Rempel
Add "ply" entry for Plymovent Group BV: https://www.plymovent.com/ Signed-off-by: Oleksij Rempel Acked-by: Rob Herring --- Documentation/devicetree/bindings/vendor-prefixes.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Docum

Re: [PATCH v2 3/5] thermal: devfreq_cooling: add new registration functions with Energy Model

2020-12-03 Thread Lukasz Luba
Hi Ionela, On 12/2/20 10:24 AM, Ionela Voinescu wrote: Hi Lukasz, On Wednesday 18 Nov 2020 at 12:03:56 (+), Lukasz Luba wrote: [snip] + struct device_node *np = NULL; [snip] + + if (dev->of_node) + np = of_node_get(dev->of_node); + Should np be checked be

Re: [PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread James Park
If the definitions in drm_fourcc.h make sense to live there, and we can't remove drm.h from that header for backward compatibility, and the code that I'm trying to compile on Windows needs the definitions in drm_fourcc.h, then doesn't it make sense to adjust drm.h? The patch that I'm proposing doe

[PATCH drm/hisilicon 0/3] Add the new api to install irq

2020-12-03 Thread Tian Tao
patch #1 is code refactorings to use devm_drm_dev_alloc. patch #2 add the new api to install irq, patch #3 is hibmc driver uses the newly added api to register interrupts. Tian Tao (3): drm/hisilicon: Code refactoring for hibmc_drm_drv drm/irq: Add the new api to install irq drm/hisilicon: U

[PATCH v2 1/6] dt-bindings: display: imx: Add i.MX8qxp/qm DPU binding

2020-12-03 Thread Liu Ying
This patch adds bindings for i.MX8qxp/qm Display Processing Unit. Signed-off-by: Liu Ying --- Note that this depends on the 'two cell binding' clock patch set which has already landed in Shawn's i.MX clk/imx git branch. Otherwise, imx8-lpcg.h won't be found. v1->v2: * Fix yamllint warnings. * R

Re: [PATCH 2/2] powerpc/ps3: make system bus's remove and shutdown callbacks return void

2020-12-03 Thread Michael Ellerman
Uwe Kleine-König writes: > Hello Michael, > > On Sat, Nov 28, 2020 at 09:48:30AM +0100, Takashi Iwai wrote: >> On Thu, 26 Nov 2020 17:59:50 +0100, >> Uwe Kleine-König wrote: >> > >> > The driver core ignores the return value of struct device_driver::remove >> > because there is only little that c

Re: [PATCH drm/hisilicon 1/3] drm/hisilicon: Code refactoring for hibmc_drm_drv

2020-12-03 Thread tiantao (H)
Hi 在 2020/12/2 17:02, Thomas Zimmermann 写道: Am 02.12.20 um 09:47 schrieb Tian Tao: Use the devm_drm_dev_alloc provided by the drm framework to alloc a structure hibmc_drm_private. Signed-off-by: Tian Tao This looks good now. Thanks for sticking to it. Thank you for your continued patie

[PATCH drm/hisilicon 2/3] drm/irq: Add the new api to install irq

2020-12-03 Thread Tian Tao
Add new api devm_drm_irq_install() to register interrupts, no need to call drm_irq_uninstall() when the drm module is removed. Signed-off-by: Tian Tao --- drivers/gpu/drm/drm_irq.c | 35 +++ include/drm/drm_irq.h | 2 +- 2 files changed, 36 insertions(+), 1 d

[PATCH v6 8/8] ARM: dts: add Plymovent BAS board

2020-12-03 Thread Oleksij Rempel
Plymovent BAS is a base system controller produced for the Plymovent filter systems. Co-Developed-by: David Jander Signed-off-by: David Jander Signed-off-by: Oleksij Rempel --- arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/imx6dl-plybas.dts | 394

[PATCH v6 7/8] dt-bindings: arm: fsl: add Plymovent BAS board

2020-12-03 Thread Oleksij Rempel
Add Plymovent Group BV BAS iMX6dl based board Signed-off-by: Oleksij Rempel Acked-by: Rob Herring --- Documentation/devicetree/bindings/arm/fsl.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml b/Documentation/devicetree/bindings/arm/fsl.ya

[PATCH v2 4/6] drm/atomic: Avoid unused-but-set-variable warning on for_each_old_plane_in_state

2020-12-03 Thread Liu Ying
Artifically use 'plane' and 'old_plane_state' to avoid 'not used' warning. The precedent has already been set by other macros in the same file. Signed-off-by: Liu Ying --- v1->v2: * No change. include/drm/drm_atomic.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/includ

[PATCH drm/hisilicon 3/3] drm/hisilicon: Use the new api devm_drm_irq_install

2020-12-03 Thread Tian Tao
Use devm_drm_irq_install to register interrupts so that drm_irq_uninstall is not called when hibmc is removed. Signed-off-by: Tian Tao Reviewed-by: Thomas Zimmermann --- drivers/gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drive

Re: [PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread Michel Dänzer
On 2020-12-02 8:47 p.m., James Park wrote: If we're trying to cut ties with the drm-uapi folder entirely, the stuff ac_surface.c need includes the AMD_FMT_MOD stuff in drm_fourcc.h, and AMDGPU_TILING_* under amdgpu_drm.h. Is there a better spot for these definitions? The Mesa src/amd/ code

[PATCH] gma500: clean up error handling in init

2020-12-03 Thread Dan Carpenter
The main problem with this error handling was that it didn't clean up if i2c_add_numbered_adapter() failed. This code is pretty old, and doesn't match with today's checkpatch.pl standards so I took the opportunity to tidy it up a bit. I changed the NULL comparison, and removed the WARNING message

Re: [PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread Pekka Paalanen
On Wed, 2 Dec 2020 23:25:58 +0100 Daniel Vetter wrote: > Also kinda disappointing that drm_fourcc.h includes drm.h and isn't > standalone, but I guess that sailed (at least for linux). Hi, FWIW, libweston core needs drm_fourcc.h too, even if nothing would ever use DRM or need libdrm otherwise.

Re: [PATCH] drm/hisilicon: Deletted the entire file hibmc_ttm.c

2020-12-03 Thread Thomas Zimmermann
Hi Am 03.12.20 um 10:05 schrieb Tian Tao: Deletted the entire file hibmc_ttm.c. drmm_vram_helper_init() can be Deletted -> Delete Here and in the subject line. called directly from hibmc_load(). hibmc_dumb_create() and hibmc_mode_funcs can go to hibmc_drm_drv.c Signed-off-by: Tian Tao C

Re: [PATCH] gma500: clean up error handling in init

2020-12-03 Thread Patrik Jakobsson
On Thu, Dec 3, 2020 at 9:41 AM Dan Carpenter wrote: > > The main problem with this error handling was that it didn't clean up if > i2c_add_numbered_adapter() failed. This code is pretty old, and doesn't > match with today's checkpatch.pl standards so I took the opportunity to > tidy it up a bit.

Re: [PATCH] drm/i915: Check the correct variable

2020-12-03 Thread Chris Wilson
Quoting Dan Carpenter (2020-12-03 08:45:17) > There is a copy and paste bug in this code. It's supposed to check > "obj2" instead of checking "obj" a second time. > > Fixes: 80f0b679d6f0 ("drm/i915: Add an implementation for i915_gem_ww_ctx > locking, v2.") > Signed-off-by: Dan Carpenter > ---

Re: [PATCH] drm/i915: Check the correct variable

2020-12-03 Thread Andi Shyti
Hi Dan, > There is a copy and paste bug in this code. It's supposed to check > "obj2" instead of checking "obj" a second time. > > Fixes: 80f0b679d6f0 ("drm/i915: Add an implementation for i915_gem_ww_ctx > locking, v2.") > Signed-off-by: Dan Carpenter > --- > drivers/gpu/drm/i915/selftests/i

[Bug 210465] New: Vega 3 lock MCLK on 1200mhz

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=210465 Bug ID: 210465 Summary: Vega 3 lock MCLK on 1200mhz Product: Drivers Version: 2.5 Kernel Version: 5.9.11, 5.4.80-2 Hardware: x86-64 OS: Linux Tree: Mainline

[Bug 210467] New: Vega 3 lock MCLK on 1200mhz

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=210467 Bug ID: 210467 Summary: Vega 3 lock MCLK on 1200mhz Product: Drivers Version: 2.5 Kernel Version: 5.9.11, 5.4.80-2 Hardware: x86-64 OS: Linux Tree: Mainline

[Bug 210465] Vega 3 lock MCLK on 1200mhz

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=210465 Alexey (intervio...@gmail.com) changed: What|Removed |Added Status|NEW |RESOLVED Resolut

[Bug 210467] Vega 3 lock MCLK on 1200mhz

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=210467 --- Comment #1 from Alexey (intervio...@gmail.com) --- *** Bug 210465 has been marked as a duplicate of this bug. *** -- You are receiving this mail because: You are watching the assignee of the bug. _

[Bug 210467] Vega 3 lock MCLK on 1200mhz

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=210467 --- Comment #2 from Alexey (intervio...@gmail.com) --- Created attachment 293915 --> https://bugzilla.kernel.org/attachment.cgi?id=293915&action=edit dmesg -- You are receiving this mail because: You are watching the assignee of the bug. _

[Bug 210467] Vega 3 lock MCLK on 1200mhz

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=210467 --- Comment #3 from Alexey (intervio...@gmail.com) --- Created attachment 293917 --> https://bugzilla.kernel.org/attachment.cgi?id=293917&action=edit dmesg debug mode -- You are receiving this mail because: You are watching the assignee of the

[Bug 210467] amdgpu Vega 3 lock MCLK on 1200mhz

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=210467 Alexey (intervio...@gmail.com) changed: What|Removed |Added Summary|Vega 3 lock MCLK on 1200mhz |amdgpu Vega 3 lock MCLK o

Re: [PATCH] iommu/io-pgtable: Remove tlb_flush_leaf

2020-12-03 Thread Will Deacon
On Wed, Nov 25, 2020 at 05:29:39PM +, Robin Murphy wrote: > The only user of tlb_flush_leaf is a particularly hairy corner of the > Arm short-descriptor code, which wants a synchronous invalidation to > minimise the races inherent in trying to split a large page mapping. > This is already far e

Re: [PATCH v2 1/5] drm: add legacy support for using degamma for gamma

2020-12-03 Thread Ville Syrjälä
On Wed, Dec 02, 2020 at 01:38:42PM +0100, Daniel Vetter wrote: > On Wed, Dec 2, 2020 at 12:52 PM Tomi Valkeinen wrote: > > > > On 30/11/2020 16:10, Daniel Vetter wrote: > > > > > The thing is, the legacy helpers should be able to pull off what userspace > > > needs to do when it's using atomic any

Re: [PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread Ville Syrjälä
On Wed, Dec 02, 2020 at 11:25:58PM +0100, Daniel Vetter wrote: > On Wed, Dec 2, 2020 at 8:48 PM James Park wrote: > > > > I can avoid modifying drm.h by doing this to drm_fourcc.h: > > > > #ifdef _WIN32 > > #include > > typedef uint64_t __u64; > > #else > > #include "drm.h" > > #endif > > > > And

Re: [PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread Simon Ser
On Thursday, December 3, 2020 1:54 PM, Ville Syrjälä wrote: > > The drm_fourcc.h maybe makes some sense (I think in some places mesa > > uses these internally, and many drivers use the modifiers directly in > > the main driver). But the amdgpu header should be all ioctl stuff, > > which should b

Re: [Intel-gfx] [PATCH] drm/i915: Check the correct variable

2020-12-03 Thread Chris Wilson
Quoting Andi Shyti (2020-12-03 11:12:24) > Hi Dan, > > > There is a copy and paste bug in this code. It's supposed to check > > "obj2" instead of checking "obj" a second time. > > > > Fixes: 80f0b679d6f0 ("drm/i915: Add an implementation for i915_gem_ww_ctx > > locking, v2.") > > Signed-off-by:

[PATCH AUTOSEL 5.9 20/39] drm/exynos: depend on COMMON_CLK to fix compile tests

2020-12-03 Thread Sasha Levin
From: Krzysztof Kozlowski [ Upstream commit e2d3d2e904ad3d381753798dcd5cae03e3c47242 ] The Exynos DRM uses Common Clock Framework thus it cannot be built on platforms without it (e.g. compile test on MIPS with RALINK and SOC_RT305X): /usr/bin/mips-linux-gnu-ld: drivers/gpu/drm/exynos/exynos

[PATCH] drm/i915: Check the correct variable

2020-12-03 Thread Dan Carpenter
There is a copy and paste bug in this code. It's supposed to check "obj2" instead of checking "obj" a second time. Fixes: 80f0b679d6f0 ("drm/i915: Add an implementation for i915_gem_ww_ctx locking, v2.") Signed-off-by: Dan Carpenter --- drivers/gpu/drm/i915/selftests/i915_gem.c | 4 ++-- 1 fil

[PULL] drm-intel-fixes v2

2020-12-03 Thread Rodrigo Vivi
Hi Dave and Daniel, Please ignore the pull request I had sent yesterday and use only this one. I had missed one patch: 14d1eaf08845 ("drm/i915/gt: Protect context lifetime with RCU"). Also, please notice that the commit 6db58901c2aa ("drm/i915/display: return earlier from +intel_modeset_init()

[PATCH v2 1/7] drm/ast: Don't pin cursor source BO explicitly during update

2020-12-03 Thread Thomas Zimmermann
Vmapping the cursor source BO contains an implicit pin operation, so there's no need to do this manually. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/ast/ast_cursor.c | 10 +- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/drivers/gpu/drm/ast/ast_cursor.c b/drivers

[PATCH v2 0/7] drm/vram-helper: Lock GEM BOs while they are mapped

2020-12-03 Thread Thomas Zimmermann
GEM VRAM helpers used to pin the BO in their implementation of vmap, so that they could not be relocated. In a recent discussion, [1] it became clear that this is incorrect and that vmap should rather repend on the reservation lock to prevent relocation. This patchset addresses the issue. Version

[PATCH v2 4/7] drm/vram-helper: Remove pinning from drm_gem_vram_{vmap, vunmap}()

2020-12-03 Thread Thomas Zimmermann
BO pinning was never meant to be part of a GEM object's vmap operation. Remove it from the related code in VRAM helpers. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/drm_gem_vram_helper.c | 16 +--- 1 file changed, 1 insertion(+), 15 deletions(-) diff --git a/drivers/gpu/drm

[PATCH v2 6/7] drm/vram-helper: Simplify vmap implementation

2020-12-03 Thread Thomas Zimmermann
After removing the pinning operations, the vmap/vunmap code as been reduced to what used to be an internal helper. Inline the helper to simplify the implementation. Signed-off-by: Thomas Zimmermann Acked-by: Christian König --- drivers/gpu/drm/drm_gem_vram_helper.c | 52 +++-

[PATCH v2 3/7] drm/vram-helper: Move BO locking from vmap code into callers

2020-12-03 Thread Thomas Zimmermann
Implementations of the vmap/vunmap callbacks may expect that the caller holds the reservation lock. Therefore push the locking from vmap and vunmap into the callers. This affects fbdev emulation, and cursor updates in ast and vboxvideo. Ast and vboxvideo acquire the BO's reservation lock directly.

[PATCH v2 5/7] drm/vram-helper: Remove vmap reference counting

2020-12-03 Thread Thomas Zimmermann
Overlapping or nested mappings of the same BO are not allowed by the semantics of the GEM vmap/vunmap operations. Concurent access to the GEM object is prevented by reservation locks. So we don't need the reference counter in the GEM VRAM object. Remove it. Signed-off-by: Thomas Zimmermann ---

[PATCH v2 2/7] drm/ast: Only map cursor BOs during updates

2020-12-03 Thread Thomas Zimmermann
The HW cursor's BO used to be mapped permanently into the kernel's address space. GEM's vmap operation will be protected by locks, and we don't want to lock the BO's for an indefinate period of time. Change the cursor code to map the HW BOs only during updates. The vmap operation in VRAM helpers i

[PATCH v2 7/7] dma-buf: Write down some rules for vmap usage

2020-12-03 Thread Thomas Zimmermann
Dma-buf's vmap and vunmap callbacks are undocumented and various exporters currently have slightly different semantics for them. Add documentation on how to implement and use these interfaces correctly. v2: * document vmap semantics in struct dma_buf_ops * add TODO item for reviewi

Re: [PATCH v2 0/7] drm/vram-helper: Lock GEM BOs while they are mapped

2020-12-03 Thread Thomas Zimmermann
Am 03.12.20 um 15:02 schrieb Thomas Zimmermann: GEM VRAM helpers used to pin the BO in their implementation of vmap, so that they could not be relocated. In a recent discussion, [1] it became Grrr, [1] was again supposed to point to the discussion at https://patchwork.freedesktop.org/patch

Re: [PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread Daniel Vetter
On Thu, Dec 3, 2020 at 9:18 AM Michel Dänzer wrote: > > On 2020-12-02 8:47 p.m., James Park wrote: > > > > If we're trying to cut ties with the drm-uapi folder entirely, the stuff > > ac_surface.c need includes the AMD_FMT_MOD stuff in drm_fourcc.h, > > and AMDGPU_TILING_* under amdgpu_drm.h. Is t

Re: [PATCH 14/15] drm/vmwgfx: Remove references to struct drm_device.pdev

2020-12-03 Thread Daniel Vetter
On Thu, Dec 03, 2020 at 03:06:20AM +, Zack Rusin wrote: > > > > On Dec 2, 2020, at 11:03, Daniel Vetter wrote: > > > > On Wed, Dec 2, 2020 at 4:37 PM Zack Rusin wrote: > >> > >> > >> > >>> On Dec 2, 2020, at 09:27, Thomas Zimmermann wrote: > >>> > >>> Hi > >>> > >>> Am 02.12.20 um 09

Re: [PATCH 0/8] drm/vc4: DSI improvements and BCM2711 support

2020-12-03 Thread Dave Stevenson
Hi Maxime On Thu, 3 Dec 2020 at 13:25, Maxime Ripard wrote: > > Hi, > > Here's a series adding support for the DSI0 controller in the BCM2835 and the > DSI1 controller found in the BCM2711. > > Let me know what you think, > Maxime Thanks for that series - your using a variant structure is much c

Re: [PATCH v2 7/7] dma-buf: Write down some rules for vmap usage

2020-12-03 Thread Daniel Vetter
On Thu, Dec 03, 2020 at 03:02:59PM +0100, Thomas Zimmermann wrote: > Dma-buf's vmap and vunmap callbacks are undocumented and various > exporters currently have slightly different semantics for them. Add > documentation on how to implement and use these interfaces correctly. > > v2: > * docu

[PATCH 2/2] drm: automatic legacy gamma support

2020-12-03 Thread Tomi Valkeinen
To support legacy gamma ioctls the drivers need to set drm_crtc_funcs.gamma_set either to a custom implementation or to drm_atomic_helper_legacy_gamma_set. Most of the atomic drivers do the latter. We can simplify this by making the core handle it automatically. Add two functions: drm_crtc_suppor

[PATCH 0/2] drm: fix and cleanup legacy gamma support

2020-12-03 Thread Tomi Valkeinen
Hi, The first patch fix legacy gamma table for HW which have a degamma lut block before CTM block, but no gamma lut after the CTM. The second one cleans up the legacy gamma support a bit by handling legacy gamma for modern drivers in the drm core. Tomi Tomi Valkeinen (2): drm: add legacy sup

[PATCH 1/2] drm: add legacy support for using degamma for gamma

2020-12-03 Thread Tomi Valkeinen
We currently have drm_atomic_helper_legacy_gamma_set() helper which can be used to handle legacy gamma-set ioctl. drm_atomic_helper_legacy_gamma_set() sets GAMMA_LUT, and clears CTM and DEGAMMA_LUT. This works fine on HW where we have either: degamma -> ctm -> gamma -> out or ctm -> gamma -> out

Re: [PATCH v4 0/2] drm: add DisplayPort connector

2020-12-03 Thread Tomi Valkeinen
Hi DRM Bridge maintainers, On 30/11/2020 13:29, Tomi Valkeinen wrote: > Hi, > > This series adds the DT bindings and a driver for DisplayPort connector. > > Minor changes since v3: > - Added Laurent's reviewed-bys > - Added $ref to graph schema > - Use 'ret' instead of 'r' > - Add the missing pe

[PATCH] drm/sched: remove superfluous whitespace in drm_sched_resubmit_jobs

2020-12-03 Thread Lucas Stach
Signed-off-by: Lucas Stach --- drivers/gpu/drm/scheduler/sched_main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/scheduler/sched_main.c b/drivers/gpu/drm/scheduler/sched_main.c index 9a0d77a68018..f517ffd53847 100644 --- a/drivers/gpu/drm/scheduler/sched_main.c +++ b/dr

Re: [PATCH] drm/sched: remove superfluous whitespace in drm_sched_resubmit_jobs

2020-12-03 Thread Christian König
Dim will probably reject that without a one line as commit message. Am 03.12.20 um 17:02 schrieb Lucas Stach: Signed-off-by: Lucas Stach Apart from that Reviewed-by: Christian König --- drivers/gpu/drm/scheduler/sched_main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/g

Re: [PATCH v2 1/3] drm/msm: adreno: Make speed-bin support generic

2020-12-03 Thread Akhil P Oommen
On 12/2/2020 10:00 PM, Jordan Crouse wrote: On Wed, Dec 02, 2020 at 08:53:51PM +0530, Akhil P Oommen wrote: On 11/30/2020 10:32 PM, Jordan Crouse wrote: On Fri, Nov 27, 2020 at 06:19:44PM +0530, Akhil P Oommen wrote: So far a530v2 gpu has support for detecting its supported opps based on a fus

Re: [PATCH v2 1/5] drm: add legacy support for using degamma for gamma

2020-12-03 Thread Tomi Valkeinen
On 03/12/2020 14:31, Ville Syrjälä wrote: > BTW I have some gamma related stuff here > git://github.com/vsyrjala/linux.git fb_helper_c8_lut_4 > > which tries to fix some fb_helper gamma stuff, and I'm also > getting rid of the gamma_store stuff for the leacy uapi for > drivers which implement the

Re: [PATCH] drm/fb-helper: Add missed unlocks in setcmap_legacy()

2020-12-03 Thread Daniel Vetter
On Thu, Dec 3, 2020 at 5:11 PM Peter Rosin wrote: > > Hi! > > On 2020-12-03 15:42, Chuhong Yuan wrote: > > setcmap_legacy() does not call drm_modeset_unlock_all() in some exits, > > add the missed unlocks with goto to fix it. > > > > Fixes: 964c60063bff ("drm/fb-helper: separate the fb_setcmap hel

[PATCH v5] drm/bridge: tfp410: Set input_bus_flags in atomic_check

2020-12-03 Thread Nikhil Devshatwar
input_bus_flags are specified in drm_bridge_timings (legacy) as well as drm_bridge_state->input_bus_cfg.flags The flags from the timings will be deprecated. Bridges are supposed to validate and set the bridge state flags from atomic_check. Implement atomic_check hook for the same. Signed-off-by:

[Bug 210479] New: amdgpu: Monitor connected to RX 6800XT loses signal when module is loaded

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=210479 Bug ID: 210479 Summary: amdgpu: Monitor connected to RX 6800XT loses signal when module is loaded Product: Drivers Version: 2.5 Kernel Version: 5.10.0-rc6 Hardware: All

[Bug 78111] APU turbo core boost not working when radeon.dpm=1

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=78111 mirh (m...@protonmail.ch) changed: What|Removed |Added CC||m...@protonmail.ch --- Comment

[Bug 94061] [radeon - Kaveri] dpm works badly - much to high power consumption compared to catalyst

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=94061 mirh (m...@protonmail.ch) changed: What|Removed |Added CC||m...@protonmail.ch --- Comment

Re: [PATCH V3 1/2] dt-bindings: Add DT bindings for Toshiba TC358762 DSI-to-DPI bridge

2020-12-03 Thread Dave Stevenson
Hi Marek On Wed, 12 Aug 2020 at 21:07, Sam Ravnborg wrote: > > Hi Marek. > > On Sun, Aug 09, 2020 at 12:57:04PM +0200, Marek Vasut wrote: > > Add DT bindings for Toshiba TC358762 DSI-to-DPI bridge, this > > one is used in the Raspberry Pi 7" touchscreen display unit. > > > > Signed-off-by: Marek

[Bug 94061] [radeon - Kaveri] dpm works badly - much to high power consumption compared to catalyst

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=94061 --- Comment #11 from Alex Deucher (alexdeuc...@gmail.com) --- The dpm code for kaveri is identical for both radeon and amdgpu. -- You are receiving this mail because: You are watching the assignee of the bug. _

[Bug 210479] amdgpu: Monitor connected to RX 6800XT loses signal when module is loaded

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

Re: [PATCH v2 7/7] dma-buf: Write down some rules for vmap usage

2020-12-03 Thread Thomas Zimmermann
Hi Am 03.12.20 um 16:26 schrieb Daniel Vetter: On Thu, Dec 03, 2020 at 03:02:59PM +0100, Thomas Zimmermann wrote: Dma-buf's vmap and vunmap callbacks are undocumented and various exporters currently have slightly different semantics for them. Add documentation on how to implement and use these

[PATCH] drm/vkms: Add setup and testing information

2020-12-03 Thread Sumera Priyadarsini
Update the vkms documentation to contain steps to: - setup the vkms driver - run tests using igt Signed-off-by: Sumera Priyadarsini --- Documentation/gpu/vkms.rst | 47 ++ 1 file changed, 47 insertions(+) diff --git a/Documentation/gpu/vkms.rst b/Documenta

[Bug 94061] [radeon - Kaveri] dpm works badly - much to high power consumption compared to catalyst

2020-12-03 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=94061 --- Comment #12 from mirh (m...@protonmail.ch) --- I see that [amdgpu_]kv_smc_bapm_enable is equivalent, but the condition guarding it seems quite different. Then I'm not really pretending to understand much else about the two kv_dpms. -- You a

Re: [PATCH drm/hisilicon 2/3] drm/irq: Add the new api to install irq

2020-12-03 Thread Sam Ravnborg
Hi Tian. On Wed, Dec 02, 2020 at 04:47:14PM +0800, Tian Tao wrote: > Add new api devm_drm_irq_install() to register interrupts, > no need to call drm_irq_uninstall() when the drm module is removed. > > Signed-off-by: Tian Tao Just a few details to fix. Sam > --- > drivers/gpu/drm/drm

Re: [PATCH] Revert "i2c: qcom-geni: Disable DMA processing on the Lenovo Yoga C630"

2020-12-03 Thread Wolfram Sang
On Tue, Nov 24, 2020 at 12:57:43PM -0600, Bjorn Andersson wrote: > A combination of recent bug fixes by Doug Anderson and the proper > definition of iommu streams means that this hack is no longer needed. > Let's clean up the code by reverting '127068abe85b ("i2c: qcom-geni: > Disable DMA processin

Re: [PATCH v2 7/7] dma-buf: Write down some rules for vmap usage

2020-12-03 Thread Daniel Vetter
On Thu, Dec 03, 2020 at 07:59:04PM +0100, Thomas Zimmermann wrote: > Hi > > Am 03.12.20 um 16:26 schrieb Daniel Vetter: > > On Thu, Dec 03, 2020 at 03:02:59PM +0100, Thomas Zimmermann wrote: > > > Dma-buf's vmap and vunmap callbacks are undocumented and various > > > exporters currently have sligh

Re: [PATCH] drm: Fix drm.h uapi header for Windows

2020-12-03 Thread Daniel Vetter
On Thu, Dec 3, 2020 at 7:55 PM James Park wrote: > > The trailing underscore for DRM_FOURCC_STANDALONE_ isn't intentional, right? > Should I put all the integer types, or just the ones that are used in that > file? Yeah that trailing _ just slipped in. And I'd just do the types already used. I

Re: [PATCH] drm/vkms: Add setup and testing information

2020-12-03 Thread Daniel Vetter
On Thu, Dec 3, 2020 at 8:13 PM Sumera Priyadarsini wrote: > > Update the vkms documentation to contain steps to: > > - setup the vkms driver > - run tests using igt > > Signed-off-by: Sumera Priyadarsini > --- > Documentation/gpu/vkms.rst | 47 ++ > 1 file c

  1   2   >