Re: [PATCH v2] phy/qcom-qmp-combo: propagate correct return value at phy_power_on()

2024-03-29 Thread Abhinav Kumar
On 3/29/2024 9:41 AM, Kuogee Hsieh wrote: Currently qmp_combo_dp_power_on() always return 0 in regardless of return value of cfg->configure_dp_phy(). This patch propagate return value of cfg->configure_dp_phy() all the way back to caller. Fixes: 52e013d0bffa ("phy: qcom-qmp: Add support for

Re: [PATCH v2] drm/msm/dp: assign correct DP controller ID to x1e80100 interface table

2024-03-29 Thread Dmitry Baryshkov
On Fri, 29 Mar 2024 at 18:48, Kuogee Hsieh wrote: > > At current x1e80100 interface table, interface #3 is wrongly > connected to DP controller #0 and interface #4 wrongly connected > to DP controller #2. Fix this problem by connect Interface #3 to > DP controller #0 and interface #4 connect to

[PATCH v0 03/14] drm/gma500, drm/i915: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 02/14] drm/amdgpu, drm/radeon: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 04/14] media: au0828: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 05/14] media: cobalt: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 06/14] media: cx18: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 07/14] media: cx25821: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 11/14] fbdev/smscufx: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

Re: [PATCH] drm/msm: fix the `CRASHDUMP_READ` target of `a6xx_get_shader_block()`

2024-03-29 Thread Abhinav Kumar
On 3/26/2024 2:23 PM, Miguel Ojeda wrote: Clang 14 in an (essentially) defconfig arm64 build for next-20240326 reports [1]: drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c:843:6: error: variable 'out' set but not used [-Werror,-Wunused-but-set-variable] The variable `out` in these

[PATCH v3] drm/msm/dp: assign correct DP controller ID to x1e80100 interface table

2024-03-29 Thread Kuogee Hsieh
At current x1e80100 interface table, interface #3 is wrongly connected to DP controller #0 and interface #4 wrongly connected to DP controller #2. Fix this problem by connect Interface #3 to DP controller #0 and interface #4 connect to DP controller #1. Also add interface #6, #7 and #8 connections

[git pull] drm fixes for 6.9-rc2

2024-03-29 Thread Dave Airlie
Hi Linus, Regular fixes for rc2, quite a few i915/amdgpu as usual, some xe, and then mostly scattered around. rc3 might be quieter with the holidays but we shall see. Regards. Dave. drm-fixes-2024-03-30: drm fixes for 6.9-rc2 bridge: - select DRM_KMS_HELPER dma-buf: - fix NULL-pointer deref

Re: [PATCH] drm/msm/dp: allow voltage swing / pre emphasis of 3

2024-03-29 Thread Doug Anderson
Hi, On Sat, Feb 3, 2024 at 5:47 AM Dmitry Baryshkov wrote: > > Both dp_link_adjust_levels() and dp_ctrl_update_vx_px() limit swing and > pre-emphasis to 2, while the real maximum value for the sum of the > voltage swing and pre-emphasis is 3. Fix the DP code to remove this > limitation. > >

Re: [PATCH v2 12/14] sh: Add support for suppressing warning backtraces

2024-03-29 Thread Guenter Roeck
On Wed, Mar 27, 2024 at 07:39:20PM +, Simon Horman wrote: [ ... ] > > > > > > Hi Guenter, > > > > > > a minor nit from my side: this change results in a Kernel doc warning. > > > > > > .../bug.h:29: warning: expecting prototype for _EMIT_BUG_ENTRY(). > > > Prototype was for

[PATCH v3] phy/qcom-qmp-combo: propagate correct return value at phy_power_on()

2024-03-29 Thread Kuogee Hsieh
Currently qmp_combo_dp_power_on() always return 0 in regardless of return value of cfg->configure_dp_phy(). This patch propagate return value of cfg->configure_dp_phy() all the way back to caller. Changes in V3: -- add v2 changes log Changes in V2: -- add Fixes tag -- add dev_err() to

[PATCH v2] phy/qcom-qmp-combo: propagate correct return value at phy_power_on()

2024-03-29 Thread Kuogee Hsieh
Currently qmp_combo_dp_power_on() always return 0 in regardless of return value of cfg->configure_dp_phy(). This patch propagate return value of cfg->configure_dp_phy() all the way back to caller. Fixes: 52e013d0bffa ("phy: qcom-qmp: Add support for DP in USB3+DP combo phy") Signed-off-by: Kuogee

Re: [PATCH v0 02/14] drm/amdgpu, drm/radeon: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
On 3/29/2024 10:38 AM, Andi Shyti wrote: > Hi, > > with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

Re: [git pull] drm fixes for 6.9-rc2

2024-03-29 Thread pr-tracker-bot
The pull request you sent on Sat, 30 Mar 2024 06:29:04 +1000: > https://gitlab.freedesktop.org/drm/kernel.git tags/drm-fixes-2024-03-30 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/486291a0e6246364936df1ecd64c90affef4b9c5 Thank you! -- Deet-doot-dot, I am a bot.

