Message-ID: <CAGXv+5HJKur9vWq4j_x7-fuL5H0SY_oQNYFWTxFCC1EBxF9yQw@mail.gmail.com>
Date: Wed, 5 Jun 2024 16:01:06 +0800
From: Chen-Yu Tsai <wenst@...omium.org>
To: Nícolas F. R. A. Prado <nfraprado@...labora.com>
Cc: linux-mediatek@...ts.infradead.org, 
	AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, 
	Matthias Brugger <matthias.bgg@...il.com>, Rob Herring <robh@...nel.org>, 
	Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, linux-kernel@...r.kernel.org, 
	linux-arm-kernel@...ts.infradead.org, kernel@...labora.com
Subject: Re: Probe failures on mt8183-kukui-jacuzzi-juniper-sku16

Hi,

On Tue, Jun 4, 2024 at 11:52 PM Nícolas F. R. A. Prado
<nfraprado@...labora.com> wrote:
>
> Hi,
>
> We're monitoring the status of device probe on the
> mt8183-kukui-jacuzzi-juniper-sku16 platform in KernelCI, and I thought it'd be
> helpful to share the results.
>
> These are the current failures:
>
> * /soc/i2c@...09000/trackpad@15
> * /soc/svs@...0bc00
> * /thermal-sensor1
> * /thermal-sensor2
>
> In more details:
>
> * /soc/i2c@...09000/trackpad@15: I believe this is a second source trackpad and
>   is thus waiting for the "platform/chrome: Introduce DT hardware prober" series
>   [1] to be merged before it can be handled. Although looks like there haven't
>   been updates to the series in a while.

I've been working on other stuff and haven't had time to respin this series.

Just to confirm, is the failure due to pinctrl conflicts when the trackpad
variants probe asynchronously?

> * /soc/svs@...0bc00: On the logs the following is reported:
>
>     SVSB_GPU: cannot get "gpu-thermal" thermal zone
>
>   MT8183 uses the auxadc_thermal driver, which apparently only registers one
>   thermal zone (tz 0, for the CPU):
>
>     devm_thermal_of_zone_register(&pdev->dev, 0, mt,
>                               &mtk_thermal_ops);
>
>   As a quick hack I changed the 0 to 2, so the GPU thermal zone got registered,
>   and that did allow SVS to go further in its probe, but still resulted in an
>   error:
>
>     [    9.330993] mtk-svs 1100bc00.svs: M_HW_RES0: 0x00320090
>     [    9.336700] mtk-svs 1100bc00.svs: M_HW_RES1: 0xa3ffff5f
>     [    9.342628] mtk-svs 1100bc00.svs: M_HW_RES2: 0x47cf47cf
>     [    9.347898] mtk-svs 1100bc00.svs: M_HW_RES3: 0xa3ffff5f
>     [    9.353164] mtk-svs 1100bc00.svs: M_HW_RES4: 0xa3fff1aa
>     [    9.358400] mtk-svs 1100bc00.svs: M_HW_RES5: 0x47ec4b82
>     [    9.358411] mtk-svs 1100bc00.svs: M_HW_RES6: 0xa3ff9aac
>     [    9.358416] mtk-svs 1100bc00.svs: M_HW_RES7: 0xa3ff1c13
>     [    9.358420] mtk-svs 1100bc00.svs: M_HW_RES8: 0x4bec4beb
>     [    9.358424] mtk-svs 1100bc00.svs: M_HW_RES9: 0xa3ff7b02
>     [    9.358429] mtk-svs 1100bc00.svs: M_HW_RES14: 0x06ce7583
>     [    9.370719] mtk-svs 1100bc00.svs: M_HW_RES15: 0x00220010
>     [    9.381134] mtk-svs 1100bc00.svs: M_HW_RES16: 0xa3ffdf46
>     [    9.381141] mtk-svs 1100bc00.svs: M_HW_RES17: 0x47ec47eb
>     [    9.391612] mtk-svs 1100bc00.svs: M_HW_RES18: 0xa3ffd443
>     [    9.477419]  SVSB_CPU_LITTLE: svs_init01_isr_handler: VDN74~30:0x16202931~0x363b4147, DC:0x02bb0013
>     [    9.495313] cpu cpu4: cpu4: failed to find OPP for 1781000000
>     [    9.495549]  SVSB_CPU_BIG: svs_init01_isr_handler: VDN74~30:0x16212a33~0x393e444b, DC:0x02d000d0
>     [    9.495847]  SVSB_CCI: svs_init01_isr_handler: VDN74~30:0x141b252e~0x33394147, DC:0x02c10049
>     [    9.495897]  SVSB_GPU: buck voltage: 700000uV, expected vboot: 800000uV
>     [    9.496184] mtk-svs 1100bc00.svs: error -EPERM: svs start fail
>     [    9.496204] mtk-svs 1100bc00.svs: probe with driver mtk-svs failed with error -1
>
>   so it seems that more changes will be required in addition to updating
>   auxadc_thermal to register multiple thermal zones.
>
> * /thermal-sensor1, /thermal-sensor2: The following is reported:
>
>     [    2.813605] thermal_sys: Failed to find 'trips' node
>     [    2.813611] thermal_sys: Failed to find trip points for thermal-sensor1 id=0
>     [    2.813619] generic-adc-thermal thermal-sensor1: Thermal zone sensor register failed: -22
>     [    2.813624] generic-adc-thermal thermal-sensor1: probe with driver generic-adc-thermal failed with error -22
>
>   And the same for thermal-sensor2. So both are missing the required 'trips'
>   subnode.

