RE: [RFC v1 0/4] drm: Add support for DRM_CAP_DEFERRED_OUT_FENCE capability

2021-08-01 Thread Zhang, Tina
> -Original Message- > From: Daniel Vetter > Sent: Friday, July 30, 2021 6:26 PM > To: Kasireddy, Vivek > Cc: dri-devel@lists.freedesktop.org; Daniel Vetter ; Gerd > Hoffmann ; Pekka Paalanen ; > Simon Ser ; Michel Dänzer ; > Zhang, Tina ; Kim, Dongwon > > Subject: Re: [RFC v1 0/4]

[PATCH] drm/vmwgfx: Replace "vmw_num_pages" with "PFN_UP"

2021-08-01 Thread Cai Huoqing
we counld use PFN_UP instead of vmw_num_pages() Signed-off-by: Cai Huoqing --- .../gpu/drm/vmwgfx/device_include/vm_basic_types.h | 13 ++--- drivers/gpu/drm/vmwgfx/vmwgfx_msg.c | 6 +++--- 2 files changed, 9 insertions(+), 10 deletions(-) diff --git

Re: [PATCH v2 14/14] drm/mediatek: add mediatek-drm of vdosys1 support for MT8195

2021-08-01 Thread CK Hu
Hi, Nancy: On Thu, 2021-07-22 at 17:45 +0800, Nancy.Lin wrote: > Add driver data of mt8195 vdosys1 to mediatek-drm and modify drm for > multi-mmsys support. The two mmsys (vdosys0 and vdosys1) will bring > up two drm drivers, only one drm driver register as the drm device. > Each drm driver binds

[PATCH] drm/vmwgfx: Make use of PFN_ALIGN/PFN_UP helper macro

2021-08-01 Thread Cai Huoqing
it's a refactor to make use of PFN_ALIGN/PFN_UP helper macro Signed-off-by: Cai Huoqing --- drivers/gpu/drm/vmwgfx/vmwgfx_bo.c | 5 ++--- drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c | 2 +- drivers/gpu/drm/vmwgfx/vmwgfx_cotable.c | 3 +-- drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 2 +-

Re: [PATCH] drm/amdgpu: fix possible null-pointer dereference in amdgpu_ttm_tt_populate()

2021-08-01 Thread Li Tuo
Thanks for your feedback! We will remove the null tests according to your advice and prepare a V2 patch. Best wishes, Tuo Li On 2021/8/2 1:19, Christian König wrote: Am 31.07.21 um 10:04 schrieb Tuo Li: The variable ttm is assigned to the variable gtt, and the variable gtt is checked in:   

Re: [BUG] video: fbdev: sis: possible uninitialized-variable access in SiS_SetCRT2FIFO_300()

2021-08-01 Thread Li Tuo
Thanks for your feedback! Should I commit a patch to fix this problem? Best wishes, Tuo Li On 2021/8/2 1:59, Sam Ravnborg wrote: Hi Tuo Li, On Sat, Jul 31, 2021 at 02:28:39PM +0800, Li Tuo wrote: Hello, Our static analysis tool finds a possible uninitialized-variable access in the sis

[PATCH] drm:This patch fixes the checkpatch.pl error to msm_drv.c

2021-08-01 Thread zhaoxiao
ERROR: do not initialise statics to false Signed-off-by: zhaoxiao --- drivers/gpu/drm/msm/msm_drv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/msm/msm_drv.c b/drivers/gpu/drm/msm/msm_drv.c index 9b8fa2ad0d84..d9ca4bc9620b 100644 ---

[PATCH v2] drm:Fixes the following checkpatch error:

2021-08-01 Thread zhaoxiao
ERROR: do not initialise statics to false FILE: :drivers/gpu/drm/msm/msm_drv.c:21: static bool reglog = false; FILE: :drivers/gpu/drm/msm/msm_drv.c:31: -bool dumpstate = false; Signed-off-by: zhaoxiao --- v2: add the more detailed patch description drivers/gpu/drm/msm/msm_drv.c | 4 ++-- 1

[PATCH v5 2/2] drm/mediatek: force hsa hbp hfp packets multiple of lanenum to avoid screen shift

2021-08-01 Thread Jitao Shi
The bridge chip ANX7625 requires the packets on lanes aligned at the end, or ANX7625 will shift the screen. Signed-off-by: Jitao Shi --- drivers/gpu/drm/mediatek/mtk_dsi.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/gpu/drm/mediatek/mtk_dsi.c

[PATCH v5 0/2] force hsa hbp hfp packets multiple of lanenum to avoid screen shift

2021-08-01 Thread Jitao Shi
Changes since v4: - Move "dt-bindings: drm/bridge: anx7625: add force_dsi_end_without_null" before "drm/mediatek: force hsa hbp hfp packets multiple of lanenum to avoid". - Retitle "dt-bindings: drm/bridge: anx7625: add force_dsi_end_without_null". Jitao Shi (2): dt-bindings:

[PATCH v5 1/2] dt-bindings: drm/bridge: anx7625: add force_dsi_end_without_null

