Re: [Intel-gfx] [PATCH] drm/i915/dpcd_bl: Don't try vesa interface unless specified by VBT

2021-04-06 Thread Jani Nikula
On Tue, 23 Mar 2021, Lyude Paul wrote: > On Tue, 2021-03-23 at 16:06 +0200, Jani Nikula wrote: >> On Thu, 18 Mar 2021, Lyude Paul wrote: >> > Actually-NAK this. I just realized I've been misreading the bug and that >> > this >> > doesn't actually s

Re: Call for an EDID parsing library

2021-04-07 Thread Jani Nikula
t yet support that in the kernel either.) BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [Intel-gfx] [PATCH v2 00/20] drm: Use new DRM printk funcs (like drm_dbg_*()) in DP helpers

2021-04-08 Thread Jani Nikula
t, and are all in sync, it's only the drm-next -> drm-misc-next backmerge that's still needed. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedeskt

Re: Call for an EDID parsing library

2021-04-08 Thread Jani Nikula
On Wed, 07 Apr 2021, Hans Verkuil wrote: > On 07/04/2021 12:31, Jani Nikula wrote: >> On Wed, 07 Apr 2021, Hans Verkuil wrote: >>> It is the most complete EDID parser I know based on the various standards. >> >> Does it support pure DisplayID in addition to Displa

Re: Call for an EDID parsing library

2021-04-08 Thread Jani Nikula
On Thu, 08 Apr 2021, Pekka Paalanen wrote: > On Thu, 08 Apr 2021 16:49:37 +0300 > Jani Nikula wrote: > >> Anyway, this is only tangentially related to the library. I just think >> we need to take DisplayID better into account also in the *users* of the >> library, as

Re: Call for an EDID parsing library

2021-04-08 Thread Jani Nikula
On Thu, 08 Apr 2021, Simon Ser wrote: > On Thursday, April 8th, 2021 at 4:58 PM, Jani Nikula > wrote: > >> Perhaps that should be the takeaway; try to minimize parsed data >> where the consumer needs to know whether it originated from DisplayID or >> EDID? > >

Re: [PATCH 1/3] drm/aperture: Add infrastructure for aperture ownership

2021-04-12 Thread Jani Nikula
or tab, resulting in rst where the code-block contents are only partially indented. Please test the documentation build before applying. Otherwise, the series seems like a nice cleanup. Acked-by: Jani Nikula -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 1/3] drm/aperture: Add infrastructure for aperture ownership

2021-04-12 Thread Jani Nikula
On Mon, 12 Apr 2021, Thomas Zimmermann wrote: > Hi > > Am 12.04.21 um 11:36 schrieb Jani Nikula: >> Please test the documentation build before applying. > > I did and I'm pretty sure it looked correct. But I'll double check. Cool, maybe someone added more sma

Re: [Intel-gfx] [PATCH 18/19] drm/i915/gtt: map the PD up front

2021-04-12 Thread Jani Nikula
lly in the mmio macros), and it would be a terrible > flag day. FWIW, we had the dev_priv/dev macro trickery for a while to not have that flag day conversion, until everything used i915 or &i915->drm. But we got rid of it afterwards. BR, Jani. -- Jani Nikula, I

[PULL] topic/intel-gen-to-ver -> drm-intel-next and drm-intel-gt-next

2021-04-19 Thread Jani Nikula
915/intel_pm.c| 50 - drivers/gpu/drm/i915/intel_uncore.c| 8 ++- drivers/gpu/drm/i915/selftests/intel_uncore.c | 8 ++- drivers/gpu/drm/i915/selftests/mock_gem_device.c | 2 +- 51 files changed, 376 inser

Re: [v1 0/3] drm: Add support for backlight control of eDP panel on ti-sn65dsi86 bridge

2021-04-20 Thread Jani Nikula
/drm/bridge/ti-sn65dsi86.c | 26 +++ > drivers/gpu/drm/drm_dp_aux_backlight.c | 191 > + > include/drm/drm_dp_aux_backlight.h | 29 > 7 files changed, 264 insertions(+) > create mode 100644 drivers/gpu/drm/drm_dp_

Re: [PATCH] vfio/gvt: fix DRM_I915_GVT dependency on VFIO_MDEV

2021-04-22 Thread Jani Nikula
_attributes") >> Signed-off-by: Arnd Bergmann >> --- >> drivers/gpu/drm/i915/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > Oh kconfig stuff like this makes my head hurt, thanks for finding it > > I also can't see an alternati

Re: [PATCH v2] drm/i915: Invoke BXT _DSM to enable MUX on HP Workstation laptops

2021-04-23 Thread Jani Nikula
disable_rpm_wakeref_asserts(&dev_priv->runtime_pm); > @@ -1271,6 +1274,8 @@ static int i915_drm_resume(struct drm_device *dev) > > intel_gvt_resume(dev_priv); > > + intel_bxt_dsm_detect(pdev); > + In intel_display_resume() perhaps? (Yay for confusing naming wrt display and modeset, it's a work-in-progress.) BR, Jani. > enable_rpm_wakeref_asserts(&dev_priv->runtime_pm); > > return 0; -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [v3 2/2] backlight: Add DisplayPort aux backlight driver

2021-04-26 Thread Jani Nikula
gt; + "failed to register backlight (%d)\n", ret); > + goto free_ddc; > + } > + > + platform_set_drvdata(pdev, bd); > + > + return 0; > + > +free_ddc: > + if (ddc) > + put_device(&ddc->dev); > + >

