Re: [RFC PATCH 03/10] drm/mipi-dsi: add API for manual control over the DSI link power state

2023-11-28 Thread Michael Walle
I'm facing similar issues with the tc358775 bridge. This bridge needs to release its reset while both clock and data lanes are in LP-11 mode. But then it needs to be configured (via I2C) while the clock lane is in enabled (HS mode), but the data lanes are still in LP-11 mode. This is quite an

[PATCH] drm/imagination: DRM_POWERVR should depend on ARCH_K3

2023-11-28 Thread Geert Uytterhoeven
The Imagination Technologies PowerVR Series 6 GPU is currently only supported on Texas Instruments K3 AM62x SoCs. Hence add a dependency on ARCH_K3, to prevent asking the user about this driver when configuring a kernel without Texas Instruments K3 Multicore SoC support. Fixes: 4babef0708656c54

Re: [PATCH] drm/msm/dpu: Capture dpu snapshot when frame_done_timer timeouts

2023-11-28 Thread Dmitry Baryshkov
On Tue, 28 Nov 2023 at 19:43, Paloma Arellano wrote: > > > On 11/27/2023 5:48 PM, Dmitry Baryshkov wrote: > > On Tue, 28 Nov 2023 at 03:12, Paloma Arellano > > wrote: > >> Trigger a devcoredump to dump dpu registers and capture the drm atomic > >> state when the frame_done_timer timeouts. > >>

Re: (subset) [PATCH 00/17] dt-bindings: samsung: add specific compatibles for existing SoC

2023-11-28 Thread Uwe Kleine-König
On Tue, Nov 28, 2023 at 06:49:23PM +0100, Thierry Reding wrote: > > On Wed, 08 Nov 2023 11:43:26 +0100, Krzysztof Kozlowski wrote: > > Merging > > === > > I propose to take entire patchset through my tree (Samsung SoC), because:

Re: [PATCH v18 04/26] drm/shmem-helper: Refactor locked/unlocked functions

2023-11-28 Thread Dmitry Osipenko
On 11/28/23 15:37, Boris Brezillon wrote: > On Tue, 28 Nov 2023 12:14:42 +0100 > Maxime Ripard wrote: > >> Hi, >> >> On Fri, Nov 24, 2023 at 11:59:11AM +0100, Boris Brezillon wrote: >>> On Fri, 24 Nov 2023 11:40:06 +0100 >>> Maxime Ripard wrote: >>> On Mon, Oct 30, 2023 at 02:01:43AM

Re: Radeon regression in 6.6 kernel

2023-11-28 Thread Alex Deucher
On Mon, Nov 27, 2023 at 6:24 PM Phillip Susi wrote: > > Alex Deucher writes: > > >> In that case those are the already known problems with the scheduler > >> changes, aren't they? > > > > Yes. Those changes went into 6.7 though, not 6.6 AFAIK. Maybe I'm > > misunderstanding what the original

Re: [RFC PATCH 03/10] drm/mipi-dsi: add API for manual control over the DSI link power state

2023-11-28 Thread Dmitry Baryshkov
On Wed, 29 Nov 2023 at 00:20, Michael Walle wrote: > > [sorry I fat fingered my former reply and converted all CCs to BCCs..] > > >> >> >> > DSI device lifetime has three different stages: > >> >> >> > 1. before the DSI link being powered up and clocking, > >> >> >> > 2. when the DSI link is in

Re: [PATCH v7 3/3] drm/panel-edp: Avoid adding multiple preferred modes

2023-11-28 Thread Doug Anderson
Hi, On Fri, Nov 17, 2023 at 1:51 PM Hsin-Yi Wang wrote: > > If a non generic edp-panel is under aux-bus, the mode read from edid would > still be selected as preferred and results in multiple preferred modes, > which is ambiguous. > > If both hard-coded mode and edid exists, only add mode from

Re: [PATCH v7 2/3] drm/panel-edp: Add auo_b116xa3_mode

2023-11-28 Thread Doug Anderson
Hi, On Fri, Nov 17, 2023 at 1:51 PM Hsin-Yi Wang wrote: > > Add auo_b116xa3_mode to override the original modes parsed from edid > of the panels 0x405c B116XAK01.0 and 0x615c B116XAN06.1 which result > in glitches on panel. > > Signed-off-by: Hsin-Yi Wang > --- > v6->v7: split usecase to

