[PATCH v2 0/6] drm: Prepare to use a GPIO on ti-sn65dsi86 for Hot Plug Detect

2020-04-20 Thread Douglas Anderson
As talked about in commit c2bfc223882d ("drm/bridge: ti-sn65dsi86: Remove the mystery delay"), the normal HPD pin on ti-sn65dsi86 is kinda useless, at least for embedded DisplayPort (eDP). However, despite the fact that the actual HPD pin on the bridge is mostly useless for eDP, the concept of H

[PATCH v2 6/6] arm64: dts: sdm845: Add "no-hpd" to sn65dsi86 on cheza

2020-04-20 Thread Douglas Anderson
We don't have the HPD line hooked up to the bridge chip. Add it as suggested in the patch ("dt-bindings: drm/bridge: ti-sn65dsi86: Document no-hpd"). NOTE: this patch isn't expected to have any effect but just keeps us cleaner for the future. Currently the driver in Linux just assumes that nobod

[PATCH v2 2/6] dt-bindings: display: Add hpd-gpios to panel-common bindings

2020-04-20 Thread Douglas Anderson
In the cases where there is no connector in a system there's no great place to put "hpd-gpios". As per discussion [1] the best place to put it is in the panel. Add this to the device tree bindings. [1] https://lore.kernel.org/r/20200417180819.ge5...@pendragon.ideasonboard.com Signed-off-by: Dou

[PATCH v2 4/6] dt-bindings: drm/bridge: ti-sn65dsi86: Convert to yaml

2020-04-20 Thread Douglas Anderson
This moves the bindings over, based a lot on toshiba,tc358768.yaml. Unless there's someone known to be better, I've set the maintainer in the yaml as the first person to submit bindings. Signed-off-by: Douglas Anderson --- Changes in v2: - specification => specifier. - power up => power. - Added

[PATCH v2 3/6] drm/panel-simple: Support hpd-gpios for delaying prepare()

2020-04-20 Thread Douglas Anderson
People use panel-simple when they have panels that are builtin to their device. In these cases the HPD (Hot Plug Detect) signal isn't really used for hotplugging devices but instead is used for power sequencing. Panel timing diagrams (especially for eDP panels) usually have the HPD signal in them

[PATCH v2 1/6] drm/bridge: ti-sn65dsi86: Export bridge GPIOs to Linux

2020-04-20 Thread Douglas Anderson
The ti-sn65dsi86 MIPI DSI to eDP bridge chip has 4 pins on it that can be used as GPIOs in a system. Each pin can be configured as input, output, or a special function for the bridge chip. These are: - GPIO1: SUSPEND Input - GPIO2: DSIA VSYNC - GPIO3: DSIA HSYNC or VSYNC - GPIO4: PWM Let's expos

Re: [PATCH 1/3] dt-bindings: drm/bridge: ti-sn65dsi86: Convert to yaml

2020-04-20 Thread Doug Anderson
Hi, On Wed, Apr 15, 2020 at 1:31 PM Laurent Pinchart wrote: > > Hi Douglas, > > Thank you for the patch. > > On Wed, Apr 15, 2020 at 08:48:39AM -0700, Douglas Anderson wrote: > > This moves the bindings over, based a lot on toshiba,tc358768.yaml. > > Unless there's someone known to be better, I'v

Re: [PATCH 2/3] dt-bindings: drm/bridge: ti-sn65dsi86: Add hpd-gpios to the bindings

2020-04-20 Thread Doug Anderson
Hi, On Fri, Apr 17, 2020 at 11:08 AM Laurent Pinchart wrote: > > As for the hpd-gpios, it should be specified in the DT node of the > component that provides the HPD signal, and contain a GPIO specifier > describing what the signal is connected to. When dealing with a physical > DP connector and

Re: [PATCH 5/5] backlight: led_bl: rewrite led_bl_parse_levels()

2020-04-20 Thread Tomi Valkeinen
On 20/04/2020 19:01, Daniel Thompson wrote: On Fri, Apr 17, 2020 at 02:33:12PM +0300, Tomi Valkeinen wrote: led_bl_parse_levels() is rather difficult to follow. Rewrite it with a more obvious code flow. ... that introduces new behaviour. There's a couple of new behaviours here but the one tha

Re: [PATCH 4/5] backlight: led_bl: fix led -> backlight brightness mapping

2020-04-20 Thread Tomi Valkeinen
On 20/04/2020 18:55, Daniel Thompson wrote: On Fri, Apr 17, 2020 at 02:33:11PM +0300, Tomi Valkeinen wrote: The code that maps the LED default brightness to backlight levels has two issues: 1) if the default brightness is the first backlight level (usually 0), the code fails to find it, and 2) w

Re: linux-next: manual merge of the drm-misc tree with the drm-misc-fixes tree

2020-04-20 Thread Tomi Valkeinen
Hi, On 21/04/2020 04:52, Stephen Rothwell wrote: Hi all, Today's linux-next merge of the drm-misc tree got a conflict in:he drm-misc tree with the drm-misc-fixes tree drivers/gpu/drm/tidss/tidss_encoder.c between commit: 9da67433f64e ("drm/tidss: fix crash related to accessing freed m

Re: [PATCH 1/3] drm/exynos: gem: Remove dead-code

2020-04-20 Thread Inki Dae
20. 4. 7. 오후 10:42에 Marek Szyprowski 이(가) 쓴 글: > The ExynosDRM page fault handler is never used, drm_gem_mmap() > always calls exynos_drm_gem_mmap() function, which perform > complete mapping for the given virtual address-space area. Right, never used. Picked it up. Thanks, Inki Dae > > Signe

Re: [PATCH] staging: android: ion: Skip sync if not mapped