Re: [PATCH v2] drm/i915: Invoke BXT _DSM to enable MUX on HP Workstation laptops

2021-04-26 Thread Jani Nikula
On Mon, 26 Apr 2021, Kai-Heng Feng wrote: > On Fri, Apr 23, 2021 at 3:35 PM Jani Nikula > wrote: >> >> On Fri, 23 Apr 2021, Kai-Heng Feng wrote: >> > On HP Fury G7 Workstations, graphics output is re-routed from Intel GFX >> > to discrete GFX after S3. This

Re: [drm-intel:drm-intel-next 3/4] drivers/gpu/drm/i915/display/intel_dp_hdcp.c:582 intel_dp_hdcp2_read_msg() error: uninitialized symbol 'msg_end'.

2021-04-26 Thread Jani Nikula
i915/hdcp: Fix uninitialized symbol 'msg_end'"). BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH] display: Fix typo issue

2021-04-26 Thread Jani Nikula
> + * recommendation, port should be enabled before plane & pipe */ > if (is_cmd_mode(intel_dsi)) { > for_each_dsi_port(port, intel_dsi->ports) > intel_de_write(dev_priv, -- Jani Nikula, Intel Open Source Graphics Center _

Re: [PATCH] drm: i915: fix build when ACPI is disabled and BACKLIGHT=m

2021-04-27 Thread Jani Nikula
> class is a module") > Fixes: 44c1220a441c ("drm/i915: extract intel_panel.h from intel_drv.h") > Signed-off-by: Randy Dunlap > Cc: Ville Syrjälä > Cc: Jani Nikula > Cc: Damien Lespiau > Cc: Daniel Vetter > Cc: Jani Nikula > Cc: Joonas Lahtinen &

[PULL] drm-intel-next-fixes for the merge window

2021-04-27 Thread Jani Nikula
n fix (Maarten) Dan Carpenter (2): drm/i915/gvt: Fix error code in intel_gvt_init_device() drm/i915: fix an error code in intel_overlay_do_put_image() Jani Nikula (1): Merge tag 'gvt-next-fixes-2021-04-21' of http

Re: [PATCH v7 0/4] drm: Move struct drm_device.pdev to legacy

2021-04-27 Thread Jani Nikula
elevant for legacy drivers. In a later patchset, we may want to > convert these as well and remove pdev entirely. On the series, Reviewed-by: Jani Nikula How should we merge these? > > v7: > * fix instances of pdev that have benn added under i915/ > v6: > *

Re: [PATCH] drm/i915/gem: Remove reference to struct drm_device.pdev

2021-04-27 Thread Jani Nikula
quot;drm/i915: Create stolen memory region from local > memory") > Cc: CQ Tang > Cc: Matthew Auld > Cc: Tvrtko Ursulin > Cc: Xinyun Liu > Cc: Tvrtko Ursulin > Cc: Jani Nikula > Cc: Joonas Lahtinen > Cc: Rodrigo Vivi > Cc: Chris Wilson > Cc: Mika Kuo

Re: [PATCH v7 0/4] drm: Move struct drm_device.pdev to legacy

2021-04-27 Thread Jani Nikula
On Tue, 27 Apr 2021, Thomas Zimmermann wrote: > Hi Jani > > Am 27.04.21 um 14:04 schrieb Jani Nikula: >> On Tue, 27 Apr 2021, Thomas Zimmermann wrote: >>> V7 of the patchset fixes some bitrot in the intel driver. >>> >>> The pdev field in struct drm_d

Re: New warnings with gcc-11

2021-04-28 Thread Jani Nikula
nn reported some of these a while back. I think we have some of them fixed in our -next already, but not all. Thanks for the reminder. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org

Re: [PATCH 1/1] i915/query: Correlate engine and cpu timestamps with better accuracy

2021-04-28 Thread Jani Nikula
+ /** Engine for which command streamer cycles is queried. */ > + struct i915_engine_class_instance engine; > + > + /** Must be zero. */ > + __u32 flags; > + > + /** > + * Command streamer cycles as read from the command streamer > + * register at 0x358 o

Re: Display notch support

2021-04-28 Thread Jani Nikula
of that? This is a bit off-topic, but DisplayID 2.0 defines primary use cases for head mounted VR and AR, so we wouldn't have to quirk them. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists

RE: [PATCH v2] drm/i915/gem: Remove reference to struct drm_device.pdev

2021-04-29 Thread Jani Nikula
longer as >>the field will be moved into the struct's legacy section. Add a fix >>for the rsp commit. > > Commit message match the patch. > > Reviewed-by: Michael J. Ruhl > M > >>v2: >> * fix an error in the commit description (Michael) >> >&g

Re: [Intel-gfx] [PATCH v8 1/1] drm/drm_mst: Use Extended Base Receiver Capability DPCD space

2021-04-29 Thread Jani Nikula
patch, with a benign looking subject, and none of the i915 maintainers in Cc. Not cool. BR, Jani. > ret = drm_dp_mst_topology_mgr_init(&intel_dp->mst_mgr, &i915->drm, > -&intel_dp->aux, 16, 3, conn_base_id); > +

Re: [Heads up to maintainers] Re: [PATCH v8 1/1] drm/drm_mst: Use Extended Base Receiver Capability DPCD space

2021-04-29 Thread Jani Nikula
t;  radeon_dp_mst_init(struct radeon_connector *radeon_connector) >>  { >> struct drm_device *dev = radeon_connector->base.dev; >> +   int max_link_rate; >>   >> if (!radeon_connector->ddc_bus->has_aux) >> return 0;

[PULL] drm-intel-next-fixes for the merge window #2

2021-04-30 Thread Jani Nikula
te calculation Colin Xu (1): drm/i915/gvt: Prevent divided by zero when calculating refresh rate Jani Nikula (1): Merge tag 'gvt-next-fixes-2021-04-29' of https://github.com/intel/gvt-linux into drm-intel-next-fixes drivers/gpu/drm/i915/gvt/handl

Re: [Heads up to maintainers] Re: [PATCH v8 1/1] drm/drm_mst: Use Extended Base Receiver Capability DPCD space

2021-05-03 Thread Jani Nikula
On Fri, 30 Apr 2021, Jani Nikula wrote: > On Thu, 29 Apr 2021, Lyude Paul wrote: >> JFYI Jani and Ben: I will be pushing this patch to drm-misc-next sometime >> today if there's no objections > > Thanks for the heads-up, I think this breaks i915. See my review > com

Re: [Heads up to maintainers] Re: [PATCH v8 1/1] drm/drm_mst: Use Extended Base Receiver Capability DPCD space

2021-05-03 Thread Jani Nikula
On Mon, 03 May 2021, Jani Nikula wrote: > On Fri, 30 Apr 2021, Jani Nikula wrote: >> On Thu, 29 Apr 2021, Lyude Paul wrote: >>> JFYI Jani and Ben: I will be pushing this patch to drm-misc-next sometime >>> today if there's no objections >> >> Thank

RE: [PATCH v2 1/1] drm/i915: Use the correct max source link rate for MST

2021-05-03 Thread Jani Nikula
CC all the maintainers. The drivers we have are monsters, and it can be tricky to get the details right. All the more important to get the Cc's right; then at least you can blame us afterwards. ;) Thanks for reacting quickly, though. BR, Jani. -- Jani Nikula,

Re: [PATCH] nouveau: make backlight support non optional

2021-08-09 Thread Jani Nikula
iver for all known devices. > > config DRM_I915_BACKLIGHT > - tristate "Control backlight support" > - depends on DRM_I915 > - default DRM_I915 > - select BACKLIGHT_CLASS_DEVICE > - help > - Say Y here if you want to control the backlight of your display > - (e.g. a laptop panel). > + def_tristate DRM_I915 > > config DRM_I915_CAPTURE_ERROR > bool "Enable capturing GPU state following a hang" -- Jani Nikula, Intel Open Source Graphics Center

[PATCH] drm/i915/gem/ttm: prefer kernel types

2021-08-10 Thread Jani Nikula
Avoid uintXX_t types in the driver. Fixes: 213d50927763 ("drm/i915/ttm: Introduce a TTM i915 gem object backend") Cc: Thomas Hellström Cc: Matthew Auld Cc: Maarten Lankhorst Signed-off-by: Jani Nikula --- drivers/gpu/drm/i915/gem/i915_gem_ttm.c | 2 +- 1 file changed, 1 inser

Re: [Intel-gfx] [PATCH] drm/i915/gem/ttm: prefer kernel types

2021-08-10 Thread Jani Nikula
On Tue, 10 Aug 2021, Daniel Vetter wrote: > On Tue, Aug 10, 2021 at 11:41:28AM +0300, Jani Nikula wrote: >> Avoid uintXX_t types in the driver. >> >> Fixes: 213d50927763 ("drm/i915/ttm: Introduce a TTM i915 gem object backend") >> Cc: Thomas Hellström

[PULL] drm-intel-next

2021-08-10 Thread Jani Nikula
using MCHBAR Gwan-gyeong Mun (1): drm/i915/dg2: Update lane disable power state during PSR Imre Deak (2): drm/i915/adlp: Add workaround to disable CMTG clock gating drm/i915: Apply CMTG clock disabling WA while DPLL0 is enabled Jani Nikula (1): drm/i915/plane: add intel_plane_

Re: missing signoff on drm-intel-gt-next pull

2021-08-11 Thread Jani Nikula
ow much pain it is to fix that up, but >> commit db47fe727e1fc516cf60fc9ab8299605ef3c2d54 >> Author: Anusha Srivatsa >> Commit: Matt Roper >> >> drm/i915/step: s/_revid_tbl/_revids >> >> Simplify the stepping info array name. >> >> Cc: Jani

Re: [Intel-gfx] linux-next: Signed-off-by missing for commit in the drm-intel tree

2021-08-11 Thread Jani Nikula
7:18:54) >> > > > Hi all, >> > > > >> > > > Commit >> > > > >> > > > db47fe727e1f ("drm/i915/step: >> > > > s/_revid_tbl/_revids") >> > > > >> > > > is missing a Signed-off-by from its committer. >> > > > >> > > > -- >> > > > Cheers, >> > > > Stephen Rothwell >> > >> > -- >> > Daniel Vetter >> > Software Engineer, Intel Corporation >> > http://blog.ffwll.ch >> >> -- >> Matt Roper >> Graphics Software Engineer >> VTT-OSGC Platform Enablement >> Intel Corporation >> (916) 356-2795 -- Jani Nikula, Intel Open Source Graphics Center

Re: [PATCH v3 0/6] add fixes to pass DP Link Layer compliance test cases

2021-08-11 Thread Jani Nikula
+- > drivers/gpu/drm/msm/dp/dp_display.c | 14 ++-- > drivers/gpu/drm/msm/dp/dp_panel.c | 9 ++- > 4 files changed, 102 insertions(+), 61 deletions(-) -- Jani Nikula, Intel Open Source Graphics Center

Re: missing signoff on drm-intel-gt-next pull

2021-08-11 Thread Jani Nikula
On Wed, 11 Aug 2021, Dave Airlie wrote: > On Wed, 11 Aug 2021 at 17:11, Jani Nikula wrote: >> >> On Wed, 11 Aug 2021, Joonas Lahtinen wrote: >> > Quoting Dave Airlie (2021-08-11 06:48:39) >> >> dim: db47fe727e1f ("drm/i915/step: >> >

Re: [Intel-gfx] linux-next: Signed-off-by missing for commit in the drm-intel tree

2021-08-11 Thread Jani Nikula
On Wed, 11 Aug 2021, Daniel Vetter wrote: > On Wed, Aug 11, 2021 at 10:16:41AM +0300, Jani Nikula wrote: >> On Tue, 10 Aug 2021, Daniel Vetter wrote: >> > On Mon, Aug 09, 2021 at 09:19:39AM -0700, Matt Roper wrote: >> >> On Mon, Aug 09, 2021 at 04:05:59PM +0200, D

Re: [Intel-gfx] [PATCH v6 01/15] drm/i915/pxp: Define PXP component interface

2021-08-13 Thread Jani Nikula
r. > + */ > +struct i915_pxp_component { > + struct device *tee_dev; > + const struct i915_pxp_component_ops *ops; > + > + /* To protect the above members. */ > + struct mutex mutex; > +}; > + > +#endif /* _I915_TEE_PXP_INTERFACE_H_ */ -- Jani Nikula, Intel Open Source Graphics Center

[PATCH 0/4] drm/dp: add some defines and prep for DP 2.0

2021-08-13 Thread Jani Nikula
I'll probably want to merge these to drm-intel-next and drm-misc-next via a topic branch. Jani Nikula (4): drm/dp: add DP 2.0 UHBR link rate and bw code conversions drm/dp: use more of the extended receiver cap drm/dp: add LTTPR DP 2.0 DPCD addresses drm/dp: add helper for extra

[PATCH 1/4] drm/dp: add DP 2.0 UHBR link rate and bw code conversions

2021-08-13 Thread Jani Nikula
The bw code equals link_rate / 0.27 Gbps only for 8b/10b link rates. Handle DP 2.0 UHBR rates as special cases, though this is not pretty. Cc: Manasi Navare Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_dp_helper.c | 26 ++ 1 file changed, 22 insertions(+), 4

[PATCH 2/4] drm/dp: use more of the extended receiver cap

2021-08-13 Thread Jani Nikula
Extend the use of extended receiver cap at 0x2200 to cover MAIN_LINK_CHANNEL_CODING_CAP in 0x2206, in case an implementation hides the DP 2.0 128b/132b channel encoding cap. Cc: Manasi Navare Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_dp_helper.c | 2 +- 1 file changed, 1 insertion

[PATCH 3/4] drm/dp: add LTTPR DP 2.0 DPCD addresses

2021-08-13 Thread Jani Nikula
DP 2.0 brings some new DPCD addresses for PHY repeaters. Cc: Manasi Navare Signed-off-by: Jani Nikula --- include/drm/drm_dp_helper.h | 4 1 file changed, 4 insertions(+) diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h index 1d5b3dbb6e56..f3a61341011d 100644 --- a

[PATCH 4/4] drm/dp: add helper for extracting adjust 128b/132b TX FFE preset

2021-08-13 Thread Jani Nikula
The DP 2.0 128b/132b channel coding uses TX FFE presets instead of vswing and pre-emphasis. Cc: Manasi Navare Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_dp_helper.c | 14 ++ include/drm/drm_dp_helper.h | 2 ++ 2 files changed, 16 insertions(+) diff --git a/drivers/gpu

Re: [PATCH] drm/fb: Fix randconfig builds

2021-08-16 Thread Jani Nikula
FB > select DRM_KMS_HELPER > select FB_CFB_FILLRECT > select FB_CFB_COPYAREA -- Jani Nikula, Intel Open Source Graphics Center

Re: [PATCH] drm/fb: Fix randconfig builds

2021-08-16 Thread Jani Nikula
elper.ko is the module, enabled via DRM_KMS_HELPER, and it has an implicit dependency on FB, and DRM_FBDEV_EMULATION selects DRM_KMS_HELPER. Select just breaks dependencies in all kinds of ways. This might help in config DRM_KMS_HELPER, and it might help the reader because it'

Re: [PATCH] drm/fb: Fix randconfig builds

2021-08-16 Thread Jani Nikula
setting driver" > depends on DRM > - depends on FB > + depends on FB && FB != m > select DRM_KMS_HELPER > select FB_CFB_FILLRECT > select FB_CFB_COPYAREA > > -- > Jackie Liu > > 在 2021/8/16 下午3:01, Jani N

Re: [PATCH] drm/fb: Fix randconfig builds

2021-08-16 Thread Jani Nikula
depends on FB || FB=n > > -- > Thanks, BR, Jackie Liu > > 在 2021/8/16 下午4:33, Jani Nikula 写道: >> On Mon, 16 Aug 2021, Jackie Liu wrote: >>> Hi Jani. >>> >>> My CI report an randconfigs build failed. there are: >>> >>> drm_fb_h

Re: [PATCH] drm/fb: Fix randconfig builds

2021-08-16 Thread Jani Nikula
On Mon, 16 Aug 2021, Jani Nikula wrote: > On Mon, 16 Aug 2021, Jackie Liu wrote: >> Hi Jani. >> >> Your suggestion is that? >> >> diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig >> index 7ff89690a976..ba179a539497 100644 >> --- a/dri

Re: [PATCH v2] drm/fb: Fix randconfig builds

2021-08-16 Thread Jani Nikula
bool "Enable legacy fbdev support for your modesetting driver" > depends on DRM > - depends on FB > + depends on FB && FB != m > select DRM_KMS_HELPER > select FB_CFB_FILLRECT > select FB_CFB_COPYAREA -- Jani Nikula, Intel Open Source Graphics Center

[PATCH 01/17] drm/dp: add DP 2.0 UHBR link rate and bw code conversions

2021-08-18 Thread Jani Nikula
The bw code equals link_rate / 0.27 Gbps only for 8b/10b link rates. Handle DP 2.0 UHBR rates as special cases, though this is not pretty. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_dp_helper.c | 26 ++ 1 file changed, 22

[PATCH 02/17] drm/dp: use more of the extended receiver cap

2021-08-18 Thread Jani Nikula
Extend the use of extended receiver cap at 0x2200 to cover MAIN_LINK_CHANNEL_CODING_CAP in 0x2206, in case an implementation hides the DP 2.0 128b/132b channel encoding cap. Cc: dri-devel@lists.freedesktop.org Reviewed-by: Manasi Navare Signed-off-by: Jani Nikula --- drivers/gpu/drm

[PATCH 04/17] drm/dp: add helper for extracting adjust 128b/132b TX FFE preset

2021-08-18 Thread Jani Nikula
The DP 2.0 128b/132b channel coding uses TX FFE presets instead of vswing and pre-emphasis. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_dp_helper.c | 14 ++ include/drm/drm_dp_helper.h | 2 ++ 2 files changed, 16 insertions(+) diff

[PATCH 03/17] drm/dp: add LTTPR DP 2.0 DPCD addresses

2021-08-18 Thread Jani Nikula
DP 2.0 brings some new DPCD addresses for PHY repeaters. Cc: dri-devel@lists.freedesktop.org Reviewed-by: Manasi Navare Signed-off-by: Jani Nikula --- include/drm/drm_dp_helper.h | 4 1 file changed, 4 insertions(+) diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h

Re: [Intel-gfx] [PATCH v2 11/11] drm/i915: Extract i915_module.c

2021-08-23 Thread Jani Nikula
funcs); i++) { >> -err = init_funcs[i].init(); >> - if (err < 0) { >> -while (i--) { >> -if (init_funcs[i].exit) >> -init_funcs[i].exit(); >> -} >> -return err; >> -} else if (err > 0) { >> -/* >> - * Early-exit success is reserved for things which >> - * don't have an exit() function because we have no >> - * idea how far they got or how to partially tear >> - * them down. >> - */ >> -WARN_ON(init_funcs[i].exit); >> - >> -/* >> - * We don't want to advertise devices with an only >> - * partially initialized driver. >> - */ >> -WARN_ON(i915_pci_driver.driver.owner); >> -break; >> -} >> -} >> - >> -init_progress = i; >> - >> -return 0; >> -} >> - >> -static void __exit i915_exit(void) >> -{ >> -int i; >> - >> -for (i = init_progress - 1; i >= 0; i--) { >> -GEM_BUG_ON(i >= ARRAY_SIZE(init_funcs)); >> -if (init_funcs[i].exit) >> -init_funcs[i].exit(); >> -} >> -} >> - >> -module_init(i915_init); >> -module_exit(i915_exit); >> - >> -MODULE_AUTHOR("Tungsten Graphics, Inc."); >> -MODULE_AUTHOR("Intel Corporation"); >> - >> -MODULE_DESCRIPTION(DRIVER_DESC); >> -MODULE_LICENSE("GPL and additional rights"); >> diff --git a/drivers/gpu/drm/i915/i915_pci.h >> b/drivers/gpu/drm/i915/i915_pci.h >> new file mode 100644 >> index ..b386f319f52e >> --- /dev/null >> +++ b/drivers/gpu/drm/i915/i915_pci.h >> @@ -0,0 +1,8 @@ >> +/* >> + * SPDX-License-Identifier: MIT >> + * >> + * Copyright © 2021 Intel Corporation >> + */ >> + >> +int i915_register_pci_driver(void); >> +void i915_unregister_pci_driver(void); -- Jani Nikula, Intel Open Source Graphics Center