Re: [PATCH v3 8/9] dt-bindings: xlnx: Add VTC and TPG bindings

2024-03-29 Thread Conor Dooley
On Fri, Mar 29, 2024 at 12:38:33AM +, Klymenko, Anatoliy wrote: > Thank you for the feedback. > > From: Krzysztof Kozlowski > > Subject: Re: [PATCH v3 8/9] dt-bindings: xlnx: Add VTC and TPG bindings > > On 22/03/2024 20:12, Klymenko, Anatoliy wrote: > > >> From: Krzysztof Kozlowski > > >>

Re: [PATCH] drm/meson: vclk: fix calculation of 59.94 fractional rates

2024-03-29 Thread Neil Armstrong
On 10/01/2024 00:07, Christian Hewitt wrote: Playing 4K media with 59.94 fractional rate (typically VP9) causes the screen to lose sync with the following error reported in the system log: [ 89.610280] Fatal Error, invalid HDMI vclk freq 593406 Modetest shows the following: 3840x2160 59.94

Re: [PATCH v11 3/7] clk: meson: g12a: make VCLK2 and ENCL clock path configurable by CCF

2024-03-29 Thread Neil Armstrong
On 29/03/2024 13:35, Jerome Brunet wrote: On Mon 25 Mar 2024 at 12:09, Neil Armstrong wrote: In order to setup the DSI clock, let's make the unused VCLK2 clock path configuration via CCF. The nocache option is removed from following clocks: - vclk2_sel - vclk2_input - vclk2_div - vclk2 -

Re: [PATCH v11 2/7] clk: meson: add vclk driver

2024-03-29 Thread Neil Armstrong
On 29/03/2024 13:33, Jerome Brunet wrote: On Mon 25 Mar 2024 at 12:09, Neil Armstrong wrote: The VCLK and VCLK_DIV clocks have supplementary bits. The VCLK gate has a "SOFT RESET" bit to toggle after the whole VCLK sub-tree rate has been set, this is implemented in the gate enable callback.

Re: [PATCH 4/4] drm/panel: simple: Add Innolux G121XCE-L01 LVDS display support

2024-03-29 Thread Jessica Zhang
On 3/28/2024 3:27 AM, Marek Vasut wrote: G121XCE-L01 is a Color Active Matrix Liquid Crystal Display composed of a TFT LCD panel, a driver circuit, and LED backlight system. The screen format is intended to support the 4:3, 1024(H) x 768(V) screen and either 262k/16.7M colors (RGB 6-bits or

Re: [PATCH] dmabuf: fix dmabuf file poll uaf issue

2024-03-29 Thread T.J. Mercier
On Tue, Mar 26, 2024 at 7:29 PM Zhiguo Jiang wrote: > > The issue is a UAF issue of dmabuf file fd. Throght debugging, we found > that the dmabuf file fd is added to the epoll event listener list, and > when it is released, it is not removed from the epoll list, which leads > to the

Re: [PATCH 3/4] drm/panel: simple: Convert Innolux G121X1-L03 to display_timing

2024-03-29 Thread Jessica Zhang
On 3/28/2024 3:27 AM, Marek Vasut wrote: Use display_timing instead of drm_display_mode to define a range of possible display timings supported by this panel. This makes the panel support more flexible and improves compatibility. No functional change is expected. The settings are picked from

[PATCH v0 14/14] i2c and treewide: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the specification.

[PATCH v0 09/14] media: cx23885: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 13/14] drm/nouveau: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

Re: [PATCH 2/4] drm/panel: simple: Add missing Innolux G121X1-L03 format, flags, connector

2024-03-29 Thread Jessica Zhang
On 3/28/2024 3:27 AM, Marek Vasut wrote: The .bpc = 6 implies .bus_format = MEDIA_BUS_FMT_RGB666_1X7X3_SPWG , add the missing bus_format. Add missing connector type and bus_flags as well. Documentation [1] 1.4 GENERAL SPECIFICATI0NS indicates this panel is capable of both RGB 18bit/24bit

[PATCH v0 12/14] fbdev/viafb: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 08/14] media: ivtv: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v0 10/14] sfc: falcon: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

[PATCH v3 1/3] arch: Select fbdev helpers with CONFIG_VIDEO

2024-03-29 Thread Thomas Zimmermann
Various Kconfig options selected the per-architecture helpers for fbdev. But none of the contained code depends on fbdev. Standardize on CONFIG_VIDEO, which will allow to add more general helpers for video functionality. CONFIG_VIDEO protects each architecture's video/ directory. This allows for

[PATCH v3 3/3] arch: Rename fbdev header and source files

2024-03-29 Thread Thomas Zimmermann
The per-architecture fbdev code has no dependencies on fbdev and can be used for any video-related subsystem. Rename the files to 'video'. Use video-sti.c on parisc as the source file depends on CONFIG_STI_CORE. On arc, arm, arm64, sh, and um the asm header file is an empty wrapper around the