Re: [PATCH v7 1/3] drm/panel-edp: Add override_edid_mode quirk for generic edp

2023-11-28 Thread Doug Anderson
Hi, On Fri, Nov 17, 2023 at 1:51 PM Hsin-Yi Wang wrote: > > Generic edp gets mode from edid. However, some panels report incorrect > mode in this way, resulting in glitches on panel. Introduce a new quirk > additional_mode to the generic edid to pick a correct hardcoded mode. > > Signed-off-by:

Re: [PATCH] drm/imagination: DRM_POWERVR should depend on ARCH_K3

2023-11-28 Thread Geert Uytterhoeven
Hi Javier, On Tue, Nov 28, 2023 at 8:03 PM Javier Martinez Canillas wrote: > Geert Uytterhoeven writes: > > The Imagination Technologies PowerVR Series 6 GPU is currently only > > supported on Texas Instruments K3 AM62x SoCs. Hence add a dependency on > > ARCH_K3, to prevent asking the user

Re: [RFC PATCH 03/10] drm/mipi-dsi: add API for manual control over the DSI link power state

2023-11-28 Thread Michael Walle
>> >> > DSI device lifetime has three different stages: >> >> > 1. before the DSI link being powered up and clocking, >> >> > 2. when the DSI link is in LP state (for the purpose of this question, >> >> > this is the time between the DSI link being powered up and the video >> >> > stream start) >>

Re: Radeon regression in 6.6 kernel

2023-11-28 Thread Luben Tuikov
On 2023-11-28 17:13, Alex Deucher wrote: > On Mon, Nov 27, 2023 at 6:24 PM Phillip Susi wrote: >> >> Alex Deucher writes: >> In that case those are the already known problems with the scheduler changes, aren't they? >>> >>> Yes. Those changes went into 6.7 though, not 6.6 AFAIK.

Re: [RFC PATCH 0/6] Supporting GMEM (generalized memory management) for external memory devices

2023-11-28 Thread Dave Airlie
On Tue, 28 Nov 2023 at 23:07, Christian König wrote: > > Am 28.11.23 um 13:50 schrieb Weixi Zhu: > > The problem: > > > > Accelerator driver developers are forced to reinvent external MM subsystems > > case by case, because Linux core MM only considers host memory resources. > > These reinvented

Re: [PATCH 07/10] acpi: Do not return struct iommu_ops from acpi_iommu_configure_id()

2023-11-28 Thread Moritz Fischer
On Tue, Nov 28, 2023 at 08:48:03PM -0400, Jason Gunthorpe wrote: Nothing needs this pointer. Return a normal error code with the usual IOMMU semantic that ENODEV means 'there is no IOMMU driver'. Acked-by: Rafael J. Wysocki Reviewed-by: Jerry Snitselaar Tested-by: Hector Martin

Re: [PATCH 09/10] ACPI: IORT: Cast from ULL to phys_addr_t

2023-11-28 Thread Moritz Fischer
On Tue, Nov 28, 2023 at 08:48:05PM -0400, Jason Gunthorpe wrote: gcc on i386 (when compile testing) warns: This is a weird test. The Makefile for drivers/acpi/arm64 is conditional on CONFIG_ARM64. How does this happen? 8->8 obj-$(CONFIG_ARM64) += arm64/ 8->8

Re: [REGRESSION]: nouveau: Asynchronous wait on fence

2023-11-28 Thread Owen T. Heisler
On 11/21/23 14:23, Owen T. Heisler wrote: On 11/21/23 09:16, Linux regression tracking (Thorsten Leemhuis) wrote: On 15.11.23 07:19, Owen T. Heisler wrote: On 10/31/23 04:18, Linux regression tracking (Thorsten Leemhuis) wrote: On 28.10.23 04:46, Owen T. Heisler wrote: #regzbot introduced:

[PATCH 10/10] ACPI: IORT: Allow COMPILE_TEST of IORT

