Re: [PATCH] drm/exynos: hdmi: report safe 640x480 mode as a fallback when no EDID found

2024-05-14 Thread Inki Dae
Hi Marek, 2024년 4월 25일 (목) 오후 6:49, Marek Szyprowski 님이 작성: > > When reading EDID fails and driver reports no modes available, the DRM > core adds an artificial 1024x786 mode to the connector. Unfortunately > some variants of the Exynos HDMI (like the one in Exynos4 SoCs) are not > able to drive

Re: [PATCH 1/4] dt-bindings: display: ti,am65x-dss: Minor Cleanup

2024-05-13 Thread Aradhya Bhatia
On 14/05/24 00:49, Rob Herring wrote: > On Sun, May 12, 2024 at 01:00:52AM +0530, Aradhya Bhatia wrote: >> Reduce tab size from 8 spaces to 4 spaces to make the bindings >> consistent, and easy to expand. > > "Re-indent the example" would be more specific than "minor cleanups" in > the

Re: [PATCH 3/4] dt-bindings: display: ti,am65x-dss: Add OLDI properties for AM625 DSS

2024-05-13 Thread Aradhya Bhatia
tes per second >> >> + oldi-txes: >> +type: object >> +properties: >> + "#address-cells": >> +const: 1 >> + >> + "#size-cells": >> +const: 0 >> + >> +patternProperties: >> + '^oldi_tx@[0-1]$': >> +type: object >> +$ref: ti,am625-oldi.yaml# >> +unevaluatedProperties: false >> +description: OLDI transmitters connected to the DSS VPs > > Connected to is not part of the DSS. I don't think these nodes belong > here as mentioned in the other patch. > Replied to this in patch 2/4 to keep the discussion in one thread. Regards Aradhya

Re: [PATCH 2/4] dt-bindings: display: ti: Add schema for AM625 OLDI Transmitter

2024-05-13 Thread Aradhya Bhatia
>>> Hi Aradhya, >>> >>> Thank you for the patch. >>> >>> On Sun, May 12, 2024 at 01:00:53AM +0530, Aradhya Bhatia wrote: >>>> Add devicetree binding schema for AM625 OLDI Transmitters. >>>> >>>> Signed-off-by: Aradh

Re: [PATCH v2 2/2] Add dp PHY dt-bindings

2024-05-13 Thread 杨连坤

RE: [PATCH v7 1/1] drm/bridge: it6505: fix hibernate to resume no display issue

2024-05-13 Thread kuro.chung
; Thomas Zimmermann ; David Airlie ; Daniel Vetter ; open list:DRM DRIVERS ; open list Subject: Re: [PATCH v7 1/1] drm/bridge: it6505: fix hibernate to resume no display issue On Mon, May 13, 2024 at 7:42 PM Robert Foss wrote: > > On Mon, May 6, 2024 at 11:36 AM kuro wrote: > > &g

[PATCH v2] drm/bridge: tc358767: Enable FRMSYNC timing generator

2024-05-13 Thread Marek Vasut
TC9595 datasheet Video Path0 Control (VPCTRL0) Register bit FRMSYNC description says "This bit should be disabled only in video mode transmission where Host transmits video timing together with video data and where pixel clock source is from DSI clock." . This driver always sources pixel clock

Re: [PATCH net-next v9 00/14] Device Memory TCP