[PATCH v2] drm/msm/dp: assign correct DP controller ID to x1e80100 interface table

2024-03-29 Thread Kuogee Hsieh
At current x1e80100 interface table, interface #3 is wrongly connected to DP controller #0 and interface #4 wrongly connected to DP controller #2. Fix this problem by connect Interface #3 to DP controller #0 and interface #4 connect to DP controller #1. Also add interface #6, #7 and #8 connections

[PATCH v0 01/14] IB/hfi1, IB/qib: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c[1], fix the terminology where I had a role to play, now that the approved verbiage exists in the specification. Compile tested,

[PATCH v0 00/14] Make I2C terminology more inclusive for I2C Algobit and consumers

2024-03-29 Thread Easwar Hariharan
I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" with more appropriate terms. Inspired by and following on to Wolfram's series to fix drivers/i2c/[1], fix the terminology for users of the I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists in the

Re: [PATCH v0 02/14] drm/amdgpu,drm/radeon: Make I2C terminology more inclusive

2024-03-29 Thread Andi Shyti
Hi Easwar, On Fri, Mar 29, 2024 at 05:00:26PM +, Easwar Hariharan wrote: > I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" I don't understand why we forget that i3c is 1.1.1 :-) > with more appropriate terms. Inspired by and following on to Wolfram's > series to fix

[PATCH] drm/i915/guc: Fix the fix for reset lock confusion

2024-03-29 Thread John . C . Harrison
From: John Harrison The previous fix for the circlular lock splat about the busyness worker wasn't quite complete. Even though the reset-in-progress flag is cleared at the start of intel_uc_reset_finish, the entire function is still inside the reset mutex lock. Not sure why the patch appeared to

Re: [PATCH] drm/msm/dp: Remove now unused connector_type from desc