2023-11-28 Thread Jason Gunthorpe
The arm-smmu driver can COMPILE_TEST on x86, so expand this to also enable the IORT code so it can be COMPILE_TEST'd too. Signed-off-by: Jason Gunthorpe --- drivers/acpi/Kconfig| 2 -- drivers/acpi/Makefile | 2 +- drivers/acpi/arm64/Kconfig | 1 + drivers/acpi/arm64/Makefile | 2

[PATCH 05/10] iommu: Mark dev_iommu_priv_set() with a lockdep

2023-11-28 Thread Jason Gunthorpe
A perfect driver would only call dev_iommu_priv_set() from its probe callback. We've made it functionally correct to call it from the of_xlate by adding a lock around that call. lockdep assert that iommu_probe_device_lock is held to discourage misuse. Exclude PPC kernels with CONFIG_FSL_PAMU

[PATCH 09/10] ACPI: IORT: Cast from ULL to phys_addr_t

2023-11-28 Thread Jason Gunthorpe
gcc on i386 (when compile testing) warns: drivers/acpi/arm64/iort.c:2014:18: warning: implicit conversion from 'unsigned long long' to 'phys_addr_t' (aka 'unsigned int') changes value from 18446744073709551615 to 4294967295 [-Wconstant-conversion] local_limit =

[PATCH 03/10] iommu/of: Use -ENODEV consistently in of_iommu_configure()

2023-11-28 Thread Jason Gunthorpe
Instead of returning 1 and trying to handle positive error codes just stick to the convention of returning -ENODEV. Remove references to ops from of_iommu_configure(), a NULL ops will already generate an error code. There is no reason to check dev->bus, if err=0 at this point then the called

[PATCH 02/10] iommmu/of: Do not return struct iommu_ops from of_iommu_configure()

2023-11-28 Thread Jason Gunthorpe
Nothing needs this pointer. Return a normal error code with the usual IOMMU semantic that ENODEV means 'there is no IOMMU driver'. Reviewed-by: Jerry Snitselaar Acked-by: Rob Herring Tested-by: Hector Martin Signed-off-by: Jason Gunthorpe --- drivers/iommu/of_iommu.c | 31

[PATCH 04/10] iommu: Mark dev_iommu_get() with lockdep

2023-11-28 Thread Jason Gunthorpe
Allocation of dev->iommu must be done under the iommu_probe_device_lock. Mark this with lockdep to discourage future mistakes. Reviewed-by: Jerry Snitselaar Tested-by: Hector Martin Signed-off-by: Jason Gunthorpe --- drivers/iommu/iommu.c | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH 08/10] iommu/tegra: Use tegra_dev_iommu_get_stream_id() in the remaining places

2023-11-28 Thread Jason Gunthorpe
This API was defined to formalize the access to internal iommu details on some Tegra SOCs, but a few callers got missed. Add them. The helper already masks by 0x so remove this code from the callers. Suggested-by: Thierry Reding Signed-off-by: Jason Gunthorpe ---

[PATCH 06/10] iommu: Replace iommu_device_lock with iommu_probe_device_lock

2023-11-28 Thread Jason Gunthorpe
The iommu_device_lock protects the iommu_device_list which is only read by iommu_ops_from_fwnode(). This is now always called under the iommu_probe_device_lock, so we don't need to double lock the linked list. Use the iommu_probe_device_lock on the write side too. Signed-off-by: Jason Gunthorpe

Re: [Nouveau] [PATCH][next] nouveau/gsp: replace zero-length array with flex-array member and use __counted_by

2023-11-28 Thread Danilo Krummrich
On 11/16/23 20:55, Timur Tabi wrote: On Thu, 2023-11-16 at 20:45 +0100, Danilo Krummrich wrote: As I already mentioned for Timur's patch [2], I'd prefer to get a fix upstream (meaning [1] in this case). Of course, that's probably more up to Timur to tell if this will work out. Don't count on

Friendly ping. I think this patch was forgotten.//回复: [PATCH] drm/qxl: remove unused declaration

2023-11-28 Thread 何敏红
Friendly ping. I think this patch was forgotten.   主 题:[PATCH] drm/qxl: remove unused declaration 日 期:2023-11-10 13:50 发件人:何敏红 收件人:airlied;kraxel;maarten.lankhorst;mripard;tzimmermann;airlied;daniel; Some functions are never used by the driver,removing the functions declaration, it can

