[Nouveau] [Bug 105319] DRM: EVO timeout with kernel 4.15.x
https://bugs.freedesktop.org/show_bug.cgi?id=105319 --- Comment #30 from Stef Bon --- I can confirm this nast bug. I also have errormessages like: nouveau :01:00.0: DRM: base-0: timeout (a lot of them). The X systenm becomnes very slow, and strange garph. effects occur. But after seconds it stablizes again. It happens specially when moving windows. I've got an arch system, but they don't seem to care there. My system: uname -a Linux ws-001 4.19.8-arch1-1-ARCH #1 SMP PREEMPT Sat Dec 8 13:49:11 UTC 2018 x86_64 GNU/Linux I've got a dual monitor setup, with 01:00.0 VGA compatible controller: NVIDIA Corporation GP107 [GeForce GTX 1050 Ti] (rev a1) I think it's somewhere in the dri code is my intuition. Stef Bon -- You are receiving this mail because: You are the assignee for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
[Nouveau] [Bug 108980] GF117: MMIO write of 0000001f FAULT at 6013d4 [ IBUS ]
https://bugs.freedesktop.org/show_bug.cgi?id=108980 --- Comment #12 from Ilia Mirkin --- Created attachment 142798 --> https://bugs.freedesktop.org/attachment.cgi?id=142798=edit fix gf117 volt speedo fuse id Legit bug here -- looks like the speedo fuse moved on GF117 to the kepler location. (Still in the old place on GF119, which logically came before GF117.) -- You are receiving this mail because: You are the assignee for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
[Nouveau] [Bug 108980] GF117: MMIO write of 0000001f FAULT at 6013d4 [ IBUS ]
https://bugs.freedesktop.org/show_bug.cgi?id=108980 --- Comment #11 from Ilia Mirkin --- (In reply to Johnny B. Goode from comment #10) > [ 15.106637] nouveau :08:00.0: volt: couldn't find speedo value, > volting not possible > [ 15.106651] nouveau :08:00.0: bus: MMIO read of FAULT at > 0212cc [ IBUS ] $ lookup -a d7 212cc PFUSE.FUSES.SPEEDO => 0 Ben - do we need to enable something more? We're going through gf100_fuse_read, which is pretty careful about enabling / disabling fuses around the read. Is fuse 0x1cc just not available on GF117? -- You are receiving this mail because: You are the assignee for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
[Nouveau] [Bug 108980] GF117: MMIO write of 0000001f FAULT at 6013d4 [ IBUS ]
https://bugs.freedesktop.org/show_bug.cgi?id=108980 --- Comment #10 from Johnny B. Goode --- It's like chasing rabbits. dmesg | grep nouveau [ 14.869172] nouveau: detected PR support, will not use DSM [ 14.869201] nouveau :08:00.0: enabling device (0006 -> 0007) [ 14.979493] nouveau :08:00.0: NVIDIA GF117 (0d7000a2) [ 15.005578] nouveau :08:00.0: bios: version 75.17.86.00.04 [ 15.089370] nouveau :08:00.0: fb: 2048 MiB DDR3 [ 15.106637] nouveau :08:00.0: volt: couldn't find speedo value, volting not possible [ 15.106651] nouveau :08:00.0: bus: MMIO read of FAULT at 0212cc [ IBUS ] [ 15.753323] nouveau :08:00.0: DRM: VRAM: 2048 MiB [ 15.754432] nouveau :08:00.0: DRM: GART: 1048576 MiB [ 15.755693] nouveau :08:00.0: DRM: Pointer to TMDS table invalid [ 15.756797] nouveau :08:00.0: DRM: DCB version 4.0 [ 15.760007] nouveau :08:00.0: DRM: MM: using COPY0 for buffer copies [ 15.761582] [drm] Initialized nouveau 1.3.1 20120801 for :08:00.0 on minor 1 -- You are receiving this mail because: You are the assignee for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
Re: [Nouveau] [PATCH] drm/nouveau: Fix memory leak in nv50_mstm_del()
v4.19.8, it's ok to skip v4.14 for this On Wed, 2018-12-12 at 05:46 +, Sasha Levin wrote: > Hi, > > [This is an automated email] > > This commit has been processed because it contains a "Fixes:" tag, > fixing commit: f479c0ba4a17 drm/nouveau/kms/nv50: initial support for DP 1.2 > multi-stream. > > The bot has tested the following trees: v4.19.8, v4.14.87, > > v4.19.8: Build OK! > v4.14.87: Failed to apply! Possible dependencies: > Unable to calculate > > > How should we proceed with this patch? > > -- > Thanks, > Sasha -- Cheers, Lyude Paul ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
[Nouveau] [Bug 109035] amazon echo dot technical support phone number
https://bugs.freedesktop.org/show_bug.cgi?id=109035 Roy changed: What|Removed |Added Version|18.1|unspecified QA Contact|nouveau@lists.freedesktop.o | |rg | Assignee|nouveau@lists.freedesktop.o |dan...@fooishbar.org |rg | URL|https://www.customercarenum | |ber-usa.com/amazon-echo-ale | |xa-customer-support/| Product|Mesa|Spam Component|Drivers/DRI/nouveau |Two -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
Re: [Nouveau] [PATCH 4/7] drm: Move the legacy kms disable_all helper to crtc helpers
On Mon, Dec 10, 2018 at 10:58:20AM -0500, Alex Deucher wrote: > On Mon, Dec 10, 2018 at 5:04 AM Daniel Vetter wrote: > > > > It's not a core function, and the matching atomic functions are also > > not in the core. Plus the suspend/resume helper is also already there. > > > > Needs a tiny bit of open-coding, but less midlayer beats that I think. > > > > Cc: Sam Bobroff > > Signed-off-by: Daniel Vetter > > Cc: Maarten Lankhorst > > Cc: Maxime Ripard > > Cc: Sean Paul > > Cc: David Airlie > > Cc: Ben Skeggs > > Cc: Alex Deucher > > Cc: "Christian König" > > Cc: "David (ChunMing) Zhou" > > Cc: Rex Zhu > > Cc: Andrey Grodzovsky > > Cc: Huang Rui > > Cc: Shaoyun Liu > > Cc: Monk Liu > > Cc: nouveau@lists.freedesktop.org > > Cc: amd-...@lists.freedesktop.org > > --- > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- > > drivers/gpu/drm/drm_crtc.c | 31 --- > > drivers/gpu/drm/drm_crtc_helper.c | 35 ++ > > drivers/gpu/drm/nouveau/nouveau_display.c | 2 +- > > drivers/gpu/drm/radeon/radeon_display.c| 2 +- > > include/drm/drm_crtc.h | 2 -- > > include/drm/drm_crtc_helper.h | 1 + > > 7 files changed, 39 insertions(+), 36 deletions(-) > > /snip > > diff --git a/drivers/gpu/drm/drm_crtc_helper.c > > b/drivers/gpu/drm/drm_crtc_helper.c > > index a3c81850e755..23159eb494f1 100644 > > --- a/drivers/gpu/drm/drm_crtc_helper.c > > +++ b/drivers/gpu/drm/drm_crtc_helper.c > > @@ -984,3 +984,38 @@ void drm_helper_resume_force_mode(struct drm_device > > *dev) > > drm_modeset_unlock_all(dev); > > } > > EXPORT_SYMBOL(drm_helper_resume_force_mode); > > + > > +/** > > + * drm_helper_force_disable_all - Forcibly turn off all enabled CRTCs > > + * @dev: DRM device whose CRTCs to turn off > > + * > > + * Drivers may want to call this on unload to ensure that all displays are > > + * unlit and the GPU is in a consistent, low power state. Takes modeset > > locks. > > + * > > + * Note: This should only be used by non-atomic legacy drivers. For an > > atomic > > + * version look at drm_atomic_helper_shutdown(). > > + * > > + * Returns: > > + * Zero on success, error code on failure. > > + */ > > +int drm_helper_force_disable_all(struct drm_device *dev) > > Maybe put crtc somewhere in the function name so it's clear what we > are disabling. FWIW, I think it's more clear this way. set_config_internal will turn off everything attached to the crtc, so _everything_ will be disabled in this case. Either way, Reviewed-by: Sean Paul Sean > With that fixed: > Reviewed-by: Alex Deucher > > > +{ > > + struct drm_crtc *crtc; > > + int ret = 0; > > + > > + drm_modeset_lock_all(dev); > > + drm_for_each_crtc(crtc, dev) > > + if (crtc->enabled) { > > + struct drm_mode_set set = { > > + .crtc = crtc, > > + }; > > + > > + ret = drm_mode_set_config_internal(); > > + if (ret) > > + goto out; > > + } > > +out: > > + drm_modeset_unlock_all(dev); > > + return ret; > > +} > > +EXPORT_SYMBOL(drm_helper_force_disable_all); > > diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c > > b/drivers/gpu/drm/nouveau/nouveau_display.c > > index f326ffd86766..5d273a655479 100644 > > --- a/drivers/gpu/drm/nouveau/nouveau_display.c > > +++ b/drivers/gpu/drm/nouveau/nouveau_display.c > > @@ -453,7 +453,7 @@ nouveau_display_fini(struct drm_device *dev, bool > > suspend, bool runtime) > > if (drm_drv_uses_atomic_modeset(dev)) > > drm_atomic_helper_shutdown(dev); > > else > > - drm_crtc_force_disable_all(dev); > > + drm_helper_force_disable_all(dev); > > } > > > > /* disable flip completion events */ > > diff --git a/drivers/gpu/drm/radeon/radeon_display.c > > b/drivers/gpu/drm/radeon/radeon_display.c > > index e6912eb99b42..92332226e5cf 100644 > > --- a/drivers/gpu/drm/radeon/radeon_display.c > > +++ b/drivers/gpu/drm/radeon/radeon_display.c > > @@ -1643,7 +1643,7 @@ void radeon_modeset_fini(struct radeon_device *rdev) > > if (rdev->mode_info.mode_config_initialized) { > > drm_kms_helper_poll_fini(rdev->ddev); > > radeon_hpd_fini(rdev); > > - drm_crtc_force_disable_all(rdev->ddev); > > + drm_helper_force_disable_all(rdev->ddev); > > radeon_fbdev_fini(rdev); > > radeon_afmt_fini(rdev); > > drm_mode_config_cleanup(rdev->ddev); > > diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h > > index b45bec0b7a9c..85abd3fe9e83 100644 > > --- a/include/drm/drm_crtc.h > > +++ b/include/drm/drm_crtc.h > > @@ -1149,8 +1149,6 @@ static inline uint32_t drm_crtc_mask(const struct > > drm_crtc
Re: [Nouveau] [PATCH 4/7] drm: Move the legacy kms disable_all helper to crtc helpers
On Mon, Dec 10, 2018 at 5:04 AM Daniel Vetter wrote: > > It's not a core function, and the matching atomic functions are also > not in the core. Plus the suspend/resume helper is also already there. > > Needs a tiny bit of open-coding, but less midlayer beats that I think. > > Cc: Sam Bobroff > Signed-off-by: Daniel Vetter > Cc: Maarten Lankhorst > Cc: Maxime Ripard > Cc: Sean Paul > Cc: David Airlie > Cc: Ben Skeggs > Cc: Alex Deucher > Cc: "Christian König" > Cc: "David (ChunMing) Zhou" > Cc: Rex Zhu > Cc: Andrey Grodzovsky > Cc: Huang Rui > Cc: Shaoyun Liu > Cc: Monk Liu > Cc: nouveau@lists.freedesktop.org > Cc: amd-...@lists.freedesktop.org > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- > drivers/gpu/drm/drm_crtc.c | 31 --- > drivers/gpu/drm/drm_crtc_helper.c | 35 ++ > drivers/gpu/drm/nouveau/nouveau_display.c | 2 +- > drivers/gpu/drm/radeon/radeon_display.c| 2 +- > include/drm/drm_crtc.h | 2 -- > include/drm/drm_crtc_helper.h | 1 + > 7 files changed, 39 insertions(+), 36 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > index c75badfa5c4c..e669297ffefb 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > @@ -2689,7 +2689,7 @@ void amdgpu_device_fini(struct amdgpu_device *adev) > amdgpu_irq_disable_all(adev); > if (adev->mode_info.mode_config_initialized){ > if (!amdgpu_device_has_dc_support(adev)) > - drm_crtc_force_disable_all(adev->ddev); > + drm_helper_force_disable_all(adev->ddev); > else > drm_atomic_helper_shutdown(adev->ddev); > } > diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c > index f660819d406e..7dabbaf033a1 100644 > --- a/drivers/gpu/drm/drm_crtc.c > +++ b/drivers/gpu/drm/drm_crtc.c > @@ -104,37 +104,6 @@ int drm_crtc_force_disable(struct drm_crtc *crtc) > return drm_mode_set_config_internal(); > } > > -/** > - * drm_crtc_force_disable_all - Forcibly turn off all enabled CRTCs > - * @dev: DRM device whose CRTCs to turn off > - * > - * Drivers may want to call this on unload to ensure that all displays are > - * unlit and the GPU is in a consistent, low power state. Takes modeset > locks. > - * > - * Note: This should only be used by non-atomic legacy drivers. For an atomic > - * version look at drm_atomic_helper_shutdown(). > - * > - * Returns: > - * Zero on success, error code on failure. > - */ > -int drm_crtc_force_disable_all(struct drm_device *dev) > -{ > - struct drm_crtc *crtc; > - int ret = 0; > - > - drm_modeset_lock_all(dev); > - drm_for_each_crtc(crtc, dev) > - if (crtc->enabled) { > - ret = drm_crtc_force_disable(crtc); > - if (ret) > - goto out; > - } > -out: > - drm_modeset_unlock_all(dev); > - return ret; > -} > -EXPORT_SYMBOL(drm_crtc_force_disable_all); > - > static unsigned int drm_num_crtcs(struct drm_device *dev) > { > unsigned int num = 0; > diff --git a/drivers/gpu/drm/drm_crtc_helper.c > b/drivers/gpu/drm/drm_crtc_helper.c > index a3c81850e755..23159eb494f1 100644 > --- a/drivers/gpu/drm/drm_crtc_helper.c > +++ b/drivers/gpu/drm/drm_crtc_helper.c > @@ -984,3 +984,38 @@ void drm_helper_resume_force_mode(struct drm_device *dev) > drm_modeset_unlock_all(dev); > } > EXPORT_SYMBOL(drm_helper_resume_force_mode); > + > +/** > + * drm_helper_force_disable_all - Forcibly turn off all enabled CRTCs > + * @dev: DRM device whose CRTCs to turn off > + * > + * Drivers may want to call this on unload to ensure that all displays are > + * unlit and the GPU is in a consistent, low power state. Takes modeset > locks. > + * > + * Note: This should only be used by non-atomic legacy drivers. For an atomic > + * version look at drm_atomic_helper_shutdown(). > + * > + * Returns: > + * Zero on success, error code on failure. > + */ > +int drm_helper_force_disable_all(struct drm_device *dev) Maybe put crtc somewhere in the function name so it's clear what we are disabling. With that fixed: Reviewed-by: Alex Deucher > +{ > + struct drm_crtc *crtc; > + int ret = 0; > + > + drm_modeset_lock_all(dev); > + drm_for_each_crtc(crtc, dev) > + if (crtc->enabled) { > + struct drm_mode_set set = { > + .crtc = crtc, > + }; > + > + ret = drm_mode_set_config_internal(); > + if (ret) > + goto out; > + } > +out: > + drm_modeset_unlock_all(dev); > + return ret; > +} > +EXPORT_SYMBOL(drm_helper_force_disable_all); > diff
Re: [Nouveau] [PATCH 7/7] drm: Split out drm_probe_helper.h
On Mon, Dec 10, 2018 at 11:11:33AM +0100, Daniel Vetter wrote: > Having the probe helper stuff (which pretty much everyone needs) in > the drm_crtc_helper.h file (which atomic drivers should never need) is > confusing. Split them out. > > To make sure I actually achieved the goal here I went through all > drivers. And indeed, all atomic drivers are now free of > drm_crtc_helper.h includes. > > Signed-off-by: Daniel Vetter > Cc: linux-arm-ker...@lists.infradead.org > Cc: virtualizat...@lists.linux-foundation.org > Cc: etna...@lists.freedesktop.org > Cc: linux-samsung-...@vger.kernel.org > Cc: intel-...@lists.freedesktop.org > Cc: linux-media...@lists.infradead.org > Cc: linux-amlo...@lists.infradead.org > Cc: linux-arm-...@vger.kernel.org > Cc: freedr...@lists.freedesktop.org > Cc: nouveau@lists.freedesktop.org > Cc: spice-de...@lists.freedesktop.org > Cc: amd-...@lists.freedesktop.org > Cc: linux-renesas-...@vger.kernel.org > Cc: linux-rockc...@lists.infradead.org > Cc: linux-st...@st-md-mailman.stormreply.com > Cc: linux-te...@vger.kernel.org > Cc: xen-de...@lists.xen.org > --- > .../gpu/drm/amd/amdgpu/amdgpu_connectors.c| 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c| 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 + > .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 2 +- > .../amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c | 2 +- > .../display/amdgpu_dm/amdgpu_dm_services.c| 2 +- > drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- > drivers/gpu/drm/arc/arcpgu_drv.c | 2 +- > drivers/gpu/drm/arc/arcpgu_sim.c | 2 +- > drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +- > drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- > drivers/gpu/drm/arm/malidp_crtc.c | 2 +- > drivers/gpu/drm/arm/malidp_drv.c | 2 +- > drivers/gpu/drm/arm/malidp_mw.c | 2 +- > drivers/gpu/drm/armada/armada_510.c | 2 +- > drivers/gpu/drm/armada/armada_crtc.c | 2 +- > drivers/gpu/drm/armada/armada_drv.c | 2 +- > drivers/gpu/drm/armada/armada_fb.c| 2 +- > drivers/gpu/drm/ast/ast_drv.c | 1 + > drivers/gpu/drm/ast/ast_mode.c| 1 + > .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c| 2 +- > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h | 2 +- > drivers/gpu/drm/bochs/bochs_drv.c | 1 + > drivers/gpu/drm/bochs/bochs_kms.c | 1 + > drivers/gpu/drm/bridge/adv7511/adv7511.h | 2 +- > drivers/gpu/drm/bridge/analogix-anx78xx.c | 3 +- > .../drm/bridge/analogix/analogix_dp_core.c| 2 +- > drivers/gpu/drm/bridge/cdns-dsi.c | 2 +- > drivers/gpu/drm/bridge/dumb-vga-dac.c | 2 +- > .../bridge/megachips-stdp-ge-b850v3-fw.c | 2 +- > drivers/gpu/drm/bridge/nxp-ptn3460.c | 2 +- > drivers/gpu/drm/bridge/panel.c| 2 +- > drivers/gpu/drm/bridge/parade-ps8622.c| 2 +- > drivers/gpu/drm/bridge/sii902x.c | 2 +- > drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 2 +- > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- > drivers/gpu/drm/bridge/tc358764.c | 2 +- > drivers/gpu/drm/bridge/tc358767.c | 2 +- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +- > drivers/gpu/drm/bridge/ti-tfp410.c| 2 +- > drivers/gpu/drm/cirrus/cirrus_drv.c | 1 + > drivers/gpu/drm/cirrus/cirrus_mode.c | 1 + > drivers/gpu/drm/drm_atomic_helper.c | 1 - > drivers/gpu/drm/drm_dp_mst_topology.c | 2 +- > drivers/gpu/drm/drm_modeset_helper.c | 2 +- > drivers/gpu/drm/drm_probe_helper.c| 2 +- > drivers/gpu/drm/drm_simple_kms_helper.c | 2 +- > drivers/gpu/drm/etnaviv/etnaviv_drv.h | 1 - > drivers/gpu/drm/exynos/exynos_dp.c| 2 +- > drivers/gpu/drm/exynos/exynos_drm_crtc.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_dpi.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_fb.c| 2 +- > drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_vidi.c | 2 +- > drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c| 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c | 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c | 2 +- > drivers/gpu/drm/gma500/psb_intel_drv.h| 1 + > .../gpu/drm/hisilicon/hibmc/hibmc_drm_de.c| 2 +- > .../gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 2 +- > .../gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c | 2 +- > .../gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c | 2 +- >
[Nouveau] TK1: DRM, Nouveau and VIC
Hi Thierry et al. I noticed that since commit 3dde5a2342cd ("ARM: tegra: Add VIC on Tegra124") graphics on Apalis TK1 is broken. During boot it fails loading the vic firmware: [1.595824] tegra-vic 5434.vic: Direct firmware load for nvidia/tegra124/vic03_ucode.bin failed with error -2 [1.606140] tegra-vic: probe of 5434.vic failed with error -2 Subsequently Tegra HDMI seems to fail completely: [2.379860] tegra-hdmi 5428.hdmi: failed to get PLL regulator And finally, Nouveau even crashes: [8.241115] nouveau 5700.gpu: Linked as a consumer to regulator.31 [8.247889] nouveau 5700.gpu: NVIDIA GK20A (0ea000a1) [8.253396] nouveau 5700.gpu: imem: using IOMMU [8.270210] Unable to handle kernel NULL pointer dereference at virtual address 006c [8.278340] pgd = (ptrval) [8.281250] [006c] *pgd= [8.284944] Internal error: Oops: 5 [#1] PREEMPT SMP ARM [8.290260] Modules linked in: nouveau(+) ttm [8.294625] CPU: 2 PID: 203 Comm: systemd-udevd Not tainted 4.20.0- rc5-next-20181207-8-g85b0f8e25f86-dirty #110 [8.305055] Hardware name: NVIDIA Tegra SoC (Flattened Device Tree) [8.311331] PC is at drm_plane_register_all+0x18/0x50 [8.316373] LR is at drm_modeset_register_all+0xc/0x70 [8.321513] pc : []lr : []psr: a0060013 [8.327768] sp : ed527c70 ip : ecc43ec0 fp : [8.332993] r10: 0016 r9 : ecc43e80 r8 : [8.338209] r7 : bf182c80 r6 : r5 : ed61b24c r4 : fffc [8.344735] r3 : 0002f000 r2 : r1 : 2e124000 r0 : ed61b000 [8.351260] Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none [8.358383] Control: 10c5387d Table: ad64c06a DAC: 0051 [8.364127] Process systemd-udevd (pid: 203, stack limit = 0x(ptrval)) [8.370654] Stack: (0xed527c70 to 0xed528000) [8.375004] 7c60: ed61b000 ed61b000 c0564cc8 [8.383177] 7c80: ed61b000 c054b5b8 0001 0001 [8.391355] 7ca0: ed527cc0 c0f08c48 ed61b000 bf180c5c bf0dc900 [8.399531] 7cc0: eda29208 5dfe844b ee9f2a10 bf180c5c c05a9328 [8.407695] 7ce0: c1006828 ee9f2a10 c100682c c05a744c ee9f2a10 bf180c5c [8.415871] 7d00: ee9f2a44 c05a77a8 c0f08c48 bf182980 c05a769c eefd14d0 c05a77a8 [8.424048] 7d20: ee9f2a10 bf180c5c ee9f2a44 c05a77a8 c0f08c48 bf182980 [8.432226] 7d40: c05a7884 ee9ebfb4 c0f08c48 bf180c5c c05a5790 ee88135c [8.440405] 7d60: ee9ebfb4 5dfe844b c0f71168 bf180c5c ee379e80 c0f71168 c05a692c [8.448570] 7d80: bf15dc00 bf180ac8 e000 bf180c5c bf180ac8 e000 bf1aa000 c05a84a0 [8.456746] 7da0: bf182b80 bf180ac8 e000 bf1aa170 c0fbd220 c0f08c48 e000 c0102ed0 [8.464924] 7dc0: ed53f4c0 006000c0 c01b3d98 000c 6113 bf182980 0040 c02592d0 [8.473102] 7de0: eda60200 2e124000 ee80 006000c0 006000c0 c01b3d98 000c c025a8cc [8.481281] 7e00: c024ce54 a113 bf182980 5dfe844b bf182980 0002 ed53f4c0 0002 [8.489459] 7e20: eceba000 c01b3dd4 c0f08c48 bf182980 ed527f40 0002 eceb9fc0 [8.497625] 7e40: 0002 c01b61a4 bf18298c 7fff bf182980 c01b2f88 c01b279c [8.505800] 7e60: bf1829c8 bf182a80 bf182b6c bf182ab0 c0b03ab0 c0d58964 c0ca726c c0ca7278 [8.513978] 7e80: c0ca72d0 c0f08c48 c02654a0 e000 bf00 [8.522157] 7ea0: 6e72656b 6c65 [8.530336] 7ec0: [8.538502] 7ee0: 5dfe844b 7fff c0f08c48 [8.546677] 7f00: 000f b6f761cc c0101204 ed526000 017b 004a3270 c01b66a4 [8.554855] 7f20: 7fff 0003 0001 004a3270 f0ced000 06e8994c [8.563032] 7f40: f0e37f3a f0e50a40 f0ced000 06e8994c f7b75f9c f7b75d34 f63e62dc 0016b000 [8.571209] 7f60: 0017f6f0 00050a48 003b 003c 0023 [8.579388] 7f80: 0014 5dfe844b 004c0ec0 0001 [8.587554] 7fa0: 017b c0101000 004c0ec0 000f b6f761cc 0002 [8.595730] 7fc0: 004c0ec0 0001 017b 0048e114 004a3270 [8.603908] 7fe0: bea8f990 bea8f980 b6f71269 b6e9f6c0 400d0010 000f [8.612096] [] (drm_plane_register_all) from [] (drm_modeset_register_all+0xc/0x70) [8.621499] [] (drm_modeset_register_all) from [] (drm_dev_register+0x168/0x1c4) [8.630855] [] (drm_dev_register) from [] (nouveau_platform_probe+0x6c/0x88 [nouveau]) [8.640739] [] (nouveau_platform_probe [nouveau]) from [] (platform_drv_probe+0x48/0x98) [8.650574] [] (platform_drv_probe) from []
[Nouveau] [PATCH 4/7] drm: Move the legacy kms disable_all helper to crtc helpers
It's not a core function, and the matching atomic functions are also not in the core. Plus the suspend/resume helper is also already there. Needs a tiny bit of open-coding, but less midlayer beats that I think. Cc: Sam Bobroff Signed-off-by: Daniel Vetter Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Sean Paul Cc: David Airlie Cc: Ben Skeggs Cc: Alex Deucher Cc: "Christian König" Cc: "David (ChunMing) Zhou" Cc: Rex Zhu Cc: Andrey Grodzovsky Cc: Huang Rui Cc: Shaoyun Liu Cc: Monk Liu Cc: nouveau@lists.freedesktop.org Cc: amd-...@lists.freedesktop.org --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- drivers/gpu/drm/drm_crtc.c | 31 --- drivers/gpu/drm/drm_crtc_helper.c | 35 ++ drivers/gpu/drm/nouveau/nouveau_display.c | 2 +- drivers/gpu/drm/radeon/radeon_display.c| 2 +- include/drm/drm_crtc.h | 2 -- include/drm/drm_crtc_helper.h | 1 + 7 files changed, 39 insertions(+), 36 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c index c75badfa5c4c..e669297ffefb 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c @@ -2689,7 +2689,7 @@ void amdgpu_device_fini(struct amdgpu_device *adev) amdgpu_irq_disable_all(adev); if (adev->mode_info.mode_config_initialized){ if (!amdgpu_device_has_dc_support(adev)) - drm_crtc_force_disable_all(adev->ddev); + drm_helper_force_disable_all(adev->ddev); else drm_atomic_helper_shutdown(adev->ddev); } diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c index f660819d406e..7dabbaf033a1 100644 --- a/drivers/gpu/drm/drm_crtc.c +++ b/drivers/gpu/drm/drm_crtc.c @@ -104,37 +104,6 @@ int drm_crtc_force_disable(struct drm_crtc *crtc) return drm_mode_set_config_internal(); } -/** - * drm_crtc_force_disable_all - Forcibly turn off all enabled CRTCs - * @dev: DRM device whose CRTCs to turn off - * - * Drivers may want to call this on unload to ensure that all displays are - * unlit and the GPU is in a consistent, low power state. Takes modeset locks. - * - * Note: This should only be used by non-atomic legacy drivers. For an atomic - * version look at drm_atomic_helper_shutdown(). - * - * Returns: - * Zero on success, error code on failure. - */ -int drm_crtc_force_disable_all(struct drm_device *dev) -{ - struct drm_crtc *crtc; - int ret = 0; - - drm_modeset_lock_all(dev); - drm_for_each_crtc(crtc, dev) - if (crtc->enabled) { - ret = drm_crtc_force_disable(crtc); - if (ret) - goto out; - } -out: - drm_modeset_unlock_all(dev); - return ret; -} -EXPORT_SYMBOL(drm_crtc_force_disable_all); - static unsigned int drm_num_crtcs(struct drm_device *dev) { unsigned int num = 0; diff --git a/drivers/gpu/drm/drm_crtc_helper.c b/drivers/gpu/drm/drm_crtc_helper.c index a3c81850e755..23159eb494f1 100644 --- a/drivers/gpu/drm/drm_crtc_helper.c +++ b/drivers/gpu/drm/drm_crtc_helper.c @@ -984,3 +984,38 @@ void drm_helper_resume_force_mode(struct drm_device *dev) drm_modeset_unlock_all(dev); } EXPORT_SYMBOL(drm_helper_resume_force_mode); + +/** + * drm_helper_force_disable_all - Forcibly turn off all enabled CRTCs + * @dev: DRM device whose CRTCs to turn off + * + * Drivers may want to call this on unload to ensure that all displays are + * unlit and the GPU is in a consistent, low power state. Takes modeset locks. + * + * Note: This should only be used by non-atomic legacy drivers. For an atomic + * version look at drm_atomic_helper_shutdown(). + * + * Returns: + * Zero on success, error code on failure. + */ +int drm_helper_force_disable_all(struct drm_device *dev) +{ + struct drm_crtc *crtc; + int ret = 0; + + drm_modeset_lock_all(dev); + drm_for_each_crtc(crtc, dev) + if (crtc->enabled) { + struct drm_mode_set set = { + .crtc = crtc, + }; + + ret = drm_mode_set_config_internal(); + if (ret) + goto out; + } +out: + drm_modeset_unlock_all(dev); + return ret; +} +EXPORT_SYMBOL(drm_helper_force_disable_all); diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c b/drivers/gpu/drm/nouveau/nouveau_display.c index f326ffd86766..5d273a655479 100644 --- a/drivers/gpu/drm/nouveau/nouveau_display.c +++ b/drivers/gpu/drm/nouveau/nouveau_display.c @@ -453,7 +453,7 @@ nouveau_display_fini(struct drm_device *dev, bool suspend, bool runtime) if (drm_drv_uses_atomic_modeset(dev)) drm_atomic_helper_shutdown(dev);
[Nouveau] [PATCH 7/7] drm: Split out drm_probe_helper.h
Having the probe helper stuff (which pretty much everyone needs) in the drm_crtc_helper.h file (which atomic drivers should never need) is confusing. Split them out. To make sure I actually achieved the goal here I went through all drivers. And indeed, all atomic drivers are now free of drm_crtc_helper.h includes. Signed-off-by: Daniel Vetter Cc: linux-arm-ker...@lists.infradead.org Cc: virtualizat...@lists.linux-foundation.org Cc: etna...@lists.freedesktop.org Cc: linux-samsung-...@vger.kernel.org Cc: intel-...@lists.freedesktop.org Cc: linux-media...@lists.infradead.org Cc: linux-amlo...@lists.infradead.org Cc: linux-arm-...@vger.kernel.org Cc: freedr...@lists.freedesktop.org Cc: nouveau@lists.freedesktop.org Cc: spice-de...@lists.freedesktop.org Cc: amd-...@lists.freedesktop.org Cc: linux-renesas-...@vger.kernel.org Cc: linux-rockc...@lists.infradead.org Cc: linux-st...@st-md-mailman.stormreply.com Cc: linux-te...@vger.kernel.org Cc: xen-de...@lists.xen.org --- .../gpu/drm/amd/amdgpu/amdgpu_connectors.c| 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c| 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 + .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 2 +- .../amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c | 2 +- .../display/amdgpu_dm/amdgpu_dm_services.c| 2 +- drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- drivers/gpu/drm/arc/arcpgu_drv.c | 2 +- drivers/gpu/drm/arc/arcpgu_sim.c | 2 +- drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +- drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- drivers/gpu/drm/arm/malidp_crtc.c | 2 +- drivers/gpu/drm/arm/malidp_drv.c | 2 +- drivers/gpu/drm/arm/malidp_mw.c | 2 +- drivers/gpu/drm/armada/armada_510.c | 2 +- drivers/gpu/drm/armada/armada_crtc.c | 2 +- drivers/gpu/drm/armada/armada_drv.c | 2 +- drivers/gpu/drm/armada/armada_fb.c| 2 +- drivers/gpu/drm/ast/ast_drv.c | 1 + drivers/gpu/drm/ast/ast_mode.c| 1 + .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c| 2 +- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h | 2 +- drivers/gpu/drm/bochs/bochs_drv.c | 1 + drivers/gpu/drm/bochs/bochs_kms.c | 1 + drivers/gpu/drm/bridge/adv7511/adv7511.h | 2 +- drivers/gpu/drm/bridge/analogix-anx78xx.c | 3 +- .../drm/bridge/analogix/analogix_dp_core.c| 2 +- drivers/gpu/drm/bridge/cdns-dsi.c | 2 +- drivers/gpu/drm/bridge/dumb-vga-dac.c | 2 +- .../bridge/megachips-stdp-ge-b850v3-fw.c | 2 +- drivers/gpu/drm/bridge/nxp-ptn3460.c | 2 +- drivers/gpu/drm/bridge/panel.c| 2 +- drivers/gpu/drm/bridge/parade-ps8622.c| 2 +- drivers/gpu/drm/bridge/sii902x.c | 2 +- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 2 +- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- drivers/gpu/drm/bridge/tc358764.c | 2 +- drivers/gpu/drm/bridge/tc358767.c | 2 +- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +- drivers/gpu/drm/bridge/ti-tfp410.c| 2 +- drivers/gpu/drm/cirrus/cirrus_drv.c | 1 + drivers/gpu/drm/cirrus/cirrus_mode.c | 1 + drivers/gpu/drm/drm_atomic_helper.c | 1 - drivers/gpu/drm/drm_dp_mst_topology.c | 2 +- drivers/gpu/drm/drm_modeset_helper.c | 2 +- drivers/gpu/drm/drm_probe_helper.c| 2 +- drivers/gpu/drm/drm_simple_kms_helper.c | 2 +- drivers/gpu/drm/etnaviv/etnaviv_drv.h | 1 - drivers/gpu/drm/exynos/exynos_dp.c| 2 +- drivers/gpu/drm/exynos/exynos_drm_crtc.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_dpi.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_fb.c| 2 +- drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_vidi.c | 2 +- drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c| 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c | 2 +- drivers/gpu/drm/gma500/psb_intel_drv.h| 1 + .../gpu/drm/hisilicon/hibmc/hibmc_drm_de.c| 2 +- .../gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 2 +- .../gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c | 2 +- .../gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c | 2 +- drivers/gpu/drm/hisilicon/kirin/dw_drm_dsi.c | 2 +- .../gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 2 +- .../gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 2 +- drivers/gpu/drm/i2c/ch7006_priv.h | 2 +- drivers/gpu/drm/i2c/sil164_drv.c | 2 +-
Re: [Nouveau] [PATCH 4/7] drm: Move the legacy kms disable_all helper to crtc helpers
On Tue, Dec 11, 2018 at 10:53 AM Sean Paul wrote: > > On Mon, Dec 10, 2018 at 10:58:20AM -0500, Alex Deucher wrote: > > On Mon, Dec 10, 2018 at 5:04 AM Daniel Vetter > > wrote: > > > > > > It's not a core function, and the matching atomic functions are also > > > not in the core. Plus the suspend/resume helper is also already there. > > > > > > Needs a tiny bit of open-coding, but less midlayer beats that I think. > > > > > > Cc: Sam Bobroff > > > Signed-off-by: Daniel Vetter > > > Cc: Maarten Lankhorst > > > Cc: Maxime Ripard > > > Cc: Sean Paul > > > Cc: David Airlie > > > Cc: Ben Skeggs > > > Cc: Alex Deucher > > > Cc: "Christian König" > > > Cc: "David (ChunMing) Zhou" > > > Cc: Rex Zhu > > > Cc: Andrey Grodzovsky > > > Cc: Huang Rui > > > Cc: Shaoyun Liu > > > Cc: Monk Liu > > > Cc: nouveau@lists.freedesktop.org > > > Cc: amd-...@lists.freedesktop.org > > > --- > > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 2 +- > > > drivers/gpu/drm/drm_crtc.c | 31 --- > > > drivers/gpu/drm/drm_crtc_helper.c | 35 ++ > > > drivers/gpu/drm/nouveau/nouveau_display.c | 2 +- > > > drivers/gpu/drm/radeon/radeon_display.c| 2 +- > > > include/drm/drm_crtc.h | 2 -- > > > include/drm/drm_crtc_helper.h | 1 + > > > 7 files changed, 39 insertions(+), 36 deletions(-) > > > > > /snip > > > > diff --git a/drivers/gpu/drm/drm_crtc_helper.c > > > b/drivers/gpu/drm/drm_crtc_helper.c > > > index a3c81850e755..23159eb494f1 100644 > > > --- a/drivers/gpu/drm/drm_crtc_helper.c > > > +++ b/drivers/gpu/drm/drm_crtc_helper.c > > > @@ -984,3 +984,38 @@ void drm_helper_resume_force_mode(struct drm_device > > > *dev) > > > drm_modeset_unlock_all(dev); > > > } > > > EXPORT_SYMBOL(drm_helper_resume_force_mode); > > > + > > > +/** > > > + * drm_helper_force_disable_all - Forcibly turn off all enabled CRTCs > > > + * @dev: DRM device whose CRTCs to turn off > > > + * > > > + * Drivers may want to call this on unload to ensure that all displays > > > are > > > + * unlit and the GPU is in a consistent, low power state. Takes modeset > > > locks. > > > + * > > > + * Note: This should only be used by non-atomic legacy drivers. For an > > > atomic > > > + * version look at drm_atomic_helper_shutdown(). > > > + * > > > + * Returns: > > > + * Zero on success, error code on failure. > > > + */ > > > +int drm_helper_force_disable_all(struct drm_device *dev) > > > > Maybe put crtc somewhere in the function name so it's clear what we > > are disabling. > > FWIW, I think it's more clear this way. set_config_internal will turn off > everything attached to the crtc, so _everything_ will be disabled in this > case. I'm not pressed. RB either way for me as well. Alex > > Either way, > > Reviewed-by: Sean Paul > > Sean > > > With that fixed: > > Reviewed-by: Alex Deucher > > > > > +{ > > > + struct drm_crtc *crtc; > > > + int ret = 0; > > > + > > > + drm_modeset_lock_all(dev); > > > + drm_for_each_crtc(crtc, dev) > > > + if (crtc->enabled) { > > > + struct drm_mode_set set = { > > > + .crtc = crtc, > > > + }; > > > + > > > + ret = drm_mode_set_config_internal(); > > > + if (ret) > > > + goto out; > > > + } > > > +out: > > > + drm_modeset_unlock_all(dev); > > > + return ret; > > > +} > > > +EXPORT_SYMBOL(drm_helper_force_disable_all); > > > diff --git a/drivers/gpu/drm/nouveau/nouveau_display.c > > > b/drivers/gpu/drm/nouveau/nouveau_display.c > > > index f326ffd86766..5d273a655479 100644 > > > --- a/drivers/gpu/drm/nouveau/nouveau_display.c > > > +++ b/drivers/gpu/drm/nouveau/nouveau_display.c > > > @@ -453,7 +453,7 @@ nouveau_display_fini(struct drm_device *dev, bool > > > suspend, bool runtime) > > > if (drm_drv_uses_atomic_modeset(dev)) > > > drm_atomic_helper_shutdown(dev); > > > else > > > - drm_crtc_force_disable_all(dev); > > > + drm_helper_force_disable_all(dev); > > > } > > > > > > /* disable flip completion events */ > > > diff --git a/drivers/gpu/drm/radeon/radeon_display.c > > > b/drivers/gpu/drm/radeon/radeon_display.c > > > index e6912eb99b42..92332226e5cf 100644 > > > --- a/drivers/gpu/drm/radeon/radeon_display.c > > > +++ b/drivers/gpu/drm/radeon/radeon_display.c > > > @@ -1643,7 +1643,7 @@ void radeon_modeset_fini(struct radeon_device *rdev) > > > if (rdev->mode_info.mode_config_initialized) { > > > drm_kms_helper_poll_fini(rdev->ddev); > > > radeon_hpd_fini(rdev); > > > - drm_crtc_force_disable_all(rdev->ddev); > > > + drm_helper_force_disable_all(rdev->ddev); > > >
Re: [Nouveau] TK1: DRM, Nouveau and VIC
Hi Thierry On Mon, 2018-12-10 at 12:00 +0100, Thierry Reding wrote: > On Mon, Dec 10, 2018 at 11:21:47AM +0100, Thierry Reding wrote: > > On Sat, Dec 08, 2018 at 02:54:45PM +, Marcel Ziswiler wrote: > > > Hi Thierry et al. > > > > > > I noticed that since commit 3dde5a2342cd ("ARM: tegra: Add VIC on > > > Tegra124") graphics on Apalis TK1 is broken. During boot it fails > > > loading the vic firmware: > > > > > > [1.595824] tegra-vic 5434.vic: Direct firmware load for > > > nvidia/tegra124/vic03_ucode.bin failed with error -2 > > > [1.606140] tegra-vic: probe of 5434.vic failed with error > > > -2 > > > > > > Subsequently Tegra HDMI seems to fail completely: > > > > > > [2.379860] tegra-hdmi 5428.hdmi: failed to get PLL > > > regulator > > > > > > And finally, Nouveau even crashes: > > > > > > [8.241115] nouveau 5700.gpu: Linked as a consumer to > > > regulator.31 > > > [8.247889] nouveau 5700.gpu: NVIDIA GK20A (0ea000a1) > > > [8.253396] nouveau 5700.gpu: imem: using IOMMU > > > [8.270210] Unable to handle kernel NULL pointer dereference > > > at > > > virtual address 006c > > > [8.278340] pgd = (ptrval) > > > [8.281250] [006c] *pgd= > > > [8.284944] Internal error: Oops: 5 [#1] PREEMPT SMP ARM > > > [8.290260] Modules linked in: nouveau(+) ttm > > > [8.294625] CPU: 2 PID: 203 Comm: systemd-udevd Not tainted > > > 4.20.0- > > > rc5-next-20181207-8-g85b0f8e25f86-dirty #110 > > > [8.305055] Hardware name: NVIDIA Tegra SoC (Flattened Device > > > Tree) > > > [8.311331] PC is at drm_plane_register_all+0x18/0x50 > > > [8.316373] LR is at drm_modeset_register_all+0xc/0x70 > > > [8.321513] pc : []lr : []psr: > > > a0060013 > > > [8.327768] sp : ed527c70 ip : ecc43ec0 fp : > > > [8.332993] r10: 0016 r9 : ecc43e80 r8 : > > > [8.338209] r7 : bf182c80 r6 : r5 : ed61b24c r4 : > > > fffc > > > [8.344735] r3 : 0002f000 r2 : r1 : 2e124000 r0 : > > > ed61b000 > > > [8.351260] Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA > > > ARM Segment none > > > [8.358383] Control: 10c5387d Table: ad64c06a DAC: 0051 > > > [8.364127] Process systemd-udevd (pid: 203, stack limit = > > > 0x(ptrval)) > > > [8.370654] Stack: (0xed527c70 to 0xed528000) > > > [8.375004] 7c60: ed61b000 > > > ed61b000 c0564cc8 > > > [8.383177] 7c80: ed61b000 c054b5b8 0001 > > > 0001 > > > [8.391355] 7ca0: ed527cc0 c0f08c48 ed61b000 > > > bf180c5c bf0dc900 > > > [8.399531] 7cc0: eda29208 5dfe844b ee9f2a10 > > > bf180c5c c05a9328 > > > [8.407695] 7ce0: c1006828 ee9f2a10 c100682c > > > c05a744c ee9f2a10 bf180c5c > > > [8.415871] 7d00: ee9f2a44 c05a77a8 c0f08c48 bf182980 > > > c05a769c eefd14d0 c05a77a8 > > > [8.424048] 7d20: ee9f2a10 bf180c5c ee9f2a44 c05a77a8 > > > c0f08c48 bf182980 > > > [8.432226] 7d40: c05a7884 ee9ebfb4 c0f08c48 bf180c5c > > > c05a5790 ee88135c > > > [8.440405] 7d60: ee9ebfb4 5dfe844b c0f71168 bf180c5c ee379e80 > > > c0f71168 c05a692c > > > [8.448570] 7d80: bf15dc00 bf180ac8 e000 bf180c5c bf180ac8 > > > e000 bf1aa000 c05a84a0 > > > [8.456746] 7da0: bf182b80 bf180ac8 e000 bf1aa170 c0fbd220 > > > c0f08c48 e000 c0102ed0 > > > [8.464924] 7dc0: ed53f4c0 006000c0 c01b3d98 000c 6113 > > > bf182980 0040 c02592d0 > > > [8.473102] 7de0: eda60200 2e124000 ee80 006000c0 006000c0 > > > c01b3d98 000c c025a8cc > > > [8.481281] 7e00: c024ce54 a113 bf182980 5dfe844b bf182980 > > > 0002 ed53f4c0 0002 > > > [8.489459] 7e20: eceba000 c01b3dd4 c0f08c48 bf182980 > > > ed527f40 0002 eceb9fc0 > > > [8.497625] 7e40: 0002 c01b61a4 bf18298c 7fff bf182980 > > > c01b2f88 c01b279c > > > [8.505800] 7e60: bf1829c8 bf182a80 bf182b6c bf182ab0 c0b03ab0 > > > c0d58964 c0ca726c c0ca7278 > > > [8.513978] 7e80: c0ca72d0 c0f08c48 c02654a0 > > > e000 bf00 > > > [8.522157] 7ea0: > > > 6e72656b 6c65 > > > [8.530336] 7ec0: > > > > > > [8.538502] 7ee0: > > > 5dfe844b 7fff c0f08c48 > > > [8.546677] 7f00: 000f b6f761cc c0101204 ed526000 > > > 017b 004a3270 c01b66a4 > > > [8.554855] 7f20: 7fff 0003 0001 004a3270 > > > f0ced000 06e8994c > > > [8.563032] 7f40: f0e37f3a f0e50a40 f0ced000 06e8994c f7b75f9c > > > f7b75d34 f63e62dc 0016b000 > > > [8.571209] 7f60: 0017f6f0 00050a48 > > > 003b 003c
Re: [Nouveau] TK1: DRM, Nouveau and VIC
Hi Thierry On Mon, 2018-12-10 at 11:21 +0100, Thierry Reding wrote: > On Sat, Dec 08, 2018 at 02:54:45PM +, Marcel Ziswiler wrote: > > Hi Thierry et al. > > > > I noticed that since commit 3dde5a2342cd ("ARM: tegra: Add VIC on > > Tegra124") graphics on Apalis TK1 is broken. During boot it fails > > loading the vic firmware: > > > > [1.595824] tegra-vic 5434.vic: Direct firmware load for > > nvidia/tegra124/vic03_ucode.bin failed with error -2 > > [1.606140] tegra-vic: probe of 5434.vic failed with error > > -2 > > > > Subsequently Tegra HDMI seems to fail completely: > > > > [2.379860] tegra-hdmi 5428.hdmi: failed to get PLL > > regulator > > > > And finally, Nouveau even crashes: > > > > [8.241115] nouveau 5700.gpu: Linked as a consumer to > > regulator.31 > > [8.247889] nouveau 5700.gpu: NVIDIA GK20A (0ea000a1) > > [8.253396] nouveau 5700.gpu: imem: using IOMMU > > [8.270210] Unable to handle kernel NULL pointer dereference at > > virtual address 006c > > [8.278340] pgd = (ptrval) > > [8.281250] [006c] *pgd= > > [8.284944] Internal error: Oops: 5 [#1] PREEMPT SMP ARM > > [8.290260] Modules linked in: nouveau(+) ttm > > [8.294625] CPU: 2 PID: 203 Comm: systemd-udevd Not tainted > > 4.20.0- > > rc5-next-20181207-8-g85b0f8e25f86-dirty #110 > > [8.305055] Hardware name: NVIDIA Tegra SoC (Flattened Device > > Tree) > > [8.311331] PC is at drm_plane_register_all+0x18/0x50 > > [8.316373] LR is at drm_modeset_register_all+0xc/0x70 > > [8.321513] pc : []lr : []psr: > > a0060013 > > [8.327768] sp : ed527c70 ip : ecc43ec0 fp : > > [8.332993] r10: 0016 r9 : ecc43e80 r8 : > > [8.338209] r7 : bf182c80 r6 : r5 : ed61b24c r4 : > > fffc > > [8.344735] r3 : 0002f000 r2 : r1 : 2e124000 r0 : > > ed61b000 > > [8.351260] Flags: NzCv IRQs on FIQs on Mode SVC_32 ISA > > ARM Segment none > > [8.358383] Control: 10c5387d Table: ad64c06a DAC: 0051 > > [8.364127] Process systemd-udevd (pid: 203, stack limit = > > 0x(ptrval)) > > [8.370654] Stack: (0xed527c70 to 0xed528000) > > [8.375004] 7c60: ed61b000 > > ed61b000 c0564cc8 > > [8.383177] 7c80: ed61b000 c054b5b8 0001 > > 0001 > > [8.391355] 7ca0: ed527cc0 c0f08c48 ed61b000 > > bf180c5c bf0dc900 > > [8.399531] 7cc0: eda29208 5dfe844b ee9f2a10 > > bf180c5c c05a9328 > > [8.407695] 7ce0: c1006828 ee9f2a10 c100682c > > c05a744c ee9f2a10 bf180c5c > > [8.415871] 7d00: ee9f2a44 c05a77a8 c0f08c48 bf182980 > > c05a769c eefd14d0 c05a77a8 > > [8.424048] 7d20: ee9f2a10 bf180c5c ee9f2a44 c05a77a8 > > c0f08c48 bf182980 > > [8.432226] 7d40: c05a7884 ee9ebfb4 c0f08c48 bf180c5c > > c05a5790 ee88135c > > [8.440405] 7d60: ee9ebfb4 5dfe844b c0f71168 bf180c5c ee379e80 > > c0f71168 c05a692c > > [8.448570] 7d80: bf15dc00 bf180ac8 e000 bf180c5c bf180ac8 > > e000 bf1aa000 c05a84a0 > > [8.456746] 7da0: bf182b80 bf180ac8 e000 bf1aa170 c0fbd220 > > c0f08c48 e000 c0102ed0 > > [8.464924] 7dc0: ed53f4c0 006000c0 c01b3d98 000c 6113 > > bf182980 0040 c02592d0 > > [8.473102] 7de0: eda60200 2e124000 ee80 006000c0 006000c0 > > c01b3d98 000c c025a8cc > > [8.481281] 7e00: c024ce54 a113 bf182980 5dfe844b bf182980 > > 0002 ed53f4c0 0002 > > [8.489459] 7e20: eceba000 c01b3dd4 c0f08c48 bf182980 > > ed527f40 0002 eceb9fc0 > > [8.497625] 7e40: 0002 c01b61a4 bf18298c 7fff bf182980 > > c01b2f88 c01b279c > > [8.505800] 7e60: bf1829c8 bf182a80 bf182b6c bf182ab0 c0b03ab0 > > c0d58964 c0ca726c c0ca7278 > > [8.513978] 7e80: c0ca72d0 c0f08c48 c02654a0 > > e000 bf00 > > [8.522157] 7ea0: > > 6e72656b 6c65 > > [8.530336] 7ec0: > > > > [8.538502] 7ee0: > > 5dfe844b 7fff c0f08c48 > > [8.546677] 7f00: 000f b6f761cc c0101204 ed526000 > > 017b 004a3270 c01b66a4 > > [8.554855] 7f20: 7fff 0003 0001 004a3270 > > f0ced000 06e8994c > > [8.563032] 7f40: f0e37f3a f0e50a40 f0ced000 06e8994c f7b75f9c > > f7b75d34 f63e62dc 0016b000 > > [8.571209] 7f60: 0017f6f0 00050a48 > > 003b 003c 0023 > > [8.579388] 7f80: 0014 5dfe844b > > 004c0ec0 0001 > > [8.587554] 7fa0: 017b c0101000 004c0ec0 000f > > b6f761cc 0002 > > [8.595730] 7fc0: 004c0ec0 0001 017b
Re: [Nouveau] TK1: DRM, Nouveau and VIC
Hi Thierry On Mon, 2018-12-10 at 17:23 +0100, Thierry Reding wrote: Snip. > > Looks like with pci_disable_device() it may take a rather strange > > path... > > Yikes... it has no business at all calling pci_disable_device() on > Tegra. Unless if you happen to have a GPU plugged into the PCIe slot. > I'm assuming that's not what you're doing? Nope, I only have a Wi-Fi card behind a PCIe switch though (;-p). > I'll see if I can reproduce (and fix) that crash on unload. > Admittedly > it's not something that I regularly test. Perhaps that's something > that > I should change... Don't worry. After a couple of years working on this I happen to try this the first time myself just now (;-p). > Thierry Cheers Marcel ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
Re: [Nouveau] drm/nouveau: tegra: Call nouveau_drm_device_init()
On Fri, 2018-11-23 at 13:11 +0100, Thierry Reding wrote: > From: Thierry Reding > > As part of commit cfea88a4d866 ("drm/nouveau: Start using new drm_dev > initialization helpers"), the initialization of the Nouveau DRM > device > was reworked and along the way the platform driver initialization was > left incomplete. Add a call to nouveau_drm_device_init() to make sure > all of the structures are properly initialized. > > Signed-off-by: Thierry Reding > Reviewed-by: Lyude Paul Tested-by: Marcel Ziswiler > --- > drivers/gpu/drm/nouveau/nouveau_drm.c | 6 ++ > 1 file changed, 6 insertions(+) > > diff --git a/drivers/gpu/drm/nouveau/nouveau_drm.c > b/drivers/gpu/drm/nouveau/nouveau_drm.c > index 2b2baf6e0e0d..d2928d43f29a 100644 > --- a/drivers/gpu/drm/nouveau/nouveau_drm.c > +++ b/drivers/gpu/drm/nouveau/nouveau_drm.c > @@ -1171,10 +1171,16 @@ nouveau_platform_device_create(const struct > nvkm_device_tegra_func *func, > goto err_free; > } > > + err = nouveau_drm_device_init(drm); > + if (err) > + goto err_put; > + > platform_set_drvdata(pdev, drm); > > return drm; > > +err_put: > + drm_dev_put(drm); > err_free: > nvkm_device_del(pdevice); ___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
[Nouveau] [PATCH 7/7] drm: Split out drm_probe_helper.h
Having the probe helper stuff (which pretty much everyone needs) in the drm_crtc_helper.h file (which atomic drivers should never need) is confusing. Split them out. To make sure I actually achieved the goal here I went through all drivers. And indeed, all atomic drivers are now free of drm_crtc_helper.h includes. Signed-off-by: Daniel Vetter Cc: linux-arm-ker...@lists.infradead.org Cc: virtualizat...@lists.linux-foundation.org Cc: etna...@lists.freedesktop.org Cc: linux-samsung-...@vger.kernel.org Cc: intel-...@lists.freedesktop.org Cc: linux-media...@lists.infradead.org Cc: linux-amlo...@lists.infradead.org Cc: linux-arm-...@vger.kernel.org Cc: freedr...@lists.freedesktop.org Cc: nouveau@lists.freedesktop.org Cc: spice-de...@lists.freedesktop.org Cc: amd-...@lists.freedesktop.org Cc: linux-renesas-...@vger.kernel.org Cc: linux-rockc...@lists.infradead.org Cc: linux-st...@st-md-mailman.stormreply.com Cc: linux-te...@vger.kernel.org Cc: xen-de...@lists.xen.org --- .../gpu/drm/amd/amdgpu/amdgpu_connectors.c| 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c| 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 + .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 2 +- .../amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c | 2 +- .../display/amdgpu_dm/amdgpu_dm_services.c| 2 +- drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- drivers/gpu/drm/arc/arcpgu_drv.c | 2 +- drivers/gpu/drm/arc/arcpgu_sim.c | 2 +- drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +- drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- drivers/gpu/drm/arm/malidp_crtc.c | 2 +- drivers/gpu/drm/arm/malidp_drv.c | 2 +- drivers/gpu/drm/arm/malidp_mw.c | 2 +- drivers/gpu/drm/armada/armada_510.c | 2 +- drivers/gpu/drm/armada/armada_crtc.c | 2 +- drivers/gpu/drm/armada/armada_drv.c | 2 +- drivers/gpu/drm/armada/armada_fb.c| 2 +- drivers/gpu/drm/ast/ast_drv.c | 1 + drivers/gpu/drm/ast/ast_mode.c| 1 + .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c| 2 +- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h | 2 +- drivers/gpu/drm/bochs/bochs_drv.c | 1 + drivers/gpu/drm/bochs/bochs_kms.c | 1 + drivers/gpu/drm/bridge/adv7511/adv7511.h | 2 +- drivers/gpu/drm/bridge/analogix-anx78xx.c | 3 +- .../drm/bridge/analogix/analogix_dp_core.c| 2 +- drivers/gpu/drm/bridge/cdns-dsi.c | 2 +- drivers/gpu/drm/bridge/dumb-vga-dac.c | 2 +- .../bridge/megachips-stdp-ge-b850v3-fw.c | 2 +- drivers/gpu/drm/bridge/nxp-ptn3460.c | 2 +- drivers/gpu/drm/bridge/panel.c| 2 +- drivers/gpu/drm/bridge/parade-ps8622.c| 2 +- drivers/gpu/drm/bridge/sii902x.c | 2 +- drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 2 +- drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- drivers/gpu/drm/bridge/tc358764.c | 2 +- drivers/gpu/drm/bridge/tc358767.c | 2 +- drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +- drivers/gpu/drm/bridge/ti-tfp410.c| 2 +- drivers/gpu/drm/cirrus/cirrus_drv.c | 1 + drivers/gpu/drm/cirrus/cirrus_mode.c | 1 + drivers/gpu/drm/drm_atomic_helper.c | 1 - drivers/gpu/drm/drm_dp_mst_topology.c | 2 +- drivers/gpu/drm/drm_modeset_helper.c | 2 +- drivers/gpu/drm/drm_probe_helper.c| 2 +- drivers/gpu/drm/drm_simple_kms_helper.c | 2 +- drivers/gpu/drm/etnaviv/etnaviv_drv.h | 1 - drivers/gpu/drm/exynos/exynos_dp.c| 2 +- drivers/gpu/drm/exynos/exynos_drm_crtc.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_dpi.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_fb.c| 2 +- drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +- drivers/gpu/drm/exynos/exynos_drm_vidi.c | 2 +- drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c| 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 2 +- drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c | 2 +- drivers/gpu/drm/gma500/psb_intel_drv.h| 1 + .../gpu/drm/hisilicon/hibmc/hibmc_drm_de.c| 2 +- .../gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 2 +- .../gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c | 2 +- .../gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c | 2 +- drivers/gpu/drm/hisilicon/kirin/dw_drm_dsi.c | 2 +- .../gpu/drm/hisilicon/kirin/kirin_drm_ade.c | 2 +- .../gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 2 +- drivers/gpu/drm/i2c/ch7006_priv.h | 2 +- drivers/gpu/drm/i2c/sil164_drv.c | 2 +-
Re: [Nouveau] [PATCH 7/7] drm: Split out drm_probe_helper.h
Le lun. 10 déc. 2018 à 12:10, Benjamin Gaignard a écrit : > > Le lun. 10 déc. 2018 à 11:24, Thierry Reding > a écrit : > > > > On Mon, Dec 10, 2018 at 11:11:33AM +0100, Daniel Vetter wrote: > > > Having the probe helper stuff (which pretty much everyone needs) in > > > the drm_crtc_helper.h file (which atomic drivers should never need) is > > > confusing. Split them out. > > > > > > To make sure I actually achieved the goal here I went through all > > > drivers. And indeed, all atomic drivers are now free of > > > drm_crtc_helper.h includes. > > > > > I have difficulties to apply this with git on top of drm-misc-next. > It is because of that I got errors (encoder and connector types not > found) while compiling adv7511_audio.c and exynos_dp.c ? > Nack on this patch because it break compiling at least on sti driver. drm_probe_helper.h doesn't bring the same includes than drm_crtc_helper.h: #include #include #include so some types, structures and functions proptotypes are missing while compiling. > Benjamin > > > Signed-off-by: Daniel Vetter > > > Cc: linux-arm-ker...@lists.infradead.org > > > Cc: virtualizat...@lists.linux-foundation.org > > > Cc: etna...@lists.freedesktop.org > > > Cc: linux-samsung-...@vger.kernel.org > > > Cc: intel-...@lists.freedesktop.org > > > Cc: linux-media...@lists.infradead.org > > > Cc: linux-amlo...@lists.infradead.org > > > Cc: linux-arm-...@vger.kernel.org > > > Cc: freedr...@lists.freedesktop.org > > > Cc: nouveau@lists.freedesktop.org > > > Cc: spice-de...@lists.freedesktop.org > > > Cc: amd-...@lists.freedesktop.org > > > Cc: linux-renesas-...@vger.kernel.org > > > Cc: linux-rockc...@lists.infradead.org > > > Cc: linux-st...@st-md-mailman.stormreply.com > > > Cc: linux-te...@vger.kernel.org > > > Cc: xen-de...@lists.xen.org > > > --- > > > .../gpu/drm/amd/amdgpu/amdgpu_connectors.c| 2 +- > > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c| 2 +- > > > drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- > > > drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 + > > > .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 2 +- > > > .../amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c | 2 +- > > > .../display/amdgpu_dm/amdgpu_dm_services.c| 2 +- > > > drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- > > > drivers/gpu/drm/arc/arcpgu_drv.c | 2 +- > > > drivers/gpu/drm/arc/arcpgu_sim.c | 2 +- > > > drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +- > > > drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- > > > drivers/gpu/drm/arm/malidp_crtc.c | 2 +- > > > drivers/gpu/drm/arm/malidp_drv.c | 2 +- > > > drivers/gpu/drm/arm/malidp_mw.c | 2 +- > > > drivers/gpu/drm/armada/armada_510.c | 2 +- > > > drivers/gpu/drm/armada/armada_crtc.c | 2 +- > > > drivers/gpu/drm/armada/armada_drv.c | 2 +- > > > drivers/gpu/drm/armada/armada_fb.c| 2 +- > > > drivers/gpu/drm/ast/ast_drv.c | 1 + > > > drivers/gpu/drm/ast/ast_mode.c| 1 + > > > .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c| 2 +- > > > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h | 2 +- > > > drivers/gpu/drm/bochs/bochs_drv.c | 1 + > > > drivers/gpu/drm/bochs/bochs_kms.c | 1 + > > > drivers/gpu/drm/bridge/adv7511/adv7511.h | 2 +- > > > drivers/gpu/drm/bridge/analogix-anx78xx.c | 3 +- > > > .../drm/bridge/analogix/analogix_dp_core.c| 2 +- > > > drivers/gpu/drm/bridge/cdns-dsi.c | 2 +- > > > drivers/gpu/drm/bridge/dumb-vga-dac.c | 2 +- > > > .../bridge/megachips-stdp-ge-b850v3-fw.c | 2 +- > > > drivers/gpu/drm/bridge/nxp-ptn3460.c | 2 +- > > > drivers/gpu/drm/bridge/panel.c| 2 +- > > > drivers/gpu/drm/bridge/parade-ps8622.c| 2 +- > > > drivers/gpu/drm/bridge/sii902x.c | 2 +- > > > drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 2 +- > > > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- > > > drivers/gpu/drm/bridge/tc358764.c | 2 +- > > > drivers/gpu/drm/bridge/tc358767.c | 2 +- > > > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +- > > > drivers/gpu/drm/bridge/ti-tfp410.c| 2 +- > > > drivers/gpu/drm/cirrus/cirrus_drv.c | 1 + > > > drivers/gpu/drm/cirrus/cirrus_mode.c | 1 + > > > drivers/gpu/drm/drm_atomic_helper.c | 1 - > > > drivers/gpu/drm/drm_dp_mst_topology.c | 2 +- > > > drivers/gpu/drm/drm_modeset_helper.c | 2 +- > > > drivers/gpu/drm/drm_probe_helper.c| 2 +- > > > drivers/gpu/drm/drm_simple_kms_helper.c | 2 +- > > > drivers/gpu/drm/etnaviv/etnaviv_drv.h | 1 - > > > drivers/gpu/drm/exynos/exynos_dp.c| 2 +- > > > drivers/gpu/drm/exynos/exynos_drm_crtc.c | 2 +- > > > drivers/gpu/drm/exynos/exynos_drm_dpi.c | 2 +- > > >
Re: [Nouveau] [PATCH 7/7] drm: Split out drm_probe_helper.h
On 10/12/2018 11:11, Daniel Vetter wrote: > Having the probe helper stuff (which pretty much everyone needs) in > the drm_crtc_helper.h file (which atomic drivers should never need) is > confusing. Split them out. > > To make sure I actually achieved the goal here I went through all > drivers. And indeed, all atomic drivers are now free of > drm_crtc_helper.h includes. > > Signed-off-by: Daniel Vetter > Cc: linux-arm-ker...@lists.infradead.org > Cc: virtualizat...@lists.linux-foundation.org > Cc: etna...@lists.freedesktop.org > Cc: linux-samsung-...@vger.kernel.org > Cc: intel-...@lists.freedesktop.org > Cc: linux-media...@lists.infradead.org > Cc: linux-amlo...@lists.infradead.org > Cc: linux-arm-...@vger.kernel.org > Cc: freedr...@lists.freedesktop.org > Cc: nouveau@lists.freedesktop.org > Cc: spice-de...@lists.freedesktop.org > Cc: amd-...@lists.freedesktop.org > Cc: linux-renesas-...@vger.kernel.org > Cc: linux-rockc...@lists.infradead.org > Cc: linux-st...@st-md-mailman.stormreply.com > Cc: linux-te...@vger.kernel.org > Cc: xen-de...@lists.xen.org > --- > .../gpu/drm/amd/amdgpu/amdgpu_connectors.c| 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c| 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- > drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 + > .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 2 +- > .../amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c | 2 +- > .../display/amdgpu_dm/amdgpu_dm_services.c| 2 +- > drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- > drivers/gpu/drm/arc/arcpgu_drv.c | 2 +- > drivers/gpu/drm/arc/arcpgu_sim.c | 2 +- > drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +- > drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- > drivers/gpu/drm/arm/malidp_crtc.c | 2 +- > drivers/gpu/drm/arm/malidp_drv.c | 2 +- > drivers/gpu/drm/arm/malidp_mw.c | 2 +- > drivers/gpu/drm/armada/armada_510.c | 2 +- > drivers/gpu/drm/armada/armada_crtc.c | 2 +- > drivers/gpu/drm/armada/armada_drv.c | 2 +- > drivers/gpu/drm/armada/armada_fb.c| 2 +- > drivers/gpu/drm/ast/ast_drv.c | 1 + > drivers/gpu/drm/ast/ast_mode.c| 1 + > .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c| 2 +- > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h | 2 +- > drivers/gpu/drm/bochs/bochs_drv.c | 1 + > drivers/gpu/drm/bochs/bochs_kms.c | 1 + > drivers/gpu/drm/bridge/adv7511/adv7511.h | 2 +- > drivers/gpu/drm/bridge/analogix-anx78xx.c | 3 +- > .../drm/bridge/analogix/analogix_dp_core.c| 2 +- > drivers/gpu/drm/bridge/cdns-dsi.c | 2 +- > drivers/gpu/drm/bridge/dumb-vga-dac.c | 2 +- > .../bridge/megachips-stdp-ge-b850v3-fw.c | 2 +- > drivers/gpu/drm/bridge/nxp-ptn3460.c | 2 +- > drivers/gpu/drm/bridge/panel.c| 2 +- > drivers/gpu/drm/bridge/parade-ps8622.c| 2 +- > drivers/gpu/drm/bridge/sii902x.c | 2 +- > drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 2 +- > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- > drivers/gpu/drm/bridge/tc358764.c | 2 +- > drivers/gpu/drm/bridge/tc358767.c | 2 +- > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +- > drivers/gpu/drm/bridge/ti-tfp410.c| 2 +- > drivers/gpu/drm/cirrus/cirrus_drv.c | 1 + > drivers/gpu/drm/cirrus/cirrus_mode.c | 1 + > drivers/gpu/drm/drm_atomic_helper.c | 1 - > drivers/gpu/drm/drm_dp_mst_topology.c | 2 +- > drivers/gpu/drm/drm_modeset_helper.c | 2 +- > drivers/gpu/drm/drm_probe_helper.c| 2 +- > drivers/gpu/drm/drm_simple_kms_helper.c | 2 +- > drivers/gpu/drm/etnaviv/etnaviv_drv.h | 1 - > drivers/gpu/drm/exynos/exynos_dp.c| 2 +- > drivers/gpu/drm/exynos/exynos_drm_crtc.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_dpi.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_fb.c| 2 +- > drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +- > drivers/gpu/drm/exynos/exynos_drm_vidi.c | 2 +- > drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c| 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c | 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_plane.c | 2 +- > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_rgb.c | 2 +- > drivers/gpu/drm/gma500/psb_intel_drv.h| 1 + > .../gpu/drm/hisilicon/hibmc/hibmc_drm_de.c| 2 +- > .../gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 2 +- > .../gpu/drm/hisilicon/hibmc/hibmc_drm_fbdev.c | 2 +- > .../gpu/drm/hisilicon/hibmc/hibmc_drm_vdac.c | 2 +- > drivers/gpu/drm/hisilicon/kirin/dw_drm_dsi.c | 2
Re: [Nouveau] [PATCH 7/7] drm: Split out drm_probe_helper.h
Le lun. 10 déc. 2018 à 11:24, Thierry Reding a écrit : > > On Mon, Dec 10, 2018 at 11:11:33AM +0100, Daniel Vetter wrote: > > Having the probe helper stuff (which pretty much everyone needs) in > > the drm_crtc_helper.h file (which atomic drivers should never need) is > > confusing. Split them out. > > > > To make sure I actually achieved the goal here I went through all > > drivers. And indeed, all atomic drivers are now free of > > drm_crtc_helper.h includes. > > I have difficulties to apply this with git on top of drm-misc-next. It is because of that I got errors (encoder and connector types not found) while compiling adv7511_audio.c and exynos_dp.c ? Benjamin > > Signed-off-by: Daniel Vetter > > Cc: linux-arm-ker...@lists.infradead.org > > Cc: virtualizat...@lists.linux-foundation.org > > Cc: etna...@lists.freedesktop.org > > Cc: linux-samsung-...@vger.kernel.org > > Cc: intel-...@lists.freedesktop.org > > Cc: linux-media...@lists.infradead.org > > Cc: linux-amlo...@lists.infradead.org > > Cc: linux-arm-...@vger.kernel.org > > Cc: freedr...@lists.freedesktop.org > > Cc: nouveau@lists.freedesktop.org > > Cc: spice-de...@lists.freedesktop.org > > Cc: amd-...@lists.freedesktop.org > > Cc: linux-renesas-...@vger.kernel.org > > Cc: linux-rockc...@lists.infradead.org > > Cc: linux-st...@st-md-mailman.stormreply.com > > Cc: linux-te...@vger.kernel.org > > Cc: xen-de...@lists.xen.org > > --- > > .../gpu/drm/amd/amdgpu/amdgpu_connectors.c| 2 +- > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c| 2 +- > > drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 2 +- > > drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 + > > .../amd/display/amdgpu_dm/amdgpu_dm_helpers.c | 2 +- > > .../amd/display/amdgpu_dm/amdgpu_dm_pp_smu.c | 2 +- > > .../display/amdgpu_dm/amdgpu_dm_services.c| 2 +- > > drivers/gpu/drm/arc/arcpgu_crtc.c | 2 +- > > drivers/gpu/drm/arc/arcpgu_drv.c | 2 +- > > drivers/gpu/drm/arc/arcpgu_sim.c | 2 +- > > drivers/gpu/drm/arm/hdlcd_crtc.c | 2 +- > > drivers/gpu/drm/arm/hdlcd_drv.c | 2 +- > > drivers/gpu/drm/arm/malidp_crtc.c | 2 +- > > drivers/gpu/drm/arm/malidp_drv.c | 2 +- > > drivers/gpu/drm/arm/malidp_mw.c | 2 +- > > drivers/gpu/drm/armada/armada_510.c | 2 +- > > drivers/gpu/drm/armada/armada_crtc.c | 2 +- > > drivers/gpu/drm/armada/armada_drv.c | 2 +- > > drivers/gpu/drm/armada/armada_fb.c| 2 +- > > drivers/gpu/drm/ast/ast_drv.c | 1 + > > drivers/gpu/drm/ast/ast_mode.c| 1 + > > .../gpu/drm/atmel-hlcdc/atmel_hlcdc_crtc.c| 2 +- > > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.h | 2 +- > > drivers/gpu/drm/bochs/bochs_drv.c | 1 + > > drivers/gpu/drm/bochs/bochs_kms.c | 1 + > > drivers/gpu/drm/bridge/adv7511/adv7511.h | 2 +- > > drivers/gpu/drm/bridge/analogix-anx78xx.c | 3 +- > > .../drm/bridge/analogix/analogix_dp_core.c| 2 +- > > drivers/gpu/drm/bridge/cdns-dsi.c | 2 +- > > drivers/gpu/drm/bridge/dumb-vga-dac.c | 2 +- > > .../bridge/megachips-stdp-ge-b850v3-fw.c | 2 +- > > drivers/gpu/drm/bridge/nxp-ptn3460.c | 2 +- > > drivers/gpu/drm/bridge/panel.c| 2 +- > > drivers/gpu/drm/bridge/parade-ps8622.c| 2 +- > > drivers/gpu/drm/bridge/sii902x.c | 2 +- > > drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 2 +- > > drivers/gpu/drm/bridge/synopsys/dw-mipi-dsi.c | 2 +- > > drivers/gpu/drm/bridge/tc358764.c | 2 +- > > drivers/gpu/drm/bridge/tc358767.c | 2 +- > > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 +- > > drivers/gpu/drm/bridge/ti-tfp410.c| 2 +- > > drivers/gpu/drm/cirrus/cirrus_drv.c | 1 + > > drivers/gpu/drm/cirrus/cirrus_mode.c | 1 + > > drivers/gpu/drm/drm_atomic_helper.c | 1 - > > drivers/gpu/drm/drm_dp_mst_topology.c | 2 +- > > drivers/gpu/drm/drm_modeset_helper.c | 2 +- > > drivers/gpu/drm/drm_probe_helper.c| 2 +- > > drivers/gpu/drm/drm_simple_kms_helper.c | 2 +- > > drivers/gpu/drm/etnaviv/etnaviv_drv.h | 1 - > > drivers/gpu/drm/exynos/exynos_dp.c| 2 +- > > drivers/gpu/drm/exynos/exynos_drm_crtc.c | 2 +- > > drivers/gpu/drm/exynos/exynos_drm_dpi.c | 2 +- > > drivers/gpu/drm/exynos/exynos_drm_drv.c | 2 +- > > drivers/gpu/drm/exynos/exynos_drm_dsi.c | 2 +- > > drivers/gpu/drm/exynos/exynos_drm_fb.c| 2 +- > > drivers/gpu/drm/exynos/exynos_drm_fbdev.c | 2 +- > > drivers/gpu/drm/exynos/exynos_drm_vidi.c | 2 +- > > drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +- > > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_crtc.c| 2 +- > > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 2 +- > > drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_kms.c |
[Nouveau] [Bug 109035] amazon echo dot technical support phone number
https://bugs.freedesktop.org/show_bug.cgi?id=109035 Ricky Pointings changed: What|Removed |Added Keywords||cleanup0407 URL||https://www.customercarenum ||ber-usa.com/amazon-echo-ale ||xa-customer-support/ -- You are receiving this mail because: You are the assignee for the bug. You are the QA Contact for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
[Nouveau] [Bug 109035] New: amazon echo dot technical support phone number
https://bugs.freedesktop.org/show_bug.cgi?id=109035 Bug ID: 109035 Summary: amazon echo dot technical support phone number Product: Mesa Version: 18.1 Hardware: Other OS: All Status: NEW Severity: normal Priority: medium Component: Drivers/DRI/nouveau Assignee: nouveau@lists.freedesktop.org Reporter: rickypointings1...@gmail.com QA Contact: nouveau@lists.freedesktop.org With the introduction of Echo Alexa, the smart speakers have been modernized. Both Echo and Alexa make a perfect pair. Users can ask Alexa to play their favorite track, know about the live traffic and news updates and much more. As this device is quite new for many users so they come across problems in setting it up, if you are also one of them and don’t know how to do so then you can contact Echo Alexa customer support. For more information click on this link:- https://www.customercarenumber-usa.com/amazon-echo-alexa-customer-support/ -- You are receiving this mail because: You are the QA Contact for the bug. You are the assignee for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
[Nouveau] [Bug 109025] Reset Button is not working properly.
https://bugs.freedesktop.org/show_bug.cgi?id=109025 Tushar Matte changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #1 from Tushar Matte --- ok working on it. -- You are receiving this mail because: You are the QA Contact for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau
[Nouveau] [Bug 109025] New: Reset Button is not working properly.
https://bugs.freedesktop.org/show_bug.cgi?id=109025 Bug ID: 109025 Summary: Reset Button is not working properly. Product: Mesa Version: 18.3 Hardware: ARM OS: Windows (All) Status: NEW Severity: major Priority: high Component: Drivers/DRI/nouveau Assignee: tushar.matt...@gmail.com Reporter: khemrajbawanth...@gmail.com QA Contact: nouveau@lists.freedesktop.org If we click on Reset Buttton, All the text fields above the button should be cleared.But some Fields are remain unclear. -- You are receiving this mail because: You are the QA Contact for the bug.___ Nouveau mailing list Nouveau@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/nouveau