[PATCH v2 01/19] drm/dp: add DP 2.0 UHBR link rate and bw code conversions

2021-08-23 Thread Jani Nikula
The bw code equals link_rate / 0.27 Gbps only for 8b/10b link rates. Handle DP 2.0 UHBR rates as special cases, though this is not pretty. Cc: dri-devel@lists.freedesktop.org Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_dp_helper.c | 26 ++ 1 file changed, 22

[PATCH v2 02/19] drm/dp: use more of the extended receiver cap

2021-08-23 Thread Jani Nikula
Extend the use of extended receiver cap at 0x2200 to cover MAIN_LINK_CHANNEL_CODING_CAP in 0x2206, in case an implementation hides the DP 2.0 128b/132b channel encoding cap. Cc: dri-devel@lists.freedesktop.org Reviewed-by: Manasi Navare Signed-off-by: Jani Nikula --- drivers/gpu/drm

[PATCH v2 03/19] drm/dp: add LTTPR DP 2.0 DPCD addresses

2021-08-23 Thread Jani Nikula
DP 2.0 brings some new DPCD addresses for PHY repeaters. Cc: dri-devel@lists.freedesktop.org Reviewed-by: Manasi Navare Signed-off-by: Jani Nikula --- include/drm/drm_dp_helper.h | 4 1 file changed, 4 insertions(+) diff --git a/include/drm/drm_dp_helper.h b/include/drm/drm_dp_helper.h