Re: [PATCH 04/10] iommu: Mark dev_iommu_get() with lockdep

2023-11-28 Thread Moritz Fischer
On Tue, Nov 28, 2023 at 08:48:00PM -0400, Jason Gunthorpe wrote: Allocation of dev->iommu must be done under the iommu_probe_device_lock. Mark this with lockdep to discourage future mistakes. Reviewed-by: Jerry Snitselaar Tested-by: Hector Martin Signed-off-by: Jason Gunthorpe ---

Re: [PATCH 10/10] ACPI: IORT: Allow COMPILE_TEST of IORT

2023-11-28 Thread Moritz Fischer
On Tue, Nov 28, 2023 at 08:48:06PM -0400, Jason Gunthorpe wrote: The arm-smmu driver can COMPILE_TEST on x86, so expand this to also enable the IORT code so it can be COMPILE_TEST'd too. Signed-off-by: Jason Gunthorpe --- drivers/acpi/Kconfig| 2 -- drivers/acpi/Makefile | 2

Re: [PATCH v2 05/12] drm/rockchip: vop2: Set YUV/RGB overlay mode

2023-11-28 Thread Andy Yan
Hi Sasha: On 11/27/23 22:16, Sascha Hauer wrote: On Wed, Nov 22, 2023 at 08:54:38PM +0800, Andy Yan wrote: From: Andy Yan Set overlay mode register according to the output mode is yuv or rgb. Signed-off-by: Andy Yan --- (no changes since v1) drivers/gpu/drm/rockchip/rockchip_drm_drv.h

Re: [PATCH v3 10/17] drm/v3d: Detach the CSD job BO setup

2023-11-28 Thread Iago Toral
El mar, 28-11-2023 a las 07:47 -0300, Maira Canal escribió: > Hi Iago, > > On 11/28/23 05:42, Iago Toral wrote: > > El lun, 27-11-2023 a las 15:48 -0300, Maíra Canal escribió: > > > From: Melissa Wen > > > > > > Detach CSD job setup from CSD submission ioctl to reuse it in CPU > > > submission

Re: [PATCH v18 04/26] drm/shmem-helper: Refactor locked/unlocked functions

2023-11-28 Thread Boris Brezillon
On Wed, 29 Nov 2023 01:05:14 +0300 Dmitry Osipenko wrote: > On 11/28/23 15:37, Boris Brezillon wrote: > > On Tue, 28 Nov 2023 12:14:42 +0100 > > Maxime Ripard wrote: > > > >> Hi, > >> > >> On Fri, Nov 24, 2023 at 11:59:11AM +0100, Boris Brezillon wrote: > >>> On Fri, 24 Nov 2023 11:40:06

Re: [PATCH 03/10] iommu/of: Use -ENODEV consistently in of_iommu_configure()

2023-11-28 Thread Moritz Fischer
On Tue, Nov 28, 2023 at 08:47:59PM -0400, Jason Gunthorpe wrote: Instead of returning 1 and trying to handle positive error codes just stick to the convention of returning -ENODEV. Remove references to ops from of_iommu_configure(), a NULL ops will already generate an error code. There is no

Re: [PATCH 3/3] drm/bridge: ti-sn65dsi86: Loosen coupling of PWM to ti-sn65dsi86 core

