Re: exynos4412: porting hdmiddc and hdmiphy node entries

[Tobias Jakobi <liquid.acid@gmx.net>]

[-- Attachment #1: Type: text/plain, Size: 1143 bytes --]

Hello,

I made some progress with the HDMI output on the board.

I updated to v4 of the exynos-simple-phy driver.
@Rahul: You can add a tested-by from me, if you want.

My dts now looks like this:
https://github.com/tobiasjakobi/linux-odroid/blob/odroid-3.15.y/arch/arm/boot/dts/exynos4412-odroidx2.dts

This works, but I have to force disabling of the LCD0 powerdomain:
https://github.com/tobiasjakobi/linux-odroid/commit/65b97415b90f54240e03a065cfea1097629fb17e

It looks like that the HDMI doesn't work properly when LCD0 pd is
switched off, which it normally is for my use case. The nodes related to
HDMI also don't seem to reference it. It seems like only the TV pd is
referenced, and from the looks of the kernel log this one gets
enabled/disabled properly.
@Inki: Is this a known problem?

Another issue which I then encountered can be easily triggered with
'modetest' from libdrm/tests:
modetest -M exynos -v -s 15@6:640x480

This triggers a kernel warning (mixer_dpms). I attached the full output.
Note that the warning appear when exiting modetest (so it's probably
triggeed by crtc restore or something).

With best wishes,
Tobias


[-- Attachment #2: exynos_mixer_warning.txt --]
[-- Type: text/plain, Size: 4443 bytes --]

[   86.760000] WARNING: CPU: 0 PID: 2512 at drivers/gpu/drm/exynos/exynos_mixer.c:620 mixer_dpms+0x54c/0x674()
[   86.760000] failed to reset Video Processor
[   86.760000] Modules linked in: bridge stp llc bnep rfcomm ecb btusb bluetooth s5p_mfc usb_storage videobuf2_dma_contig videobuf2_memops videobuf2_core
[   86.760000] CPU: 0 PID: 2512 Comm: modetest Not tainted 3.15.0-rc5+ #13
[   86.760000] Backtrace: 
[   86.760000] [<c0011a5c>] (dump_backtrace) from [<c0011bf8>] (show_stack+0x18/0x1c)
[   86.760000]  r6:0000026c r5:00000009 r4:00000000 r3:00000000
[   86.760000] [<c0011be0>] (show_stack) from [<c041f494>] (dump_stack+0x88/0xd4)
[   86.760000] [<c041f40c>] (dump_stack) from [<c0023a6c>] (warn_slowpath_common+0x6c/0x90)
[   86.760000]  r4:e5919a98 r3:e5918000
[   86.760000] [<c0023a00>] (warn_slowpath_common) from [<c0023b34>] (warn_slowpath_fmt+0x38/0x40)
[   86.760000]  r8:c05be648 r7:00000000 r6:c059e414 r5:e6395410 r4:00000000
[   86.760000] [<c0023b00>] (warn_slowpath_fmt) from [<c0260fc8>] (mixer_dpms+0x54c/0x674)
[   86.760000]  r3:c01c9268 r2:c0516aa0
[   86.760000] [<c0260a7c>] (mixer_dpms) from [<c024ffa4>] (exynos_drm_crtc_dpms+0x74/0x114)
[   86.760000]  r10:00000001 r9:e612c9d4 r8:c05be648 r7:e5399780 r6:00000000 r5:c05ff240
[   86.760000]  r4:e612d000
[   86.760000] [<c024ff30>] (exynos_drm_crtc_dpms) from [<c02500f8>] (exynos_drm_crtc_commit+0x1c/0x4c)
[   86.760000]  r8:e612c800 r7:e5399780 r6:e6129e40 r5:c05be648 r4:e612d000
[   86.760000] [<c02500dc>] (exynos_drm_crtc_commit) from [<c0230954>] (drm_crtc_helper_set_mode+0x3c4/0x518)
[   86.760000]  r5:e612c9d8 r4:e612d000
[   86.760000] [<c0230590>] (drm_crtc_helper_set_mode) from [<c0231354>] (drm_crtc_helper_set_config+0x778/0x9d0)
[   86.760000]  r10:e618fc80 r9:00000000 r8:c05ff240 r7:e612c9d8 r6:e612c9c0 r5:e612c9b4
[   86.760000]  r4:e612d000
[   86.760000] [<c0230bdc>] (drm_crtc_helper_set_config) from [<c0241e74>] (drm_mode_set_config_internal+0x60/0xec)
[   86.760000]  r10:c05ff240 r9:e612c88c r8:00000000 r7:e6384300 r6:e618fc80 r5:e612d000
[   86.760000]  r4:e6398900
[   86.760000] [<c0241e14>] (drm_mode_set_config_internal) from [<c0234a10>] (drm_fb_helper_restore_fbdev_mode+0xb8/0xd8)
[   86.760000]  r6:e618fc80 r5:00000000 r4:00000001 r3:00000000
[   86.760000] [<c0234958>] (drm_fb_helper_restore_fbdev_mode) from [<c0250e00>] (exynos_drm_fbdev_restore_mode+0x34/0x40)
[   86.760000]  r8:e612c800 r7:e612c800 r6:e612c838 r5:e612c800 r4:e618f200
[   86.760000] [<c0250dcc>] (exynos_drm_fbdev_restore_mode) from [<c024f6f4>] (exynos_drm_lastclose+0x10/0x14)
[   86.760000]  r5:e612c850 r4:e5a61480
[   86.760000] [<c024f6e4>] (exynos_drm_lastclose) from [<c02389cc>] (drm_lastclose+0x38/0x154)
[   86.760000] [<c0238994>] (drm_lastclose) from [<c0238eac>] (drm_release+0x3c4/0x5bc)
[   86.760000]  r10:e6127f80 r9:e612c88c r8:e5a614f4 r7:e612c800 r6:e612c838 r5:e612c850
[   86.760000]  r4:e5a61480 r3:00000000
[   86.760000] [<c0238ae8>] (drm_release) from [<c00d7a8c>] (__fput+0x88/0x1c4)
[   86.760000]  r10:00000000 r9:e5a616c8 r8:00000008 r7:e5c46990 r6:e59483d0 r5:e632b100
[   86.760000]  r4:e5a616c0
[   86.760000] [<c00d7a04>] (__fput) from [<c00d7c2c>] (____fput+0x10/0x14)
[   86.760000]  r10:e5990ff8 r9:418004fc r8:e5918000 r7:e50ac800 r6:00000000 r5:c05ceb70
[   86.760000]  r4:e50acbb0
[   86.760000] [<c00d7c1c>] (____fput) from [<c003c528>] (task_work_run+0xb0/0xe4)
[   86.760000] [<c003c478>] (task_work_run) from [<c0025894>] (do_exit+0x2b4/0x8e4)
[   86.760000]  r7:e5990fc0 r6:e50ac800 r5:00000002 r4:e50acbc8
[   86.760000] [<c00255e0>] (do_exit) from [<c0025fd8>] (do_group_exit+0x44/0xb8)
[   86.760000]  r7:e5bee8c4
[   86.760000] [<c0025f94>] (do_group_exit) from [<c00310e8>] (get_signal_to_deliver+0x1e4/0x558)
[   86.760000]  r7:e5bee8c4 r6:e5918000 r5:e5919edc r4:e5918000
[   86.760000] [<c0030f04>] (get_signal_to_deliver) from [<c041c554>] (do_signal+0xb4/0x3d0)
[   86.760000]  r10:b6def2ec r9:e5918000 r8:b6def2f0 r7:fffffdfe r6:e5918000 r5:00000001
[   86.760000]  r4:e5919fb0
[   86.760000] [<c041c4a0>] (do_signal) from [<c0011568>] (do_work_pending+0x8c/0xcc)
[   86.760000]  r10:00000000 r8:e5919fb0 r7:c000eca4 r6:e5918000 r5:e5918000 r4:bec65e18
[   86.760000] [<c00114dc>] (do_work_pending) from [<c000eb60>] (work_pending+0xc/0x20)
[   86.760000]  r8:c000eca4 r7:0000008e r6:bec65e18 r5:00000003 r4:bec65e18 r3:00000000
[   86.760000] ---[ end trace de2bb103e2dbf872 ]---