2021-08-01 Thread Jitao Shi
The force_dsi_end_without_null requires the dsi host ent at the same time in line. Signed-off-by: Jitao Shi --- .../bindings/display/bridge/analogix,anx7625.yaml | 6 ++ 1 file changed, 6 insertions(+) diff --git

Re: [PATCH v4 2/2] dt-bindings: mediatek: add force_dsi_end_without_null

2021-08-01 Thread Chun-Kuang Hu
Hi, Jitao: Jitao Shi 於 2021年8月1日 週日 下午12:06寫道: Move this patch before the patch "drm/mediatek: force hsa hbp hfp packets multiple of lanenum to avoid screen shift", and this patch's title should be "dt-bindings: drm/bridge: anx7625: add force_dsi_end_without_null" Regards, Chun-Kuang. > > The

Re: [PATCH v4 2/3] drm/loongson: Add GPIO and I2C driver for loongson drm.

2021-08-01 Thread Sam Ravnborg
Hi lichenyang, On Fri, Jul 30, 2021 at 05:41:47PM +0800, lichenyang wrote: > Implement use GPIO and I2C driver to detect connector > and fetch EDID via DDC. > > v3: > - Change some driver log to the drm_ version. > > v2: > - Optimize the error handling process. > - Delete loongson_i2c_bus_match

Re: [PATCH 00/14] drm: Make DRM's IRQ helpers legacy

2021-08-01 Thread Sam Ravnborg
Hi Thomas, > > > > 1) IRQ_NOTCONNECTED > > > > We do not have this check in drm_irq today and we should avoid spreading > > it all over. We are either carrying it forever or we wil lsee patches > > floating in to drop the check again. > > The current use in the kernel is minimal: > >

Re: [PATCH 00/14] drm: Make DRM's IRQ helpers legacy

2021-08-01 Thread Thomas Zimmermann
Hi Sam Am 31.07.21 um 20:50 schrieb Sam Ravnborg: Hi Thomas, On Tue, Jul 27, 2021 at 08:27:07PM +0200, Thomas Zimmermann wrote: DRM's IRQ helpers are only helpful for old, non-KMS drivers. Move the code behind CONFIG_DRM_LEGACY. Convert KMS drivers to Linux IRQ interfaces. DRM provides IRQ

[Bug 205089] amdgpu : drm:amdgpu_cs_ioctl : Failed to initialize parser -125

2021-08-01 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=205089 mcmar...@gmx.net changed: What|Removed |Added CC||mcmar...@gmx.net --- Comment #16 from

Re: [BUG] video: fbdev: sis: possible uninitialized-variable access in SiS_SetCRT2FIFO_300()

2021-08-01 Thread Sam Ravnborg
Hi Tuo Li, On Sat, Jul 31, 2021 at 02:28:39PM +0800, Li Tuo wrote: > Hello, > > Our static analysis tool finds a possible uninitialized-variable access in > the sis driver in Linux 5.14.0-rc3: > > At the beginning of the function SiS_SetCRT2FIFO_300(), the variable > modeidindex is not

Re: [PATCH] drm/amdgpu: fix possible null-pointer dereference in amdgpu_ttm_tt_unpopulate()

2021-08-01 Thread Christian König
Am 31.07.21 um 10:13 schrieb Tuo Li: The variable ttm is assigned to the variable gtt, and the variable gtt is checked in: if (gtt && gtt->userptr) This indicates that both ttm and gtt can be NULL. If so, a null-pointer dereference will occur: if (ttm->page_flags & TTM_PAGE_FLAG_SG)

Re: [PATCH] drm/amdgpu: fix possible null-pointer dereference in amdgpu_ttm_tt_populate()

2021-08-01 Thread Christian König
Am 31.07.21 um 10:04 schrieb Tuo Li: The variable ttm is assigned to the variable gtt, and the variable gtt is checked in: if (gtt && gtt->userptr) This indicates that both ttm and gtt can be NULL. If so, a null-pointer dereference will occur: if (ttm->page_flags & TTM_PAGE_FLAG_SG)

[PATCH V5 2/2] drm/vkms: Add support for virtual hardware mode

2021-08-01 Thread Sumera Priyadarsini
Add a virtual hardware or vblank-less mode as a module to enable VKMS to emulate virtual hardware drivers. This means no vertical blanking events occur and pageflips are completed arbitrarily and when required for updating the frame. Add a new drm_crtc_helper_funcs struct,

[PATCH V5 1/2] drm/vkms: Refactor vkms_composer_worker() to prep for virtual_hw mode

2021-08-01 Thread Sumera Priyadarsini
Add a new function vkms_composer_common(). The actual plane composition work has been moved to the helper function, vkms_composer_common() which is called by vkms_composer_worker() and will be called in the implementation of virtual_hw mode as well. Signed-off-by: Sumera Priyadarsini --- Changes

[PATCH V5 0/2] drm/vkms: Add virtual hardware module

