diff mbox series

[v2,4/4] arm64: dts: qcom: qrb5165-rb5: enable DP altmode

Message ID 20230817145940.9887-5-dmitry.baryshkov@linaro.org
State Accepted
Commit b3dea914127e9065df003002ed13a2ef40d19877
Headers show
Series arm64: dts: qcom: qrb5165-rb5: enable DP support | expand

Commit Message

Dmitry Baryshkov Aug. 17, 2023, 2:59 p.m. UTC
Add displayport altmode declaration to the Type-C controller node to
enable DP altmode negotiation.

Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Aishwarya TCV Oct. 13, 2023, 4:05 p.m. UTC | #1
Hi Dmitry


On 17/08/2023 15:59, Dmitry Baryshkov wrote:
> Add displayport altmode declaration to the Type-C controller node to
> enable DP altmode negotiation.
> 
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---
>  arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 7 +++++++
>  1 file changed, 7 insertions(+)
> 

At present RB5 runs against next-master are consistently throwing
warning messages (attached below with call trace) on "Memory manager not
clean during takedown". I can send full logs if required. However, the
runs are booting successfully so the logs are quite big. Tried running
with older dtb built, where these
warnings was not observed.


A bisect (full log below) identified this patch as
introducing the failure.

git bisect start
# good: [8a749fd1a8720d4619c91c8b6e7528c0a355c0aa] Linux 6.6-rc4
git bisect good 8a749fd1a8720d4619c91c8b6e7528c0a355c0aa
# bad: [2a860505b617cf8fda4ebff6cf05d3f774145440] Add linux-next
specific files for 20231009
git bisect bad 2a860505b617cf8fda4ebff6cf05d3f774145440
# bad: [11a92414d65ec3ea50b86004baa9b54e03fc454a] Merge branch 'master'
of git://linuxtv.org/media_tree.git
git bisect bad 11a92414d65ec3ea50b86004baa9b54e03fc454a
# good: [d4ec2b3f0e34a40991fb8d5fb374c07382fe0892] bcachefs:
stack_trace_save_tsk() depends on CONFIG_STACKTRACE
git bisect good d4ec2b3f0e34a40991fb8d5fb374c07382fe0892
# good: [94b6eadf2db4b7ec5b6e5c4dfd3d552ea388e441] Merge branch
'for-next' of
git://git.kernel.org/pub/scm/linux/kernel/git/tmlind/linux-omap.git
git bisect good 94b6eadf2db4b7ec5b6e5c4dfd3d552ea388e441
# bad: [15c1906eb69acb82e6992d701f9522216fdaf234] Merge branch
'xtensa-for-next' of git://github.com/jcmvbkbc/linux-xtensa.git
git bisect bad 15c1906eb69acb82e6992d701f9522216fdaf234
# bad: [b837ae5f505e0a05329ab9c72ed1a6c55f70456c] Merge branch
'for-next' of git://git.kernel.org/pub/scm/linux/kernel/git/krzk/linux.git
git bisect bad b837ae5f505e0a05329ab9c72ed1a6c55f70456c
# bad: [33e9032a1875bb1aee3c68a4540f5a577ff44130] arm64: dts: qcom:
apq8016-sbc: Add missing ADV7533 regulators
git bisect bad 33e9032a1875bb1aee3c68a4540f5a577ff44130
# bad: [a6546460ca439bade19d64eb63cee2d97c29fb72] arm64: dts: qcom:
sc8180x: switch PCIe QMP PHY to new style of bindings
git bisect bad a6546460ca439bade19d64eb63cee2d97c29fb72
# good: [4988881ec067c3e6d382de1583b7f5b1095ddea2] arm64: dts: qcom:
sm6125: Add dispcc node
git bisect good 4988881ec067c3e6d382de1583b7f5b1095ddea2
# good: [ea96b90a58cf5d2e91ac177f081118ff26b85c1d] arm64: dts: qcom:
sm8250: Define ports for qmpphy orientation-switching
git bisect good ea96b90a58cf5d2e91ac177f081118ff26b85c1d
# bad: [b3dea914127e9065df003002ed13a2ef40d19877] arm64: dts: qcom:
qrb5165-rb5: enable DP altmode
git bisect bad b3dea914127e9065df003002ed13a2ef40d19877
# good: [25defdca4d902b338c05bc01a1de1064a6d3b7f3] arm64: dts: qcom:
qrb5165-rb5: Switch on TCPM usb-role-switching for usb_1
git bisect good 25defdca4d902b338c05bc01a1de1064a6d3b7f3
# good: [956aa24b16350a50d3a6beb9237bc35aa2f447d6] arm64: dts: qcom:
sm8250: Add DisplayPort device node
git bisect good 956aa24b16350a50d3a6beb9237bc35aa2f447d6
# bad: [96387ee7534dc449be35a9bb98b7668da2bed545] arm64: dts: qcom:
qrb5165-rb5: enable displayport controller
git bisect bad 96387ee7534dc449be35a9bb98b7668da2bed545
# good: [d342e1c993bd7589cad9d2da099c6a9c652ecb9f] arm64: dts: qcom:
qrb5165-rb5: add onboard USB-C redriver
git bisect good d342e1c993bd7589cad9d2da099c6a9c652ecb9f
# first bad commit: [96387ee7534dc449be35a9bb98b7668da2bed545] arm64:
dts: qcom: qrb5165-rb5: enable displayport controller