[PATCH v2 04/19] drm/dp: add helper for extracting adjust 128b/132b TX FFE preset

2021-08-23 Thread Jani Nikula
The DP 2.0 128b/132b channel coding uses TX FFE presets instead of vswing and pre-emphasis. Cc: dri-devel@lists.freedesktop.org Reviewed-by: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_dp_helper.c | 14 ++ include/drm/drm_dp_helper.h | 2 ++ 2 files

Re: [GIT PULL] drm-misc + drm-intel: Add support for out-of-band hotplug notification

2021-08-24 Thread Jani Nikula
typec: altmodes/displayport: Notify drm subsys of hotplug events > > drivers/gpu/drm/drm_connector.c | 79 + > drivers/gpu/drm/drm_crtc_internal.h | 2 + > drivers/gpu/drm/drm_sysfs.c | 87 > +++- > driver

Re: [Intel-gfx] [PATCH 2/4] drm/dp: use more of the extended receiver cap

2021-08-25 Thread Jani Nikula
On Thu, 19 Aug 2021, Ville Syrjälä wrote: > On Fri, Aug 13, 2021 at 01:43:20PM +0300, Jani Nikula wrote: >> Extend the use of extended receiver cap at 0x2200 to cover >> MAIN_LINK_CHANNEL_CODING_CAP in 0x2206, in case an implementation hides >> the DP 2.0 128b/132b channel en