2023-11-28 Thread Laurent Pinchart
On Tue, Nov 28, 2023 at 04:32:10PM -0800, Doug Anderson wrote: > On Thu, Nov 23, 2023 at 9:54 AM Uwe Kleine-König wrote: > > > > Introduce a dedicated private data structure for the pwm aux driver > > provided by the sn65dsi86 driver. This way data needed for PWM operation > > is (to a certain

Re: [PATCH 2/3] drm/bridge: ti-sn65dsi86: Change parameters of ti_sn65dsi86_{read,write}_u16

2023-11-28 Thread Laurent Pinchart
On Tue, Nov 28, 2023 at 04:34:30PM -0800, Doug Anderson wrote: > On Thu, Nov 23, 2023 at 9:54 AM Uwe Kleine-König wrote: > > > > This aligns the function's parameters to regmap_{read,write} and > > simplifies the next change that takes pwm driver data out of struct > > ti_sn65dsi86. > > > >

Re: [PATCH 0/3] drm/bridge: ti-sn65dsi86: Some updates

2023-11-28 Thread Laurent Pinchart
Hello, On Fri, Nov 24, 2023 at 09:56:55AM +0100, Neil Armstrong wrote: > On 23/11/2023 18:54, Uwe Kleine-König wrote: > > Hello, > > > > this is a series I created while starring at the ti-sn65dsi86 driver in > > the context of my pwm-lifetime series. > > > > The first patch should be fine. The

Re: [PATCH 1/3] drm/bridge: ti-sn65dsi86: Simplify using pm_runtime_resume_and_get()

2023-11-28 Thread Doug Anderson
Hi, On Thu, Nov 23, 2023 at 9:54 AM Uwe Kleine-König wrote: > > pm_runtime_resume_and_get() already drops the runtime PM usage counter > in the error case. So a call to pm_runtime_put_sync() can be dropped. > > Signed-off-by: Uwe Kleine-König > --- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 6

Re: [Nouveau] [PATCH][next] nouveau/gsp: replace zero-length array with flex-array member and use __counted_by

2023-11-28 Thread Gustavo A. R. Silva
On 11/28/23 19:01, Danilo Krummrich wrote: On 11/16/23 20:55, Timur Tabi wrote: On Thu, 2023-11-16 at 20:45 +0100, Danilo Krummrich wrote: As I already mentioned for Timur's patch [2], I'd prefer to get a fix upstream (meaning [1] in this case). Of course, that's probably more up to Timur

Re: [PATCH v2 1/6] dt-bindings: display: Add yamls for JH7110 display system

2023-11-28 Thread Keith Zhao
On 2023/10/25 20:50, Krzysztof Kozlowski wrote: > On 25/10/2023 12:39, Keith Zhao wrote: >> StarFive SoCs JH7110 display system: > > A nit, subject: drop second/last, redundant "yamls for". The > "dt-bindings" prefix is already stating that these are bindings, so > format is fixed. > >>

Re: [PATCH 04/10] iommu: Mark dev_iommu_get() with lockdep

2023-11-28 Thread Baolu Lu
On 2023/11/29 8:48, Jason Gunthorpe wrote: Allocation of dev->iommu must be done under the iommu_probe_device_lock. Mark this with lockdep to discourage future mistakes. Reviewed-by: Jerry Snitselaar Tested-by: Hector Martin Signed-off-by: Jason Gunthorpe --- drivers/iommu/iommu.c | 2 ++ 1

Re: [PATCH 2/3] drm/bridge: ti-sn65dsi86: Change parameters of ti_sn65dsi86_{read, write}_u16

2023-11-28 Thread Doug Anderson
Hi, On Thu, Nov 23, 2023 at 9:54 AM Uwe Kleine-König wrote: > > This aligns the function's parameters to regmap_{read,write} and > simplifies the next change that takes pwm driver data out of struct > ti_sn65dsi86. > > Signed-off-by: Uwe Kleine-König > --- >

[PATCH 01/10] iommu: Remove struct iommu_ops *iommu from arch_setup_dma_ops()

2023-11-28 Thread Jason Gunthorpe
This is not being used to pass ops, it is just a way to tell if an iommu driver was probed. These days this can be detected directly via device_iommu_mapped(). Call device_iommu_mapped() in the two places that need to check it and remove the iommu parameter everywhere. Reviewed-by: Jerry

[PATCH 07/10] acpi: Do not return struct iommu_ops from acpi_iommu_configure_id()

2023-11-28 Thread Jason Gunthorpe
Nothing needs this pointer. Return a normal error code with the usual IOMMU semantic that ENODEV means 'there is no IOMMU driver'. Acked-by: Rafael J. Wysocki Reviewed-by: Jerry Snitselaar Tested-by: Hector Martin Signed-off-by: Jason Gunthorpe --- drivers/acpi/scan.c | 29

[PATCH 00/10] IOMMU related FW parsing cleanup

2023-11-28 Thread Jason Gunthorpe
These are the patches from the from the prior series without the "fwspec polishing": https://lore.kernel.org/r/0-v2-36a0088ecaa7+22c6e-iommu_fwspec_...@nvidia.com Rebased onto Robin's patch:

回复: [PATCH v2] drm/i915: correct the input parameter on _intel_dsb_commit()

2023-11-28 Thread 何敏红
Friendly ping. I think this patch was forgotten.   主 题:[PATCH v2] drm/i915: correct the input parameter on _intel_dsb_commit() 日 期:2023-11-14 10:43 发件人:何敏红 收件人:何敏红; Current, the dewake_scanline variable is defined as unsigned int,an unsigned int variable that is always greater than or

Re: [PATCH 3/3] drm/bridge: ti-sn65dsi86: Loosen coupling of PWM to ti-sn65dsi86 core

2023-11-28 Thread Doug Anderson
Hi, On Thu, Nov 23, 2023 at 9:54 AM Uwe Kleine-König wrote: > > Introduce a dedicated private data structure for the pwm aux driver > provided by the sn65dsi86 driver. This way data needed for PWM operation > is (to a certain degree) nicely separated and doesn't occupy memory in > the

Re: [PATCH 1/3] drm/bridge: ti-sn65dsi86: Simplify using pm_runtime_resume_and_get()

2023-11-28 Thread Laurent Pinchart
Hi Uwe, Thank you for the patch. On Thu, Nov 23, 2023 at 06:54:27PM +0100, Uwe Kleine-König wrote: > pm_runtime_resume_and_get() already drops the runtime PM usage counter > in the error case. So a call to pm_runtime_put_sync() can be dropped. > > Signed-off-by: Uwe Kleine-König I wonder if

Re: [Nouveau] [PATCH][next] nouveau/gsp: replace zero-length array with flex-array member and use __counted_by

2023-11-28 Thread Danilo Krummrich
On 11/29/23 02:06, Gustavo A. R. Silva wrote: On 11/28/23 19:01, Danilo Krummrich wrote: On 11/16/23 20:55, Timur Tabi wrote: On Thu, 2023-11-16 at 20:45 +0100, Danilo Krummrich wrote: As I already mentioned for Timur's patch [2], I'd prefer to get a fix upstream (meaning [1] in this case).

Re: [PATCH 07/10] acpi: Do not return struct iommu_ops from acpi_iommu_configure_id()

2023-11-28 Thread Baolu Lu
On 2023/11/29 8:48, Jason Gunthorpe wrote: Nothing needs this pointer. Return a normal error code with the usual IOMMU semantic that ENODEV means 'there is no IOMMU driver'. Acked-by: Rafael J. Wysocki Reviewed-by: Jerry Snitselaar Tested-by: Hector Martin Signed-off-by: Jason Gunthorpe ---

Re: [PATCH 02/10] iommmu/of: Do not return struct iommu_ops from of_iommu_configure()

2023-11-28 Thread Baolu Lu
On 2023/11/29 8:47, Jason Gunthorpe wrote: Nothing needs this pointer. Return a normal error code with the usual IOMMU semantic that ENODEV means 'there is no IOMMU driver'. Reviewed-by: Jerry Snitselaar Acked-by: Rob Herring Tested-by: Hector Martin Signed-off-by: Jason Gunthorpe ---

Re: [PATCH] nouveau/gsp/r535: remove a stray unlock in r535_gsp_rpc_send()

2023-11-28 Thread Danilo Krummrich
On 11/27/23 13:56, Dan Carpenter wrote: This unlock doesn't belong here and it leads to a double unlock in the caller, r535_gsp_rpc_push(). Fixes: 176fdcbddfd2 ("drm/nouveau/gsp/r535: add support for booting GSP-RM") Signed-off-by: Dan Carpenter Good catch - applied to drm-misc-fixes. -

Re: [PATCH v2 04/12] drm/rockchip: vop2: clear afbc en and transform bit for cluster window at linear mode

2023-11-28 Thread Andy Yan
Hi Heiko: On 11/27/23 23:02, Heiko Stübner wrote: Am Mittwoch, 22. November 2023, 13:54:25 CET schrieb Andy Yan: From: Andy Yan The enable bit and transform offset of cluster windows should be cleared when it work at linear mode, or we may have a iommu fault issue. Signed-off-by: Andy Yan

Re: [PATCH v2 01/12] dt-bindings: display: msm: qcm2290-mdss: Use the non-deprecated DSI compat

2023-11-28 Thread Krzysztof Kozlowski
On 27/11/2023 16:28, Konrad Dybcio wrote: > The "qcom,dsi-ctrl-6g-qcm2290" has been deprecated in commit 0c0f65c6dd44 > ("dt-bindings: msm: dsi-controller-main: Add compatible strings for every > current SoC"), but the example hasn't been updated to reflect that. > > Fix that. Reviewed-by:

Re: [PATCH v2 02/12] dt-bindings: display: msm: Add reg bus and rotator interconnects

2023-11-28 Thread Krzysztof Kozlowski
On 27/11/2023 16:28, Konrad Dybcio wrote: > Apart from the already handled data bus (MAS_MDP_Pn<->DDR), there are > other connection paths: > - a path that connects rotator block to the DDR. > - a path that needs to be handled to ensure MDSS register access > functions properly, namely the "reg

Re: [PATCH 3/3] Revert "drm/bridge: panel: Add a device link between drm device and panel device"

2023-11-28 Thread Neil Armstrong
On 28/11/2023 00:10, Linus Walleij wrote: This reverts commit 199cf07ebd2b0d41185ac79b895547d45610b681. This patch creates bugs on devices where the DRM device is the ancestor of the panel devices. Attempts to fix this have failed because it leads to using device core functionality which is

Re: [PATCH 1/3] Revert "driver core: Export device_is_dependent() to modules"

2023-11-28 Thread Neil Armstrong
On 28/11/2023 00:10, Linus Walleij wrote: This reverts commit 1d5e8f4bf06da86b71cc9169110d1a0e1e7af337. Greg says: "why exactly is this needed? Nothing outside of the driver core should be needing this function, it shouldn't be public at all (I missed that before.) So please, revert it for

RE: [bug report] backlight: mp3309c: Add support for MPS MP3309C

2023-11-28 Thread Flavio Suligoi
Hi Dan, Thanks for the report, I'll fix the bug as soon as possible. Regards, Flavio > -Original Message- > From: Dan Carpenter > Sent: Tuesday, November 28, 2023 8:20 AM > To: Flavio Suligoi > Cc: dri-devel@lists.freedesktop.org > Subject: [bug report] backlight: mp3309c: Add support

Re: [PATCH 2/3] Revert "drm/bridge: panel: Check device dependency before managing device link"

2023-11-28 Thread Neil Armstrong
On 28/11/2023 00:10, Linus Walleij wrote: This reverts commit 39d5b6a64ace77d0c11c398d272218df5f939abb. This patch was causing build errors by using an unexported function from the device core, which Greg questions the saneness in exporting. Link:

Re: [PATCH v2] drm/mxsfb: Switch to drmm_mode_config_init

2023-11-28 Thread Maxime Ripard
Hi, On Mon, Nov 27, 2023 at 10:25:38PM +0100, Marek Vasut wrote: > Switch from deprecated unmanaged drm_mode_config_init() to > managed drmm_mode_config_init(). No functional change. > > Signed-off-by: Marek Vasut > --- > Cc: Daniel Vetter > Cc: David Airlie > Cc: Fabio Estevam > Cc: Maarten

Re: [PATCH 0/3] Revert panel fixes and original buggy patch

2023-11-28 Thread Greg Kroah-Hartman
On Tue, Nov 28, 2023 at 01:52:44AM +0200, Laurent Pinchart wrote: > On Tue, Nov 28, 2023 at 12:36:15AM +0100, Linus Walleij wrote: > > On Tue, Nov 28, 2023 at 12:25 AM Laurent Pinchart wrote: > > > On Tue, Nov 28, 2023 at 12:10:18AM +0100, Linus Walleij wrote: > > > > This series reverts the

Re: [PATCH 0/3] Revert panel fixes and original buggy patch

2023-11-28 Thread Neil Armstrong
Hi Linus, On 28/11/2023 00:36, Linus Walleij wrote: On Tue, Nov 28, 2023 at 12:25 AM Laurent Pinchart wrote: On Tue, Nov 28, 2023 at 12:10:18AM +0100, Linus Walleij wrote: This series reverts the attempts to fix the bug that went into v6.7-rc1 in commit 199cf07ebd2b "drm/bridge: panel: Add a

Re: [PATCH v2 04/12] drm/rockchip: vop2: clear afbc en and transform bit for cluster window at linear mode

2023-11-28 Thread Heiko Stübner
Am Dienstag, 28. November 2023, 09:03:46 CET schrieb Andy Yan: > Hi Heiko: > > On 11/27/23 23:02, Heiko Stübner wrote: > > Am Mittwoch, 22. November 2023, 13:54:25 CET schrieb Andy Yan: > >> From: Andy Yan > >> > >> The enable bit and transform offset of cluster windows should be > >> cleared

Re: [PATCH v3 10/17] drm/v3d: Detach the CSD job BO setup

2023-11-28 Thread Iago Toral
El lun, 27-11-2023 a las 15:48 -0300, Maíra Canal escribió: > From: Melissa Wen > > Detach CSD job setup from CSD submission ioctl to reuse it in CPU > submission ioctl for indirect CSD job. > > Signed-off-by: Melissa Wen > Co-developed-by: Maíra Canal > Signed-off-by: Maíra Canal > --- >  

Re: [PATCH v3 12/17] drm/v3d: Create a CPU job extension for a indirect CSD job

2023-11-28 Thread Iago Toral
El lun, 27-11-2023 a las 15:48 -0300, Maíra Canal escribió: (...) > @@ -860,19 +913,66 @@ v3d_submit_cpu_ioctl(struct drm_device *dev, > void *data, >   > mutex_lock(>sched_lock); > v3d_push_job(_job->base); > + > +   switch (cpu_job->job_type) { > +   case

Re: [PATCH v3 15/17] drm/v3d: Create a CPU job extension to copy timestamp query to a buffer

2023-11-28 Thread Iago Toral
El lun, 27-11-2023 a las 15:48 -0300, Maíra Canal escribió: (...) >  /** > diff --git a/include/uapi/drm/v3d_drm.h b/include/uapi/drm/v3d_drm.h > index 930f8d07f088..a3ae1f220291 100644 > --- a/include/uapi/drm/v3d_drm.h > +++ b/include/uapi/drm/v3d_drm.h > @@ -75,6 +75,7 @@ struct

Re: [PATCH v3 16/17] drm/v3d: Create a CPU job extension for the reset performance query job

2023-11-28 Thread Iago Toral
El lun, 27-11-2023 a las 15:48 -0300, Maíra Canal escribió: (...) > diff --git a/include/uapi/drm/v3d_drm.h b/include/uapi/drm/v3d_drm.h > index a3ae1f220291..76a02d2c01e6 100644 > --- a/include/uapi/drm/v3d_drm.h > +++ b/include/uapi/drm/v3d_drm.h > @@ -76,6 +76,7 @@ struct drm_v3d_extension { >  

Re: [PATCH v3 17/17] drm/v3d: Create a CPU job extension for the copy performance query job

2023-11-28 Thread Iago Toral
El lun, 27-11-2023 a las 15:48 -0300, Maíra Canal escribió: > A CPU job is a type of job that performs operations that requires CPU > intervention. A copy performance query job is a job that copy the > complete > or partial result of a query to a buffer. In order to copy the result > of > a

Re: [PATCH v3 00/17] drm/v3d: Introduce CPU jobs

2023-11-28 Thread Iago Toral
Hi Maíra, El lun, 27-11-2023 a las 15:48 -0300, Maíra Canal escribió: > This patchset implements the basic infrastructure for a new type of > V3D job, a CPU job. A CPU job is a job that requires CPU > intervention. > It would be nice to perform this operations on the kernel space as we > can

[PATCH v3] drm/atomic-helpers: Invoke end_fb_access while owning plane state

2023-11-28 Thread Thomas Zimmermann
Invoke drm_plane_helper_funcs.end_fb_access before drm_atomic_helper_commit_hw_done(). The latter function hands over ownership of the plane state to the following commit, which might free it. Releasing resources in end_fb_access then operates on undefined state. This bug has been observed with

<    1   2   3