2020-04-20 Thread Christian Brauner
On Thu, Apr 16, 2020 at 12:25:08PM +0200, Greg Kroah-Hartman wrote: > On Tue, Apr 14, 2020 at 09:41:31PM -0700, John Stultz wrote: > > On Tue, Apr 14, 2020 at 7:28 AM Greg Kroah-Hartman > > wrote: > > > > > > On Tue, Apr 14, 2020 at 04:18:47PM +0200, Ørjan Eide wrote: > > > > Only sync the sg-list

[PATCH 4/9] drm/msm/a6xx: HFI v2 for A640 and A650

2020-04-20 Thread Jonathan Marek
Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 68 --- drivers/gpu/drm/msm/adreno/a6xx_gmu.h | 7 ++ drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 6 +- drivers/gpu/drm/msm/adreno/a6xx_hfi.c | 117 -- drivers/gpu/drm/msm/adreno/a6xx_hf

[PATCH 2/5] powerpc: Replace _ALIGN_DOWN() by ALIGN_DOWN()

2020-04-20 Thread Christophe Leroy
_ALIGN_DOWN() is specific to powerpc ALIGN_DOWN() is generic and does the same Replace _ALIGN_DOWN() by ALIGN_DOWN() Signed-off-by: Christophe Leroy --- arch/powerpc/include/asm/book3s/32/pgtable.h | 2 +- arch/powerpc/include/asm/nohash/32/pgtable.h | 2 +- arch/powerpc/kernel/pci_64.c

[PATCH 6/9] drm/msm/a6xx: add support for A650 gmu rscc registers

2020-04-20 Thread Jonathan Marek
Some of the RSCC registers on A650 are in a separate region. Note this also changes the address of these registers: RSCC_TCS1_DRV0_STATUS RSCC_TCS2_DRV0_STATUS RSCC_TCS3_DRV0_STATUS Based on the values in msm-4.14 and msm-4.19 kernels. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adr

[PATCH v11 1/2] drm/panel: add support for rm69299 visionox panel driver

2020-04-20 Thread Harigovindan P
Add support for Visionox panel driver. Signed-off-by: Harigovindan P --- Changes in v2: - Dropping redundant space in Kconfig(Sam Ravnborg). - Changing structure for include files(Sam Ravnborg). - Removing backlight related code and functions(Sam Ravnborg). - Remo

[PATCH v11 2/2] dt-bindings: documenting compatible string vendor "visionox"

2020-04-20 Thread Harigovindan P
Documenting compatible string vendor "visionox" in vendor-prefix yaml file. Signed-off-by: Harigovindan P --- Changes in v11: - Added visionox compatible string in vendor-prefixes.yaml - Added as a part of checkpatch script error for panel driver. Documentation/devicetree/bindin

Re: [PATCH v6 00/12] ARM/MIPS: DTS: add child nodes describing the PVRSGX GPU present in some OMAP SoC and JZ4780 (and many more)

2020-04-20 Thread Maxime Ripard
Hi, On Fri, Apr 17, 2020 at 02:09:06PM +0200, Philipp Rossak wrote: > > > I'm a bit skeptical on that one since it doesn't even list the > > > interrupts connected to the GPU that the binding mandates. > > > > I think he left it out for a future update. > > But best he comments himself. > > I'm

[PATCH -next] drm/amdgpu/gfx10: remove unused variables

