Date: Tue, 4 Jun 2024 11:51:58 -0400
From: Nícolas F. R. A. Prado <nfraprado@...labora.com>
To: linux-mediatek@...ts.infradead.org,
	AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
	Matthias Brugger <matthias.bgg@...il.com>
Cc: Chen-Yu Tsai <wenst@...omium.org>, 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: Probe failures on mt8183-kukui-jacuzzi-juniper-sku16

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.

* /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.

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