Re: [Nouveau] [PATCH v2 09/20] drm/i915: Remove references to struct drm_device.pdev

2020-12-10 Thread Jani Nikula
On Tue, 08 Dec 2020, Thomas Zimmermann wrote: > ping for a review of the i915 patches What did you have in mind regarding merging the series? Should we just pick the patches up? BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ___ Nouveau

Re: [Nouveau] [PATCH v2 13/20] drm/nouveau: Remove references to struct drm_device.pdev

2020-12-10 Thread Thomas Zimmermann
Hi Am 09.12.20 um 19:04 schrieb Jeremy Cline: Hi, On Tue, Dec 01, 2020 at 11:35:35AM +0100, Thomas Zimmermann wrote: Using struct drm_device.pdev is deprecated. Convert nouveau to struct drm_device.dev. No functional changes. Signed-off-by: Thomas Zimmermann Cc: Ben Skeggs ---

[Nouveau] [RFC 3/5] drm/i915/dp: Remove redundant AUX backlight frequency calculations

2020-12-10 Thread Lyude Paul
Noticed this while moving all of the VESA backlight code in i915 over to DRM helpers: it would appear that we calculate the frequency value we want to write to DP_EDP_BACKLIGHT_FREQ_SET twice even though this value never actually changes during runtime. So, let's simplify things by just caching

[Nouveau] [RFC 4/5] drm/dp: Extract i915's eDP backlight code into DRM helpers

2020-12-10 Thread Lyude Paul
Since we're about to implement eDP backlight support in nouveau using the standard protocol from VESA, we might as well just take the code that's already written for this and move it into a set of shared DRM helpers. Note that these helpers are intended to handle DPCD related backlight control

Re: [Nouveau] [PATCH v2 09/20] drm/i915: Remove references to struct drm_device.pdev

2020-12-10 Thread Thomas Zimmermann
Hi Am 10.12.20 um 10:03 schrieb Jani Nikula: On Tue, 08 Dec 2020, Thomas Zimmermann wrote: ping for a review of the i915 patches What did you have in mind regarding merging the series? Should we just pick the patches up? Originally I thought that individual trees would merge their rsp

Re: [Nouveau] [PATCH v2 13/20] drm/nouveau: Remove references to struct drm_device.pdev

2020-12-10 Thread Jeremy Cline
On Thu, Dec 10, 2020 at 08:56:08AM +0100, Thomas Zimmermann wrote: > Hi > > Am 09.12.20 um 19:04 schrieb Jeremy Cline: > > Hi, > > > > On Tue, Dec 01, 2020 at 11:35:35AM +0100, Thomas Zimmermann wrote: > > > Using struct drm_device.pdev is deprecated. Convert nouveau to struct > > >

Re: [Nouveau] [PATCH 2/7] Revert "ACPI / OSI: Add OEM _OSI string to enable NVidia HDMI audio"

2020-12-10 Thread Lyude Paul
Hi, bumping the discussion again here because we just ran into this again, as we just had a laptop OEM try to fix another issue by adding an OSI check for these strings instead of helping us fix the issue in nouveau. Note as well the issue I'm referring to is entirely independent of runtime D3,