Boot log:

<4>[   87.062915] ------------[ cut here ]------------
<4>[   87.067944] Memory manager not clean during takedown.
<4>[   87.074791] WARNING: CPU: 7 PID: 70 at
drivers/gpu/drm/drm_mm.c:999 drm_mm_takedown+0x28/0x38 [drm]
<4>[   87.084382] Modules linked in: fuse ip_tables x_tables ipv6
r8153_ecm cdc_ether snd_soc_hdmi_codec ax88179_178a venus_dec venus_enc
r8152 videobuf2_dma_contig usbnet videobuf2_memops onboard_usb_hub
mcp251xfd nb7vpq904m can_dev lontium_lt9611uxc msm leds_qcom_lpg
qcom_spmi_adc_tm5 qcom_spmi_adc5 rtc_pm8xxx led_class_multicolor ocmem
qcom_pon qcom_spmi_temp_alarm qcom_vadc_common venus_core v4l2_mem2mem
gpu_sched videobuf2_v4l2 crct10dif_ce drm_dp_aux_bus videodev
snd_soc_sm8250 snd_soc_qcom_sdw drm_display_helper videobuf2_common
snd_soc_qcom_common qrtr qcom_stats mc coresight_stm stm_core
qcom_q6v5_pas spi_geni_qcom llcc_qcom phy_qcom_qmp_combo qcom_pil_info
typec icc_bwmon i2c_qcom_geni phy_qcom_qmp_usb qcom_q6v5
coresight_replicator coresight_tmc coresight_etm4x coresight_funnel
phy_qcom_snps_femto_v2 qcom_rng qcom_sysmon snd_soc_lpass_va_macro
qcom_common lpass_gfm_sm8250 qcrypto pinctrl_sm8250_lpass_lpi
qcom_glink_smem snd_soc_lpass_macro_common coresight ufs_qcom
sha256_generic mdt_loader libsha256 qmi_helpers
<4>[   87.084435]  pinctrl_lpass_lpi soundwire_qcom soundwire_bus
authenc display_connector qcom_wdt phy_qcom_qmp_ufs
snd_soc_lpass_wsa_macro phy_qcom_qmp_pcie slimbus icc_osm_l3 libdes
drm_kms_helper drm socinfo backlight qnoc_sm8250
<4>[   87.198040] CPU: 7 PID: 70 Comm: kworker/u16:3 Tainted: G        W
         6.6.0-rc5-next-20231013 #1