2024-05-13 Thread Jakub Kicinski
On Fri, 10 May 2024 16:21:11 -0700 Mina Almasry wrote: > Device Memory TCP Sorry Mina, this is too big to apply during the merge window :( -- pw-bot: defer

[PATCH] drm: Combine identical if/elif code blocks

2024-05-13 Thread Thorsten Blum
Merge the identical if/elif code blocks and remove the following two warnings reported by make includecheck: asm/ioctl.h is included more than once linux/types.h is included more than once Signed-off-by: Thorsten Blum --- include/uapi/drm/drm.h | 8 +--- 1 file changed, 1

Re: [PATCH v4] drm/nouveau: use tile_mode and pte_kind for VM_BIND bo allocations

2024-05-13 Thread Mohamed Ahmed
Hey, Understood. Thanks a lot and sorry for any inconvenience. Mohamed On Mon, May 13, 2024 at 11:28 PM Danilo Krummrich wrote: > Hi Mohamed, > > Thank you for fixing this up! > > On 5/9/24 22:43, Mohamed Ahmed wrote: > > Allows PTE kind and tile mode on BO create with VM_BIND, > > and adds a

Re: [PATCH 2/3] dt-bindings: display: panel: constrain 'reg' in SPI panels

2024-05-13 Thread Linus Walleij
On Thu, May 9, 2024 at 11:43 AM Krzysztof Kozlowski wrote: > SPI-attached devices could have more than one chip-select, thus their > bindings are supposed to constrain the 'reg' property to match hardware. > Add missing 'reg' constrain for SPI-attached display panels. > > Signed-off-by:

Re: [PATCH v6 7/7] drm/panel: himax-hx83102: Support for IVO t109nw41 MIPI-DSI panel

2024-05-13 Thread Linus Walleij
On Sat, May 11, 2024 at 4:14 AM Cong Yang wrote: > The IVO t109nw41 is a 11.0" WUXGA TFT LCD panel, use hx83102 controller > which fits in nicely with the existing panel-himax-hx83102 driver. Hence, > we add a new compatible with panel specific config. > > Signed-off-by: Cong Yang Reviewed-by:

Re: [PATCH v6 5/7] drm/panel: himax-hx83102: Support for BOE nv110wum-l60 MIPI-DSI panel

2024-05-13 Thread Linus Walleij
On Sat, May 11, 2024 at 4:13 AM Cong Yang wrote: > The BOE nv110wum-l60 is a 11.0" WUXGA TFT LCD panel, use hx83102 controller > which fits in nicely with the existing panel-himax-hx83102 driver. Hence, > we add a new compatible with panel specific config. > > Signed-off-by: Cong Yang

Re: [PATCH v4] drm/nouveau: use tile_mode and pte_kind for VM_BIND bo allocations

2024-05-13 Thread Danilo Krummrich
Hi Mohamed, Thank you for fixing this up! On 5/9/24 22:43, Mohamed Ahmed wrote: Allows PTE kind and tile mode on BO create with VM_BIND, and adds a GETPARAM to indicate this change. This is needed to support It's usually better to use imperative verb form for commit messages. No need to send

[PATCH] drm/edid: remove drm_do_get_edid()

2024-05-13 Thread Jani Nikula
All users of drm_do_get_edid() have been converted to drm_edid_read_custom(). Remove the unused function to prevent new users from creeping in. Signed-off-by: Jani Nikula --- Cc: Robert Foss Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Thomas Zimmermann --- drivers/gpu/drm/drm_edid.c | 28

Re: [PATCH v6 2/7] drm/panel: himax-hx83102: Break out as separate driver

2024-05-13 Thread Linus Walleij
On Sat, May 11, 2024 at 4:13 AM Cong Yang wrote: > The Starry HX83102 based mipi panel should never have been part of the boe > tv101wum-n16 driver. Discussion with Doug and Linus in V1 [1], we need a > separate driver to enable the hx83102 controller. > > In hx83102 driver, add DSI commands as

Re: [PATCH 3/3] dt-bindings: display: panel: constrain 'reg' in DSI panels

2024-05-13 Thread Linus Walleij
On Thu, May 9, 2024 at 11:43 AM Krzysztof Kozlowski wrote: > DSI-attached devices could respond to more than one virtual channel > number, thus their bindings are supposed to constrain the 'reg' property > to match hardware. Add missing 'reg' constrain for DSI-attached display > panels, based

Re: [PATCH v4 6/9] drm/panel: novatek-nt36672e: Switch to mipi_dsi_dcs_write_seq_multi()

2024-05-13 Thread Linus Walleij
On Wed, May 8, 2024 at 10:53 PM Douglas Anderson wrote: > This is a mechanical conversion of the novatek-nt36672e driver to use > the new mipi_dsi_dcs_write_seq_multi(). The new function is easier for > clients to understand and using it also causes smaller code to be > generated. Specifically:

Re: [PATCH 2/9] drm: Export drm_plane_has_format()

2024-05-13 Thread Jani Nikula
On Mon, 13 May 2024, Ville Syrjala wrote: > From: Ville Syrjälä > > Export drm_plane_has_format() so that drivers can use it. > > Signed-off-by: Ville Syrjälä Reviewed-by: Jani Nikula > --- > drivers/gpu/drm/drm_crtc_internal.h | 2 -- > drivers/gpu/drm/drm_plane.c | 1 + >

Re: [PATCH 1/9] drm: Rename drm_plane_check_pixel_format() to drm_plane_has_format()

2024-05-13 Thread Jani Nikula
On Mon, 13 May 2024, Ville Syrjala wrote: > From: Ville Syrjälä > > Rename drm_plane_check_pixel_format() to drm_plane_has_format() > and change the return type accordingly. Allows one to write > more natural code. > > Also matches drm_any_plane_has_format() better. > > Signed-off-by: Ville

Re: [PATCH 3/4] dt-bindings: display: ti,am65x-dss: Add OLDI properties for AM625 DSS

2024-05-13 Thread Rob Herring
+$ref: ti,am625-oldi.yaml# > + unevaluatedProperties: false > +description: OLDI transmitters connected to the DSS VPs Connected to is not part of the DSS. I don't think these nodes belong here as mentioned in the other patch. Rob

Re: [PATCH 2/4] dt-bindings: display: ti: Add schema for AM625 OLDI Transmitter

2024-05-13 Thread Rob Herring
On Mon, May 13, 2024 at 02:07:44PM +0530, Aradhya Bhatia wrote: > Hi Laurent, > > Thank you for reviewing the patches! > > On 13-May-24 01:04, Laurent Pinchart wrote: > > Hi Aradhya, > > > > Thank you for the patch. > > > > On Sun, May 12, 2

Re: [PATCH 1/4] dt-bindings: display: ti,am65x-dss: Minor Cleanup

2024-05-13 Thread Rob Herring
On Sun, May 12, 2024 at 01:00:52AM +0530, Aradhya Bhatia wrote: > Reduce tab size from 8 spaces to 4 spaces to make the bindings > consistent, and easy to expand. "Re-indent the example" would be more specific than "minor cleanups" in the subject. Otherwise, Acked-by: Rob Herring (Arm) > >

Re: [PATCH v2] drm/amd/display: fix documentation warnings for mpc.h

2024-05-13 Thread Alex Deucher
On Sat, May 11, 2024 at 12:22 AM Stephen Rothwell wrote: > > Hi Marcelo, > > On Sat, 11 May 2024 13:37:17 +1000 Stephen Rothwell > wrote: > > > > Thanks for doing this. > > > > I haven't tested it, but just a couple of little things: > > > > On Fri, 10 May 2024 21:02:02 -0300 Marcelo Mendes

[PATCH 9/9] drm/i915: Nuke the TGL+ chroma plane tile row alignment stuff

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä I don't think the display hardware really has such chroma plane tile row alignment requirements as outlined in commit d156135e6a54 ("drm/i915/tgl: Make sure a semiplanar UV plane is tile row size aligned") Bspec had the same exact thing to say about earlier hardware as well,

[PATCH 8/9] drm/i915: Update plane alignment requirements for TGL+

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä Currently we still use the SKL+ PLANE_SURF alignment even for TGL+ even though the hardware no longer needs it. Introduce a separate tgl_plane_min_alignment() and update it to more accurately reflect the hardware requirements. Signed-off-by: Ville Syrjälä ---

[PATCH 7/9] drm/i915: Move intel_surf_alignment() into skl_univerals_plane.c

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä Now that all pre-skl platforms have their own .min_alignment() functions the remainder of intel_surf_alignment() can be hoisted into skl_univerals_plane.c (and renamed appropriately). Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_fb.c | 77

[PATCH 6/9] drm/i915: Split pre-skl platforms out from intel_surf_alignment()

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä Extract the necessary chunks from intel_surf_alignment() into per-platform variants for all pre-skl primary/sprite planes. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/i9xx_plane.c | 69 - drivers/gpu/drm/i915/display/intel_fb.c |

[PATCH 4/9] drm/i915: Introduce fb->min_alignment

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä Different planes could have different alignment requirements even for the same format/modifier. Collect the alignment requirements across all planes capable of scanning out the fb such that the alignment used when pinning the normal ggtt view is satisfactory to all those

[PATCH 3/9] drm/i915: Introduce plane->min_alignment() vfunc

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä Different hardware generations have different scanout alignment requirements. Introduce a new vfunc that will allow us to make that distinction without horrible if-ladders. For now we directly plug in the existing intel_surf_alignment() and intel_cursor_alignment()

[PATCH 5/9] drm/i915: Split cursor alignment to per-platform vfuncs

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä Split intel_cursor_alignment() into per-platform variants. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/i915/display/intel_cursor.c | 40 +++-- drivers/gpu/drm/i915/display/intel_fb.c | 16 - drivers/gpu/drm/i915/display/intel_fb.h | 3

[PATCH 2/9] drm: Export drm_plane_has_format()

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä Export drm_plane_has_format() so that drivers can use it. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_crtc_internal.h | 2 -- drivers/gpu/drm/drm_plane.c | 1 + include/drm/drm_plane.h | 2 ++ 3 files changed, 3 insertions(+), 2 deletions(-)

[PATCH 1/9] drm: Rename drm_plane_check_pixel_format() to drm_plane_has_format()

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä Rename drm_plane_check_pixel_format() to drm_plane_has_format() and change the return type accordingly. Allows one to write more natural code. Also matches drm_any_plane_has_format() better. Signed-off-by: Ville Syrjälä --- drivers/gpu/drm/drm_atomic.c| 7 ++-

[PATCH 0/9] drm/i915: Polish plane surface alignment handling

2024-05-13 Thread Ville Syrjala
From: Ville Syrjälä intel_surf_alignment() in particular has devolved into a complete mess. Redesign the code so that we can handle alignment restrictions in a nicer. Also adjust alignment for TGL+ to actually match the hardware requirements. Ville Syrjälä (9): drm: Rename

Re: [PATCH v7 1/1] drm/bridge: it6505: fix hibernate to resume no display issue

2024-05-13 Thread Robert Foss
HDCP_DONE, it6505_irq_hdcp_done }, > > { BIT_INT_AUX_CMD_FAIL, it6505_irq_aux_cmd_fail }, > > { BIT_INT_HDCP_KSV_CHECK, it6505_irq_hdcp_ksv_check }, > > { BIT_INT_AUDIO_FIFO_ERROR, it6505_irq_audio_fifo_error }, > > { BIT_INT_LINK_TRAIN_FAIL, it6505_irq_link_train_fail }, > > - { BIT_INT_VID_FIFO_ERROR, it6505_irq_video_fifo_error }, > > - { BIT_INT_IO_FIFO_OVERFLOW, > > it6505_irq_io_latch_fifo_overflow }, > > }; > > int int_status[3], i; > > > > @@ -2550,6 +2574,7 @@ static irqreturn_t it6505_int_threaded_handler(int > > unused, void *data) > > if (it6505_test_bit(irq_vec[i].bit, (unsigned int > > *)int_status)) > > irq_vec[i].handler(it6505); > > } > > + it6505_irq_video_handler(it6505, (unsigned int *) > > int_status); > > } > > > > pm_runtime_put_sync(dev); > > -- > > 2.25.1 > > > > Reviewed-by: Robert Foss This patch does not apply on drm-misc-next, please fix this and then this patch is ready to be applied.