2021-08-01 Thread Sumera Priyadarsini
This patchset adds support for emulating virtual hardware with VKMS. The virtual hardware mode can be enabled by using the following command while loading the module: sudo modprobe vkms enable_virtual_hw=1 The first patch is prep work for adding virtual_hw mode and refactors the plane

Re: [Intel-gfx] [PATCH] drm/i915: Use Transparent Hugepages when IOMMU is enabled

2021-08-01 Thread kernel test robot
Hi Tvrtko, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm-intel/for-linux-next] [also build test ERROR on v5.14-rc3 next-20210730] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as

Re: [PATCH v3 0/3] Remove all strcpy() uses

2021-08-01 Thread Len Baker
Hi Andy, On Sun, Aug 01, 2021 at 02:40:40PM +0300, Andy Shevchenko wrote: > On Sun, Aug 1, 2021 at 11:53 AM Len Baker wrote: > > > > strcpy() performs no bounds checking on the destination buffer. This > > could result in linear overflows beyond the end of the buffer, leading > > to all kinds of

Re: [PATCH] DRM: ast: Fixed coding style issues of ast_mode.c

2021-08-01 Thread Thomas Zimmermann
Hi Am 31.07.21 um 02:53 schrieb Gregory Williams: Removed space before comma, fixed if statements by putting trailing statements on new line, fixed unsigned int declaration, and removed not needed else statement after return. Signed-off-by: Gregory Williams Added to drm-misc-next. Thanks

[PATCH v3 3/3] staging/fbtft: Fix braces coding style

2021-08-01 Thread Len Baker
Add braces to the "for" loop. This way, the kernel coding style is followed. Signed-off-by: Len Baker --- drivers/staging/fbtft/fbtft-core.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/staging/fbtft/fbtft-core.c b/drivers/staging/fbtft/fbtft-core.c index

Re: [PATCH v3 0/3] Remove all strcpy() uses

2021-08-01 Thread Andy Shevchenko
On Sun, Aug 1, 2021 at 11:53 AM Len Baker wrote: > > strcpy() performs no bounds checking on the destination buffer. This > could result in linear overflows beyond the end of the buffer, leading > to all kinds of misbehaviors. So, this serie removes all strcpy uses > from the "staging/fbtft"

[PATCH v3 2/3] staging/fbtft: Remove unnecessary variable initialization

2021-08-01 Thread Len Baker
Remove the initialization of the variable "i" since it is written a few lines later. Signed-off-by: Len Baker --- drivers/staging/fbtft/fbtft-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fbtft/fbtft-core.c b/drivers/staging/fbtft/fbtft-core.c index

[Bug 213935] New: AMDGPU Renoir crash/freeze while using vaapi with some video types in some apps - drm:amdgpu_dm_atomic_commit_tail [amdgpu]] *ERROR* Waiting for fences timed out!

2021-08-01 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=213935 Bug ID: 213935 Summary: AMDGPU Renoir crash/freeze while using vaapi with some video types in some apps - drm:amdgpu_dm_atomic_commit_tail [amdgpu]] *ERROR*

[Bug 211425] [drm:atom_op_jump] *ERROR* atombios stuck in loop for more than 20secs aborting

2021-08-01 Thread bugzilla-daemon
https://bugzilla.kernel.org/show_bug.cgi?id=211425 Andreas (icedragon...@web.de) changed: What|Removed |Added Kernel Version|5.13.4 |5.13.6 -- You may reply

[PATCH v3 1/3] staging/fbtft: Remove all strcpy() uses

2021-08-01 Thread Len Baker
strcpy() performs no bounds checking on the destination buffer. This could result in linear overflows beyond the end of the buffer, leading to all kinds of misbehaviors. The safe replacement is strscpy() but in this case it is simpler to use the "%*ph" format specifier. Moreover, with the "0x%02X

[PATCH v3 0/3] Remove all strcpy() uses

2021-08-01 Thread Len Baker
strcpy() performs no bounds checking on the destination buffer. This could result in linear overflows beyond the end of the buffer, leading to all kinds of misbehaviors. So, this serie removes all strcpy uses from the "staging/fbtft" subsystem. Also, refactor the code a bit to follow the kernel

Re: [PATCH v5] drm/panel: Add support for E Ink VB3300-KCA

2021-08-01 Thread Alistair Francis
On Sun, Aug 1, 2021 at 4:26 AM Sam Ravnborg wrote: > > Hi Alistair, > > On Fri, Jul 30, 2021 at 10:13:10PM +1000, Alistair Francis wrote: > > Add support for the 10.3" E Ink panel described at: > > https://www.eink.com/product.html?type=productdetail=7 > > > > Signed-off-by: Alistair Francis > >

Re: [PATCH] dyndbg: add special aux_print framework

2021-08-01 Thread kernel test robot
to use '--base' as documented in https://git-scm.com/docs/git-format-patch] url: https://github.com/0day-ci/linux/commits/Jim-Cromie/dyndbg-add-special-aux_print-framework/20210801-113749 base: git://anongit.freedesktop.org/tegra/linux.git drm/tegra/for-next config: x86_64-randconfig-a012