[PATCH] drm/i915/gvt: clean up kernel-doc in gtt.c

2021-10-02 Thread Randy Dunlap
Fix kernel-doc warnings in gtt.c: gtt.c:1152: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst * Check if can do 2M page gtt.c:1152: warning: missing initial short description on line: * Check if can do 2M page gtt.c:2209: war

Re: [PATCH v2 6/8] drm/msm/dsi: stop setting clock parents manually

2021-10-02 Thread Dmitry Baryshkov
On 10/07/2021 00:07, Dmitry Baryshkov wrote: There is no reason to set clock parents manually, use device tree to assign DSI/display clock parents to DSI PHY clocks. Dropping this manual setup allows us to drop repeating code and to move registration of hw clock providers to generic place. Signe

Re: [V2] drm: msm: adreno: use IS_ERR() instead of null pointer check

2021-10-02 Thread Dmitry Baryshkov
On 07/11/2020 06:19, Wang Qing wrote: a6xx_gmu_get_mmio() never return null in case of error, but ERR_PTR(), so we should use IS_ERR() instead of null pointer check and IS_ERR_OR_NULL(). Signed-off-by: Wang Qing As a second thought, ioremap's NULL is converted to ERR_PTR(-EINVAL), so the pat

Re: [V2] drm: msm: adreno: use IS_ERR() instead of null pointer check

2021-10-02 Thread Dmitry Baryshkov
On 07/11/2020 06:19, Wang Qing wrote: a6xx_gmu_get_mmio() never return null in case of error, but ERR_PTR(), so we should use IS_ERR() instead of null pointer check and IS_ERR_OR_NULL(). Not quite. a6xx_gmu_get_mmio() can return NULL, as it uses ioremap() internally. And ioremap returns NULL i

[PATCH v1 5/5] drm/bridge: tc358768: Correct BTACNTRL1 programming

2021-10-02 Thread Dmitry Osipenko
TXTAGOCNT and RXTASURECNT bitfields of BTACNTRL1 register are swapped in the code, correct them. Driver doesn't implement low power mode for now, so this change doesn't make a practical difference yet. Tested-by: Andreas Westman Dorcsak # Asus TF700T Tested-by: Maxim Schwalm #TF700T Signed-off-b

[PATCH v1 4/5] drm/bridge: tc358768: Disable non-continuous clock mode

2021-10-02 Thread Dmitry Osipenko
Non-continuous clock mode doesn't work because driver doesn't support it properly. The bridge driver programs wrong bitfields that are required by the non-continuous mode (BTACNTRL1 register bitfields are swapped in the code), but fixing them doesn't help. Display panel of ASUS Transformer TF700T

[PATCH v1 3/5] drm/bridge: tc358768: Calculate video start delay

2021-10-02 Thread Dmitry Osipenko
Calculate video start delay based on the display timing instead of hardcoding it to a default value. This fixes "trembling" display output on Asus Transformer TF700T which uses Panasonic VVX10F004B00 display panel. Tested-by: Andreas Westman Dorcsak # Asus TF700T Tested-by: Maxim Schwalm #TF700T

[PATCH v1 2/5] drm/bridge: tc358768: Support pulse mode

2021-10-02 Thread Dmitry Osipenko
Support pulse-mode synchronization which is supported and used by simple DSI panels like Panasonic VVX10F004B00. Tested-by: Andreas Westman Dorcsak # Asus TF700T Tested-by: Maxim Schwalm #TF700T Signed-off-by: Dmitry Osipenko --- drivers/gpu/drm/bridge/tc358768.c | 66 ++---

[PATCH v1 1/5] drm/bridge: tc358768: Enable reference clock

2021-10-02 Thread Dmitry Osipenko
The driver assumes that reference clock is always-enabled, but this is hardware-dependent and not true for ASUS Transformer TF700T. Make driver enable / disable the reference clock. Tested-by: Andreas Westman Dorcsak # Asus TF700T Tested-by: Maxim Schwalm #TF700T Signed-off-by: Dmitry Osipenko

[PATCH v1 0/5] Improvements for TC358768 DSI bridge driver