Re: [PATCH v7 1/1] drm/bridge: it6505: fix hibernate to resume no display issue

2024-05-13 Thread Robert Foss
On Mon, May 6, 2024 at 11:36 AM kuro wrote: > > From: Kuro > > ITE added a FIFO reset bit for input video. When system power resume, > the TTL input of it6505 may get some noise before video signal stable > and the hardware function reset is required. > But the input FIFO reset will also trigger

Re: [PATCH v4 6/9] drm/panel: novatek-nt36672e: Switch to mipi_dsi_dcs_write_seq_multi()

2024-05-13 Thread Dmitry Baryshkov
On Wed, May 08, 2024 at 01:51:48PM -0700, Douglas Anderson wrote: > This is a mechanical conversion of the novatek-nt36672e driver to use > the new mipi_dsi_dcs_write_seq_multi(). The new function is easier for > clients to understand and using it also causes smaller code to be > generated.

Re: [PATCH] drm/bridge: adv7511: Attach next bridge without creating connector

2024-05-13 Thread Dmitry Baryshkov
On Mon, 13 May 2024 at 10:55, Liu Ying wrote: > > The connector is created by either this ADV7511 bridge driver or > any DRM device driver/previous bridge driver, so this ADV7511 > bridge driver should not let the next bridge driver create connector. > > If the next bridge is a HDMI connector,