Re: [PATCH v2 11/11] drm/i915: Extract i915_module.c

2021-08-25 Thread Jani Nikula
7;re using GEM_BUG_ON() in generic driver code. BR, Jani. > + if (init_funcs[i].exit) > + init_funcs[i].exit(); > + } > +} -- Jani Nikula, Intel Open Source Graphics Center

Re: [PATCH 21/33] drm/i915/guc: Connect reset modparam updates to GuC policy flags

2021-08-26 Thread Jani Nikula
add this kind of checks in the middle of the generic functions. What if the type was bool or ulong, where the generic function is a debugfs helper outside of i915? See the comment in i915_debugfs_params() that I added there exactly because I envisioned someone was going to need this facility:

[PULL] drm-intel-next-fixes

2021-08-26 Thread Jani Nikula
display.c | 3 +++ drivers/gpu/drm/i915/gt/intel_migrate.c | 2 +- drivers/gpu/drm/i915/gt/selftest_hangcheck.c | 4 +-- drivers/gpu/drm/i915/i915_module.c | 37 ++-- 4 files changed, 30 insertions(+), 16 deletions(-) -- Jani Nikula, Intel Open Source Graphics Center

Re: [Intel-gfx] [PATCH 2/4] drm/dp: use more of the extended receiver cap