IIRC either the driver or the DT had comments saying "these zones don't
need trip points". In that case maybe the driver should use
thermal_tripless_zone_device_register() for them?


ChenYu

> If you have any additional information on these issues please let me know.
>
> Full test results are included below if you're curious.
>
> Note: these results are from today's next: next-20240604
>
> Thanks,
> Nícolas
>
>
> [1] [RFC PATCH v3 0/5] platform/chrome: Introduce DT hardware prober
> https://lore.kernel.org/all/20231128084236.157152-1-wenst@chromium.org/
>
>
> TAP version 13
> 1..1
> # timeout set to 45
> # selftests: dt: test_unprobed_devices.sh
> # TAP version 13
> # 1..166
> # ok 1 / # SKIP
> # ok 2 /backlight_lcd0
> # ok 3 /bt-sco
> # ok 4 /cci
> # ok 5 /cpus/cpu@0 # SKIP
> # ok 6 /cpus/cpu@1 # SKIP
> # ok 7 /cpus/cpu@100 # SKIP
> # ok 8 /cpus/cpu@101 # SKIP
> # ok 9 /cpus/cpu@102 # SKIP
> # ok 10 /cpus/cpu@103 # SKIP
> # ok 11 /cpus/cpu@2 # SKIP
> # ok 12 /cpus/cpu@3 # SKIP
> # ok 13 /cpus/idle-states/cluster-sleep-0 # SKIP
> # ok 14 /cpus/idle-states/cluster-sleep-1 # SKIP
> # ok 15 /cpus/idle-states/cpu-sleep # SKIP
> # ok 16 /cpus/l2-cache0 # SKIP
> # ok 17 /cpus/l2-cache1 # SKIP
> # ok 18 /firmware/chromeos # SKIP
> # ok 19 /firmware/coreboot
> # ok 20 /fixed-factor-clock-13m # SKIP
> # ok 21 /max98357a
> # ok 22 /mt8183-sound
> # ok 23 /opp-table-0 # SKIP
> # ok 24 /opp-table-cci # SKIP
> # ok 25 /opp-table-cluster0 # SKIP
> # ok 26 /opp-table-cluster1 # SKIP
> # ok 27 /oscillator # SKIP
> # ok 28 /oscillator1 # SKIP
> # ok 29 /pmu-a53
> # ok 30 /pmu-a73
> # ok 31 /pp1200-mipibrdg
> # ok 32 /pp1800-mipibrdg
> # ok 33 /pp3300-panel
> # ok 34 /psci
> # ok 35 /regulator-vsys
> # ok 36 /regulator0
> # ok 37 /regulator1
> # ok 38 /regulator2
> # ok 39 /regulator3
> # ok 40 /regulator4
> # ok 41 /regulator5
> # ok 42 /regulator6
> # ok 43 /reserved-memory/memory@...00000 # SKIP
> # ok 44 /reserved-memory/ramoops
> # ok 45 /soc
> # ok 46 /soc/aal@...10000
> # ok 47 /soc/audio-controller@...20000
> # ok 48 /soc/audio-controller@...20000/mt8183-afe-pcm
> # ok 49 /soc/auxadc@...01000
> # ok 50 /soc/ccorr@...0f000
> # ok 51 /soc/color@...0e000
> # ok 52 /soc/cpu-debug@...0000
> # ok 53 /soc/cpu-debug@...0000
> # ok 54 /soc/cpu-debug@...0000
> # ok 55 /soc/cpu-debug@...0000
> # ok 56 /soc/cpu-debug@...0000
> # ok 57 /soc/cpu-debug@...0000
> # ok 58 /soc/cpu-debug@...0000
> # ok 59 /soc/cpu-debug@...0000
> # ok 60 /soc/dither@...12000 # SKIP
> # ok 61 /soc/dma-controller0@...01000
> # ok 62 /soc/dma-controller@...05000 # SKIP
> # ok 63 /soc/dsi-phy@...50000
> # ok 64 /soc/dsi@...14000
> # ok 65 /soc/efuse@...10000
> # ok 66 /soc/efuse@...0000
> # ok 67 /soc/gamma@...11000
> # ok 68 /soc/gpu@...40000
> # ok 69 /soc/i2c@...05000
> # ok 70 /soc/i2c@...07000
> # ok 71 /soc/i2c@...07000/touchscreen@10
> # ok 72 /soc/i2c@...08000
> # ok 73 /soc/i2c@...08000/anx7625@58
> # ok 74 /soc/i2c@...08000/anx7625@...aux-bus/panel
> # ok 75 /soc/i2c@...09000
> # not ok 76 /soc/i2c@...09000/trackpad@15
> # ok 77 /soc/i2c@...09000/trackpad@2c
> # ok 78 /soc/i2c@...0f000
> # ok 79 /soc/i2c@...11000
> # ok 80 /soc/i2c@...16000
> # ok 81 /soc/i2c@...16000/ts3a227e@3b
> # ok 82 /soc/interrupt-controller@...0000 # SKIP
> # ok 83 /soc/interrupt-controller@...0a80 # SKIP
> # ok 84 /soc/iommu@...05000
> # ok 85 /soc/jpeg-encoder@...30000
> # ok 86 /soc/larb@...17000
> # ok 87 /soc/larb@...21000
> # ok 88 /soc/larb@...2f000
> # ok 89 /soc/larb@...10000
> # ok 90 /soc/larb@...10000
> # ok 91 /soc/larb@...01000
> # ok 92 /soc/larb@...02000
> # ok 93 /soc/mailbox@...38000
> # ok 94 /soc/mdp3-ccorr@...1c000 # SKIP
> # ok 95 /soc/mdp3-rsz0@...03000 # SKIP
> # ok 96 /soc/mdp3-rsz1@...04000 # SKIP
> # ok 97 /soc/mdp3-wdma@...06000 # SKIP
> # ok 98 /soc/mmc@...30000
> # ok 99 /soc/mmc@...40000
> # ok 100 /soc/mmc@...40000/qca-wifi@1
> # ok 101 /soc/mutex@...16000
> # ok 102 /soc/ovl@...08000
> # ok 103 /soc/ovl@...09000
> # ok 104 /soc/ovl@...0a000
> # ok 105 /soc/pinctrl@...05000
> # ok 106 /soc/pwm@...06000
> # ok 107 /soc/pwm@...0e000
> # ok 108 /soc/pwrap@...0d000
> # ok 109 /soc/pwrap@...0d000/pmic
> # ok 110 /soc/pwrap@...0d000/pmic/keys
> # ok 111 /soc/pwrap@...0d000/pmic/mt6358codec
> # ok 112 /soc/pwrap@...0d000/pmic/mt6358regulator
> # ok 113 /soc/pwrap@...0d000/pmic/rtc
> # ok 114 /soc/rdma@...0b000
> # ok 115 /soc/rdma@...0c000
> # ok 116 /soc/scp@...00000
> # ok 117 /soc/scp@...00000/cros-ec-rpmsg
> # ok 118 /soc/serial@...02000
> # ok 119 /soc/serial@...03000
> # ok 120 /soc/serial@...03000/bluetooth
> # ok 121 /soc/smi@...19000
> # ok 122 /soc/spi@...0a000
> # ok 123 /soc/spi@...0a000/tpm@0
> # ok 124 /soc/spi@...10000
> # ok 125 /soc/spi@...10000/flash@0
> # ok 126 /soc/spi@...12000
> # ok 127 /soc/spi@...12000/cros-ec@0
> # ok 128 /soc/spi@...12000/cros-ec@...xtcon0
> # ok 129 /soc/spi@...12000/cros-ec@...2c-tunnel
> # ok 130 /soc/spi@...12000/cros-ec@...2c-tunnel/sbs-battery@b
> # ok 131 /soc/spi@...12000/cros-ec@...eyboard-controller
> # ok 132 /soc/spi@...12000/cros-ec@...ypec
> # ok 133 /soc/spi@...12000/cros-ec@...ypec/connector@0 # SKIP
> # not ok 134 /soc/svs@...0bc00
> # ok 135 /soc/syscon@...00000
> # ok 136 /soc/syscon@...01000
> # ok 137 /soc/syscon@...03000
> # ok 138 /soc/syscon@...06000 # SKIP
> # ok 139 /soc/syscon@...06000/power-controller
> # ok 140 /soc/syscon@...0c000
> # ok 141 /soc/syscon@...00000
> # ok 142 /soc/syscon@...00000
> # ok 143 /soc/syscon@...20000
> # ok 144 /soc/syscon@...00000
> # ok 145 /soc/syscon@...00000
> # ok 146 /soc/syscon@...00000
> # ok 147 /soc/syscon@...10000
> # ok 148 /soc/syscon@...80000
> # ok 149 /soc/syscon@...80000
> # ok 150 /soc/syscon@...00000
> # ok 151 /soc/syscon@...0000
> # ok 152 /soc/t-phy@...40000
> # ok 153 /soc/thermal-sensor@...0b000
> # ok 154 /soc/timer@...17000 # SKIP
> # ok 155 /soc/usb@...01000
> # ok 156 /soc/usb@...01000/usb@...00000
> # ok 157 /soc/usb@...01000/usb@...00000/hub@1
> # ok 158 /soc/video-codec@...20000
> # ok 159 /soc/watchdog@...07000
> # not ok 160 /thermal-sensor1
> # not ok 161 /thermal-sensor2
> # ok 162 /timer # SKIP
> # ok 163 /vddio-mipibrdg
> # ok 164 /volume-buttons
> # ok 165 /wifi-pwrseq
> # ok 166 /wifi-wakeup
> # # Totals: pass:127 fail:4 xfail:0 xpass:0 skip:35 error:0

Powered by blists - more mailing lists