* [PATCH v2 0/7] drm: ci: fixes
@ 2023-09-04 16:15 Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 1/7] drm: ci: igt_runner: remove todo Vignesh Raman
` (6 more replies)
0 siblings, 7 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-04 16:15 UTC (permalink / raw)
To: dri-devel
Cc: helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
The patch series contains improvements, enabling new ci jobs which
enables testing for Mediatek MT8173, Qualcomm APQ 8016 and VirtIO GPU,
fixing issues with the ci jobs and updating the expectation files.
This series is intended for drm branch topic/drm-ci.
v2:
- Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
- Reworded the commit message for enabling jobs
- Added a new patch in the series to use scripts/config to enable/disable configs
Vignesh Raman (7):
drm: ci: igt_runner: remove todo
drm: ci: Force db410c to host mode
drm: ci: virtio: update ci variables
drm: ci: Enable configs to fix mt8173 boot hang issue
drm: ci: Update xfails
drm: ci: Enable new jobs
drm: ci: Use scripts/config to enable/disable configs
drivers/gpu/drm/ci/arm64.config | 2 ++
drivers/gpu/drm/ci/build.sh | 19 ++++++++++++-------
.../ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
drivers/gpu/drm/ci/gitlab-ci.yml | 2 +-
drivers/gpu/drm/ci/igt_runner.sh | 1 -
drivers/gpu/drm/ci/test.yml | 14 ++++----------
.../gpu/drm/ci/xfails/amdgpu-stoney-fails.txt | 1 -
drivers/gpu/drm/ci/xfails/i915-cml-fails.txt | 1 -
drivers/gpu/drm/ci/xfails/i915-cml-flakes.txt | 2 ++
drivers/gpu/drm/ci/xfails/i915-glk-flakes.txt | 1 +
.../drm/ci/xfails/mediatek-mt8173-fails.txt | 2 --
.../drm/ci/xfails/mediatek-mt8173-flakes.txt | 16 ++++++++++++++++
.../gpu/drm/ci/xfails/msm-apq8016-flakes.txt | 2 ++
.../drm/ci/xfails/rockchip-rk3288-flakes.txt | 1 +
.../drm/ci/xfails/rockchip-rk3399-fails.txt | 4 ++--
.../drm/ci/xfails/rockchip-rk3399-flakes.txt | 3 +++
16 files changed, 59 insertions(+), 25 deletions(-)
create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
--
2.40.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 1/7] drm: ci: igt_runner: remove todo
2023-09-04 16:15 [PATCH v2 0/7] drm: ci: fixes Vignesh Raman
@ 2023-09-04 16:15 ` Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 2/7] drm: ci: Force db410c to host mode Vignesh Raman
` (5 subsequent siblings)
6 siblings, 0 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-04 16:15 UTC (permalink / raw)
To: dri-devel
Cc: helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
/sys/kernel/debug/dri/*/state exist for every atomic KMS driver.
We do not test non-atomic drivers, so remove the todo.
Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
---
v2:
- No changes
---
drivers/gpu/drm/ci/igt_runner.sh | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/ci/igt_runner.sh b/drivers/gpu/drm/ci/igt_runner.sh
index 2bb759165063..5bf130ac57c9 100755
--- a/drivers/gpu/drm/ci/igt_runner.sh
+++ b/drivers/gpu/drm/ci/igt_runner.sh
@@ -15,7 +15,6 @@ cat /sys/kernel/debug/device_component/*
'
# Dump drm state to confirm that kernel was able to find a connected display:
-# TODO this path might not exist for all drivers.. maybe run modetest instead?
set +e
cat /sys/kernel/debug/dri/*/state
set -e
--
2.40.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-04 16:15 [PATCH v2 0/7] drm: ci: fixes Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 1/7] drm: ci: igt_runner: remove todo Vignesh Raman
@ 2023-09-04 16:15 ` Vignesh Raman
2023-09-04 16:59 ` Dmitry Baryshkov
2023-09-06 12:55 ` Helen Koike
2023-09-04 16:15 ` [PATCH v2 3/7] drm: ci: virtio: update ci variables Vignesh Raman
` (4 subsequent siblings)
6 siblings, 2 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-04 16:15 UTC (permalink / raw)
To: dri-devel
Cc: helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Force db410c to host mode to fix network issue which results in failure
to mount root fs via NFS.
See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
Use fdtoverlay command to merge base device tree with an overlay
which contains the fix for USB controllers to work in host mode.
Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
---
v2:
- Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
---
drivers/gpu/drm/ci/build.sh | 5 +++++
.../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
2 files changed, 18 insertions(+)
create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
index 7b014287a041..92ffd98cd09e 100644
--- a/drivers/gpu/drm/ci/build.sh
+++ b/drivers/gpu/drm/ci/build.sh
@@ -92,6 +92,11 @@ done
if [[ -n ${DEVICE_TREES} ]]; then
make dtbs
+ if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
+ dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
+ fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
+ mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
+ fi
cp ${DEVICE_TREES} /lava-files/.
fi
diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
new file mode 100644
index 000000000000..57b7604f1c23
--- /dev/null
+++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
@@ -0,0 +1,13 @@
+/dts-v1/;
+/plugin/;
+
+/ {
+ fragment@0 {
+ target-path = "/soc@0";
+ __overlay__ {
+ usb@78d9000 {
+ dr_mode = "host";
+ };
+ };
+ };
+};
--
2.40.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 3/7] drm: ci: virtio: update ci variables
2023-09-04 16:15 [PATCH v2 0/7] drm: ci: fixes Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 1/7] drm: ci: igt_runner: remove todo Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 2/7] drm: ci: Force db410c to host mode Vignesh Raman
@ 2023-09-04 16:15 ` Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 4/7] drm: ci: Enable configs to fix mt8173 boot hang issue Vignesh Raman
` (3 subsequent siblings)
6 siblings, 0 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-04 16:15 UTC (permalink / raw)
To: dri-devel
Cc: helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Update ci variables to fix the below error,
ERROR - Igt error: malloc(): corrupted top size
ERROR - Igt error: Received signal SIGABRT.
ERROR - Igt error: Stack trace:
ERROR - Igt error: #0 [fatal_sig_handler+0x17b]
Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
---
v2:
- No changes
---
drivers/gpu/drm/ci/test.yml | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/ci/test.yml b/drivers/gpu/drm/ci/test.yml
index 6473cddaa7a9..d85add39f425 100644
--- a/drivers/gpu/drm/ci/test.yml
+++ b/drivers/gpu/drm/ci/test.yml
@@ -316,8 +316,11 @@ virtio_gpu:none:
stage: virtio-gpu
variables:
CROSVM_GALLIUM_DRIVER: llvmpipe
- DRIVER_NAME: virtio_gpu
+ DRIVER_NAME: virtio
GPU_VERSION: none
+ CROSVM_MEMORY: 12288
+ CROSVM_CPU: $FDO_CI_CONCURRENT
+ CROSVM_GPU_ARGS: "vulkan=true,gles=false,backend=virglrenderer,egl=true,surfaceless=true"
extends:
- .test-gl
tags:
--
2.40.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 4/7] drm: ci: Enable configs to fix mt8173 boot hang issue
2023-09-04 16:15 [PATCH v2 0/7] drm: ci: fixes Vignesh Raman
` (2 preceding siblings ...)
2023-09-04 16:15 ` [PATCH v2 3/7] drm: ci: virtio: update ci variables Vignesh Raman
@ 2023-09-04 16:15 ` Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 5/7] drm: ci: Update xfails Vignesh Raman
` (2 subsequent siblings)
6 siblings, 0 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-04 16:15 UTC (permalink / raw)
To: dri-devel
Cc: helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Enable regulator
Enable MT6397 RTC driver
Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
---
v2:
- No changes
---
drivers/gpu/drm/ci/arm64.config | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/ci/arm64.config b/drivers/gpu/drm/ci/arm64.config
index 817e18ddfd4f..ea7a6cceff40 100644
--- a/drivers/gpu/drm/ci/arm64.config
+++ b/drivers/gpu/drm/ci/arm64.config
@@ -184,6 +184,8 @@ CONFIG_HW_RANDOM_MTK=y
CONFIG_MTK_DEVAPC=y
CONFIG_PWM_MTK_DISP=y
CONFIG_MTK_CMDQ=y
+CONFIG_REGULATOR_DA9211=y
+CONFIG_RTC_DRV_MT6397=y
# For nouveau. Note that DRM must be a module so that it's loaded after NFS is up to provide the firmware.
CONFIG_ARCH_TEGRA=y
--
2.40.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 5/7] drm: ci: Update xfails
2023-09-04 16:15 [PATCH v2 0/7] drm: ci: fixes Vignesh Raman
` (3 preceding siblings ...)
2023-09-04 16:15 ` [PATCH v2 4/7] drm: ci: Enable configs to fix mt8173 boot hang issue Vignesh Raman
@ 2023-09-04 16:15 ` Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 6/7] drm: ci: Enable new jobs Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 7/7] drm: ci: Use scripts/config to enable/disable configs Vignesh Raman
6 siblings, 0 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-04 16:15 UTC (permalink / raw)
To: dri-devel
Cc: helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Update amdgpu-stoney-fails, mediatek-mt8173-flakes,
mediatek-mt8173-fails, rockchip-rk3399-fails, rockchip-rk3399-flakes,
rockchip-rk3288-flakes, i915-cml-fails, i915-cml-flakes,
msm-apq8016-flakes files.
Add tests that fail sometimes into the *-flakes file and tests
that are failing into the *-fails file.
Signed-off-by: Helen Koike <helen.koike@collabora.com>
Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
---
v2:
- No changes
---
.../gpu/drm/ci/xfails/amdgpu-stoney-fails.txt | 1 -
drivers/gpu/drm/ci/xfails/i915-cml-fails.txt | 1 -
drivers/gpu/drm/ci/xfails/i915-cml-flakes.txt | 2 ++
drivers/gpu/drm/ci/xfails/i915-glk-flakes.txt | 1 +
.../gpu/drm/ci/xfails/mediatek-mt8173-fails.txt | 2 --
.../gpu/drm/ci/xfails/mediatek-mt8173-flakes.txt | 16 ++++++++++++++++
drivers/gpu/drm/ci/xfails/msm-apq8016-flakes.txt | 2 ++
.../gpu/drm/ci/xfails/rockchip-rk3288-flakes.txt | 1 +
.../gpu/drm/ci/xfails/rockchip-rk3399-fails.txt | 4 ++--
.../gpu/drm/ci/xfails/rockchip-rk3399-flakes.txt | 3 +++
10 files changed, 27 insertions(+), 6 deletions(-)
diff --git a/drivers/gpu/drm/ci/xfails/amdgpu-stoney-fails.txt b/drivers/gpu/drm/ci/xfails/amdgpu-stoney-fails.txt
index bd9392536e7c..58bfded8a3fc 100644
--- a/drivers/gpu/drm/ci/xfails/amdgpu-stoney-fails.txt
+++ b/drivers/gpu/drm/ci/xfails/amdgpu-stoney-fails.txt
@@ -1,7 +1,6 @@
kms_addfb_basic@bad-pitch-65536,Fail
kms_addfb_basic@bo-too-small,Fail
kms_async_flips@invalid-async-flip,Fail
-kms_atomic@plane-immutable-zpos,Fail
kms_atomic_transition@plane-toggle-modeset-transition,Fail
kms_bw@linear-tiling-1-displays-2560x1440p,Fail
kms_bw@linear-tiling-1-displays-3840x2160p,Fail
diff --git a/drivers/gpu/drm/ci/xfails/i915-cml-fails.txt b/drivers/gpu/drm/ci/xfails/i915-cml-fails.txt
index 6139b410e767..5f513c638beb 100644
--- a/drivers/gpu/drm/ci/xfails/i915-cml-fails.txt
+++ b/drivers/gpu/drm/ci/xfails/i915-cml-fails.txt
@@ -1,4 +1,3 @@
-kms_color@ctm-0-25,Fail
kms_flip_scaled_crc@flip-32bpp-linear-to-64bpp-linear-downscaling,Fail
kms_flip_scaled_crc@flip-32bpp-linear-to-64bpp-linear-upscaling,Fail
kms_flip_scaled_crc@flip-32bpp-xtile-to-64bpp-xtile-downscaling,Fail
diff --git a/drivers/gpu/drm/ci/xfails/i915-cml-flakes.txt b/drivers/gpu/drm/ci/xfails/i915-cml-flakes.txt
index 0514a7b3fdb0..f06f1a5b16f9 100644
--- a/drivers/gpu/drm/ci/xfails/i915-cml-flakes.txt
+++ b/drivers/gpu/drm/ci/xfails/i915-cml-flakes.txt
@@ -7,6 +7,8 @@ kms_bw@linear-tiling-3-displays-3840x2160p
kms_bw@linear-tiling-4-displays-1920x1080p
kms_bw@linear-tiling-4-displays-2560x1440p
kms_bw@linear-tiling-4-displays-3840x2160p
+kms_color@ctm-0-25
+kms_cursor_legacy@torture-move
kms_draw_crc@draw-method-xrgb8888-render-xtiled
kms_flip@flip-vs-suspend
kms_flip_scaled_crc@flip-32bpp-ytile-to-64bpp-ytile-downscaling
diff --git a/drivers/gpu/drm/ci/xfails/i915-glk-flakes.txt b/drivers/gpu/drm/ci/xfails/i915-glk-flakes.txt
index fc41d13a2d56..3aee1f11ee90 100644
--- a/drivers/gpu/drm/ci/xfails/i915-glk-flakes.txt
+++ b/drivers/gpu/drm/ci/xfails/i915-glk-flakes.txt
@@ -8,6 +8,7 @@ kms_bw@linear-tiling-3-displays-3840x2160p
kms_bw@linear-tiling-4-displays-1920x1080p
kms_bw@linear-tiling-4-displays-2560x1440p
kms_bw@linear-tiling-4-displays-3840x2160p
+kms_cursor_legacy@torture-bo
kms_flip@blocking-wf_vblank
kms_flip@wf_vblank-ts-check
kms_flip@wf_vblank-ts-check-interruptible
diff --git a/drivers/gpu/drm/ci/xfails/mediatek-mt8173-fails.txt b/drivers/gpu/drm/ci/xfails/mediatek-mt8173-fails.txt
index 671916067dba..c8e64bbfd480 100644
--- a/drivers/gpu/drm/ci/xfails/mediatek-mt8173-fails.txt
+++ b/drivers/gpu/drm/ci/xfails/mediatek-mt8173-fails.txt
@@ -1,5 +1,4 @@
kms_3d,Fail
-kms_addfb_basic@addfb25-bad-modifier,Fail
kms_bw@linear-tiling-1-displays-1920x1080p,Fail
kms_bw@linear-tiling-1-displays-2560x1440p,Fail
kms_bw@linear-tiling-1-displays-3840x2160p,Fail
@@ -11,7 +10,6 @@ kms_bw@linear-tiling-3-displays-2560x1440p,Fail
kms_bw@linear-tiling-3-displays-3840x2160p,Fail
kms_color@pipe-A-invalid-gamma-lut-sizes,Fail
kms_color@pipe-B-invalid-gamma-lut-sizes,Fail
-kms_force_connector_basic@force-connector-state,Fail
kms_force_connector_basic@force-edid,Fail
kms_force_connector_basic@force-load-detect,Fail
kms_force_connector_basic@prune-stale-modes,Fail
diff --git a/drivers/gpu/drm/ci/xfails/mediatek-mt8173-flakes.txt b/drivers/gpu/drm/ci/xfails/mediatek-mt8173-flakes.txt
index e69de29bb2d1..9ed6722df2c2 100644
--- a/drivers/gpu/drm/ci/xfails/mediatek-mt8173-flakes.txt
+++ b/drivers/gpu/drm/ci/xfails/mediatek-mt8173-flakes.txt
@@ -0,0 +1,16 @@
+core_setmaster_vs_auth
+kms_addfb_basic@addfb25-bad-modifier
+kms_color@invalid-gamma-lut-sizes
+kms_cursor_legacy@cursor-vs-flip-atomic
+kms_cursor_legacy@cursor-vs-flip-legacy
+kms_force_connector_basic@force-connector-state
+kms_hdmi_inject@inject-4k
+kms_plane_scaling@plane-scaler-with-pixel-format-unity-scaling
+kms_plane_scaling@plane-upscale-with-modifiers-20x20
+kms_plane_scaling@plane-upscale-with-pixel-format-20x20
+kms_plane_scaling@plane-upscale-with-rotation-20x20
+kms_plane_scaling@planes-downscale-factor-0-25-upscale-20x20
+kms_plane_scaling@planes-downscale-factor-0-5-upscale-20x20
+kms_plane_scaling@planes-downscale-factor-0-75-upscale-20x20
+kms_prop_blob@invalid-set-prop
+kms_prop_blob@invalid-set-prop-any
diff --git a/drivers/gpu/drm/ci/xfails/msm-apq8016-flakes.txt b/drivers/gpu/drm/ci/xfails/msm-apq8016-flakes.txt
index 0e3b60d3fade..cd48a8c1d569 100644
--- a/drivers/gpu/drm/ci/xfails/msm-apq8016-flakes.txt
+++ b/drivers/gpu/drm/ci/xfails/msm-apq8016-flakes.txt
@@ -1,3 +1,5 @@
+kms_cursor_legacy@forked-bo
+kms_cursor_legacy@forked-move
kms_force_connector_basic@force-connector-state
kms_force_connector_basic@force-edid
kms_force_connector_basic@force-load-detect
diff --git a/drivers/gpu/drm/ci/xfails/rockchip-rk3288-flakes.txt b/drivers/gpu/drm/ci/xfails/rockchip-rk3288-flakes.txt
index 45c54c75c899..081fe6f8d488 100644
--- a/drivers/gpu/drm/ci/xfails/rockchip-rk3288-flakes.txt
+++ b/drivers/gpu/drm/ci/xfails/rockchip-rk3288-flakes.txt
@@ -1,6 +1,7 @@
kms_addfb_basic@addfb25-bad-modifier
kms_cursor_crc@.*
kms_flip@basic-flip-vs-wf_vblank
+kms_flip@blocking-wf_vblank
kms_invalid_mode@int-max-clock,Crash
kms_pipe_crc_basic@.*
kms_properties@connector-properties-atomic,Crash
diff --git a/drivers/gpu/drm/ci/xfails/rockchip-rk3399-fails.txt b/drivers/gpu/drm/ci/xfails/rockchip-rk3399-fails.txt
index 6db08ba6b008..029a3be3164d 100644
--- a/drivers/gpu/drm/ci/xfails/rockchip-rk3399-fails.txt
+++ b/drivers/gpu/drm/ci/xfails/rockchip-rk3399-fails.txt
@@ -2,14 +2,13 @@ kms_color@legacy-gamma,Fail
kms_color@pipe-A-legacy-gamma,Fail
kms_color@pipe-B-legacy-gamma,Fail
kms_flip@basic-flip-vs-wf_vblank,Fail
-kms_flip@blocking-wf_vblank,Fail
kms_flip@dpms-vs-vblank-race,Fail
kms_flip@flip-vs-absolute-wf_vblank,Fail
kms_flip@flip-vs-absolute-wf_vblank-interruptible,Fail
kms_flip@flip-vs-blocking-wf-vblank,Fail
kms_flip@flip-vs-panning,Fail
kms_flip@flip-vs-panning-interruptible,Fail
-kms_flip@flip-vs-wf_vblank-interruptible,Fail
+kms_flip@modeset-vs-vblank-race,Fail
kms_flip@plain-flip-fb-recreate,Fail
kms_flip@plain-flip-fb-recreate-interruptible,Fail
kms_flip@plain-flip-ts-check,Fail
@@ -35,3 +34,4 @@ kms_plane_multiple@atomic-pipe-B-tiling-none,Fail
kms_plane_multiple@tiling-none,Fail
kms_prime@basic-crc,Fail
kms_rmfb@close-fd,Fail
+kms_universal_plane@universal-plane-pipe-B-functional,Fail
diff --git a/drivers/gpu/drm/ci/xfails/rockchip-rk3399-flakes.txt b/drivers/gpu/drm/ci/xfails/rockchip-rk3399-flakes.txt
index 4c0539b4beaf..869426a02cbd 100644
--- a/drivers/gpu/drm/ci/xfails/rockchip-rk3399-flakes.txt
+++ b/drivers/gpu/drm/ci/xfails/rockchip-rk3399-flakes.txt
@@ -1,7 +1,10 @@
+kms_color@gamma
kms_cursor_crc@.*
+kms_flip@blocking-wf_vblank
kms_flip@dpms-vs-vblank-race-interruptible
kms_flip@flip-vs-expired-vblank
+kms_flip@flip-vs-wf_vblank-interruptible
kms_flip@modeset-vs-vblank-race-interruptible
kms_pipe_crc_basic@.*
kms_pipe_crc_basic@compare-crc-sanitycheck-pipe-A
--
2.40.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 6/7] drm: ci: Enable new jobs
2023-09-04 16:15 [PATCH v2 0/7] drm: ci: fixes Vignesh Raman
` (4 preceding siblings ...)
2023-09-04 16:15 ` [PATCH v2 5/7] drm: ci: Update xfails Vignesh Raman
@ 2023-09-04 16:15 ` Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 7/7] drm: ci: Use scripts/config to enable/disable configs Vignesh Raman
6 siblings, 0 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-04 16:15 UTC (permalink / raw)
To: dri-devel
Cc: helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Enable the following jobs, as the issues noted in the
TODO comments have been resolved. This will ensure that these jobs
are now included and executed as part of the CI/CD pipeline.
msm:apq8016:
TODO: current issue: it is not fiding the NFS root. Fix and remove this rule.
mediatek:mt8173:
TODO: current issue: device is hanging. Fix and remove this rule.
virtio_gpu:none:
TODO: current issue: malloc(): corrupted top size. Fix and remove this rule.
Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
---
v2:
- Reworded the commit message
---
drivers/gpu/drm/ci/gitlab-ci.yml | 2 +-
drivers/gpu/drm/ci/test.yml | 9 ---------
2 files changed, 1 insertion(+), 10 deletions(-)
diff --git a/drivers/gpu/drm/ci/gitlab-ci.yml b/drivers/gpu/drm/ci/gitlab-ci.yml
index 2c4df53f5dfe..d2aac4404914 100644
--- a/drivers/gpu/drm/ci/gitlab-ci.yml
+++ b/drivers/gpu/drm/ci/gitlab-ci.yml
@@ -248,4 +248,4 @@ sanity:
# Jobs that need to pass before spending hardware resources on further testing
.required-for-hardware-jobs:
- needs: []
\ No newline at end of file
+ needs: []
diff --git a/drivers/gpu/drm/ci/test.yml b/drivers/gpu/drm/ci/test.yml
index d85add39f425..1771af21e2d9 100644
--- a/drivers/gpu/drm/ci/test.yml
+++ b/drivers/gpu/drm/ci/test.yml
@@ -108,9 +108,6 @@ msm:apq8016:
RUNNER_TAG: google-freedreno-db410c
script:
- ./install/bare-metal/fastboot.sh
- rules:
- # TODO: current issue: it is not fiding the NFS root. Fix and remove this rule.
- - when: never
msm:apq8096:
extends:
@@ -273,9 +270,6 @@ mediatek:mt8173:
DEVICE_TYPE: mt8173-elm-hana
GPU_VERSION: mt8173
RUNNER_TAG: mesa-ci-x86-64-lava-mt8173-elm-hana
- rules:
- # TODO: current issue: device is hanging. Fix and remove this rule.
- - when: never
mediatek:mt8183:
extends:
@@ -333,6 +327,3 @@ virtio_gpu:none:
- debian/x86_64_test-gl
- testing:x86_64
- igt:x86_64
- rules:
- # TODO: current issue: malloc(): corrupted top size. Fix and remove this rule.
- - when: never
\ No newline at end of file
--
2.40.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v2 7/7] drm: ci: Use scripts/config to enable/disable configs
2023-09-04 16:15 [PATCH v2 0/7] drm: ci: fixes Vignesh Raman
` (5 preceding siblings ...)
2023-09-04 16:15 ` [PATCH v2 6/7] drm: ci: Enable new jobs Vignesh Raman
@ 2023-09-04 16:15 ` Vignesh Raman
6 siblings, 0 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-04 16:15 UTC (permalink / raw)
To: dri-devel
Cc: helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Instead of modifying files in git to enable/disable
configs, use scripts/config on the .config file which
will be used for building the kernel.
Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
---
v2:
- Added a new patch in the series to use scripts/config to enable/disable configs
---
drivers/gpu/drm/ci/build.sh | 14 +++++++-------
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
index 92ffd98cd09e..c95f4daac221 100644
--- a/drivers/gpu/drm/ci/build.sh
+++ b/drivers/gpu/drm/ci/build.sh
@@ -70,19 +70,19 @@ if [ -z "$CI_MERGE_REQUEST_PROJECT_PATH" ]; then
fi
fi
-for opt in $ENABLE_KCONFIGS; do
- echo CONFIG_$opt=y >> drivers/gpu/drm/ci/${KERNEL_ARCH}.config
-done
-for opt in $DISABLE_KCONFIGS; do
- echo CONFIG_$opt=n >> drivers/gpu/drm/ci/${KERNEL_ARCH}.config
-done
-
if [[ -n "${MERGE_FRAGMENT}" ]]; then
./scripts/kconfig/merge_config.sh ${DEFCONFIG} drivers/gpu/drm/ci/${MERGE_FRAGMENT}
else
make `basename ${DEFCONFIG}`
fi
+for opt in $ENABLE_KCONFIGS; do
+ ./scripts/config --enable CONFIG_$opt
+done
+for opt in $DISABLE_KCONFIGS; do
+ ./scripts/config --disable CONFIG_$opt
+done
+
make ${KERNEL_IMAGE_NAME}
mkdir -p /lava-files/
--
2.40.1
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-04 16:15 ` [PATCH v2 2/7] drm: ci: Force db410c to host mode Vignesh Raman
@ 2023-09-04 16:59 ` Dmitry Baryshkov
2023-09-05 8:43 ` Maxime Ripard
2023-09-06 12:55 ` Helen Koike
1 sibling, 1 reply; 20+ messages in thread
From: Dmitry Baryshkov @ 2023-09-04 16:59 UTC (permalink / raw)
To: Vignesh Raman
Cc: dri-devel, helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
mripard, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
On Mon, 4 Sept 2023 at 19:16, Vignesh Raman <vignesh.raman@collabora.com> wrote:
>
> Force db410c to host mode to fix network issue which results in failure
> to mount root fs via NFS.
> See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
>
> Use fdtoverlay command to merge base device tree with an overlay
> which contains the fix for USB controllers to work in host mode.
>
> Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
> ---
>
> v2:
> - Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
>
> ---
> drivers/gpu/drm/ci/build.sh | 5 +++++
> .../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
> 2 files changed, 18 insertions(+)
> create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>
> diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
> index 7b014287a041..92ffd98cd09e 100644
> --- a/drivers/gpu/drm/ci/build.sh
> +++ b/drivers/gpu/drm/ci/build.sh
> @@ -92,6 +92,11 @@ done
>
> if [[ -n ${DEVICE_TREES} ]]; then
> make dtbs
> + if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
> + dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> + fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
> + mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
> + fi
> cp ${DEVICE_TREES} /lava-files/.
> fi
>
> diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> new file mode 100644
> index 000000000000..57b7604f1c23
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> @@ -0,0 +1,13 @@
> +/dts-v1/;
> +/plugin/;
> +
> +/ {
> + fragment@0 {
> + target-path = "/soc@0";
> + __overlay__ {
> + usb@78d9000 {
> + dr_mode = "host";
> + };
> + };
> + };
> +};
> --
> 2.40.1
Can we use normal dtso syntax here instead of defining fragments manually?
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-04 16:59 ` Dmitry Baryshkov
@ 2023-09-05 8:43 ` Maxime Ripard
2023-09-05 10:09 ` Vignesh Raman
0 siblings, 1 reply; 20+ messages in thread
From: Maxime Ripard @ 2023-09-05 8:43 UTC (permalink / raw)
To: Dmitry Baryshkov
Cc: Vignesh Raman, dri-devel, helen.koike, guilherme.gallo,
sergi.blanch.torne, david.heidelberg, daniels, gustavo.padovan,
emma, robclark, robdclark, anholt, robdclark, airlied, daniel,
jani.nikula, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Hi,
On Mon, Sep 04, 2023 at 07:59:26PM +0300, Dmitry Baryshkov wrote:
> On Mon, 4 Sept 2023 at 19:16, Vignesh Raman <vignesh.raman@collabora.com> wrote:
> >
> > Force db410c to host mode to fix network issue which results in failure
> > to mount root fs via NFS.
> > See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
> >
> > Use fdtoverlay command to merge base device tree with an overlay
> > which contains the fix for USB controllers to work in host mode.
> >
> > Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
> > ---
> >
> > v2:
> > - Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
> >
> > ---
> > drivers/gpu/drm/ci/build.sh | 5 +++++
> > .../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
> > 2 files changed, 18 insertions(+)
> > create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> >
> > diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
> > index 7b014287a041..92ffd98cd09e 100644
> > --- a/drivers/gpu/drm/ci/build.sh
> > +++ b/drivers/gpu/drm/ci/build.sh
> > @@ -92,6 +92,11 @@ done
> >
> > if [[ -n ${DEVICE_TREES} ]]; then
> > make dtbs
> > + if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
> > + dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > + fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
> > + mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
> > + fi
> > cp ${DEVICE_TREES} /lava-files/.
> > fi
> >
> > diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > new file mode 100644
> > index 000000000000..57b7604f1c23
> > --- /dev/null
> > +++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > @@ -0,0 +1,13 @@
> > +/dts-v1/;
> > +/plugin/;
> > +
> > +/ {
> > + fragment@0 {
> > + target-path = "/soc@0";
> > + __overlay__ {
> > + usb@78d9000 {
> > + dr_mode = "host";
> > + };
> > + };
> > + };
> > +};
> > --
> > 2.40.1
>
> Can we use normal dtso syntax here instead of defining fragments manually?
What Dmitry is hinting about is to use the "Sugar Syntax". There a good documentation here:
https://source.android.com/docs/core/architecture/dto/syntax
Maxime
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-05 8:43 ` Maxime Ripard
@ 2023-09-05 10:09 ` Vignesh Raman
2023-09-05 11:00 ` Maxime Ripard
0 siblings, 1 reply; 20+ messages in thread
From: Vignesh Raman @ 2023-09-05 10:09 UTC (permalink / raw)
To: Maxime Ripard, Dmitry Baryshkov
Cc: dri-devel, helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
linux-kernel, linux-mediatek, virtualization, linux-arm-msm
Hi Dmitry, Maxime,
On 05/09/23 14:13, Maxime Ripard wrote:
> Hi,
>
> On Mon, Sep 04, 2023 at 07:59:26PM +0300, Dmitry Baryshkov wrote:
>> On Mon, 4 Sept 2023 at 19:16, Vignesh Raman <vignesh.raman@collabora.com> wrote:
>>>
>>> Force db410c to host mode to fix network issue which results in failure
>>> to mount root fs via NFS.
>>> See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
>>>
>>> Use fdtoverlay command to merge base device tree with an overlay
>>> which contains the fix for USB controllers to work in host mode.
>>>
>>> Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
>>> ---
>>>
>>> v2:
>>> - Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
>>>
>>> ---
>>> drivers/gpu/drm/ci/build.sh | 5 +++++
>>> .../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
>>> 2 files changed, 18 insertions(+)
>>> create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>>>
>>> diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
>>> index 7b014287a041..92ffd98cd09e 100644
>>> --- a/drivers/gpu/drm/ci/build.sh
>>> +++ b/drivers/gpu/drm/ci/build.sh
>>> @@ -92,6 +92,11 @@ done
>>>
>>> if [[ -n ${DEVICE_TREES} ]]; then
>>> make dtbs
>>> + if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
>>> + dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>>> + fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
>>> + mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
>>> + fi
>>> cp ${DEVICE_TREES} /lava-files/.
>>> fi
>>>
>>> diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>>> new file mode 100644
>>> index 000000000000..57b7604f1c23
>>> --- /dev/null
>>> +++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>>> @@ -0,0 +1,13 @@
>>> +/dts-v1/;
>>> +/plugin/;
>>> +
>>> +/ {
>>> + fragment@0 {
>>> + target-path = "/soc@0";
>>> + __overlay__ {
>>> + usb@78d9000 {
>>> + dr_mode = "host";
>>> + };
>>> + };
>>> + };
>>> +};
>>> --
>>> 2.40.1
>>
>> Can we use normal dtso syntax here instead of defining fragments manually?
>
> What Dmitry is hinting about is to use the "Sugar Syntax". There a good documentation here:
> https://source.android.com/docs/core/architecture/dto/syntax
With the below DTO syntax,
/dts-v1/;
/plugin/;
&usb {
usb@78d9000 {
dr_mode = "host";
};
};
Decoded dtbo file is,
/dts-v1/;
/ {
fragment@0 {
target = <0xffffffff>;
__overlay__ {
usb@78d9000 {
dr_mode = "host";
};
};
};
__fixups__ {
usb = "/fragment@0:target:0";
};
};
With the previous fix using fragment we get,
/ {
fragment@0 {
target-path = "/soc@0";
__overlay__ {
usb@78d9000 {
dr_mode = "host";
};
};
};
};
Decoded apq8016-sbc.dtb file with the fix (setting dr_mode to host) is,
/dts-v1/;
/ {
soc@0 {
usb@78d9000 {
dr_mode = "host";
};
};
};
How can set the target to "soc@0" using the DTO syntax? Otherwise
fdtoverlay fails to apply the dtbo file with the base dtb.
Regards,
Vignesh
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-05 10:09 ` Vignesh Raman
@ 2023-09-05 11:00 ` Maxime Ripard
2023-09-05 11:10 ` Dmitry Baryshkov
0 siblings, 1 reply; 20+ messages in thread
From: Maxime Ripard @ 2023-09-05 11:00 UTC (permalink / raw)
To: Vignesh Raman
Cc: Dmitry Baryshkov, dri-devel, helen.koike, guilherme.gallo,
sergi.blanch.torne, david.heidelberg, daniels, gustavo.padovan,
emma, robclark, robdclark, anholt, robdclark, airlied, daniel,
jani.nikula, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
On Tue, Sep 05, 2023 at 03:39:33PM +0530, Vignesh Raman wrote:
> Hi Dmitry, Maxime,
>
> On 05/09/23 14:13, Maxime Ripard wrote:
> > Hi,
> >
> > On Mon, Sep 04, 2023 at 07:59:26PM +0300, Dmitry Baryshkov wrote:
> > > On Mon, 4 Sept 2023 at 19:16, Vignesh Raman <vignesh.raman@collabora.com> wrote:
> > > >
> > > > Force db410c to host mode to fix network issue which results in failure
> > > > to mount root fs via NFS.
> > > > See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
> > > >
> > > > Use fdtoverlay command to merge base device tree with an overlay
> > > > which contains the fix for USB controllers to work in host mode.
> > > >
> > > > Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
> > > > ---
> > > >
> > > > v2:
> > > > - Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
> > > >
> > > > ---
> > > > drivers/gpu/drm/ci/build.sh | 5 +++++
> > > > .../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
> > > > 2 files changed, 18 insertions(+)
> > > > create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > > >
> > > > diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
> > > > index 7b014287a041..92ffd98cd09e 100644
> > > > --- a/drivers/gpu/drm/ci/build.sh
> > > > +++ b/drivers/gpu/drm/ci/build.sh
> > > > @@ -92,6 +92,11 @@ done
> > > >
> > > > if [[ -n ${DEVICE_TREES} ]]; then
> > > > make dtbs
> > > > + if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
> > > > + dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > > > + fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
> > > > + mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
> > > > + fi
> > > > cp ${DEVICE_TREES} /lava-files/.
> > > > fi
> > > >
> > > > diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > > > new file mode 100644
> > > > index 000000000000..57b7604f1c23
> > > > --- /dev/null
> > > > +++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > > > @@ -0,0 +1,13 @@
> > > > +/dts-v1/;
> > > > +/plugin/;
> > > > +
> > > > +/ {
> > > > + fragment@0 {
> > > > + target-path = "/soc@0";
> > > > + __overlay__ {
> > > > + usb@78d9000 {
> > > > + dr_mode = "host";
> > > > + };
> > > > + };
> > > > + };
> > > > +};
> > > > --
> > > > 2.40.1
> > >
> > > Can we use normal dtso syntax here instead of defining fragments manually?
> >
> > What Dmitry is hinting about is to use the "Sugar Syntax". There a good documentation here:
> > https://source.android.com/docs/core/architecture/dto/syntax
>
>
> With the below DTO syntax,
> /dts-v1/;
> /plugin/;
>
> &usb {
> usb@78d9000 {
> dr_mode = "host";
> };
> };
>
> Decoded dtbo file is,
> /dts-v1/;
>
> / {
>
> fragment@0 {
> target = <0xffffffff>;
>
> __overlay__ {
>
> usb@78d9000 {
> dr_mode = "host";
> };
> };
> };
>
> __fixups__ {
> usb = "/fragment@0:target:0";
> };
> };
>
> With the previous fix using fragment we get,
> / {
>
> fragment@0 {
> target-path = "/soc@0";
>
> __overlay__ {
>
> usb@78d9000 {
> dr_mode = "host";
> };
> };
> };
> };
>
> Decoded apq8016-sbc.dtb file with the fix (setting dr_mode to host) is,
> /dts-v1/;
> / {
> soc@0 {
> usb@78d9000 {
> dr_mode = "host";
> };
> };
> };
>
> How can set the target to "soc@0" using the DTO syntax?
To strictly answer your question, that would be something like
&{/soc@0} {
usb@78d9000 {
dr_mode = "host";
};
};
You can simplify this further however by doing:
&{/soc@0/usb@78d9000} {
dr_mode = "host";
};
Also, that node actually has a label ("usb"), defined here:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/qcom/msm8916.dtsi#n2322
So you can end up with
&usb {
dr_mode = "host";
};
All of them should be equivalent to the one you had in your patch.
Maxime
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-05 11:00 ` Maxime Ripard
@ 2023-09-05 11:10 ` Dmitry Baryshkov
2023-09-05 11:41 ` Vignesh Raman
0 siblings, 1 reply; 20+ messages in thread
From: Dmitry Baryshkov @ 2023-09-05 11:10 UTC (permalink / raw)
To: Maxime Ripard
Cc: Vignesh Raman, dri-devel, helen.koike, guilherme.gallo,
sergi.blanch.torne, david.heidelberg, daniels, gustavo.padovan,
emma, robclark, robdclark, anholt, robdclark, airlied, daniel,
jani.nikula, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
On Tue, 5 Sept 2023 at 14:00, Maxime Ripard <mripard@kernel.org> wrote:
>
> On Tue, Sep 05, 2023 at 03:39:33PM +0530, Vignesh Raman wrote:
> > Hi Dmitry, Maxime,
> >
> > On 05/09/23 14:13, Maxime Ripard wrote:
> > > Hi,
> > >
> > > On Mon, Sep 04, 2023 at 07:59:26PM +0300, Dmitry Baryshkov wrote:
> > > > On Mon, 4 Sept 2023 at 19:16, Vignesh Raman <vignesh.raman@collabora.com> wrote:
> > > > >
> > > > > Force db410c to host mode to fix network issue which results in failure
> > > > > to mount root fs via NFS.
> > > > > See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
> > > > >
> > > > > Use fdtoverlay command to merge base device tree with an overlay
> > > > > which contains the fix for USB controllers to work in host mode.
> > > > >
> > > > > Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
> > > > > ---
> > > > >
> > > > > v2:
> > > > > - Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
> > > > >
> > > > > ---
> > > > > drivers/gpu/drm/ci/build.sh | 5 +++++
> > > > > .../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
> > > > > 2 files changed, 18 insertions(+)
> > > > > create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > > > >
> > > > > diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
> > > > > index 7b014287a041..92ffd98cd09e 100644
> > > > > --- a/drivers/gpu/drm/ci/build.sh
> > > > > +++ b/drivers/gpu/drm/ci/build.sh
> > > > > @@ -92,6 +92,11 @@ done
> > > > >
> > > > > if [[ -n ${DEVICE_TREES} ]]; then
> > > > > make dtbs
> > > > > + if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
> > > > > + dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > > > > + fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
> > > > > + mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
> > > > > + fi
> > > > > cp ${DEVICE_TREES} /lava-files/.
> > > > > fi
> > > > >
> > > > > diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > > > > new file mode 100644
> > > > > index 000000000000..57b7604f1c23
> > > > > --- /dev/null
> > > > > +++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > > > > @@ -0,0 +1,13 @@
> > > > > +/dts-v1/;
> > > > > +/plugin/;
> > > > > +
> > > > > +/ {
> > > > > + fragment@0 {
> > > > > + target-path = "/soc@0";
> > > > > + __overlay__ {
> > > > > + usb@78d9000 {
> > > > > + dr_mode = "host";
> > > > > + };
> > > > > + };
> > > > > + };
> > > > > +};
> > > > > --
> > > > > 2.40.1
> > > >
> > > > Can we use normal dtso syntax here instead of defining fragments manually?
> > >
> > > What Dmitry is hinting about is to use the "Sugar Syntax". There a good documentation here:
> > > https://source.android.com/docs/core/architecture/dto/syntax
> >
> >
> > With the below DTO syntax,
> > /dts-v1/;
> > /plugin/;
> >
> > &usb {
> > usb@78d9000 {
> > dr_mode = "host";
> > };
> > };
> >
> > Decoded dtbo file is,
> > /dts-v1/;
> >
> > / {
> >
> > fragment@0 {
> > target = <0xffffffff>;
> >
> > __overlay__ {
> >
> > usb@78d9000 {
> > dr_mode = "host";
> > };
> > };
> > };
> >
> > __fixups__ {
> > usb = "/fragment@0:target:0";
> > };
> > };
> >
> > With the previous fix using fragment we get,
> > / {
> >
> > fragment@0 {
> > target-path = "/soc@0";
> >
> > __overlay__ {
> >
> > usb@78d9000 {
> > dr_mode = "host";
> > };
> > };
> > };
> > };
> >
> > Decoded apq8016-sbc.dtb file with the fix (setting dr_mode to host) is,
> > /dts-v1/;
> > / {
> > soc@0 {
> > usb@78d9000 {
> > dr_mode = "host";
> > };
> > };
> > };
> >
> > How can set the target to "soc@0" using the DTO syntax?
>
> To strictly answer your question, that would be something like
>
> &{/soc@0} {
> usb@78d9000 {
> dr_mode = "host";
> };
> };
>
> You can simplify this further however by doing:
>
>
> &{/soc@0/usb@78d9000} {
> dr_mode = "host";
> };
>
> Also, that node actually has a label ("usb"), defined here:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/qcom/msm8916.dtsi#n2322
>
> So you can end up with
>
> &usb {
> dr_mode = "host";
> };
... which is the simplest and thus more robust one.
> All of them should be equivalent to the one you had in your patch.
>
> Maxime
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-05 11:10 ` Dmitry Baryshkov
@ 2023-09-05 11:41 ` Vignesh Raman
2023-09-05 11:57 ` Maxime Ripard
0 siblings, 1 reply; 20+ messages in thread
From: Vignesh Raman @ 2023-09-05 11:41 UTC (permalink / raw)
To: Dmitry Baryshkov, Maxime Ripard
Cc: dri-devel, helen.koike, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
linux-kernel, linux-mediatek, virtualization, linux-arm-msm
Hi,
On 05/09/23 16:40, Dmitry Baryshkov wrote:
> On Tue, 5 Sept 2023 at 14:00, Maxime Ripard <mripard@kernel.org> wrote:
>>
>> On Tue, Sep 05, 2023 at 03:39:33PM +0530, Vignesh Raman wrote:
>>> Hi Dmitry, Maxime,
>>>
>>> On 05/09/23 14:13, Maxime Ripard wrote:
>>>> Hi,
>>>>
>>>> On Mon, Sep 04, 2023 at 07:59:26PM +0300, Dmitry Baryshkov wrote:
>>>>> On Mon, 4 Sept 2023 at 19:16, Vignesh Raman <vignesh.raman@collabora.com> wrote:
>>>>>>
>>>>>> Force db410c to host mode to fix network issue which results in failure
>>>>>> to mount root fs via NFS.
>>>>>> See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
>>>>>>
>>>>>> Use fdtoverlay command to merge base device tree with an overlay
>>>>>> which contains the fix for USB controllers to work in host mode.
>>>>>>
>>>>>> Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
>>>>>> ---
>>>>>>
>>>>>> v2:
>>>>>> - Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
>>>>>>
>>>>>> ---
>>>>>> drivers/gpu/drm/ci/build.sh | 5 +++++
>>>>>> .../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
>>>>>> 2 files changed, 18 insertions(+)
>>>>>> create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>>>>>>
>>>>>> diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
>>>>>> index 7b014287a041..92ffd98cd09e 100644
>>>>>> --- a/drivers/gpu/drm/ci/build.sh
>>>>>> +++ b/drivers/gpu/drm/ci/build.sh
>>>>>> @@ -92,6 +92,11 @@ done
>>>>>>
>>>>>> if [[ -n ${DEVICE_TREES} ]]; then
>>>>>> make dtbs
>>>>>> + if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
>>>>>> + dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>>>>>> + fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
>>>>>> + mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
>>>>>> + fi
>>>>>> cp ${DEVICE_TREES} /lava-files/.
>>>>>> fi
>>>>>>
>>>>>> diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>>>>>> new file mode 100644
>>>>>> index 000000000000..57b7604f1c23
>>>>>> --- /dev/null
>>>>>> +++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>>>>>> @@ -0,0 +1,13 @@
>>>>>> +/dts-v1/;
>>>>>> +/plugin/;
>>>>>> +
>>>>>> +/ {
>>>>>> + fragment@0 {
>>>>>> + target-path = "/soc@0";
>>>>>> + __overlay__ {
>>>>>> + usb@78d9000 {
>>>>>> + dr_mode = "host";
>>>>>> + };
>>>>>> + };
>>>>>> + };
>>>>>> +};
>>>>>> --
>>>>>> 2.40.1
>>>>>
>>>>> Can we use normal dtso syntax here instead of defining fragments manually?
>>>>
>>>> What Dmitry is hinting about is to use the "Sugar Syntax". There a good documentation here:
>>>> https://source.android.com/docs/core/architecture/dto/syntax
>>>
>>>
>>> With the below DTO syntax,
>>> /dts-v1/;
>>> /plugin/;
>>>
>>> &usb {
>>> usb@78d9000 {
>>> dr_mode = "host";
>>> };
>>> };
>>>
>>> Decoded dtbo file is,
>>> /dts-v1/;
>>>
>>> / {
>>>
>>> fragment@0 {
>>> target = <0xffffffff>;
>>>
>>> __overlay__ {
>>>
>>> usb@78d9000 {
>>> dr_mode = "host";
>>> };
>>> };
>>> };
>>>
>>> __fixups__ {
>>> usb = "/fragment@0:target:0";
>>> };
>>> };
>>>
>>> With the previous fix using fragment we get,
>>> / {
>>>
>>> fragment@0 {
>>> target-path = "/soc@0";
>>>
>>> __overlay__ {
>>>
>>> usb@78d9000 {
>>> dr_mode = "host";
>>> };
>>> };
>>> };
>>> };
>>>
>>> Decoded apq8016-sbc.dtb file with the fix (setting dr_mode to host) is,
>>> /dts-v1/;
>>> / {
>>> soc@0 {
>>> usb@78d9000 {
>>> dr_mode = "host";
>>> };
>>> };
>>> };
>>>
>>> How can set the target to "soc@0" using the DTO syntax?
>>
>> To strictly answer your question, that would be something like
>>
>> &{/soc@0} {
>> usb@78d9000 {
>> dr_mode = "host";
>> };
>> };
>>
>> You can simplify this further however by doing:
>>
>>
>> &{/soc@0/usb@78d9000} {
>> dr_mode = "host";
>> };
The above works. Thanks.
>>
>> Also, that node actually has a label ("usb"), defined here:
>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/qcom/msm8916.dtsi#n2322
>>
>> So you can end up with
>>
>> &usb {
>> dr_mode = "host";
>> };
>
> ... which is the simplest and thus more robust one.
>
Should it be,
&{/soc@0/usb} {
dr_mode = "host";
};
I will send a v3 version for this. Thank you.
Regards,
Vignesh
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-05 11:41 ` Vignesh Raman
@ 2023-09-05 11:57 ` Maxime Ripard
2023-09-05 13:36 ` Vignesh Raman
0 siblings, 1 reply; 20+ messages in thread
From: Maxime Ripard @ 2023-09-05 11:57 UTC (permalink / raw)
To: Vignesh Raman
Cc: Dmitry Baryshkov, dri-devel, helen.koike, guilherme.gallo,
sergi.blanch.torne, david.heidelberg, daniels, gustavo.padovan,
emma, robclark, robdclark, anholt, robdclark, airlied, daniel,
jani.nikula, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
[-- Attachment #1: Type: text/plain, Size: 599 bytes --]
On Tue, Sep 05, 2023 at 05:11:43PM +0530, Vignesh Raman wrote:
> > > Also, that node actually has a label ("usb"), defined here:
> > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/qcom/msm8916.dtsi#n2322
> > >
> > > So you can end up with
> > >
> > > &usb {
> > > dr_mode = "host";
> > > };
> >
> > ... which is the simplest and thus more robust one.
> >
>
> Should it be,
> &{/soc@0/usb} {
> dr_mode = "host";
> };
No. The &{/...} syntax refers to a path. &... refers to a label. They
are not equivalent.
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-05 11:57 ` Maxime Ripard
@ 2023-09-05 13:36 ` Vignesh Raman
2023-09-05 13:40 ` Maxime Ripard
0 siblings, 1 reply; 20+ messages in thread
From: Vignesh Raman @ 2023-09-05 13:36 UTC (permalink / raw)
To: Maxime Ripard
Cc: Dmitry Baryshkov, dri-devel, helen.koike, guilherme.gallo,
sergi.blanch.torne, david.heidelberg, daniels, gustavo.padovan,
emma, robclark, robdclark, anholt, robdclark, airlied, daniel,
jani.nikula, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Hi Maxime,
On 05/09/23 17:27, Maxime Ripard wrote:
> On Tue, Sep 05, 2023 at 05:11:43PM +0530, Vignesh Raman wrote:
>>>> Also, that node actually has a label ("usb"), defined here:
>>>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/qcom/msm8916.dtsi#n2322
>>>>
>>>> So you can end up with
>>>>
>>>> &usb {
>>>> dr_mode = "host";
>>>> };
>>>
>>> ... which is the simplest and thus more robust one.
>>>
>>
>> Should it be,
>> &{/soc@0/usb} {
>> dr_mode = "host";
>> };
>
> No. The &{/...} syntax refers to a path. &... refers to a label. They
> are not equivalent.
Sorry I was not clear before.
With,
&usb {
dr_mode = "host";
};
The target is <0xffffffff> and fdtoverlay fails to apply the dtbo.
With,
&{/soc@0/usb} {
dr_mode = "host";
};
The target-path is "/soc@0/usb" (usb: usb@78d9000)
/ {
fragment@0 {
target-path = "/soc@0/usb";
__overlay__ {
dr_mode = "host";
};
};
};
So will use &{/...} syntax in this case.
Regards,
Vignesh
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-05 13:36 ` Vignesh Raman
@ 2023-09-05 13:40 ` Maxime Ripard
2023-09-06 5:11 ` Vignesh Raman
0 siblings, 1 reply; 20+ messages in thread
From: Maxime Ripard @ 2023-09-05 13:40 UTC (permalink / raw)
To: Vignesh Raman
Cc: Dmitry Baryshkov, dri-devel, helen.koike, guilherme.gallo,
sergi.blanch.torne, david.heidelberg, daniels, gustavo.padovan,
emma, robclark, robdclark, anholt, robdclark, airlied, daniel,
jani.nikula, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
[-- Attachment #1: Type: text/plain, Size: 1391 bytes --]
On Tue, Sep 05, 2023 at 07:06:43PM +0530, Vignesh Raman wrote:
> Hi Maxime,
>
> On 05/09/23 17:27, Maxime Ripard wrote:
> > On Tue, Sep 05, 2023 at 05:11:43PM +0530, Vignesh Raman wrote:
> > > > > Also, that node actually has a label ("usb"), defined here:
> > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm64/boot/dts/qcom/msm8916.dtsi#n2322
> > > > >
> > > > > So you can end up with
> > > > >
> > > > > &usb {
> > > > > dr_mode = "host";
> > > > > };
> > > >
> > > > ... which is the simplest and thus more robust one.
> > > >
> > >
> > > Should it be,
> > > &{/soc@0/usb} {
> > > dr_mode = "host";
> > > };
> >
> > No. The &{/...} syntax refers to a path. &... refers to a label. They
> > are not equivalent.
>
> Sorry I was not clear before.
>
> With,
> &usb {
> dr_mode = "host";
> };
>
> The target is <0xffffffff> and fdtoverlay fails to apply the dtbo.
You do have /plugin/ and have compiled the base device tree with overlay
support, right?
> With,
> &{/soc@0/usb} {
> dr_mode = "host";
> };
>
> The target-path is "/soc@0/usb" (usb: usb@78d9000)
Right, and that's not the path you want to modify. The path you want to
modify is /soc@0/usb@78d9000. usb is the label, it's absolute, and you
can't mix and match a path ("/soc@0/") and a label ("usb")
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-05 13:40 ` Maxime Ripard
@ 2023-09-06 5:11 ` Vignesh Raman
0 siblings, 0 replies; 20+ messages in thread
From: Vignesh Raman @ 2023-09-06 5:11 UTC (permalink / raw)
To: Maxime Ripard
Cc: Dmitry Baryshkov, dri-devel, helen.koike, guilherme.gallo,
sergi.blanch.torne, david.heidelberg, daniels, gustavo.padovan,
emma, robclark, robdclark, anholt, robdclark, airlied, daniel,
jani.nikula, linux-kernel, linux-mediatek, virtualization,
linux-arm-msm
Hi Maxime,
On 05/09/23 19:10, Maxime Ripard wrote:
>> With,
>> &usb {
>> dr_mode = "host";
>> };
>>
>> The target is <0xffffffff> and fdtoverlay fails to apply the dtbo.
>
> You do have /plugin/ and have compiled the base device tree with overlay
> support, right?
After compiling base dtbs with overlay support (make DTC_FLAGS=-@ dtbs)
it works.
>
>> With,
>> &{/soc@0/usb} {
>> dr_mode = "host";
>> };
>>
>> The target-path is "/soc@0/usb" (usb: usb@78d9000)
>
> Right, and that's not the path you want to modify. The path you want to
> modify is /soc@0/usb@78d9000. usb is the label, it's absolute, and you
> can't mix and match a path ("/soc@0/") and a label ("usb")
Thanks for the clarification.
Regards,
Vignesh
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-04 16:15 ` [PATCH v2 2/7] drm: ci: Force db410c to host mode Vignesh Raman
2023-09-04 16:59 ` Dmitry Baryshkov
@ 2023-09-06 12:55 ` Helen Koike
2023-09-06 13:13 ` Maxime Ripard
1 sibling, 1 reply; 20+ messages in thread
From: Helen Koike @ 2023-09-06 12:55 UTC (permalink / raw)
To: Vignesh Raman, dri-devel
Cc: guilherme.gallo, sergi.blanch.torne, david.heidelberg, daniels,
gustavo.padovan, emma, robclark, robdclark, anholt, robdclark,
airlied, daniel, jani.nikula, mripard, linux-kernel,
linux-mediatek, virtualization, linux-arm-msm
Hi!
On 04/09/2023 13:15, Vignesh Raman wrote:
> Force db410c to host mode to fix network issue which results in failure
> to mount root fs via NFS.
> See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
>
> Use fdtoverlay command to merge base device tree with an overlay
> which contains the fix for USB controllers to work in host mode.
>
> Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
> ---
>
> v2:
> - Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
>
> ---
> drivers/gpu/drm/ci/build.sh | 5 +++++
> .../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
> 2 files changed, 18 insertions(+)
> create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
>
> diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
> index 7b014287a041..92ffd98cd09e 100644
> --- a/drivers/gpu/drm/ci/build.sh
> +++ b/drivers/gpu/drm/ci/build.sh
> @@ -92,6 +92,11 @@ done
>
> if [[ -n ${DEVICE_TREES} ]]; then
> make dtbs
> + if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
> + dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> + fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
> + mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
> + fi
> cp ${DEVICE_TREES} /lava-files/.
> fi
>
> diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> new file mode 100644
> index 000000000000..57b7604f1c23
> --- /dev/null
> +++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> @@ -0,0 +1,13 @@
> +/dts-v1/;
> +/plugin/;
> +
> +/ {
> + fragment@0 {
> + target-path = "/soc@0";
> + __overlay__ {
> + usb@78d9000 {
> + dr_mode = "host";
> + };
> + };
> + };
> +};
Another thing that I was discussing with David and Vignesh, since we
will need this overlay spinets not only for drm-ci but also for mesa ci
(and every body who uses the farms), would it be interesting to move it
to some place more official? like dts folders? Or would that be against
Linux policy?
Regards,
Helen
^ permalink raw reply [flat|nested] 20+ messages in thread
* Re: [PATCH v2 2/7] drm: ci: Force db410c to host mode
2023-09-06 12:55 ` Helen Koike
@ 2023-09-06 13:13 ` Maxime Ripard
0 siblings, 0 replies; 20+ messages in thread
From: Maxime Ripard @ 2023-09-06 13:13 UTC (permalink / raw)
To: Helen Koike
Cc: Vignesh Raman, dri-devel, guilherme.gallo, sergi.blanch.torne,
david.heidelberg, daniels, gustavo.padovan, emma, robclark,
robdclark, anholt, robdclark, airlied, daniel, jani.nikula,
linux-kernel, linux-mediatek, virtualization, linux-arm-msm
[-- Attachment #1: Type: text/plain, Size: 2888 bytes --]
On Wed, Sep 06, 2023 at 09:55:40AM -0300, Helen Koike wrote:
> Hi!
>
> On 04/09/2023 13:15, Vignesh Raman wrote:
> > Force db410c to host mode to fix network issue which results in failure
> > to mount root fs via NFS.
> > See https://gitlab.freedesktop.org/gfx-ci/linux/-/commit/cb72a629b8c15c80a54dda510743cefd1c4b65b8
> >
> > Use fdtoverlay command to merge base device tree with an overlay
> > which contains the fix for USB controllers to work in host mode.
> >
> > Signed-off-by: Vignesh Raman <vignesh.raman@collabora.com>
> > ---
> >
> > v2:
> > - Use fdtoverlay command to merge overlay dtbo with the base dtb instead of modifying the kernel sources
> > ---
> > drivers/gpu/drm/ci/build.sh | 5 +++++
> > .../gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts | 13 +++++++++++++
> > 2 files changed, 18 insertions(+)
> > create mode 100644 drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> >
> > diff --git a/drivers/gpu/drm/ci/build.sh b/drivers/gpu/drm/ci/build.sh
> > index 7b014287a041..92ffd98cd09e 100644
> > --- a/drivers/gpu/drm/ci/build.sh
> > +++ b/drivers/gpu/drm/ci/build.sh
> > @@ -92,6 +92,11 @@ done
> > if [[ -n ${DEVICE_TREES} ]]; then
> > make dtbs
> > + if [[ -e arch/arm64/boot/dts/qcom/apq8016-sbc.dtb ]]; then
> > + dtc -@ -I dts -O dtb -o drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > + fdtoverlay -i arch/arm64/boot/dts/qcom/apq8016-sbc.dtb -o arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dtbo
> > + mv arch/arm64/boot/dts/qcom/apq8016-sbc-overlay.dtb arch/arm64/boot/dts/qcom/apq8016-sbc.dtb
> > + fi
> > cp ${DEVICE_TREES} /lava-files/.
> > fi
> > diff --git a/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > new file mode 100644
> > index 000000000000..57b7604f1c23
> > --- /dev/null
> > +++ b/drivers/gpu/drm/ci/dt-overlays/apq8016-sbc-overlay.dts
> > @@ -0,0 +1,13 @@
> > +/dts-v1/;
> > +/plugin/;
> > +
> > +/ {
> > + fragment@0 {
> > + target-path = "/soc@0";
> > + __overlay__ {
> > + usb@78d9000 {
> > + dr_mode = "host";
> > + };
> > + };
> > + };
> > +};
>
>
> Another thing that I was discussing with David and Vignesh, since we will
> need this overlay spinets not only for drm-ci but also for mesa ci (and
> every body who uses the farms), would it be interesting to move it to some
> place more official? like dts folders? Or would that be against Linux
> policy?
AFAIK, the policy was changed recently to allow overlays to be merged,
see $(find arch/ -name *.dtso). So generally speaking, it should be ok
to send it.
Maxime
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]
^ permalink raw reply [flat|nested] 20+ messages in thread
end of thread, other threads:[~2023-09-06 13:13 UTC | newest]
Thread overview: 20+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-09-04 16:15 [PATCH v2 0/7] drm: ci: fixes Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 1/7] drm: ci: igt_runner: remove todo Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 2/7] drm: ci: Force db410c to host mode Vignesh Raman
2023-09-04 16:59 ` Dmitry Baryshkov
2023-09-05 8:43 ` Maxime Ripard
2023-09-05 10:09 ` Vignesh Raman
2023-09-05 11:00 ` Maxime Ripard
2023-09-05 11:10 ` Dmitry Baryshkov
2023-09-05 11:41 ` Vignesh Raman
2023-09-05 11:57 ` Maxime Ripard
2023-09-05 13:36 ` Vignesh Raman
2023-09-05 13:40 ` Maxime Ripard
2023-09-06 5:11 ` Vignesh Raman
2023-09-06 12:55 ` Helen Koike
2023-09-06 13:13 ` Maxime Ripard
2023-09-04 16:15 ` [PATCH v2 3/7] drm: ci: virtio: update ci variables Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 4/7] drm: ci: Enable configs to fix mt8173 boot hang issue Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 5/7] drm: ci: Update xfails Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 6/7] drm: ci: Enable new jobs Vignesh Raman
2023-09-04 16:15 ` [PATCH v2 7/7] drm: ci: Use scripts/config to enable/disable configs Vignesh Raman
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®