2021-08-26 Thread Jani Nikula
On Wed, 25 Aug 2021, Jani Nikula wrote: > On Thu, 19 Aug 2021, Ville Syrjälä wrote: >> On Fri, Aug 13, 2021 at 01:43:20PM +0300, Jani Nikula wrote: >>> Extend the use of extended receiver cap at 0x2200 to cover >>> MAIN_LINK_CHANNEL_CODING_CAP in 0x2206, in case an i

Re: [Intel-gfx] [PATCH 1/2] drm/i915/dp_link_training: Add newlines to debug messages

2021-03-15 Thread Jani Nikula
dbg_kms(&dp_to_i915(intel_dp)->drm, >> -"[CONNECTOR:%d:%s] Link Training %s at link rate = %d, lane >> count = %d, at %s", >> +"[CONNECTOR:%d:%s] Link Training %s at link rate = %d, lane >> co

[PULL] drm-intel-next

2021-03-16 Thread Jani Nikula
e 100644 drivers/gpu/drm/i915/display/skl_scaler.c create mode 100644 drivers/gpu/drm/i915/display/skl_scaler.h create mode 100644 drivers/gpu/drm/i915/display/skl_universal_plane.c create mode 100644 drivers/gpu/drm/i915/display/skl_universal_plane.h -- Jani Nikula, Intel Open Source Graphics Cen

[PULL] drm-intel-fixes

2021-03-18 Thread Jani Nikula
| 13 + drivers/gpu/drm/i915/i915_reg.h | 23 ++- drivers/gpu/drm/i915/intel_pm.c | 16 +++- 3 files changed, 42 insertions(+), 10 deletions(-) -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list

Re: [PATCH] drm: i915: Fix a typo

2021-03-19 Thread Jani Nikula
gt; val = intel_de_read(dev_priv, enable_reg); > -- > 2.26.2 > -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [Intel-gfx] [PATCH 1/4] drm/i915: Drop I915_CONTEXT_PARAM_RINGSIZE

2021-03-22 Thread Jani Nikula
) I use this monster in my .gitconfig: [alias] cite = "!f() { git log -1 '--pretty=format:%H (\"%s\")%n' $1 | sed -e 's/\\([0-f]\\{12\\}\\)[0-f]*/\\1/'; }; f" With that, 'git cite ' will give you the nice reference with 12 chars of sha1

Re: [PATCH] drm/i915: A typo fix

2021-03-23 Thread Jani Nikula
nothign here. > + * nothing here. >*/ > > val = intel_de_read(dev_priv, enable_reg); > -- > 2.31.0 > -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PATCH 1/2] drm/i915: add gem/gt TODO