2021-10-02 Thread Dmitry Osipenko
This series adds couple improvements to the TC358768 DSI bridge driver, enabling Panasonic VVX10F004B00 DSI panel support. This panel is used by ASUS Transformer TF700T tablet, which is ready for upstream kernel and display panel support is the biggest missing part. Dmitry Osipenko (5): drm/brid

Re: [Intel-gfx] [PATCH v2 00/17] drm: cleanup: Use DRM_MODESET_LOCK_ALL_* helpers where possible

2021-10-02 Thread Fernando Ramos
On 21/10/02 09:13AM, Fernando Ramos wrote: > > Sean, could you revert the whole patch series? I'll have a deeper look into > the > patch set and come up with a v3 where all these issues will be addressed. > Hi Sean, I now understand the nature of the issue that caused the problem with i915 and

Re: [PATCH 10/10] drm/gma500: Rename struct gtt_range to struct psb_gem_object

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > struct gtt_range represents a GEM object. Rename the structure to struct > psb_gem_object and update all users. No functional changes. > > Signed-off-by: Thomas Zimmermann Acked-by: Patrik Jakobsson > --- > drivers/gpu/drm/gma500/

Re: [PATCH 09/10] drm/gma500: Rewrite GTT page insert/remove without struct gtt_range

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > struct gtt_range represents a GEM object and should not be used for GTT > setup. Change psb_gtt_insert() and psb_gtt_remove() to receive all > necessary parameters from their caller. This also eliminates possible > failure from psb_gtt_

Re: [PATCH 08/10] drm/gma500: Set page-caching flags in GEM pin/unpin

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > Caching of the GEM object's backing pages are unrelated to GTT > management. Move the respective calls from GTT code to GEM code. > > Signed-off-by: Thomas Zimmermann > --- > drivers/gpu/drm/gma500/gem.c | 9 - > drivers/gpu/

Re: [PATCH 07/10] drm/gma500: Inline psb_gtt_{alloc,free}_range() into rsp callers

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > psb_gtt_alloc_range() allocates struct gtt_range, create the GTT resource > and performs some half-baked initialization. Inline the function into its > only caller psb_gem_create(). For creating the GTT resource, introduce a > new helpe

Re: [PATCH 06/10] drm/gma500: Inline psb_gtt_attach_pages() and psb_gtt_detach_pages()

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > psb_gtt_attach_pages() are not GTT functions but deal with the GEM > object's SHMEM pages. The only callers of psb_gtt_attach_pages() and > psb_gtt_detach_pages() are the GEM pin helpers. Inline the calls and > cleanup the resulting cod

Re: [PATCH 05/10] drm/gma500: Rename psb_gtt_{pin, unpin}() to psb_gem_{pin, unpin}()

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > Rename psb_gtt_pin() to psb_gem_pin() to reflect the semantics of the > function. Same for psb_gtt_unpin(). No functional changes. Acked-by: Patrik Jakobsson > > Signed-off-by: Thomas Zimmermann > --- > drivers/gpu/drm/gma500/gem.

Re: [PATCH 04/10] drm/gma500: Allocate GTT ranges in stolen memory with psb_gem_create()

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > Support private objects for stolen memory in psb_gem_create() and > convert users to psb_gem_create(). For stolen memory, psb_gem_create() > now initializes the GEM object via drm_gem_private_object_init(). > > In the fbdev setup, repla

Re: [PATCH 03/10] drm/gma500: Reimplement psb_gem_create()

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > Implement psb_gem_create() for general use. Create the GEM handle in > psb_gem_create_dumb(). Allows to use psb_gem_create() for creating all > of the GEM objects. > > While at it, clean-up drm_gem_dumb_create() to make it more readable

Re: [PATCH 02/10] drm/gma500: Use to_gtt_range() everywhere

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > Convert upcasts from struct drm_gem_object to struct gtt_range to > to_gtt_range(). Some places used container_of() directly. > > Signed-off-by: Thomas Zimmermann Acked-by: Patrik Jakobsson > --- > drivers/gpu/drm/gma500/gem.c