<4>[   87.207857] Hardware name: Qualcomm Technologies, Inc. Robotics
RB5 (DT)
<4>[   87.215009] Workqueue: events_unbound deferred_probe_work_func
<4>[   87.221273] pstate: 60400005 (nZCv daif +PAN -UAO -TCO -DIT -SSBS
BTYPE=--)
<4>[   87.228686] pc : drm_mm_takedown+0x28/0x38 [drm]
<4>[   87.233716] lr : drm_mm_takedown+0x28/0x38 [drm]
<4>[   87.238741] sp : ffff800080743810
<4>[   87.242416] x29: ffff800080743810 x28: ffff7824cd321880 x27:
ffff7824c36c9d40
<4>[   87.250007] x26: ffffb54fa83c71a0 x25: dead000000000100 x24:
dead000000000122
<4>[   87.257599] x23: ffff7824c3600810 x22: ffffb54f5a99b708 x21:
ffff7824c3600800
<4>[   87.265189] x20: ffff7824c4a4c600 x19: ffff7824c7752800 x18:
ffffb54f5a8e0048
<4>[   87.272779] x17: 0000000000010108 x16: 0000000000010108 x15:
0000483e088c93ae
<4>[   87.280371] x14: 00000000000002a4 x13: 00000000000002a4 x12:
0000000000000000
<4>[   87.287962] x11: 0000000000000001 x10: 0000000000000a60 x9 :
ffff800080743680
<4>[   87.295554] x8 : ffff7824c0320ac0 x7 : ffff78263efaf040 x6 :
00000000012105e7
<4>[   87.303140] x5 : 00000000410fd0d0 x4 : 0000000000f0000f x3 :
0000000000100000
<4>[   87.310730] x2 : 0000000000000000 x1 : 0000000000000000 x0 :
ffff7824c0320000
<4>[   87.318317] Call trace:
<4>[   87.321103]  drm_mm_takedown+0x28/0x38 [drm]
<4>[   87.325778]  drm_vma_offset_manager_destroy+0x14/0x20 [drm]
<4>[   87.331794]  drm_gem_init_release+0x14/0x20 [drm]
<4>[   87.336918]  drm_managed_release+0xac/0x140 [drm]
<4>[   87.342043]  drm_dev_put+0x94/0xbc [drm]
<4>[   87.346366]  msm_drm_uninit.isra.0+0x98/0xd0 [msm]
<4>[   87.351594]  msm_drm_bind+0x20c/0x3b0 [msm]
<4>[   87.356181]  try_to_bring_up_aggregate_device+0x168/0x1d4
<4>[   87.361994]  __component_add+0xa4/0x170
<4>[   87.366202]  component_add+0x14/0x20
<4>[   87.370142]  dsi_dev_attach+0x20/0x2c [msm]
<4>[   87.374731]  dsi_host_attach+0x9c/0x144 [msm]
<4>[   87.379493]  devm_mipi_dsi_attach+0x34/0xb4
<4>[   87.384064]  lt9611uxc_attach_dsi.isra.0+0x84/0xfc [lontium_lt9611uxc]
<4>[   87.391040]  lt9611uxc_probe+0x598/0x65c [lontium_lt9611uxc]
<4>[   87.397126]  i2c_device_probe+0x1a8/0x330
<4>[   87.401520]  really_probe+0x148/0x2b4
<4>[   87.405554]  __driver_probe_device+0x78/0x12c
<4>[   87.410302]  driver_probe_device+0x3c/0x15c
<4>[   87.414875]  __device_attach_driver+0xb8/0x134
<4>[   87.419710]  bus_for_each_drv+0x84/0xe0
<4>[   87.423929]  __device_attach+0xa8/0x1b0
<4>[   87.428138]  device_initial_probe+0x14/0x20
<4>[   87.432704]  bus_probe_device+0xa8/0xac
<4>[   87.436913]  deferred_probe_work_func+0x88/0xc0
<4>[   87.441829]  process_one_work+0x138/0x260
<4>[   87.446225]  worker_thread+0x32c/0x438
<4>[   87.450346]  kthread+0x118/0x11c
<4>[   87.453941]  ret_from_fork+0x10/0x20
<4>[   87.457888] ---[ end trace 0000000000000000 ]---

Thanks,
Aishwarya

> diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> index 3bd0c06e7315..c8cd40a462a3 100644
> --- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> +++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
> @@ -1423,6 +1423,13 @@ PDO_FIXED_DUAL_ROLE |
>  					 PDO_FIXED_USB_COMM |
>  					 PDO_FIXED_DATA_SWAP)>;
>  
> +		altmodes {
> +			displayport {
> +				svid = <0xff01>;
> +				vdo = <0x00001c46>;
> +			};
> +		};
> +
>  		ports {
>  			#address-cells = <1>;
>  			#size-cells = <0>;
Dmitry Baryshkov Oct. 13, 2023, 4:30 p.m. UTC | #2
Hello Aishwarya,

On Fri, 13 Oct 2023 at 19:05, Aishwarya TCV <aishwarya.tcv@arm.com> wrote:
>
> Hi Dmitry
>
>
> On 17/08/2023 15:59, Dmitry Baryshkov wrote:
> > Add displayport altmode declaration to the Type-C controller node to
> > enable DP altmode negotiation.
> >
> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> > ---
> >  arch/arm64/boot/dts/qcom/qrb5165-rb5.dts | 7 +++++++
> >  1 file changed, 7 insertions(+)
> >
>
> At present RB5 runs against next-master are consistently throwing
> warning messages (attached below with call trace) on "Memory manager not
> clean during takedown". I can send full logs if required. However, the
> runs are booting successfully so the logs are quite big. Tried running
> with older dtb built, where these
> warnings was not observed.

Thank you for the report. These patches should be fixed by the
patchset at https://patchwork.freedesktop.org/series/125010/
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
index 3bd0c06e7315..c8cd40a462a3 100644
--- a/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
+++ b/arch/arm64/boot/dts/qcom/qrb5165-rb5.dts
@@ -1423,6 +1423,13 @@  PDO_FIXED_DUAL_ROLE |
 					 PDO_FIXED_USB_COMM |
 					 PDO_FIXED_DATA_SWAP)>;
 
+		altmodes {
+			displayport {
+				svid = <0xff01>;
+				vdo = <0x00001c46>;
+			};
+		};
+
 		ports {
 			#address-cells = <1>;
 			#size-cells = <0>;