2021-03-23 Thread Jani Nikula
ms than via patches on the mailing list. Granted, the TODO file enforces the discussion happens at a large enough audience, but for at least some of the items I'd suggest filing gitlab issues [1], with todo label, and tracking there. BR, Jani. [1] https://gitlab.freedesktop.org/drm/intel/-

Re: [PATCH] drm/i915/dpcd_bl: Don't try vesa interface unless specified by VBT

2021-03-23 Thread Jani Nikula
;> >> So it's possible that this value might just need to be tweaked, but for now >> let's just disable the VESA backlight interface unless it's specified in >> the VBT just to be safe. We might be able to try enabling this again by >> default in the fu

Re: [PATCH 10/11] drm/i915: avoid stringop-overread warning on pri_latency

2021-03-24 Thread Jani Nikula
on't think there's an issue, but the code could use a bunch of improvements. Like, we have intel_print_wm_latency() for debug logging and wm_latency_show() for debugfs, and there's a bunch of duplication and ugh. But this seems like the easiest fix for the warning. Reviewed-by: Jani Nikula

Re: [PATCH 11/11] [RFC] drm/i915/dp: fix array overflow warning

2021-03-25 Thread Jani Nikula
#include "intel_vdsc.h" > #include "intel_vrr.h" > > -#define DP_DPRX_ESI_LEN 14 > +#define DP_DPRX_ESI_LEN 16 > > /* DP DSC throughput values used for slice count calculations KPixels/s */ > #define DP_DSC_PEAK_PIXEL_RATE

Re: [Intel-gfx] [PATCH v2 0/2] HDCP 2.2 DP errata

2021-03-25 Thread Jani Nikula
lay/intel_dp_hdcp.c | 45 ++-- > include/drm/drm_hdcp.h | 5 ++- > 2 files changed, 36 insertions(+), 14 deletions(-) Maarten, Maxime, Thomas - Can I get an ack for merging this via drm-intel-next, please? BR, Jani. -- Jani Nikula, Int

Re: [Intel-gfx] [PATCH v5 1/2] drm/dp_helper: Define options for FRL training for HDMI2.1 PCON

2021-03-26 Thread Jani Nikula
p_pcon_frl_prepare(struct drm_dp_aux *aux, bool > enable_frl_ready_hpd); > bool drm_dp_pcon_is_frl_ready(struct drm_dp_aux *aux); > int drm_dp_pcon_frl_configure_1(struct drm_dp_aux *aux, int max_frl_gbps, > - bool concurrent_mode); > + u8 frl_mode); > int drm_dp_pcon_frl_configure_2(struct drm_dp_aux *aux, int max_frl_mask, > - bool extended_train_mode); > + u8 frl_type); > int drm_dp_pcon_reset_frl_config(struct drm_dp_aux *aux); > int drm_dp_pcon_frl_enable(struct drm_dp_aux *aux); -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [PULL] topic/i915-gem-next

2021-03-26 Thread Jani Nikula
in drm-next they will backmerge, and the -gt-next tree is open for > business again. I guess worth noting is that drm-intel-gt-next is now rebased on top of current drm-next. Since the topic branch is part of drm-tip, I presume the conflicts are manageable. BR, Jani. -- Jani Nikula, Intel Ope

[PATCH v2 0/8] drm/edid: overhaul displayid iterator

2021-03-29 Thread Jani Nikula
v2 of [1], addressing Ville's review comments, and adding a couple of extra patches on top. BR, Jani. [1] https://patchwork.freedesktop.org/series/87802/ Jani Nikula (8): drm/edid: make a number of functions, parameters and variables const drm/displayid: add separate drm_displayid.c

[PATCH v2 1/8] drm/edid: make a number of functions, parameters and variables const

2021-03-29 Thread Jani Nikula
If there's no need to change it, it should be const. There's more to be done, but start off with changes that make follow-up work easier. No functional changes. Reviewed-by: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_ed

[PATCH v2 2/8] drm/displayid: add separate drm_displayid.c

2021-03-29 Thread Jani Nikula
lä Signed-off-by: Jani Nikula --- drivers/gpu/drm/Makefile| 2 +- drivers/gpu/drm/drm_displayid.c | 59 + drivers/gpu/drm/drm_edid.c | 58 ++-- include/drm/drm_displayid.h | 8 + include/drm/drm_edid.h

[PATCH v2 3/8] drm/displayid: add new displayid section/block iterators

2021-03-29 Thread Jani Nikula
ille) - remove __ prefix from displayid_iter_block Reviewed-by: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_displayid.c | 74 + include/drm/drm_displayid.h | 18 2 files changed, 92 insertions(+) diff --git a/drivers/gpu/drm/drm_displayid.c b/dr

[PATCH v2 4/8] drm/edid: use the new displayid iterator for detailed modes

2021-03-29 Thread Jani Nikula
Neatly reduce displayid boilerplate in code. No functional changes. Reviewed-by: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_edid.c | 23 ++- 1 file changed, 6 insertions(+), 17 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm

[PATCH v2 5/8] drm/edid: use the new displayid iterator for finding CEA extension

2021-03-29 Thread Jani Nikula
Neatly reduce displayid boilerplate in code. No functional changes. Reviewed-by: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_edid.c | 25 + 1 file changed, 9 insertions(+), 16 deletions(-) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm

[PATCH v2 6/8] drm/edid: use the new displayid iterator for tile info