Re: [PATCH v2 2/7] drm/mipi-dsi: wrap more functions for streamline handling

2024-05-13 Thread Doug Anderson
Hi, On Sat, May 11, 2024 at 4:00 PM Dmitry Baryshkov wrote: > > Follow the pattern of mipi_dsi_dcs_*_multi() and wrap several existing > MIPI DSI functions to use the context for processing. This simplifies > and streamlines driver code to use simpler code pattern. > > Note, msleep function is

Re: [PATCH 4/4] drm/xe/guc: Expose raw access to GuC log over debugfs

2024-05-13 Thread John Harrison
On 5/12/2024 08:36, Michal Wajdeczko wrote: We already provide the content of the GuC log in debugsfs, but it is in a text format where each log dword is printed as hexadecimal number, which does not scale well with large GuC log buffers. To allow more efficient access to the GuC log, which

Re: [PATCH v6 5/7] drm/panel: himax-hx83102: Support for BOE nv110wum-l60 MIPI-DSI panel

2024-05-13 Thread Doug Anderson
Hi, On Fri, May 10, 2024 at 7:13 PM Cong Yang wrote: > > The BOE nv110wum-l60 is a 11.0" WUXGA TFT LCD panel, use hx83102 controller > which fits in nicely with the existing panel-himax-hx83102 driver. Hence, > we add a new compatible with panel specific config. > > Signed-off-by: Cong Yang >

Re: [PATCH v6 7/7] drm/panel: himax-hx83102: Support for IVO t109nw41 MIPI-DSI panel

2024-05-13 Thread Doug Anderson
Hi, On Fri, May 10, 2024 at 7:14 PM Cong Yang wrote: > > The IVO t109nw41 is a 11.0" WUXGA TFT LCD panel, use hx83102 controller > which fits in nicely with the existing panel-himax-hx83102 driver. Hence, > we add a new compatible with panel specific config. > > Signed-off-by: Cong Yang > --- >

Re: [PATCH v6 2/7] drm/panel: himax-hx83102: Break out as separate driver