2020-04-20 Thread YueHaibing
drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c: In function gfx_v10_rlcg_wreg: drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c:1291:18: warning: variable grbm_idx set but not used [-Wunused-but-set-variable] drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c:1290:18: warning: variable grbm_cntl set but not used [-Wunused-but

Re: [PATCH v6 01/12] dt-bindings: add img, pvrsgx.yaml for Imagination GPUs

2020-04-20 Thread Maxime Ripard
On Fri, Apr 17, 2020 at 02:15:44PM +0200, H. Nikolaus Schaller wrote: > > Am 17.04.2020 um 12:25 schrieb Maxime Ripard : > > On Wed, Apr 15, 2020 at 06:42:18PM +0200, H. Nikolaus Schaller wrote: > >>> Am 15.04.2020 um 18:21 schrieb Maxime Ripard : > >>> > >>> On Wed, Apr 15, 2020 at 05:09:45PM +02

[PATCH 5/5] powerpc: Remove _ALIGN_UP(), _ALIGN_DOWN() and _ALIGN()

2020-04-20 Thread Christophe Leroy
These three powerpc macros have been replaced by equivalent generic macros and are not used anymore. Remove them. Signed-off-by: Christophe Leroy --- arch/powerpc/include/asm/page.h | 7 --- 1 file changed, 7 deletions(-) diff --git a/arch/powerpc/include/asm/page.h b/arch/powerpc/include/

[PATCH v2 2/2] panel: simple: Add Ivo M133NWF4 R0

2020-04-20 Thread Bjorn Andersson
The InfoVision Optoelectronics M133NWF4 R0 panel is a 13.3" 1920x1080 eDP panel, add support for it in panel-simple. Signed-off-by: Bjorn Andersson --- Changes since v1: - Specified bus_format, bus_flags and connector_type drivers/gpu/drm/panel/panel-simple.c | 34

Re: [PATCH] drm/ast: Allocate initial CRTC state of the correct size

2020-04-20 Thread Konstantin Khlebnikov
On 20/04/2020 15.55, Thomas Zimmermann wrote: Hi Konstantin Am 20.04.20 um 14:45 schrieb Konstantin Khlebnikov: I've stumbled upon this too. Trivial fix was posted but stuck in review. This is patch from Thomas Zimmermann changed according to suggestions by Daniel Vetter from https://patchwo

[PATCH] drm/ast: Allocate initial CRTC state of the correct size

2020-04-20 Thread Konstantin Khlebnikov
I've stumbled upon this too. Trivial fix was posted but stuck in review. This is patch from Thomas Zimmermann changed according to suggestions by Daniel Vetter from https://patchwork.kernel.org/patch/11356157/ The ast driver inherits from DRM's CRTC state, but still uses the atomic helper for str

[PATCH 3/9] drm/msm/a6xx: allow allocating GMU memory with a fixed address

2020-04-20 Thread Jonathan Marek
Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 15 --- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c b/drivers/gpu/drm/msm/adreno/a6xx_gmu.c index 748cd379065f..c6ecb3189ec5 100644 --- a/drivers/gpu/drm/

Re: [PATCH 2/9] Revert "drm/msm/a6xx: Use the DMA API for GMU memory objects"

2020-04-20 Thread Jonathan Marek
On 4/20/20 3:51 PM, Bjorn Andersson wrote: On Mon 20 Apr 07:03 PDT 2020, Jonathan Marek wrote: This reverts commit a5fb8b918920c6f7706a8b5b8ea535a7f077a7f6. Why? It removes something I need for the next patches in the series, however I'm open to suggestions on a better solution (Jordan sa

[PATCH -next] drm/amdgpu: remove set but not used variable 'priority'

2020-04-20 Thread YueHaibing
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c: In function amdgpu_job_submit: drivers/gpu/drm/amd/amdgpu/amdgpu_job.c:148:26: warning: variable priority set but not used [-Wunused-but-set-variable] commit 33abcb1f5a17 ("drm/amdgpu: set compute queue priority at mqd_init") left behind this, remove it.

[PATCH 1/2] dt-bindings: display: simple: Add BOE NV133FHM-N61

2020-04-20 Thread Bjorn Andersson
Add the BOE NV133FHM-N61 13.3" FHD (1920x1080) TFT LCD Panel to the compatible list of panel-simple. Signed-off-by: Bjorn Andersson --- .../devicetree/bindings/display/panel/panel-simple.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/Documentation/devicetree/bindings/display/

Re: [PATCH 2/9] Revert "drm/msm/a6xx: Use the DMA API for GMU memory objects"

2020-04-20 Thread Bjorn Andersson
On Mon 20 Apr 07:03 PDT 2020, Jonathan Marek wrote: > This reverts commit a5fb8b918920c6f7706a8b5b8ea535a7f077a7f6. Why? > --- > drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 115 +++--- > drivers/gpu/drm/msm/adreno/a6xx_gmu.h | 6 +- > 2 files changed, 107 insertions(+), 14 del

[PATCH 8/9] drm/msm/a6xx: enable GMU log

2020-04-20 Thread Jonathan Marek
This is required for a650 to work. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 16 drivers/gpu/drm/msm/adreno/a6xx_gmu.h | 1 + drivers/gpu/drm/msm/adreno/a6xx_gmu.xml.h | 4 3 files changed, 21 insertions(+) diff --git a/drivers/gpu

Re: [PATCH v2] drm/i915: Fix ref->mutex deadlock in i915_active_wait()

2020-04-20 Thread Sultan Alsawaf
On Mon, Apr 20, 2020 at 12:02:39PM +0300, Joonas Lahtinen wrote: > I think the the patch should be dropped for now before the issue is > properly addressed. Either by backporting the mainline fixes or if > those are too big and there indeed is a smaller alternative patch > that is properly reviewed

Re:Re: [PATCH V2] amdgpu: remove unnecessary condition check

2020-04-20 Thread 赵军奎
From: Felix Kuehling Date: 2020-04-21 12:24:19 To: 1587180037-113840-1-git-send-email-bern...@vivo.com,Alex Deucher ,"Christian König" ,"David (ChunMing) Zhou" ,David Airlie ,Daniel Vetter ,amd-...@lists.freedesktop.org,dri-devel@lists.freedesktop.org,linux-ker...@vger.kernel.org Cc: openso

[PATCH 2/2] panel: simple: Add BOE NV133FHM-N61

2020-04-20 Thread Bjorn Andersson
The BOE NV133FHM-N61 panel is a 13.3" 1920x1080 eDP panel, add support for it in panel-simple. Signed-off-by: Bjorn Andersson --- drivers/gpu/drm/panel/panel-simple.c | 33 1 file changed, 33 insertions(+) diff --git a/drivers/gpu/drm/panel/panel-simple.c b/drivers

[PATCH V2] amdgpu, fix unnessary check "info" NULL branch

2020-04-20 Thread Bernard Zhao
kvfree ensure that the null pointer will do nothing. If addr is NULL, first is_vmalloc_addr will not enter, and kfree function just return when addr is NULL. There will be no risk here. Signed-off-by: Bernard Zhao --- drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c | 3 +-- 1 file changed, 1 insertion(+)

[PATCH 4/5] powerpc: Replace _ALIGN() by ALIGN()

2020-04-20 Thread Christophe Leroy
_ALIGN() is specific to powerpc ALIGN() is generic and does the same Replace _ALIGN() by ALIGN() Signed-off-by: Christophe Leroy --- arch/powerpc/include/asm/book3s/32/pgtable.h | 2 +- arch/powerpc/include/asm/nohash/32/pgtable.h | 2 +- arch/powerpc/kernel/prom_init.c | 8 ---

[PATCH v11 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel

2020-04-20 Thread Harigovindan P
Adding support for visionox rm69299 panel driver and adding bindings for the same panel. https://patchwork.kernel.org/patch/11461943/ has already been added to drm-misc-next. Harigovindan P (2): drm/panel: add support for rm69299 visionox panel driver dt-bindings: documenting compatible stri

Re: [PATCH v4] drm/i915: Synchronize active and retire callbacks

2020-04-20 Thread Sultan Alsawaf
On Mon, Apr 20, 2020 at 11:21:42AM +0300, Joonas Lahtinen wrote: > So it seems that the patch got pulled into v5.6 and has been backported > to v5.5 but not v5.4. You're right, that's my mistake. > In doing that zeroing of ring->vaddr is removed so the test to do mdelay(1) > and "ring->vaddr = NU

[PATCH 1/9] drm/msm/adreno: add A640/A650 to gpulist

2020-04-20 Thread Jonathan Marek
Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/adreno_device.c | 24 ++ drivers/gpu/drm/msm/adreno/adreno_gpu.c| 2 +- drivers/gpu/drm/msm/adreno/adreno_gpu.h| 10 + 3 files changed, 35 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/m

[PATCH v3] of_device: removed #include that caused a recursion in included headers

2020-04-20 Thread Hadar Gat
Both of_platform.h and of_device.h were included each other. In of_device.h, removed unneeded #include to of_platform.h and added include to of_platform.h in the files that needs it. Signed-off-by: Hadar Gat Reported-by: kbuild test robot Acked-by: Jonathan Cameron #for-iio Acked-by: Stephen Bo

[PATCH v2] amdgpu: fixes error branch not return errno issue

2020-04-20 Thread Bernard Zhao
The "if(!encoder)" branch return the same value 0 of the success branch, maybe return -EINVAL is more better. Signed-off-by: Bernard Zhao --- Changes since V1: * commit message improve --- drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c | 14 +++--- 1 file changed, 7 insertions(+), 7 del

[PATCH] drm/v3d: Set pm_ops

2020-04-20 Thread YueHaibing
v3d_v3d_pm_ops is defined but forgot to set in v3d_platform_driver. Fixes: 57692c94dcbe ("drm/v3d: Introduce a new DRM driver for Broadcom V3D V3.x+") Signed-off-by: YueHaibing --- drivers/gpu/drm/v3d/v3d_drv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/gpu/drm/v3d/v3d_drv.c b/

[PATCH 3/5] powerpc: Replace _ALIGN_UP() by ALIGN()

2020-04-20 Thread Christophe Leroy
_ALIGN_UP() is specific to powerpc ALIGN() is generic and does the same Replace _ALIGN_UP() by ALIGN() Signed-off-by: Christophe Leroy --- arch/powerpc/include/asm/iommu.h | 4 ++-- arch/powerpc/kernel/head_booke.h | 2 +- arch/powerpc/kernel/nvram_64.c |

[PATCH -next] drm/mediatek: Fix Kconfig warning

2020-04-20 Thread YueHaibing
WARNING: unmet direct dependencies detected for MTK_MMSYS Depends on [n]: (ARCH_MEDIATEK [=y] || COMPILE_TEST [=n]) && COMMON_CLK_MT8173_MMSYS [=n] Selected by [y]: - DRM_MEDIATEK [=y] && HAS_IOMEM [=y] && DRM [=y] && (ARCH_MEDIATEK [=y] || ARM && COMPILE_TEST [=n]) && COMMON_CLK [=y] && HA

[PATCH V2] amdgpu: remove unnecessary condition check

2020-04-20 Thread Bernard Zhao
There is no need to if check again, maybe we could merge into the above else branch. Signed-off-by: Bernard Zhao --- Changes since V1: *commit message improve *code style refactoring Link for V1: * https://lore.kernel.org/patchwork/patch/1226587/ --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gp

[PATCH 7/9] drm/msm/a6xx: gmu_pdc register values for A640 and A650

2020-04-20 Thread Jonathan Marek
Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 22 +++--- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c b/drivers/gpu/drm/msm/adreno/a6xx_gmu.c index 3e51939eb867..b583bf6e293b 100644 --- a/drivers

[PATCH] video/fbdev/riva: Remove dead code

2020-04-20 Thread Souptick Joarder
These are dead code since 3.15. These can be removed forever if no plan to use it further. Signed-off-by: Souptick Joarder --- drivers/video/fbdev/riva/riva_hw.c | 18 -- 1 file changed, 18 deletions(-) diff --git a/drivers/video/fbdev/riva/riva_hw.c b/drivers/video/fbdev/riva/

[PATCH v2 1/2] dt-bindings: display: simple: Add IVO M133NWF4 R0

2020-04-20 Thread Bjorn Andersson
Define the vendor prefix for InfoVision Optoelectronics and add their M133NWF4 R0 13.3" FHD (1920x1080) TFT LCD panel to the compatible list of panel-simple. Signed-off-by: Bjorn Andersson --- Changes since v1: - Added this patch .../devicetree/bindings/display/panel/panel-simple.yaml

[PATCH v11 2/2] dt-bindings: documenting compatible string vendor "visionox"

2020-04-20 Thread Harigovindan P
Documenting compatible string vendor "visionox" in vendor-prefix yaml file. Signed-off-by: Harigovindan P --- Changes in v11: - Added visionox compatible string in vendor-prefixes.yaml - Added as a part of checkpatch script error for panel driver. Documentation/devicetree/bindin

[PATCH 1/5] drivers/powerpc: Replace _ALIGN_UP() by ALIGN()

2020-04-20 Thread Christophe Leroy
_ALIGN_UP() is specific to powerpc ALIGN() is generic and does the same Replace _ALIGN_UP() by ALIGN() Signed-off-by: Christophe Leroy --- drivers/ps3/ps3-lpm.c | 6 +++--- drivers/vfio/pci/vfio_pci_nvlink2.c | 2 +- drivers/video/fbdev/ps3fb.c | 4 ++-- sound/ppc/snd_ps3.

[PATCH v11 1/2] drm/panel: add support for rm69299 visionox panel driver

2020-04-20 Thread Harigovindan P
Add support for Visionox panel driver. Signed-off-by: Harigovindan P --- Changes in v2: - Dropping redundant space in Kconfig(Sam Ravnborg). - Changing structure for include files(Sam Ravnborg). - Removing backlight related code and functions(Sam Ravnborg). - Remo

[PATCH 9/9] drm/msm/a6xx: update a6xx_hw_init for A640 and A650

2020-04-20 Thread Jonathan Marek
Adreno 640 and 650 GPUs need some registers set differently. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/a6xx.xml.h | 14 +++ drivers/gpu/drm/msm/adreno/a6xx_gpu.c | 56 ++- 2 files changed, 61 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/d

[PATCH V2] amdgpu: optimization-- reduce noneed mutex_lock area

2020-04-20 Thread Bernard Zhao
Maybe we could reduce the mutex_lock(&mem->lock)`s protected code area, and noneed to protect pr_debug. Signed-off-by: Bernard Zhao Changes since V1: *commit message improve Link for V1: *https://lore.kernel.org/patchwork/patch/1226588/ --- drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd_gpuvm.c | 7

[PATCH 5/9] drm/msm/a6xx: A640/A650 GMU firmware path

2020-04-20 Thread Jonathan Marek
Newer GPUs have different gmu firmware path. Signed-off-by: Jonathan Marek --- drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 136 +++--- drivers/gpu/drm/msm/adreno/a6xx_gmu.h | 11 ++ drivers/gpu/drm/msm/adreno/a6xx_gmu.xml.h | 6 + 3 files changed, 138 insertions(+), 15 del

[PATCH v11 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel

2020-04-20 Thread Harigovindan P
Adding support for visionox rm69299 panel driver and adding bindings for the same panel. https://patchwork.kernel.org/patch/11461943/ has already been added to drm-misc-next. Harigovindan P (2): drm/panel: add support for rm69299 visionox panel driver dt-bindings: documenting compatible stri

[PATCH v11 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel

2020-04-20 Thread Harigovindan P
Adding support for visionox rm69299 panel driver and adding bindings for the same panel. https://patchwork.kernel.org/patch/11461943/ has already been added to drm-misc-next. Harigovindan P (2): drm/panel: add support for rm69299 visionox panel driver dt-bindings: documenting compatible stri

[PATCH] drm/gma500: Remove dead code

2020-04-20 Thread Souptick Joarder
These are dead code since 3.7. If there is no plan to use them further, these can be removed forever. Signed-off-by: Souptick Joarder --- drivers/gpu/drm/gma500/cdv_intel_dp.c| 31 + drivers/gpu/drm/gma500/cdv_intel_lvds.c | 83 --- drivers/gpu/drm/gma500

[PATCH 2/9] Revert "drm/msm/a6xx: Use the DMA API for GMU memory objects"

2020-04-20 Thread Jonathan Marek
This reverts commit a5fb8b918920c6f7706a8b5b8ea535a7f077a7f6. --- drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 115 +++--- drivers/gpu/drm/msm/adreno/a6xx_gmu.h | 6 +- 2 files changed, 107 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c b/drive

Re: [PATCH v4] drm/i915: Synchronize active and retire callbacks

2020-04-20 Thread Joonas Lahtinen
Quoting Sultan Alsawaf (2020-04-20 19:15:14) > On Mon, Apr 20, 2020 at 11:21:42AM +0300, Joonas Lahtinen wrote: > > So it seems that the patch got pulled into v5.6 and has been backported > > to v5.5 but not v5.4. > > You're right, that's my mistake. Did applying the patch to v5.4 fix the issue a

Re: [PATCH 06/18] drm/i915/display/dp: Prefer drm_WARN* over WARN*

2020-04-20 Thread Jani Nikula
On Mon, 06 Apr 2020, Pankaj Bharadiya wrote: > struct drm_device specific drm_WARN* macros include device information > in the backtrace, so we know what device the warnings originate from. The patch no longer applies. > > Prefer drm_WARN* over WARN* at places where struct intel_dp struct > poi

Re: [PATCH 11/18] drm/i915/display/sdvo: Prefer drm_WARN* over WARN*

2020-04-20 Thread Jani Nikula
On Mon, 06 Apr 2020, Pankaj Bharadiya wrote: > struct drm_device specific drm_WARN* macros include device information > in the backtrace, so we know what device the warnings originate from. > > Prefer drm_WARN* over WARN* calls. > > Signed-off-by: Pankaj Bharadiya > --- > drivers/gpu/drm/i915/d

Re: [PATCH 12/18] drm/i915/display/tc: Prefer drm_WARN_ON over WARN_ON

2020-04-20 Thread Jani Nikula
On Mon, 06 Apr 2020, Pankaj Bharadiya wrote: > struct drm_device specific drm_WARN* macros include device information > in the backtrace, so we know what device the warnings originate from. > > Prefer drm_WARN_ON over WARN_ON. > > Conversion is done with below sementic patch: > > @@ > identifier

[PATCH] drm/i915/selftests: Fix i915_address_space refcnt leak

2020-04-20 Thread Xiyu Yang
igt_ppgtt_pin_update() invokes i915_gem_context_get_vm_rcu(), which returns a reference of the i915_address_space object to "vm" with increased refcount. When igt_ppgtt_pin_update() returns, "vm" becomes invalid, so the refcount should be decreased to keep refcount balanced. The reference countin

Re: [PATCH V5 1/4] backlight: qcom-wled: convert the wled bindings to .yaml format

2020-04-20 Thread Lee Jones
On Fri, 17 Apr 2020, kgu...@codeaurora.org wrote: > On 2020-04-15 20:39, Rob Herring wrote: > > On Tue, Apr 07, 2020 at 09:17:07PM +0530, Kiran Gunda wrote: > > > Convert the qcom-wled bindings from .txt to .yaml format. > > > Also replace PM8941 to WLED3 and PMI8998 to WLED4. > > > > > > Signed-

Re: [PATCH 7/8] fbdev: rework backlight dependencies

2020-04-20 Thread Jani Nikula
On Fri, 17 Apr 2020, Sam Ravnborg wrote: > Hi Arnd. > > On Fri, Apr 17, 2020 at 05:55:52PM +0200, Arnd Bergmann wrote: >> Rather than having CONFIG_FB_BACKLIGHT select CONFIG_BACKLIGHT_CLASS_DEVICE, >> make any driver that needs it have a dependency on the class device >> being available, to preve

Re: [PATCH 0/8] drm, fbdev: rework dependencies

2020-04-20 Thread Jani Nikula
On Fri, 17 Apr 2020, Jason Gunthorpe wrote: > On Fri, Apr 17, 2020 at 07:14:53PM +0200, Daniel Vetter wrote: >> On Fri, Apr 17, 2020 at 05:55:45PM +0200, Arnd Bergmann wrote: >> > I tried to fix up some dependencies after the sii8620 "imply EXTCON" >> > statementn broke, trying a few things but in

Re: [PATCH v4] drm/i915: Synchronize active and retire callbacks

2020-04-20 Thread Joonas Lahtinen
Quoting Sultan Alsawaf (2020-04-20 08:24:19) > Chris, > > Could you please look at this in earnest? This is a real bug that crashes my > laptop without any kind of provocation. It is undeniably a bug in i915, and > I've > clearly described it in my patch. If you dont like the patch, I'm open to a

Operating KMS UAPI (Re: RFC: Drm-connector properties managed by another driver / privacy screen support)

2020-04-20 Thread Pekka Paalanen
On Fri, 17 Apr 2020 16:17:18 +0200 Daniel Vetter wrote: > On Fri, Apr 17, 2020 at 11:02 AM Pekka Paalanen wrote: > > > > Hi, > > > > let's think about how userspace uses atomic KMS UAPI. The simplest way > > to use atomic correctly is that userspace will for every update send the > > full, compl

Re: [PATCH v2] drm/i915: Fix ref->mutex deadlock in i915_active_wait()

2020-04-20 Thread Joonas Lahtinen
Quoting Greg KH (2020-04-14 11:23:44) > On Tue, Apr 14, 2020 at 09:15:07AM +0100, Chris Wilson wrote: > > Quoting Greg KH (2020-04-11 12:39:57) > > > On Fri, Apr 10, 2020 at 07:17:38AM -0700, Sultan Alsawaf wrote: > > > > On Fri, Apr 10, 2020 at 11:08:38AM +0200, Greg KH wrote: > > > > > On Tue, Ap

Re: [PATCH 26/29] mm: remove vmalloc_user_node_flags

2020-04-20 Thread Geert Uytterhoeven
Hi Christoph, On Tue, Apr 14, 2020 at 3:22 PM Christoph Hellwig wrote: > Open code it in __bpf_map_area_alloc, which is the only caller. Also > clean up __bpf_map_area_alloc to have a single vmalloc call with > slightly different flags instead of the current two different calls. > > For this to

Re: [PATCH 24/29] mm: remove __vmalloc_node_flags_caller

2020-04-20 Thread Geert Uytterhoeven
Hi Christoph, On Tue, Apr 14, 2020 at 3:21 PM Christoph Hellwig wrote: > Just use __vmalloc_node instead which gets and extra argument. To be > able to to use __vmalloc_node in all caller make it available outside > of vmalloc and implement it in nommu.c. > > Signed-off-by: Christoph Hellwig >

[Bug 206017] Kernel 5.4.x unusable with GUI due to crashes (some hard)

2020-04-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=206017 --- Comment #17 from udo (udo...@xs4all.nl) --- Kernel 5.6.x works very well. Git mesa might help too. -- You are receiving this mail because: You are watching the assignee of the bug. ___ dri-devel ma

[PATCH] drm: imx: Unify encoder creation

2020-04-20 Thread Adrian Ratiu
imx drivers don't require drm encoders and they all had empty/no-op implementations which got converted to simple objects to pacify the drm core which still requires something to be defined. The problem now is that each driver duplicates the same encoder initialization logic and I'm working on add

Re: Operating KMS UAPI (Re: RFC: Drm-connector properties managed by another driver / privacy screen support)

2020-04-20 Thread Pekka Paalanen
On Mon, 20 Apr 2020 11:27:04 +0300 Pekka Paalanen wrote: > On Fri, 17 Apr 2020 16:17:18 +0200 > Daniel Vetter wrote: > > > On Fri, Apr 17, 2020 at 11:02 AM Pekka Paalanen > > wrote: > > > > > > Hi, > > > > > > let's think about how userspace uses atomic KMS UAPI. The simplest way > > > to u

Re: Operating KMS UAPI (Re: RFC: Drm-connector properties managed by another driver / privacy screen support)

2020-04-20 Thread Simon Ser
On Monday, April 20, 2020 10:27 AM, Pekka Paalanen wrote: > The only "random" KMS state is the properties the userspace KMS > program does not know that are set on start-up. I have been assuming > that as long as you had fbdev active before the KMS program started, > the unknown properties have "

Re: Operating KMS UAPI (Re: RFC: Drm-connector properties managed by another driver / privacy screen support)

2020-04-20 Thread Simon Ser
I'd really like a drmModeAtomicAddDefaultProperty that resets a prop to its default value… ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH v5 1/8] drm/fourcc: Add modifier definitions for describing Amlogic Video Framebuffer Compression

2020-04-20 Thread Neil Armstrong
On 17/04/2020 20:14, Daniel Vetter wrote: > On Fri, Apr 17, 2020 at 6:05 PM Neil Armstrong > wrote: >> >> On 17/04/2020 17:07, Daniel Vetter wrote: [...] > > Yup there's a number of parametried modifiers. As long as the stuff > you get from a DRM_FORMAT_MOD_ ... macro is a full modifier with >

Re: [PATCH] drm/i915/dpcd_bl: Unbreak enable_dpcd_backlight modparam

2020-04-20 Thread Jani Nikula
On Mon, 13 Apr 2020, Lyude Paul wrote: > Looks like I accidentally made it so you couldn't force DPCD backlight > support on, whoops. Fix that. Thanks, pushed. BR, Jani. > > Signed-off-by: Lyude Paul > Fixes: 17f5d57915be ("drm/i915: Force DPCD backlight mode on X1 Extreme 2nd > Gen 4K AMOL

Rule for bridge yaml dt binding maintainers?

2020-04-20 Thread Adrian Ratiu
Hello, I got confused while doing the txt -> yaml conversion at [1] and it's still not clear to me who should be added in the "maintainers" field. Clearly not the maintainers as returned by get_maintainer.pl. :) Rob mentioned that "owners" should be manintainers but I also have trouble pic

Re: [PATCH 5/8] drm/vmwgfx: make framebuffer support optional

2020-04-20 Thread Thomas Zimmermann
Hi IMHO, at some point console support will require a change where there's a single config option enables/disables fbdev emulation for all drivers. But this patch is a step in the right direction. Thanks! Am 17.04.20 um 17:55 schrieb Arnd Bergmann: > CONFIG_FB causes unnecessary Kconfig dependenc

Re: Operating KMS UAPI (Re: RFC: Drm-connector properties managed by another driver / privacy screen support)

2020-04-20 Thread Pekka Paalanen
On Mon, 20 Apr 2020 10:15:39 + Simon Ser wrote: > On Monday, April 20, 2020 10:27 AM, Pekka Paalanen > wrote: > > > The only "random" KMS state is the properties the userspace KMS > > program does not know that are set on start-up. I have been assuming > > that as long as you had fbdev act

Re: Multiple regulators for one device [was drm/panfrost: add devfreq regulator support]

2020-04-20 Thread Mark Brown
On Sun, Apr 19, 2020 at 11:25:08AM +0200, Clément Péron wrote: > Just saw that a Lima devfreq[0] has been also introduced with I think > exactly the same logic. > Is this something that hasn't been triggered by Maintainer or I am > missing something? My understanding is that there is very little

Re: Operating KMS UAPI (Re: RFC: Drm-connector properties managed by another driver / privacy screen support)

2020-04-20 Thread Simon Ser
On Monday, April 20, 2020 2:22 PM, Pekka Paalanen wrote: > On Mon, 20 Apr 2020 10:15:39 + > Simon Ser cont...@emersion.fr wrote: > > > On Monday, April 20, 2020 10:27 AM, Pekka Paalanen ppaala...@gmail.com > > wrote: > > > > > The only "random" KMS state is the properties the userspace KMS >

Re: [PATCH] drm/ast: Allocate initial CRTC state of the correct size

2020-04-20 Thread Thomas Zimmermann
Hi Konstantin Am 20.04.20 um 14:45 schrieb Konstantin Khlebnikov: > I've stumbled upon this too. Trivial fix was posted but stuck in review. > This is patch from Thomas Zimmermann changed according to suggestions by > Daniel Vetter from https://patchwork.kernel.org/patch/11356157/ Did you find it

Re: [PATCH 05/59] drm/vboxvidoe: use managed pci functions

2020-04-20 Thread Hans de Goede
Hi, On 4/15/20 7:44 PM, Daniel Vetter wrote: On Wed, Apr 15, 2020 at 05:03:55PM +0200, Hans de Goede wrote: Hi, On 4/15/20 9:39 AM, Daniel Vetter wrote: Allows us to drop the cleanup code on the floor. Sam noticed in his review: With this change we avoid calling pci_disable_device() twise i

Re: [PATCH 01/59] drm: Add devm_drm_dev_alloc macro

2020-04-20 Thread Thomas Zimmermann
Hi Am 15.04.20 um 09:39 schrieb Daniel Vetter: > Add a new macro helper to combine the usual init sequence in drivers, > consisting of a kzalloc + devm_drm_dev_init + drmm_add_final_kfree > triplet. This allows us to remove the rather unsightly > drmm_add_final_kfree from all currently merged driv

Re: [PATCH] backlight: lms501kf03: remove unused 'seq_sleep_in' and 'seq_up_dn'

2020-04-20 Thread Daniel Thompson
On Fri, Apr 17, 2020 at 05:22:57PM +0800, Jason Yan wrote: > Fix the following gcc warning: > > drivers/video/backlight/lms501kf03.c:96:28: warning: ‘seq_sleep_in’ > defined but not used [-Wunused-const-variable=] > static const unsigned char seq_sleep_in[] = { > ^

[Bug 205291] Cannot switch off Radeon HD 4330/4350/4550 with vgaswitcheroo

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

[Bug 205291] Cannot switch off Radeon HD 4330/4350/4550 with vgaswitcheroo

2020-04-20 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205291 --- Comment #2 from Alex Deucher (alexdeuc...@gmail.com) --- Please attach your dmesg output in both the working and non-working cases. -- You are receiving this mail because: You are watching the assignee of the bug. ___

Re: [PATCH 6/8] simplefs: add file creation functions

2020-04-20 Thread Emanuele Giuseppe Esposito
On 4/14/20 2:56 PM, Greg Kroah-Hartman wrote: On Tue, Apr 14, 2020 at 02:43:00PM +0200, Emanuele Giuseppe Esposito wrote: A bunch of code is duplicated between debugfs and tracefs, unify it to the simplefs library. The code is very similar, except that dentry and inode creation are unified i

Re: [PATCH 4/8] fs: introduce simple_new_inode

2020-04-20 Thread Emanuele Giuseppe Esposito
On 4/14/20 3:01 PM, Greg Kroah-Hartman wrote: On Tue, Apr 14, 2020 at 02:42:58PM +0200, Emanuele Giuseppe Esposito wrote: It is a common special case for new_inode to initialize the time to the current time and the inode to get_next_ino(). Introduce a core function that does it and use it thr

Re: [PATCH 0/8] Simplefs: group and simplify linux fs code

2020-04-20 Thread Emanuele Giuseppe Esposito
On 4/16/20 8:59 AM, Luis Chamberlain wrote: On Tue, Apr 14, 2020 at 02:42:54PM +0200, Emanuele Giuseppe Esposito wrote: This series of patches introduce wrappers for functions, arguments simplification in functions calls and most importantly groups duplicated code in a single header, simplefs

Re: [PATCH 1/8] apparmor: just use vfs_kern_mount to make .null

2020-04-20 Thread Emanuele Giuseppe Esposito
On 4/16/20 8:44 AM, Luis Chamberlain wrote: On Tue, Apr 14, 2020 at 02:42:55PM +0200, Emanuele Giuseppe Esposito wrote: aa_mk_null_file is using simple_pin_fs/simple_release_fs with local variables as arguments, for what would amount to a simple vfs_kern_mount/mntput pair if everything was in

Re: [PATCH 0/8] drm, fbdev: rework dependencies

2020-04-20 Thread Arnd Bergmann
On Mon, Apr 20, 2020 at 10:14 AM Jani Nikula wrote: > On Fri, 17 Apr 2020, Jason Gunthorpe wrote: > > On Fri, Apr 17, 2020 at 07:14:53PM +0200, Daniel Vetter wrote: > >> On Fri, Apr 17, 2020 at 05:55:45PM +0200, Arnd Bergmann wrote: > >> > > >> > If we can agree on these changes, maybe someone ca

[PATCH][next] drm/amdgpu: fix a dereference on pointer hive before it is null checked

2020-04-20 Thread Colin King
From: Colin Ian King Currently pointer hive is being dereferenced before it is being null checked and hence can lead to null pointer dereference issues. Fix this by only dereferencing pointer hive after it has been null checked. Addresses-Coverity: ("Dereference before null check") Fixes: 25d427

[drm-intel:topic/core-for-CI 18/20] include/net/ax25.h:121:24: error: redeclaration of enumerator 'AX25_PROTO_DAMA_MASTER'

2020-04-20 Thread kbuild test robot
tree: git://anongit.freedesktop.org/drm-intel topic/core-for-CI head: d0435a9b45070b945578c093dcd363b6b73a502c commit: 198db0fc276cdf8e1bb66a4a03473dbea1400d18 [18/20] Revert "drm/i915: Don't select BROKEN" config: x86_64-allyesconfig (attached as .config) compiler: gcc-7 (Ubuntu 7.5.0-6ubuntu

Re: [PATCH] drm/ast: Allocate initial CRTC state of the correct size

2020-04-20 Thread Thomas Zimmermann
Hi Am 20.04.20 um 14:58 schrieb Konstantin Khlebnikov: > > > On 20/04/2020 15.55, Thomas Zimmermann wrote: >> Hi Konstantin >> >> Am 20.04.20 um 14:45 schrieb Konstantin Khlebnikov: >>> I've stumbled upon this too. Trivial fix was posted but stuck in review. >>> This is patch from Thomas Zimmerm

Re: [PATCH 6/8] simplefs: add file creation functions

2020-04-20 Thread Greg Kroah-Hartman
On Mon, Apr 20, 2020 at 03:57:48PM +0200, Emanuele Giuseppe Esposito wrote: > > > On 4/14/20 2:56 PM, Greg Kroah-Hartman wrote: > > On Tue, Apr 14, 2020 at 02:43:00PM +0200, Emanuele Giuseppe Esposito wrote: > > > A bunch of code is duplicated between debugfs and tracefs, unify it to the > > > si

Re: [PATCH 1/9] drm/msm/adreno: add A640/A650 to gpulist

2020-04-20 Thread Greg Kroah-Hartman
On Mon, Apr 20, 2020 at 10:03:05AM -0400, Jonathan Marek wrote: > Signed-off-by: Jonathan Marek > --- > drivers/gpu/drm/msm/adreno/adreno_device.c | 24 ++ > drivers/gpu/drm/msm/adreno/adreno_gpu.c| 2 +- > drivers/gpu/drm/msm/adreno/adreno_gpu.h| 10 + > 3 fi

Re: [PATCH 6/8] simplefs: add file creation functions

2020-04-20 Thread Paolo Bonzini
On 20/04/20 16:28, Greg Kroah-Hartman wrote: >> I assume you meant a new file. These new functions are used only by a few >> filesystems, and I didn't want to include them in vmlinux unconditionally, >> so I introduced simplefs.c and CONFIG_SIMPLEFS instead of extending libfs.c. >> In this way only

  1   2   >