Re: [PATCH 01/10] drm/gma500: Move helpers for struct gtt_range from gtt.c to gem.c

2021-10-02 Thread Patrik Jakobsson
On Tue, Sep 28, 2021 at 10:44 AM Thomas Zimmermann wrote: > > Allocation and pinning helpers for struct gtt_range are GEM functions, > so move them to gem.c. No functional changes. > > Signed-off-by: Thomas Zimmermann > --- > drivers/gpu/drm/gma500/framebuffer.c | 1 - > drivers/gpu/drm/

Re: [PATCH v13 06/35] clk: tegra: Support runtime PM and power domain

2021-10-02 Thread Dmitry Osipenko
01.10.2021 15:32, Ulf Hansson пишет: >> +static __maybe_unused int tegra_clock_pm_suspend(struct device *dev) >> +{ >> + struct tegra_clk_device *clk_dev = dev_get_drvdata(dev); >> + >> + /* >> +* Power management of the clock is entangled with the Tegra PMC >> +* GENPD

[PATCH] drm/panel-simple: Add Vivax TPC-9150 panel v5

2021-10-02 Thread Nikola Pavlica
The model and make of the LCD panel of the Vivax TPC-9150 is unknown, hence the panel settings that were retrieved with a FEX dump are named after the device NOT the actual panel. The LCD in question is a 50 pin MISO TFT LCD panel of the resolution 1024x600 used by the aforementioned device. Vers

Re: [PATCH] drm/msm: delete conversion from bool value to bool return

2021-10-02 Thread Dmitry Baryshkov
On 02/11/2020 15:50, Bernard Zhao wrote: Delete the conversion from bool value to bool function return. Signed-off-by: Bernard Zhao Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/

Re: [PATCH] drm/msm: dsi: Remove redundant null check before clk_prepare_enable/clk_disable_unprepare

2021-10-02 Thread Dmitry Baryshkov
On 31/12/2020 12:56, Xu Wang wrote: Because clk_prepare_enable() and clk_disable_unprepare() already checked NULL clock parameter, so the additional checks are unnecessary, just remove them. Signed-off-by: Xu Wang Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/dsi/dsi_host.c | 15

Re: [PATCH] drm/msm/mdp5: Remove redundant null check before clk_prepare_enable/clk_disable_unprepare

2021-10-02 Thread Dmitry Baryshkov
On 31/12/2020 12:49, Xu Wang wrote: Because clk_prepare_enable() and clk_disable_unprepare() already checked NULL clock parameter, so the additional checks are unnecessary, just remove them. Signed-off-by: Xu Wang Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/mdp5/mdp5_kms.c

Re: [PATCH] drm/msm/mdp4: Remove redundant null check before clk_prepare_enable/clk_disable_unprepare

2021-10-02 Thread Dmitry Baryshkov
On 31/12/2020 12:41, Xu Wang wrote: Because clk_prepare_enable() and clk_disable_unprepare() already checked NULL clock parameter, so the additional checks are unnecessary, just remove them. Signed-off-by: Xu Wang Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/mdp4/mdp4_kms.c

[PATCH] drm/msm/a6xx: correct cx_debugbus_read arguments

2021-10-02 Thread Dmitry Baryshkov
First argument of cx_debugbus_read() should be 'void __iomem *' rather than 'void * __iomem' to make sparse happy. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/adreno/a6xx_gpu_state.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/adreno/a6xx_g

Re: [Intel-gfx] [PATCH v2 00/17] drm: cleanup: Use DRM_MODESET_LOCK_ALL_* helpers where possible

2021-10-02 Thread Fernando Ramos
On 21/10/02 09:13AM, Fernando Ramos wrote: > On 21/10/02 05:30AM, Ville Syrjälä wrote: > > On Sat, Oct 02, 2021 at 01:05:47AM +0300, Ville Syrjälä wrote: > > > On Fri, Oct 01, 2021 at 04:48:15PM -0400, Sean Paul wrote: > > > > On Fri, Oct 01, 2021 at 10:00:50PM +0300, Ville Syrjälä wrote: > > > > >

Re: Handling DRM master transitions cooperatively

2021-10-02 Thread Hans de Goede
Hi, On 10/1/21 6:33 PM, Simon Ser wrote: > On Wednesday, September 22nd, 2021 at 11:21, Hans de Goede > wrote: > >> I would be happy to work on the plymouth side of this, so that we >> have at least one consumer of such a flag lined up for merging. > > Do you have plans to work on the kernel s

Re: [BUG 5.15-rc3] kernel BUG at drivers/gpu/drm/i915/i915_sw_fence.c:245!

2021-10-02 Thread Linus Torvalds
On Sat, Oct 2, 2021 at 5:17 AM Steven Rostedt wrote: > > On Sat, 2 Oct 2021 03:17:29 -0700 (PDT) > Hugh Dickins wrote: > > > Yes (though bisection doesn't work right on this one): the fix > > Interesting, as it appeared to be very reliable. But I didn't do the > "try before / after" on the patch.

[PATCH 06/10] platform/x86: thinkpad_acpi: Add hotkey_notify_extended_hotkey() helper

2021-10-02 Thread Hans de Goede
Factor the extended hotkey handling out of hotkey_notify_hotkey() and into a new hotkey_notify_extended_hotkey() helper. This is a preparation patch for adding support the privacy-screen hotkey toggle (which needs some special handling, it should NOT send an evdev key-event to userspace...). Revi

[PATCH 08/10] platform/x86: thinkpad_acpi: Register a privacy-screen device

2021-10-02 Thread Hans de Goede
Register a privacy-screen device on laptops with a privacy-screen, this exports the PrivacyGuard features to user-space using a standardized vendor-agnostic sysfs interface. Note the sysfs interface is read-only. Registering a privacy-screen device with the new privacy-screen class code will also

[PATCH 09/10] drm/i915: Add intel_modeset_probe_defer() helper

2021-10-02 Thread Hans de Goede
The upcoming privacy-screen support adds another check for deferring probe till some other drivers have bound first. Factor out the current vga_switcheroo_client_probe_defer() check into an intel_modeset_probe_defer() helper, so that further probe-deferral checks can be added there. Signed-off-by

[PATCH 10/10] drm/i915: Add privacy-screen support (v2)

2021-10-02 Thread Hans de Goede
Add support for eDP panels with a built-in privacy screen using the new drm_privacy_screen class. Changes in v2: - Call drm_connector_update_privacy_screen() from intel_enable_ddi_dp() / intel_ddi_update_pipe_dp() instead of adding a for_each_new_connector_in_state() loop to intel_atomic_commi

[PATCH 07/10] platform/x86: thinkpad_acpi: Get privacy-screen / lcdshadow ACPI handles only once

2021-10-02 Thread Hans de Goede
Get the privacy-screen / lcdshadow ACPI handles once and cache them, instead of retrieving them every time we need them. Reviewed-by: Emil Velikov Reviewed-by: Lyude Paul Signed-off-by: Hans de Goede --- drivers/platform/x86/thinkpad_acpi.c | 18 -- 1 file changed, 8 insertions

[PATCH 05/10] drm/connector: Add a drm_connector privacy-screen helper functions (v2)

2021-10-02 Thread Hans de Goede
Add 2 drm_connector privacy-screen helper functions: 1. drm_connector_attach_privacy_screen_provider(), this function creates and attaches the standard privacy-screen properties and registers a generic notifier for generating sysfs-connector-status-events on external changes to the privacy-screen

[PATCH 04/10] drm/privacy-screen: Add notifier support (v2)

2021-10-02 Thread Hans de Goede
Add support for privacy-screen consumers to register a notifier to be notified of external (e.g. done by the hw itself on a hotkey press) state changes. Changes in v2: - Drop WARN_ON(mutex_is_locked(&priv->lock)) check in drm_privacy_screen_call_notifier_chain() it may be locked by another thr

[PATCH 03/10] drm/privacy-screen: Add X86 specific arch init code

2021-10-02 Thread Hans de Goede
Add X86 specific arch init code, which fills the privacy-screen lookup table by checking for various vendor specific ACPI interfaces for controlling the privacy-screen. This initial version only checks for the Lenovo Thinkpad specific ACPI methods for privacy-screen control. Reviewed-by: Emil Vel

[PATCH 01/10] drm/connector: Add support for privacy-screen properties (v4)

2021-10-02 Thread Hans de Goede
From: Rajat Jain Add support for generic electronic privacy screen properties, that can be added by systems that have an integrated EPS. Changes in v2 (Hans de Goede) - Create 2 properties, "privacy-screen sw-state" and "privacy-screen hw-state", to deal with devices where the OS might be lo

[PATCH 02/10] drm: Add privacy-screen class (v4)

2021-10-02 Thread Hans de Goede
On some new laptops the LCD panel has a builtin electronic privacy-screen. We want to export this functionality as a property on the drm connector object. But often this functionality is not exposed on the GPU but on some other (ACPI) device. This commit adds a privacy-screen class allowing the dr

[PATCH 00/10] drm: Add privacy-screen class and connector properties

2021-10-02 Thread Hans de Goede
Hi all, Here is a new version of my privacy-screen series, addressing the (few) review-remarks from the version posted on September 6th. This new version contains the following changes: - drm: Add privacy-screen class (v4) - Make drm_privacy_screen_set_sw_state() skip calling out to the hw if

[PATCH 16/16] Revert "drm: cleanup: drm_modeset_lock_all_ctx() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 21dde40902d2636f70766b3154931de57e1215e9. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/drm_client_modeset.c | 9 ++--- 1 file changed, 6 i

[PATCH 15/16] Revert "drm/i915: cleanup: drm_modeset_lock_all_ctx() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 399190e70816886e2bca1f3f3bc3d9c544af88e7. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/i915/display/intel_display.c | 18 +- 1

[PATCH 13/16] Revert "drm: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 7c5f2eecc21f44fba1b1f13ce09c2533db9d781a. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/drm_client_modeset.c | 5 ++--- drivers/gpu/drm/drm_cr

[PATCH 14/16] Revert "drm/msm: cleanup: drm_modeset_lock_all_ctx() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 4c048437ef7adb2d81fe4ddc5c04179126eefcf2. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/msm_disp_snapshot_util.c | 12 ++-- 1

[PATCH 12/16] Revert "drm/vmwgfx: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 6b92e77156c5adf6606c8ad825c71404417d88af. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/vmwgfx/vmwgfx_ioctl.c | 11 --- drivers/gpu/drm

[PATCH 11/16] Revert "drm/tegra: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit a2cd9947d99b54c959fce20dc19d81af53f4674e. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/tegra/dsi.c | 6 ++ drivers/gpu/drm/tegra/hdmi.c

[PATCH 10/16] Revert "drm/shmobile: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 9b8c437ef1a5f34686fc96b391c201a80f1a2ea0. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/shmobile/shmob_drm_drv.c | 6 ++ 1 file changed, 2

[PATCH 09/16] Revert "drm/radeon: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 26723c3d6b930775f9a85521d09655c533a839e6. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/radeon/radeon_device.c | 21 ++--- driv

[PATCH 08/16] Revert "drm/omapdrm: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 6067fddc1a4fdf155787ab6182c192e4b360b73c. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/omapdrm/omap_fb.c | 9 +++-- 1 file changed, 3 inse

[PATCH 07/16] Revert "drm/nouveau: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 6aa2daae589b63e8b39fe6f7f2b59fb3063efa05. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/nouveau/dispnv50/disp.c | 15 +-- 1 file ch

[PATCH 06/16] Revert "drm/msm: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit fd49ef52e2db015ce69ad02bab0702489d141a41. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c | 10 -- 1 file change

[PATCH 05/16] Revert "drm/i915: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 746826bcf8fdf682668ff1c415d6b91dbf5858be. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/i915/display/intel_audio.c| 16 +++ drivers/gpu

[PATCH 04/16] Revert "drm/i915: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN() part 2"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 984c9949f1c4cf36c35aa2de5ee2b65c39379fd9. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/i915/display/intel_overlay.c | 40 ++--

[PATCH 03/16] Revert "drm/gma500: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 4f9e860e6ad65ff4ea8ce165a1407d96ff1b2211. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/gma500/psb_device.c | 18 ++ 1 file cha

[PATCH 02/16] Revert "drm/amd: cleanup: drm_modeset_lock_all() --> DRM_MODESET_LOCK_ALL_BEGIN()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 299f040e855b69c29522cde446777902381a07f5. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 21 +++- .../gpu/dr

[PATCH 01/16] Revert "drm: cleanup: remove drm_modeset_(un)lock_all()"

2021-10-02 Thread Sean Paul
From: Sean Paul This reverts commit 8d813d1a535c8c7503a5f4061654a73026be8c5e. This patchset breaks on intel platforms and was previously NACK'd by Ville. Cc: Ville Syrjälä Cc: Fernando Ramos Signed-off-by: Sean Paul --- drivers/gpu/drm/drm_modeset_lock.c | 94 +-

Re: [PATCH 1/2] drm: Add Gamma and Degamma LUT sizes props to drm_crtc to validate.

2021-10-02 Thread Sean Paul
On Fri, Oct 01, 2021 at 04:34:34PM -0400, Sean Paul wrote: > On Wed, Sep 29, 2021 at 03:39:25PM -0400, Mark Yacoub wrote: > > From: Mark Yacoub > > > > [Why] > > 1. drm_atomic_helper_check doesn't check for the LUT sizes of either Gamma > > or Degamma props in the new CRTC state, allowing any inv

Re: [PATCH v2 0/3] drm/etnaviv: IOMMU related fixes

2021-10-02 Thread Michael Walle
Am 2021-09-07 18:49, schrieb Michael Walle: This patch series fixes usage of the etnaviv driver with GPUs behind a IOMMU. It was tested on a NXP LS1028A SoC. Together with Lucas' MMU patches [1] there are not more (GPU internal) MMU nor (system) IOMMU faults on the LS1028A. [1] https://lists

Re: [BUG 5.15-rc3] kernel BUG at drivers/gpu/drm/i915/i915_sw_fence.c:245!

2021-10-02 Thread Steven Rostedt
On Sat, 2 Oct 2021 03:17:29 -0700 (PDT) Hugh Dickins wrote: > Yes (though bisection doesn't work right on this one): the fix Interesting, as it appeared to be very reliable. But I didn't do the "try before / after" on the patch. > https://lore.kernel.org/lkml/1f955bff-fd9e-d2ee-132a-f758add9e..

Re: [Intel-gfx] [PATCH v2 0/4] drm/dp, drm/i915: Finish basic PWM support for VESA backlight helpers

2021-10-02 Thread Hans de Goede
Hi Lyude, On 10/2/21 12:53 AM, Lyude Paul wrote: > When I originally moved all of the VESA backlight code in i915 into DRM > helpers, one of the things I didn't have the hardware or time for > testing was machines that used a combination of PWM and DPCD in order to > control their backlights. This

[PATCH] drm: panel-orientation-quirks: Add quirk for Aya Neo 2021

2021-10-02 Thread Bryant Mairs
Signed-off-by: Bryant Mairs --- drivers/gpu/drm/drm_panel_orientation_quirks.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/drm_panel_orientation_quirks.c b/drivers/gpu/drm/drm_panel_orientation_quirks.c index f6bdec7fa925..30c17a76f49a 100644 --- a/drivers/gpu/drm/d

Re: [Intel-gfx] [PATCH v2 00/17] drm: cleanup: Use DRM_MODESET_LOCK_ALL_* helpers where possible

2021-10-02 Thread Fernando Ramos
On 21/10/02 05:30AM, Ville Syrjälä wrote: > On Sat, Oct 02, 2021 at 01:05:47AM +0300, Ville Syrjälä wrote: > > On Fri, Oct 01, 2021 at 04:48:15PM -0400, Sean Paul wrote: > > > On Fri, Oct 01, 2021 at 10:00:50PM +0300, Ville Syrjälä wrote: > > > > On Fri, Oct 01, 2021 at 02:36:55PM -0400, Sean Paul