2024-05-13 Thread Doug Anderson
Hi, On Fri, May 10, 2024 at 7:13 PM Cong Yang wrote: > > +static int hx83102_prepare(struct drm_panel *panel) > +{ > + struct hx83102 *ctx = panel_to_hx83102(panel); > + struct mipi_dsi_device *dsi = ctx->dsi; > + struct device *dev = >dev; > + int ret; > + > +

Re: [PATCH] drm/bridge: tc358767: Enable FRMSYNC timing generator

2024-05-13 Thread Robert Foss
On Mon, May 13, 2024 at 4:16 AM Marek Vasut wrote: > > TC9595 datasheet Video Path0 Control (VPCTRL0) Register bit FRMSYNC > description > says "This bit should be disabled only in video mode transmission where Host > transmits video timing together with video data and where pixel clock source >

Re: [PATCH 0/3] dt-bindings: display: panel: constrain 'reg'

2024-05-13 Thread Dmitry Baryshkov
display/panel/raydium,rm69380.yaml Because those were added to drm-misc during the last cycle. So ideally the patch should have gone through drm-misc. -- With best wishes Dmitry

Re: [PATCH v2 00/12] Remove redundant checks on existence of 'bridge->encoder'

2024-05-13 Thread Robert Foss
On Mon, 13 May 2024 23:30:57 +0800, Sui Jingfeng wrote: > The checks on the existence of bridge->encoder in the implementation of > drm_bridge_funcs::attach() is not necessary, as it has already been checked > in the drm_bridge_attach() function call by previous bridge or KMS driver. > The

Re: [PATCH v2 2/3] dt-bindings: arm: mediatek: mmsys: Add OF graph support for board path

2024-05-13 Thread Conor Dooley
On Mon, May 13, 2024 at 03:44:00PM +0200, AngeloGioacchino Del Regno wrote: > Il 13/05/24 08:15, CK Hu (胡俊光) ha scritto: > > On Fri, 2024-05-10 at 12:14 +0200, AngeloGioacchino Del Regno wrote: > > > Il 10/05/24 11:34, CK Hu (胡俊光) ha scritto: > > > > On Thu, 2024-05-09 at 11:27 +0200,

Re: [PATCH v6 0/7] Adds support for ConfigFS to VKMS!

2024-05-13 Thread José Expósito
On Mon, May 13, 2024 at 12:03:07PM +0300, Marius Vlad wrote: > Hi all, > On Mon, May 13, 2024 at 10:08:38AM +0200, José Expósito wrote: > > On Fri, May 10, 2024 at 06:19:45PM +0200, Louis Chauvet wrote: > > > Le 09/05/24 - 18:18, Jim Shargo a écrit : > > > > Sima--thanks SO MUCH for going through

Re: [PATCH] drm/i915/gem/i915_gem_ttm_move: Fix typo

2024-05-13 Thread Rodrigo Vivi
On Mon, May 13, 2024 at 02:14:51AM -0400, Deming Wang wrote: > The mapings should be replaced by mappings. > > Signed-off-by: Deming Wang Reviewed-by: Rodrigo Vivi and pushed to drm-intel-gt-next thanks for the patch > --- > drivers/gpu/drm/i915/gem/i915_gem_ttm_move.c |

[PATCH] drm/msm: Add obj flags to gpu devcoredump

2024-05-13 Thread Rob Clark
From: Rob Clark When debugging faults, it is useful to know how the BO is mapped (cached vs WC, gpu readonly, etc). Signed-off-by: Rob Clark --- drivers/gpu/drm/msm/adreno/adreno_gpu.c | 1 + drivers/gpu/drm/msm/msm_gpu.c | 6 -- drivers/gpu/drm/msm/msm_gpu.h | 1 + 3

Re: [PATCH v4 4/9] drm/mipi-dsi: Reduce driver bloat of mipi_dsi_*_write_seq()

2024-05-13 Thread Doug Anderson
r > > mipi_dsi_dcs_write_seq(), let's also print the buffer for > > mipi_dsi_generic_write_seq() in the error case. > > > > It should be noted that the current consensus of DRM folks is that the > > mipi_dsi_*_write_seq() should be deprecated due to the non-intuitive > > re

Re: [PATCH 01/20] drm/drm_managed: try to improve the drmm DOC

2024-05-13 Thread Rodrigo Vivi
On Fri, May 10, 2024 at 07:12:14PM +0100, Matthew Auld wrote: > Hopefully make it clearer when to use devm vs drmm. > > Signed-off-by: Matthew Auld > Cc: Daniel Vetter > Cc: dri-devel@lists.freedesktop.org > --- > drivers/gpu/drm/drm_managed.c | 42 +++ > 1 file

Patch "drm/vmwgfx: Fix invalid reads in fence signaled events" has been added to the 6.1-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/vmwgfx: Fix invalid reads in fence signaled events to the 6.1-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

[PATCH v2 12/12] drm/bridge: analogix: Remove redundant checks on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
The checks on the existence of bridge->encoder in the implementation of drm_bridge_funcs::attach() is not necessary, as it has already been checked in the drm_bridge_attach() function call by previous bridge or KMS driver. The drm_bridge_attach() will quit with a negative error code returned if it

[PATCH v2 11/12] drm/bridge: imx: Remove redundant checks on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
The checks on the existence of bridge->encoder in the implementation of drm_bridge_funcs::attach() is not necessary, as it has already been checked in the drm_bridge_attach() function call by previous bridge or KMS driver. The drm_bridge_attach() will quit with a negative error code returned if it

[PATCH v2 10/12] drm/bridge: lt9611uxc: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
In the lt9611uxc_connector_init() function, the check on the existence of bridge->encoder is not necessary, as it has already been checked in the drm_bridge_attach() function. And the check on the drm bridge core happens before check in the implementation. Hence, it is guaranteed that the .encoder

[PATCH v2 09/12] drm/bridge: synopsys: dw-mipi-dsi: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
In the dw_mipi_dsi_bridge_attach() function, the check on the existence of bridge->encoder is not necessary, as it has already been checked in the drm_bridge_attach() function invocked by previous bridge or KMS driver. The previous drm_bridge_attach() will quit with a negative error code returned

[PATCH v2 08/12] drm/bridge: megachips-stdpxxxx-ge-b850v3-fw: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
In the ge_b850v3_lvds_create_connector function, the check on the existence of bridge->encoder is not necessary, as it has already been checked in the drm_bridge_attach() function called by upstream bridge or driver. Hence, it is guaranteed that the .encoder member of the drm_bridge instance is

[PATCH v2 07/12] drm/bridge: cdns-mhdp8546: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
In the cdns_mhdp_connector_init() function, the check on the existence of bridge->encoder is not necessary, as it has already been checked in the drm_bridge_attach() function. As the cdns_mhdp_connector_init() is only called by cdns_mhdp_attach(), it is guaranteed that the .encoder member of the

[PATCH v2 06/12] drm/bridge: adv7511: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
In the adv7511_connector_init() function, the check on the existence of bridge->encoder is not necessary. As it has already been checked in the drm_bridge_attach() which happens prior to the adv7511_bridge_attach() get called. Also note that the adv7511_connector_init() is only called by

[PATCH v2 05/12] drm/bridge: it6505: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
In it6505_bridge_attach(), the check on the existence of 'bridge->encoder' is not necessary, as it has already been checked in the drm_bridge_attach() which happens prior to it6505_bridge_attach() get called. Note that the it6505_bridge_attach() will only be called by .attach() of the previous

[PATCH v2 04/12] drm/bridge: panel: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
Because the existence of 'bridge->encoder' has already been checked before the panel_bridge_attach() function get called, and the drm_bridge_attach() will quit with a negative error code returned if it fails for some reasons. Hence, it is guaranteed that the .encoder member of the drm_bridge

[PATCH v2 03/12] drm/bridge: nxp-ptn3460: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
Because the existence of 'bridge->encoder' has already been checked before the ptn3460_bridge_attach() function get called, and drm_bridge_attach() will quit with a negative error code returned if it fails for some reasons. Hence, it is guaranteed that the .encoder member of the drm_bridge

[PATCH v2 01/12] drm/bridge: simple-bridge: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
Because the existence of 'bridge->encoder' has already been checked before the simple_bridge_attach() function get called, and drm_bridge_attach() will quit with a negative error code returned if it fails for some reasons. Hence, it is guaranteed that the .encoder member of the drm_bridge instance

[PATCH v2 02/12] drm/bridge: tfp410: Remove a redundant check on existence of bridge->encoder

2024-05-13 Thread Sui Jingfeng
Because the existence of bridge->encoder has already been checked before the simple_bridge_attach() function get called, And drm_bridge_attach() will quit with a negative error code returned if it fails for some reasons. Hence, it is guaranteed that the .encoder member of the drm_bridge instance

[PATCH v2 00/12] Remove redundant checks on existence of 'bridge->encoder'

2024-05-13 Thread Sui Jingfeng
The checks on the existence of bridge->encoder in the implementation of drm_bridge_funcs::attach() is not necessary, as it has already been checked in the drm_bridge_attach() function call by previous bridge or KMS driver. The drm_bridge_attach() will quit with a negative error code returned if it

Patch "drm/vmwgfx: Fix invalid reads in fence signaled events" has been added to the 5.15-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/vmwgfx: Fix invalid reads in fence signaled events to the 5.15-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

Patch "drm/vmwgfx: Fix invalid reads in fence signaled events" has been added to the 5.10-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/vmwgfx: Fix invalid reads in fence signaled events to the 5.10-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

Patch "drm/vmwgfx: Fix invalid reads in fence signaled events" has been added to the 5.4-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/vmwgfx: Fix invalid reads in fence signaled events to the 5.4-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

Patch "drm/vmwgfx: Fix invalid reads in fence signaled events" has been added to the 4.19-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/vmwgfx: Fix invalid reads in fence signaled events to the 4.19-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

Patch "drm/vmwgfx: Fix invalid reads in fence signaled events" has been added to the 6.6-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/vmwgfx: Fix invalid reads in fence signaled events to the 6.6-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

Patch "drm/ttm: Print the memory decryption status just once" has been added to the 6.6-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/ttm: Print the memory decryption status just once to the 6.6-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

Patch "drm/vmwgfx: Fix invalid reads in fence signaled events" has been added to the 6.8-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/vmwgfx: Fix invalid reads in fence signaled events to the 6.8-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

Patch "drm/ttm: Print the memory decryption status just once" has been added to the 6.8-stable tree

2024-05-13 Thread gregkh
This is a note to let you know that I've just added the patch titled drm/ttm: Print the memory decryption status just once to the 6.8-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch

Re: [PATCH v2 2/2] drm/tests: Add a unit test for range bias allocation

2024-05-13 Thread Matthew Auld
On 13/05/2024 16:11, Paneer Selvam, Arunpravin wrote: Hi Matthew, On 5/13/2024 1:49 PM, Matthew Auld wrote: On 12/05/2024 08:59, Arunpravin Paneer Selvam wrote: Allocate cleared blocks in the bias range when the DRM buddy's clear avail is zero. This will validate the bias range allocation in

Re: [PATCH] nouveau/firmware: using dma non-coherent interfaces for fw loading.

2024-05-13 Thread kernel test robot
Hi Dave, kernel test robot noticed the following build errors: [auto build test ERROR on drm-misc/drm-misc-next] [also build test ERROR on drm-tip/drm-tip linus/master v6.9 next-20240513] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we

Re: [PATCH v2 2/2] drm/tests: Add a unit test for range bias allocation

2024-05-13 Thread Paneer Selvam, Arunpravin
Hi Matthew, On 5/13/2024 1:49 PM, Matthew Auld wrote: On 12/05/2024 08:59, Arunpravin Paneer Selvam wrote: Allocate cleared blocks in the bias range when the DRM buddy's clear avail is zero. This will validate the bias range allocation in scenarios like system boot when no cleared blocks are

Re: [PATCH 1/2] drm: bridge: samsung-dsim: Initialize bridge on attach

2024-05-13 Thread Marek Vasut
callback, and recovered out of the debug mode just before TC9595 performs first AUX channel access later in its attach callback. Signed-off-by: Marek Vasut This patch breaks driver operation on Samsung TM2e board with S6E3HF2 DSI panel. The initialization procedure is very fragile and it looks

Re: [PATCH v3 05/10] media: dt-bindings: video-interfaces: Support DisplayPort MST

2024-05-13 Thread Rob Herring (Arm)
On Tue, 07 May 2024 15:54:08 +, Paweł Anikiel wrote: > Add a DisplayPort bus type and a multi-stream-support property > indicating whether the interface supports MST. > > Signed-off-by: Paweł Anikiel > --- > .../devicetree/bindings/media/video-interfaces.yaml| 7 +++ >

[PATCH v2] drm/i915: Fix memory leak by correcting cache object name in error handler

2024-05-13 Thread Jiasheng Jiang
Replace "slab_priorities" with "slab_dependencies" in the error handler to avoid memory leak. Fixes: 32eb6bcfdda9 ("drm/i915: Make request allocation caches global") Cc: # v5.2+ Reviewed-by: Nirmoy Das Signed-off-by: Jiasheng Jiang --- Changelog: v1 -> v2: 1. Alter the subject. ---

[PATCH v2] drm/i915: Fix memory leak by correcting cache object name in error handler

2024-05-13 Thread Jiasheng Jiang
Replace "slab_priorities" with "slab_dependencies" in the error handler to avoid memory leak. Fixes: 32eb6bcfdda9 ("drm/i915: Make request allocation caches global") Signed-off-by: Jiasheng Jiang --- Changelog: v1 -> v2: 1. Alter the subject. --- drivers/gpu/drm/i915/i915_scheduler.c | 2 +-

Re: [PATCH v2 1/2] Add write DP phyd register from parse dts

2024-05-13 Thread 杨连坤

[PATCH] drm/i915: Correct error handler

2024-05-13 Thread Jiasheng Jiang
> On 5/11/2024 5:48 PM, Jiasheng Jiang wrote: >> Replace "slab_priorities" with "slab_dependencies" in the error handler to >> avoid memory leak. > > Nice catch. I would make the subject more like: > > drm/i915: Fix memory leak by correcting cache object name in error handler > >> >> Fixes:

Re: [PATCH v2 1/2] Add write DP phyd register from parse dts

2024-05-13 Thread 杨连坤

Re: [PATCH v2 2/2] Add dp PHY dt-bindings

2024-05-13 Thread Krzysztof Kozlowski
t; >>> Please reach internally to Mediatek or collaborating companies to get >>> basic training and instructions how to write patches and bindings. >>> >>> Otherwise it is waste of our time. Mediatek is not a small company so >>> there is no excuse in sending s

Re: [PATCH v2 2/2] Add dp PHY dt-bindings

2024-05-13 Thread Krzysztof Kozlowski
nkun Yang wrote: >>> Add dp PHY dt-bindings. >> >> This tells me nothing. Read submitting patches. >> >>> >>> Changeds in v2: >> >> This does goes to changelog, not commit log. >> >>> - Add dp PHY dt-bindings. >>> ht

Re: [PATCH v2 2/2] Add dp PHY dt-bindings

2024-05-13 Thread 杨连坤

[PATCH] drm/i915: Correct error handler

2024-05-13 Thread Jiasheng Jiang
> On 5/11/2024 5:48 PM, Jiasheng Jiang wrote: >> Replace "slab_priorities" with "slab_dependencies" in the error handler to >> avoid memory leak. > > Nice catch. I would make the subject more like: > > drm/i915: Fix memory leak by correcting cache object name in error handler > >> >> Fixes:

Re: [PATCH] i2c: mux: Remove class argument from i2c_mux_add_adapter()

2024-05-13 Thread Wolfram Sang
On Thu, Apr 18, 2024 at 10:55:39PM +0200, Heiner Kallweit wrote: > 99a741aa7a2d ("i2c: mux: gpio: remove support for class-based device > instantiation") removed the last call to i2c_mux_add_adapter() with a > non-null class argument. Therefore the class argument can be removed. > > Note:

Re: [PATCH v2 2/2] Add dp PHY dt-bindings

2024-05-13 Thread 杨连坤

Re: [PATCH v2 0/2] Add PHY-dp bindings

2024-05-13 Thread 杨连坤

Re: [PATCH v1 1/1] Support YUV422 for DPTX.

2024-05-13 Thread 杨连坤

Re: [RFC PATCH 00/18] TTM interface for managing VRAM oversubscription

2024-05-13 Thread Friedrich Vock
Hi, On 02.05.24 16:23, Maarten Lankhorst wrote: Hey, [snip] For Xe, I've been loking at using cgroups. A small prototype is available at https://cgit.freedesktop.org/~mlankhorst/linux/log/?h=dumpcg To stimulate discussion, I've added amdgpu support as well. This should make it possible to

Re: [PATCH v2 2/3] dt-bindings: arm: mediatek: mmsys: Add OF graph support for board path

2024-05-13 Thread AngeloGioacchino Del Regno
Il 13/05/24 08:15, CK Hu (胡俊光) ha scritto: On Fri, 2024-05-10 at 12:14 +0200, AngeloGioacchino Del Regno wrote: Il 10/05/24 11:34, CK Hu (胡俊光) ha scritto: On Thu, 2024-05-09 at 11:27 +0200, AngeloGioacchino Del Regno wrote: Il 09/05/24 07:42, CK Hu (胡俊光) ha scritto: On Wed, 2024-05-08 at

Re: [PATCH 0/3] dt-bindings: display: panel: constrain 'reg'

2024-05-13 Thread Rob Herring
On Thu, May 09, 2024 at 11:42:50AM +0200, Krzysztof Kozlowski wrote: > Hi, > > Cleanups for display panel bindings. > > Rob, maybe you could take entire set if it applies? I based it on > linux-next, so letl me know if I need to rebase on your for-next. Applied. These 2 don't exist in my tree:

Re: [PATCH v7 6/8] math.h Add macros to round to closest specified power of 2

2024-05-13 Thread Andy Shevchenko
On Mon, May 13, 2024 at 06:34:19PM +0530, Devarsh Thakkar wrote: > On 13/05/24 17:55, Andy Shevchenko wrote: > > On Mon, May 13, 2024 at 04:55:58PM +0530, Devarsh Thakkar wrote: > >> On 13/05/24 14:29, Andy Shevchenko wrote: > >>> On Sat, May 11, 2024 at 11:11:14PM +0530, Devarsh Thakkar wrote: >

Re: [PATCH v2 1/2] Add write DP phyd register from parse dts

2024-05-13 Thread AngeloGioacchino Del Regno
by adjusting the SSC gear. Obtaining the DPTX node, parsing the dts to obtain PHY register address and value can adapt to settings of different manufacturers projects. Changeds in v2: - Optimized method of writing to DP PHY register https://patchwork.kernel.org/project/linux-mediatek/patch

Re: [PATCH v7 6/8] math.h Add macros to round to closest specified power of 2

2024-05-13 Thread Devarsh Thakkar
know if you have any other suggestions for naming? > - add examples into commit message of the math.h patch Agreed > - add test cases (you need to create lib/math_kunit.c for that) Agreed. > - elaborate in the commit message of the IPU3 change what you posted above > (some kind of a summ

Re: [PATCH v2 2/2] drm/panel/lg-sw43408: mark sw43408_backlight_ops as static

2024-05-13 Thread Neil Armstrong
On 09/05/2024 23:20, Dmitry Baryshkov wrote: Fix sparse warning regarding symbol 'sw43408_backlight_ops' not being declared. Reported-by: kernel test robot Closes: https://lore.kernel.org/oe-kbuild-all/202404200739.hbwzvohr-...@intel.com/ Signed-off-by: Dmitry Baryshkov ---

[PATCH 04/10] drm/mgag200: Allocate instance of struct mga_i2c_chan dynamically

2024-05-13 Thread Thomas Zimmermann
Allocate instances of struct mga_i2c_chan in mgag200_ddc_create() and return a pointer to the contained i2c adapter. The callers of the function are now independent from struct mga_i2c_chan. Signed-off-by: Thomas Zimmermann --- drivers/gpu/drm/mgag200/mgag200_ddc.h | 11 +++

<    1   2   3   4   5   6   7   8   9   10   >