2024-03-29 Thread Bjorn Andersson
On Fri, Mar 29, 2024 at 07:23:07AM +0200, Dmitry Baryshkov wrote: > On Fri, 29 Mar 2024 at 06:02, Bjorn Andersson > wrote: > > diff --git a/drivers/gpu/drm/msm/dp/dp_display.c > > b/drivers/gpu/drm/msm/dp/dp_display.c [..] > > static const struct msm_dp_desc sc8280xp_edp_descs[] = { > > This

Re: [PATCH] drm/meson: vclk: fix calculation of 59.94 fractional rates

2024-03-29 Thread Neil Armstrong
Hi, On Tue, 09 Jan 2024 23:07:04 +, Christian Hewitt wrote: > Playing 4K media with 59.94 fractional rate (typically VP9) causes the screen > to lose > sync with the following error reported in the system log: > > [ 89.610280] Fatal Error, invalid HDMI vclk freq 593406 > > Modetest shows

Re: (subset) [PATCH 0/5] Add Support for RK3326 GameForce Chi

2024-03-29 Thread Neil Armstrong
Hi, On Mon, 25 Mar 2024 08:49:54 -0500, Chris Morgan wrote: > From: Chris Morgan > > Add support for the GameForce Chi [1]. > > The GameForce Chi has the following hardware: > Tested: > - 3.5" dual lane 640x480 DSI display. > - 15 GPIO based face buttons. > - 2 ADC based face buttons. > -

Re: [PATCH v0 02/14] drm/amdgpu, drm/radeon: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
On 3/29/2024 10:16 AM, Andi Shyti wrote: > Hi Easwar, > > On Fri, Mar 29, 2024 at 05:00:26PM +, Easwar Hariharan wrote: >> I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" > > I don't understand why we forget that i3c is 1.1.1 :-) That's because it's a copy-paste error

Re: [PATCH v0 02/14] drm/amdgpu, drm/radeon: Make I2C terminology more inclusive

2024-03-29 Thread Easwar Hariharan
On 3/29/2024 10:28 AM, Easwar Hariharan wrote: > On 3/29/2024 10:16 AM, Andi Shyti wrote: >> Hi Easwar, >> >> >> The specification talks about: >> >> - master -> controller >> - slave -> target (and not client) >> >> But both you and Wolfram have used client. I'd like to reach >> some more

Re: [PATCH v0 02/14] drm/amdgpu,drm/radeon: Make I2C terminology more inclusive

2024-03-29 Thread Andi Shyti
Hi, On Fri, Mar 29, 2024 at 10:28:14AM -0700, Easwar Hariharan wrote: > On 3/29/2024 10:16 AM, Andi Shyti wrote: > > Hi Easwar, > > > > On Fri, Mar 29, 2024 at 05:00:26PM +, Easwar Hariharan wrote: > >> I2C v7, SMBus 3.2, and I3C specifications have replaced "master/slave" > > > > I don't

Re: [PATCH 0/9] enabled -Wformat-truncation for clang

2024-03-29 Thread patchwork-bot+netdevbpf
Hello: This series was applied to netdev/net-next.git (main) by Jakub Kicinski : On Tue, 26 Mar 2024 23:37:59 +0100 you wrote: > From: Arnd Bergmann > > With randconfig build testing, I found only eight files that produce > warnings with clang when -Wformat-truncation is enabled. This means >

[PATCH v3 2/3] arch: Remove struct fb_info from video helpers

2024-03-29 Thread Thomas Zimmermann
The per-architecture video helpers do not depend on struct fb_info or anything else from fbdev. Remove it from the interface and replace fb_is_primary_device() with video_is_primary_device(). The new helper is similar in functionality, but can operate on non-fbdev devices. Signed-off-by: Thomas

[PATCH v3 0/3] arch: Remove fbdev dependency from video helpers

2024-03-29 Thread Thomas Zimmermann
Make architecture helpers for display functionality depend on general video functionality instead of fbdev. This avoids the dependency on fbdev and makes the functionality available for non-fbdev code. Patch 1 replaces the variety of Kconfig options that control the Makefiles with CONFIG_VIDEO.

RE: [PATCH v3 8/9] dt-bindings: xlnx: Add VTC and TPG bindings

2024-03-29 Thread Klymenko, Anatoliy
> -Original Message- > From: Conor Dooley > Sent: Friday, March 29, 2024 8:47 AM > To: Klymenko, Anatoliy > Cc: Krzysztof Kozlowski ; Laurent Pinchart > ; Maarten Lankhorst > ; Maxime Ripard > ; Thomas Zimmermann ; > David Airlie ; Daniel Vetter ; Simek, > Michal ; Andrzej Hajda > ;

Re: [PATCH] drm/msm/dp: allow voltage swing / pre emphasis of 3

2024-03-29 Thread Abhinav Kumar
Hi Doug On 3/29/2024 4:07 PM, Doug Anderson wrote: Hi, On Sat, Feb 3, 2024 at 5:47 AM Dmitry Baryshkov wrote: Both dp_link_adjust_levels() and dp_ctrl_update_vx_px() limit swing and pre-emphasis to 2, while the real maximum value for the sum of the voltage swing and pre-emphasis is 3. Fix

[PATCH] drm/msm/dpu: fix vblank IRQ handling for command panels

2024-03-29 Thread Dmitry Baryshkov
In case of CMD DSI panels, the vblank IRQ can be used outside of irq_enable/irq_disable pair. This results in the following kind of messages. Move assignment of IRQ indices to atomic_enable / atomic_disable callbacks. [dpu error]invalid IRQ=[134217727, 31]

[PATCH] drm/msm/dpu: make error messages at dpu_core_irq_register_callback() more sensible

2024-03-29 Thread Dmitry Baryshkov
There is little point in using %ps to print a value known to be NULL. On the other hand it makes sense to print the callback symbol in the 'invalid IRQ' message. Correct those two error messages to make more sense. Fixes: 6893199183f8 ("drm/msm/dpu: stop using raw IRQ indices in the kernel

[PATCH 1/3] dt-bindings: panel: Add LG SW43408 MIPI-DSI panel

2024-03-29 Thread Dmitry Baryshkov
From: Sumit Semwal LG SW43408 is 1080x2160, 4-lane MIPI-DSI panel present on Google Pixel 3 phones. Signed-off-by: Vinod Koul Signed-off-by: Sumit Semwal [caleb: convert to yaml] Signed-off-by: Caleb Connolly Signed-off-by: Dmitry Baryshkov --- .../bindings/display/panel/lg,sw43408.yaml

[PATCH 3/3] drm: panel: Add LG sw43408 panel driver

2024-03-29 Thread Dmitry Baryshkov
From: Sumit Semwal LG SW43408 is 1080x2160, 4-lane MIPI-DSI panel, used in some Pixel3 phones. Whatever init sequence we have for this panel isn't capable of initialising it completely, toggling the reset gpio ever causes the panel to die. Until this is resolved we avoid resetting the panel.

[PATCH 0/3] drm/panel: add support for LG SW43408 panel

2024-03-29 Thread Dmitry Baryshkov
The LG SW43408 panel is used on Google Pixel3 devices. For a long time we could not submit the driver, as the panel was not coming up from the reset. The panel seems to be picky about some of the delays during init and it also uses non-standard payload for MIPI_DSI_COMPRESSION_MODE.

[PATCH 2/3] drm/mipi-dsi: add mipi_dsi_compression_mode_raw()

2024-03-29 Thread Dmitry Baryshkov
The LG SW43408 panel requires sending non-standard data as a part of the MIPI_DSI_COMPRESSION_MODE packet. Rather than hacking existing mipi_dsi_compression_mode() add mipi_dsi_compression_mode_raw(), which accepts raw data buffer and length. Signed-off-by: Dmitry Baryshkov ---

Re: [PATCH] drm/vkms: call drm_atomic_helper_shutdown before drm_dev_put()

2024-03-29 Thread Greg KH
On Thu, Mar 21, 2024 at 03:55:37PM +0800, guomengqi (A) wrote: > > 在 2024/3/21 15:39, Greg KH 写道: > > On Thu, Mar 21, 2024 at 03:07:52PM +0800, Guo Mengqi wrote: > > > commit 73a82b22963d ("drm/atomic: Fix potential use-after-free > > > in nonblocking commits") introduced drm_dev_get/put() to > >

RE: [PULL] drm-intel-fixes

2024-03-29 Thread Musial, Ewelina
Hi Rodrigo, When I opened dashboard early morning all results were available, so I don't think there was any issue during night. It could be long reporting queue - reporting through AWS takes ages but this is not an issue with reporting, this is how AWS works. Regards, Ewelina -Original

Re: [PATCH] drm/ttm: remove unused paramter

2024-03-29 Thread Christian König
Am 29.03.24 um 12:10 schrieb Christian König: Am 25.03.24 um 08:45 schrieb Jesse Zhang: remove the unsed the paramter in the function ttm_bo_bounce_temp_buffer and ttm_bo_add_move_fence. Signed-off-by: Jesse Zhang Good catch, Reviewed-by: Christian König Please rebase that patch on top

Re: [PATCH 09/22] gpio: virtio: drop owner assignment

2024-03-29 Thread Krzysztof Kozlowski
On 29/03/2024 11:27, Bartosz Golaszewski wrote: > On Wed, Mar 27, 2024 at 1:45 PM Krzysztof Kozlowski > wrote: >> >> virtio core already sets the .owner, so driver does not need to. >> >> Signed-off-by: Krzysztof Kozlowski >> >> --- >> >> Depends on the first patch. >> --- >>

Re: [PATCH 09/22] gpio: virtio: drop owner assignment

2024-03-29 Thread Bartosz Golaszewski
On Fri, Mar 29, 2024 at 12:35 PM Krzysztof Kozlowski wrote: > > On 29/03/2024 11:27, Bartosz Golaszewski wrote: > > On Wed, Mar 27, 2024 at 1:45 PM Krzysztof Kozlowski > > wrote: > >> > >> virtio core already sets the .owner, so driver does not need to. > >> > >> Signed-off-by: Krzysztof

[PATCH AUTOSEL 6.8 64/98] drm: Check output polling initialized before disabling

2024-03-29 Thread Sasha Levin
From: Shradha Gupta [ Upstream commit 5abffb66d12bcac84bf7b66389c571b8bb6e82bd ] In drm_kms_helper_poll_disable() check if output polling support is initialized before disabling polling. If not flag this as a warning. Additionally in drm_mode_config_helper_suspend() and

[PATCH AUTOSEL 6.8 59/98] accel/habanalabs: increase HL_MAX_STR to 64 bytes to avoid warnings

2024-03-29 Thread Sasha Levin
From: Koby Elbaz [ Upstream commit 8c075401f2dbda43600c61f780a165abde77877a ] Fix a warning of a buffer overflow: ‘snprintf’ output between 38 and 47 bytes into a destination of size 32 Signed-off-by: Koby Elbaz Reviewed-by: Oded Gabbay Signed-off-by: Oded Gabbay Signed-off-by: Sasha Levin

Re: [PATCH 09/22] gpio: virtio: drop owner assignment

2024-03-29 Thread Stefano Garzarella
On Fri, Mar 29, 2024 at 11:27:19AM +0100, Bartosz Golaszewski wrote: On Wed, Mar 27, 2024 at 1:45 PM Krzysztof Kozlowski wrote: virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. ---

Re: [PATCH] drm/ttm: remove unused paramter

2024-03-29 Thread Christian König
Am 25.03.24 um 08:45 schrieb Jesse Zhang: remove the unsed the paramter in the function ttm_bo_bounce_temp_buffer and ttm_bo_add_move_fence. Signed-off-by: Jesse Zhang Good catch, Reviewed-by: Christian König Regards, Christian. --- drivers/gpu/drm/ttm/ttm_bo.c | 10 -- 1 file

Re: [PATCH v12 0/8] mm/gup: Introduce memfd_pin_folios() for pinning memfd folios

2024-03-29 Thread David Hildenbrand
On 29.03.24 06:38, Kasireddy, Vivek wrote: Hi David, On 25.02.24 08:56, Vivek Kasireddy wrote: Currently, some drivers (e.g, Udmabuf) that want to longterm-pin the pages/folios associated with a memfd, do so by simply taking a reference on them. This is not desirable because the pages/folios

Re: [PATCH 01/22] virtio: store owner from modules with register_virtio_driver()

2024-03-29 Thread Stefano Garzarella
On Wed, Mar 27, 2024 at 01:40:54PM +0100, Krzysztof Kozlowski wrote: Modules registering driver with register_virtio_driver() might forget to set .owner field. i2c-virtio.c for example has it missing. The field is used by some of other kernel parts for reference counting (try_module_get()), so

Re: [PATCH 01/22] virtio: store owner from modules with register_virtio_driver()

2024-03-29 Thread Krzysztof Kozlowski
On 29/03/2024 12:42, Stefano Garzarella wrote: >> }; >> >> -int register_virtio_driver(struct virtio_driver *driver) >> +int __register_virtio_driver(struct virtio_driver *driver, struct module >> *owner) >> { >> /* Catch this early. */ >> BUG_ON(driver->feature_table_size &&

[PATCH AUTOSEL 6.8 05/98] drm: panel-orientation-quirks: Add quirk for GPD Win Mini

2024-03-29 Thread Sasha Levin
From: Samuel Dionne-Riel [ Upstream commit 2f862fdc0fd802e728b6ca96bc78ec3f01bf161e ] This adds a DMI orientation quirk for the GPD Win Mini panel. Signed-off-by: Samuel Dionne-Riel Signed-off-by: Linus Walleij Link:

[PATCH AUTOSEL 6.8 10/98] drm/amd/display: For FPO and SubVP/DRR configs program vmin/max sel

2024-03-29 Thread Sasha Levin
From: Alvin Lee [ Upstream commit fc9f47455ae143e8831415a46eab3fbc69e408aa ] [Why & How] For FPO and SubVP/DRR cases we need to ensure to program OTG_V_TOTAL_MIN/MAX_SEL, otherwise stretching the vblank in FPO / SubVP / DRR cases will not have any effect and we could hit underflow / corruption.

[PATCH AUTOSEL 6.8 24/98] drm/amd/display: Disable idle reallow as part of command/gpint execution

2024-03-29 Thread Sasha Levin
From: Nicholas Kazlauskas [ Upstream commit 6226a5aa77370329e01ee8abe50a95e60618ce97 ] [Why] Workaroud for a race condition where DMCUB is in the process of committing to IPS1 during the handshake causing us to miss the transition into IPS2 and touch the INBOX1 RPTR causing a HW hang. [How]

[PATCH AUTOSEL 6.8 46/98] drm/panel: simple: Add BOE BP082WX1-100 8.2" panel

2024-03-29 Thread Sasha Levin
From: Tony Lindgren [ Upstream commit dc90214ff58be575fdceb549f901506cdef5d093 ] The BOE BP082WX1-100 is a 8.2" panel similar to the 10.1" panel BP101WX1-100. Both panels use the same timings. Acked-by: Conor Dooley Reviewed-by: Dmitry Baryshkov Signed-off-by: Tony Lindgren Link:

Re: [PATCH 09/22] gpio: virtio: drop owner assignment

2024-03-29 Thread Bartosz Golaszewski
On Wed, Mar 27, 2024 at 1:45 PM Krzysztof Kozlowski wrote: > > virtio core already sets the .owner, so driver does not need to. > > Signed-off-by: Krzysztof Kozlowski > > --- > > Depends on the first patch. > --- > drivers/gpio/gpio-virtio.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git

Re: [PATCH v2 3/3] arm64: dts: qcom: sm6350: Add DisplayPort controller

2024-03-29 Thread Konrad Dybcio
On 29.03.2024 8:45 AM, Luca Weiss wrote: > Add the node for the DisplayPort controller found on the SM6350 SoC. > > Reviewed-by: Dmitry Baryshkov > Signed-off-by: Luca Weiss > --- Reviewed-by: Konrad Dybcio Konrad

Re: [PATCH 16/22] net: vmw_vsock: virtio: drop owner assignment

2024-03-29 Thread Stefano Garzarella
On Wed, Mar 27, 2024 at 01:41:09PM +0100, Krzysztof Kozlowski wrote: virtio core already sets the .owner, so driver does not need to. Signed-off-by: Krzysztof Kozlowski --- Depends on the first patch. --- net/vmw_vsock/virtio_transport.c | 1 - 1 file changed, 1 deletion(-) Acked-by:

Re: [PATCH v11 2/7] clk: meson: add vclk driver

2024-03-29 Thread Jerome Brunet
On Mon 25 Mar 2024 at 12:09, Neil Armstrong wrote: > The VCLK and VCLK_DIV clocks have supplementary bits. > > The VCLK gate has a "SOFT RESET" bit to toggle after the whole > VCLK sub-tree rate has been set, this is implemented in > the gate enable callback. > > The VCLK_DIV clocks as enable

[PATCH AUTOSEL 6.8 03/98] drm/amdgpu: Skip do PCI error slot reset during RAS recovery

2024-03-29 Thread Sasha Levin
From: "Stanley.Yang" [ Upstream commit 601429cca96b4af3be44172c3b64e4228515dbe1 ] Why: The PCI error slot reset maybe triggered after inject ue to UMC multi times, this caused system hang. [ 557.371857] amdgpu :af:00.0: amdgpu: GPU reset succeeded, trying to resume [

[PATCH AUTOSEL 6.8 01/98] drm/vc4: don't check if plane->state->fb == state->fb

2024-03-29 Thread Sasha Levin
From: Maíra Canal [ Upstream commit 5ee0d47dcf33efd8950b347dcf4d20bab12a3fa9 ] Currently, when using non-blocking commits, we can see the following kernel warning: [ 110.908514] [ cut here ] [ 110.908529] refcount_t: underflow; use-after-free. [ 110.908620] WARNING:

[PATCH AUTOSEL 6.8 02/98] drm/ci: uprev mesa version: fix kdl commit fetch

2024-03-29 Thread Sasha Levin
From: Vignesh Raman [ Upstream commit d315a68e94a76310c349add3f9c914cefda0a87f ] build-kdl.sh was doing a `clone --depth 1` of the default branch, then checking out a commit that might not be the latest of that branch, resulting in container build error.

[PATCH AUTOSEL 6.8 07/98] drm/ttm: return ENOSPC from ttm_bo_mem_space v3

2024-03-29 Thread Sasha Levin
From: Christian König [ Upstream commit 28e5126718c7b306b8c29d2ae8f48417e9303aa1 ] Only convert it to ENOMEM in ttm_bo_validate. This allows ttm_bo_validate to distinguish between an out of memory situation and just out of space in a placement domain. v2: improve commit message v3: fix

[PATCH AUTOSEL 6.8 09/98] drm/amd/display: increased min_dcfclk_mhz and min_fclk_mhz

2024-03-29 Thread Sasha Levin
From: Sohaib Nadeem [ Upstream commit d46fb0068c54d3dc95ae8298299c4d9edb0fb7c1 ] [why] Originally, PMFW said min FCLK is 300Mhz, but min DCFCLK can be increased to 400Mhz because min FCLK is now 600Mhz so FCLK >= 1.5 * DCFCLK hardware requirement will still be satisfied. Increasing min DCFCLK

[PATCH AUTOSEL 6.8 62/98] drm/amd/amdgpu: Fix potential ioremap() memory leaks in amdgpu_device_init()

2024-03-29 Thread Sasha Levin
From: Srinivasan Shanmugam [ Upstream commit eb4f139888f636614dab3bcce97ff61cefc4b3a7 ] This ensures that the memory mapped by ioremap for adev->rmmio, is properly handled in amdgpu_device_init(). If the function exits early due to an error, the memory is unmapped. If the function completes

[PATCH AUTOSEL 6.8 54/98] drm/amd/display: Fix nanosec stat overflow

2024-03-29 Thread Sasha Levin
From: Aric Cyr [ Upstream commit 14d68acfd04b39f34eea7bea65dda652e6db5bf6 ] [Why] Nanosec stats can overflow on long running systems potentially causing statistic logging issues. [How] Use 64bit types for nanosec stats to ensure no overflow. Reviewed-by: Rodrigo Siqueira Tested-by: Daniel

[PATCH AUTOSEL 6.6 03/75] drm: panel-orientation-quirks: Add quirk for GPD Win Mini

2024-03-29 Thread Sasha Levin
From: Samuel Dionne-Riel [ Upstream commit 2f862fdc0fd802e728b6ca96bc78ec3f01bf161e ] This adds a DMI orientation quirk for the GPD Win Mini panel. Signed-off-by: Samuel Dionne-Riel Signed-off-by: Linus Walleij Link:

[PATCH AUTOSEL 6.6 05/75] drm/ttm: return ENOSPC from ttm_bo_mem_space v3

2024-03-29 Thread Sasha Levin
From: Christian König [ Upstream commit 28e5126718c7b306b8c29d2ae8f48417e9303aa1 ] Only convert it to ENOMEM in ttm_bo_validate. This allows ttm_bo_validate to distinguish between an out of memory situation and just out of space in a placement domain. v2: improve commit message v3: fix

[PATCH AUTOSEL 6.6 38/75] drm/amd/display: Fix nanosec stat overflow

2024-03-29 Thread Sasha Levin
From: Aric Cyr [ Upstream commit 14d68acfd04b39f34eea7bea65dda652e6db5bf6 ] [Why] Nanosec stats can overflow on long running systems potentially causing statistic logging issues. [How] Use 64bit types for nanosec stats to ensure no overflow. Reviewed-by: Rodrigo Siqueira Tested-by: Daniel

Re: [PATCH v11 3/7] clk: meson: g12a: make VCLK2 and ENCL clock path configurable by CCF

2024-03-29 Thread Jerome Brunet
On Mon 25 Mar 2024 at 12:09, Neil Armstrong wrote: > In order to setup the DSI clock, let's make the unused VCLK2 clock path > configuration via CCF. > > The nocache option is removed from following clocks: > - vclk2_sel > - vclk2_input > - vclk2_div > - vclk2 > - vclk_div1 > - vclk2_div2_en >

[PATCH AUTOSEL 6.6 41/75] accel/habanalabs: increase HL_MAX_STR to 64 bytes to avoid warnings

2024-03-29 Thread Sasha Levin
From: Koby Elbaz [ Upstream commit 8c075401f2dbda43600c61f780a165abde77877a ] Fix a warning of a buffer overflow: ‘snprintf’ output between 38 and 47 bytes into a destination of size 32 Signed-off-by: Koby Elbaz Reviewed-by: Oded Gabbay Signed-off-by: Oded Gabbay Signed-off-by: Sasha Levin

[PATCH v2 0/3] DisplayPort support for SM6350/SM7225

2024-03-29 Thread Luca Weiss
Add the required changes to support DisplayPort (normally(?) available via the USB-C connector) on the SM6350/SM7225 SoC. This has been tested on a Fairphone 4 smartphone with additional changes not included in this series (mostly just wiring up TCPM and the SBU mux). Signed-off-by: Luca Weiss

[PATCH v2 3/3] arm64: dts: qcom: sm6350: Add DisplayPort controller

2024-03-29 Thread Luca Weiss
Add the node for the DisplayPort controller found on the SM6350 SoC. Reviewed-by: Dmitry Baryshkov Signed-off-by: Luca Weiss --- arch/arm64/boot/dts/qcom/sm6350.dtsi | 88 1 file changed, 88 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sm6350.dtsi

[PATCH v2 1/3] dt-bindings: display: msm: dp-controller: document SM6350 compatible

2024-03-29 Thread Luca Weiss
Add the compatible string for the DisplayPort controller on SM6350 which is compatible with the one on SM8350. Acked-by: Rob Herring Signed-off-by: Luca Weiss --- Documentation/devicetree/bindings/display/msm/dp-controller.yaml | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH v2 2/3] dt-bindings: display: msm: sm6350-mdss: document DP controller subnode

2024-03-29 Thread Luca Weiss
Document the displayport controller subnode of the SM6350 MDSS. Signed-off-by: Luca Weiss --- .../devicetree/bindings/display/msm/qcom,sm6350-mdss.yaml| 9 + 1 file changed, 9 insertions(+) diff --git a/Documentation/devicetree/bindings/display/msm/qcom,sm6350-mdss.yaml

[RFC PATCH 6/8] drm/ttm/pool: Provide a helper to shrink pages.

2024-03-29 Thread Thomas Hellström
Provide a helper to shrink ttm_tt page-vectors on a per-page basis. A ttm_backup backend could then in theory get away with allocating a single temporary page for each struct ttm_tt. This is accomplished by splitting larger pages before trying to back them up. In the future we could allow

[PATCH 4/8] drm/ttm: Allow continued swapout after -ENOSPC falure

2024-03-29 Thread Thomas Hellström
The -ENOSPC failure from ttm_bo_swapout() meant that the lru_lock was dropped and simply restarting the iteration meant we'd likely hit the same error again on the same resource. Now that we can restart the iteration even if the lock was dropped, do that. Cc: Christian König Cc: Somalapuram

[PATCH 2/8] drm/ttm: Use LRU hitches

2024-03-29 Thread Thomas Hellström
Have iterators insert themselves into the list they are iterating over using hitch list nodes. Since only the iterator owner can remove these list nodes from the list, it's safe to unlock the list and when continuing, use them as a starting point. Due to the way LRU bumping works in TTM, newly

[RFC PATCH 0/8] TTM shrinker helpers and xe buffer object shrinker

2024-03-29 Thread Thomas Hellström
This series implements TTM shrinker / eviction helpers and an xe bo shrinker. It builds on two previous series. First https://www.mail-archive.com/dri-devel@lists.freedesktop.org/msg484425.html for patch 1-4, which IMO still could be reviewed and pushed as a separate series. Second the previous

[RFC PATCH 7/8] drm/xe, drm/ttm: Provide a generic LRU walker helper

2024-03-29 Thread Thomas Hellström
Export the needed functions from TTM and provide a generic LRU walker in xe, in the spirit of drm_gem_lru_scan() but building on the restartable TTM LRU functionality. The LRU walker optionally supports locking objects as part of a drm_exec locking transaction, and can thus be used for both

[RFC PATCH 5/8] drm/ttm: Add a virtual base class for graphics memory backup

2024-03-29 Thread Thomas Hellström
Initially intended for experimenting with different backup solutions (shmem vs direct swap cache insertion), abstract the backup destination using a virtual base class. Also provide a sample implementation for shmem. While when settling on a preferred backup solution, one could perhaps skip the

[PATCH 3/8] drm/ttm, drm/amdgpu, drm/xe: Consider hitch moves within bulk sublist moves

2024-03-29 Thread Thomas Hellström
To address the problem with hitches moving when bulk move sublists are lru-bumped, register the list cursors with the ttm_lru_bulk_move structure when traversing its list, and when lru-bumping the list, move the cursor hitch to the tail. This also means it's mandatory for drivers to call

[PATCH 1/8] drm/ttm: Allow TTM LRU list nodes of different types

2024-03-29 Thread Thomas Hellström
To be able to handle list unlocking while traversing the LRU list, we want the iterators not only to point to the next position of the list traversal, but to insert themselves as list nodes at that point to work around the fact that the next node might otherwise disappear from the list while the

  1   2   >