2021-03-29 Thread Jani Nikula
Neatly reduce displayid boilerplate in code. Remove excessive debug logging while at it, no other functional changes. The old displayid iterator becomes unused; remove it as well as make drm_find_displayid_extension() static. Reviewed-by: Ville Syrjälä Signed-off-by: Jani Nikula --- drivers

[PATCH v2 7/8] drm/displayid: allow data blocks with 0 payload length

2021-03-29 Thread Jani Nikula
The DisplayID specifications explicitly call out 0 as a valid payload length for data blocks. The mere presence of a data block, or the information coded in the block specific data (bits 7:3 in offset 1), may be enough to convey the necessary information. Signed-off-by: Jani Nikula --- drivers

[PATCH v2 8/8] drm/displayid: rename displayid_hdr to displayid_header

2021-03-29 Thread Jani Nikula
Avoid any confusion with High Dynamic Range. No functional changes. Signed-off-by: Jani Nikula --- drivers/gpu/drm/drm_displayid.c | 10 +- include/drm/drm_displayid.h | 2 +- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/drm_displayid.c b/drivers

Re: [Intel-gfx] [PATCH v5 1/2] drm/dp_helper: Define options for FRL training for HDMI2.1 PCON

2021-03-31 Thread Jani Nikula
On Fri, 26 Mar 2021, Maxime Ripard wrote: > Hi, > > On Fri, Mar 26, 2021 at 11:47:58AM +0200, Jani Nikula wrote: >> On Tue, 23 Mar 2021, Ankit Nautiyal wrote: >> > Currently the FRL training mode (Concurrent, Sequential) and >> > training type (Normal, Exten

Re: [Intel-gfx] [PATCH v2 0/2] HDCP 2.2 DP errata

2021-03-31 Thread Jani Nikula
On Thu, 25 Mar 2021, Jani Nikula wrote: > On Wed, 24 Mar 2021, Anshuman Gupta wrote: >> HDCP DP 2.2 errata is part of HDCP DP 2.3 specs >> as well. >> >> Anshuman Gupta (2): >> drm/i915/hdcp: Add DP HDCP2.2 timeout to read entire msg >> drm/hdc

Re: [Intel-gfx] [PATCH v2 1/2] drm/i915/hdcp: Add DP HDCP2.2 timeout to read entire msg

2021-03-31 Thread Jani Nikula
} >> +} >> + >> byte = buf; >> *byte = msg_id; >> >> diff --git a/include/drm/drm_hdcp.h b/include/drm/drm_hdcp.h >> index ac22c246542a..2b165a0f434f 100644 >> --- a/include/drm/drm_hdcp.h >> +++ b/include/drm/drm_hdcp.h >>

Re: [PATCH v2 0/8] drm/edid: overhaul displayid iterator

2021-03-31 Thread Jani Nikula
On Mon, 29 Mar 2021, Jani Nikula wrote: > v2 of [1], addressing Ville's review comments, and adding a couple of > extra patches on top. > > BR, > Jani. > > [1] https://patchwork.freedesktop.org/series/87802/ Pushed to drm-misc-next, with Ville's review and Maxim

[PULL] drm-intel-next

2021-04-01 Thread Jani Nikula
dst_stride!=width/height drm/i915: Fix docbook header for __intel_runtime_pm_get_if_active() Jani Nikula (23): drm/i915: remove unused ADLS_REVID_* macros drm/i915/bios: mass convert dev_priv to i915 drm/i915/bios: store bdb version in i915 drm/i915/bios: re

Re: [PATCH v2 05/20] drm/dp: Add backpointer to drm_device in drm_dp_aux

2021-04-01 Thread Jani Nikula
c; > struct device *dev; > + struct drm_device *drm_dev; Bikeshed, I would probably have called it just drm for brevity, but no strong feelings. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center ___ dri-devel mailing lis

Re: [Intel-gfx] [PATCH v2 00/20] drm: Use new DRM printk funcs (like drm_dbg_*()) in DP helpers

2021-04-01 Thread Jani Nikula
drm_dp_mst_topology_mgr > to drm_dp_get_vc_payload_bw()" as this was the style in use > previously, and 2 chars over the limit looks nicer then trying to > line-wrap this > * Don't rewrap comments For anything touching i915, and for merging via whichever tree or branch see

Re: [PATCH] pwm: Rename pwm_get_state() to better reflect its semantic

2021-04-06 Thread Jani Nikula
by the consumer (opposed to what was actually implemented in > hardware in reply to the last request). To make this semantic obvious > rename the function. > > Signed-off-by: Uwe Kleine-König > --- > drivers/gpu/drm/i915/display/intel_panel.c | 4 +-- Acked-by: Jani Nikula -

Re: [PATCH] drm/i915/sysfs: convert snprintf to sysfs_emit

2021-04-06 Thread Jani Nikula
gt_max_freq_mhz_store(struct device *kdev, > @@ -387,8 +382,7 @@ static ssize_t gt_min_freq_mhz_show(struct device *kdev, > struct device_attribute > struct drm_i915_private *dev_priv = kdev_minor_to_i915(kdev); > struct intel_rps *rps = &dev_priv->gt.rps; > > -

Re: [PATCH v6 1/4] drm/i915: Keep track of pwm-related backlight hooks separately

2021-01-13 Thread Jani Nikula
7;t: if (!panel->backlight.level) panel->backlight.level = intel_pwm_get_backlight(connector); What do you think? BR, Jani. > - val = intel_panel_compute_brightness(connector, val); > - panel->backlight.level = clamp(val, panel->backlight.min, > -

  1   2   3   4   5   6   7   8   9   10   >