Message-ID: <f804c19f-622e-41af-8be5-3751532c4e9c@collabora.com>
Date: Mon, 28 Jul 2025 12:55:08 +0200
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
To: Rob Herring <robh@...nel.org>, Matthias Brugger
 <matthias.bgg@...nel.org>, Julien Massot <julien.massot@...labora.com>,
 Ariel D'Alessandro <ariel.dalessandro@...labora.com>
Cc: linux-mediatek@...ts.infradead.org, herbert@...dor.apana.org.au,
 davem@...emloft.net, krzk+dt@...nel.org, conor+dt@...nel.org,
 chunkuang.hu@...nel.org, p.zabel@...gutronix.de, airlied@...il.com,
 simona@...ll.ch, maarten.lankhorst@...ux.intel.com, mripard@...nel.org,
 tzimmermann@...e.de, jassisinghbrar@...il.com, mchehab@...nel.org,
 matthias.bgg@...il.com, chunfeng.yun@...iatek.com, vkoul@...nel.org,
 kishon@...nel.org, sean.wang@...nel.org, linus.walleij@...aro.org,
 lgirdwood@...il.com, broonie@...nel.org, andersson@...nel.org,
 mathieu.poirier@...aro.org, daniel.lezcano@...aro.org, tglx@...utronix.de,
 atenart@...nel.org, jitao.shi@...iatek.com, ck.hu@...iatek.com,
 houlong.wei@...iatek.com, kyrie.wu@...iatek.corp-partner.google.com,
 andy.teng@...iatek.com, tinghan.shen@...iatek.com, jiaxin.yu@...iatek.com,
 shane.chien@...iatek.com, olivia.wen@...iatek.com, granquet@...libre.com,
 eugen.hristev@...aro.org, arnd@...db.de, sam.shih@...iatek.com,
 jieyy.yang@...iatek.com, frank-w@...lic-files.de, mwalle@...nel.org,
 fparent@...libre.com, linux-crypto@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 dri-devel@...ts.freedesktop.org, linux-media@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org, linux-phy@...ts.infradead.org,
 linux-gpio@...r.kernel.org, linux-remoteproc@...r.kernel.org,
 linux-sound@...r.kernel.org
Subject: Re: [PATCH 00/38] MediaTek devicetree/bindings warnings sanitization

Il 25/07/25 15:52, Rob Herring ha scritto:
> On Thu, Jul 24, 2025 at 3:39 AM AngeloGioacchino Del Regno
> <angelogioacchino.delregno@...labora.com> wrote:
>>
>> As Rob pointed out, MediaTek devicetrees are *poor* in the dtbs_check
>> tests, and got an infinite load of warnings.
>>
>> This series starts attacking this situation.
>>
>> I didn't really count how many warnings I have resolved - it's a lot
>> of them anyway - and I think that this is a good start in any case.
> 
> 40 out of 125 (on arm64) fixed! Thanks! FYI, here's the ones that
> remain (first number is number of times the warning occurs):
> 

Thanks Rob!

I know that there are more warnings to be fixed - but I am unable to do that
right now, as I'm going on holiday in a few days.

However, I will ask to some colleagues of mine (Ariel and Julien, added to this
thread so that they also get your log and can take action) to try to solve
some more warnings while I am away, starting with the easiest ones so that we
reduce this bad number.

In that pinctrl dt fixes commit, you made me realize how many (old leftover and
new) warnings we had on MediaTek - it's bad, and must be fixed.. but in the
meanwhile, thanks again for making me notice.

Speaking of holidays, I'll also ask to Matthias to manage this merge window, so
that those commits won't have to wait for the next one (as I'll be back in early
September, I'm not sure how much time will be left until the window closes).

Cheers!
Angelo

>       29 (mediatek,mt8183-mfgcfg): 'power-domains' does not match any
> of the regexes: '^pinctrl-[0-9]+$'
>       29 failed to match any schema with compatible:
> ['mediatek,mt8183-audiosys', 'syscon']
>       29 failed to match any schema with compatible: ['mediatek,mt8183-audio']
>       27 (mediatek,mt8183-pinctrl): 'gpio-line-names' does not match
> any of the regexes: '-pins(-[a-z]+)?$', '^pinctrl-[0-9]+$'
>       22 (mediatek,mt6359): '#sound-dai-cells' does not match any of
> the regexes: '^pinctrl-[0-9]+$'
>       14 failed to match any schema with compatible:
> ['mediatek,mt8183_mt6358_ts3a227_max98357']
>       12 (mediatek,mt8186-mt6366-rt1019-rt5682s-sound): 'model' is a
> required property
>       12 failed to match any schema with compatible: ['mediatek,mt8173-mdp-rsz']
>        9 (mediatek,mt8195-iommu-infra): interrupts: [[0, 795, 4, 0],
> [0, 796, 4, 0], [0, 797, 4, 0], [0, 798, 4, 0], [0, 799, 4, 0]] is too
> long
>        8 failed to match any schema with compatible: ['mediatek,mt8173-mdp-wrot']
>        8 failed to match any schema with compatible:
> ['mediatek,mt8173-mdp-rdma', 'mediatek,mt8173-mdp']
>        6 failed to match any schema with compatible:
> ['mediatek,mt8183_da7219_rt1015p']
>        5 (mediatek,mt7986-eth): interrupts: [[0, 196, 4], [0, 197, 4],
> [0, 198, 4], [0, 199, 4]] is too short
>        5 failed to match any schema with compatible:
> ['mediatek,mt8183_mt6358_ts3a227_rt1015p']
>        4 (mediatek,mt8173-mmsys): 'assigned-clock-rates',
> 'assigned-clocks' do not match any of the regexes: '^pinctrl-[0-9]+$'
>        4 (mediatek,mt8173-disp-ufoe): 'mediatek,gce-client-reg' does
> not match any of the regexes: '^pinctrl-[0-9]+$'
>        4 (mediatek,mt8173-disp-od): 'mediatek,gce-client-reg' does not
> match any of the regexes: '^pinctrl-[0-9]+$'
>        4 (mediatek,mt6360): #interrupt-cells: 1 was expected
>        4 failed to match any schema with compatible: ['mediatek,mt8173-vpu']
>        4 failed to match any schema with compatible: ['mediatek,mt8173-mdp-wdma']
>        4 failed to match any schema with compatible: ['mediatek,mt8173-mdp-rdma']
>        4 failed to match any schema with compatible: ['mediatek,mt7622-pcie']
>        3 (mediatek,mt8192-audsys): 'mt8192-afe-pcm' does not match any
> of the regexes: '^pinctrl-[0-9]+$'
>        3 (mediatek,mt8173-thermal): Unevaluated properties are not
> allowed ('bank0-supply', 'bank1-supply' were unexpected)
>        3 (mediatek,mt8173-pinctrl): 'gpio-line-names' does not match
> any of the regexes: '^pinctrl-[0-9]+$', 'pins$'
>        3 (mediatek,mt8173-dsi): Unevaluated properties are not allowed
> ('ports' was unexpected)
>        3 (mediatek,mt8173-dsi): ports: 'port@1' is a required property
>        3 (mediatek,mt8173-dsi): ports: 'port@0' is a required property
>        3 failed to match any schema with compatible: ['mediatek,mt8173-rt5650']
>        2 (mediatek,mt8192_mt6359_rt1015p_rt5682): 'model' is a required property
>        2 (mediatek,mt8192-i2c): Unevaluated properties are not allowed
> ('clock-stretch-ns' was unexpected)
>        2 (mediatek,mt8186-spmi): Unevaluated properties are not allowed
> ('interrupts' was unexpected)
>        2 (mediatek,mt7986-tphy): usb-phy@700:reg: [[0, 1792], [0,
> 2304]] is too long
>        2 (mediatek,mt7622-pwrap): 'regulators' does not match any of
> the regexes: '^pinctrl-[0-9]+$'
>        2 (mediatek,mt7622-pciesys): compatible: 'oneOf' conditional
> failed, one must be fixed:
>        2 (mediatek,mt7622-audsys): audio-controller: 'power-domains' is
> a required property
>        2 (mediatek,mt7622-audsys): audio-controller:clock-names:
> ['infra_sys_audio_clk', 'top_audio_mux1_sel
> ', 'top_audio_mux2_sel', 'top_audio_a1sys_hp', 'top_audio_a2sys_hp',
> 'i2s0_src_sel', 'i2s1_src_sel', 'i2s2_
> src_sel', 'i2s3_src_sel', 'i2s0_src_div', 'i2s1_src_div',
> 'i2s2_src_div', 'i2s3_src_div', 'i2s0_mclk_en','
> i2s1_mclk_en', 'i2s2_mclk_en', 'i2s3_mclk_en', 'i2so0_hop_ck',
> 'i2so1_hop_ck', 'i2so2_hop_ck', 'i2so3_hop_c
> k', 'i2si0_hop_ck', 'i2si1_hop_ck', 'i2si2_hop_ck', 'i2si3_hop_ck',
> 'asrc0_out_ck', 'asrc1_out_ck', 'asrc2_
> out_ck', 'asrc3_out_ck', 'audio_afe_pd', 'audio_afe_conn_pd',
> 'audio_a1sys_pd', 'audio_a2sys_pd'] is too sh
> ort
>        2 (mediatek,mt7622-audio): 'power-domains' is a required property
>        2 (mediatek,mt7622-audio): clock-names: ['infra_sys_audio_clk',
> 'top_audio_mux1_sel', 'top_audio_mux2
> _sel', 'top_audio_a1sys_hp', 'top_audio_a2sys_hp', 'i2s0_src_sel',
> 'i2s1_src_sel', 'i2s2_src_sel', 'i2s3_sr
> c_sel', 'i2s0_src_div', 'i2s1_src_div', 'i2s2_src_div',
> 'i2s3_src_div', 'i2s0_mclk_en', 'i2s1_mclk_en', 'i2
> s2_mclk_en', 'i2s3_mclk_en', 'i2so0_hop_ck', 'i2so1_hop_ck',
> 'i2so2_hop_ck', 'i2so3_hop_ck', 'i2si0_hop_ck'
> , 'i2si1_hop_ck', 'i2si2_hop_ck', 'i2si3_hop_ck', 'asrc0_out_ck',
> 'asrc1_out_ck', 'asrc2_out_ck', 'asrc3_ou
> t_ck', 'audio_afe_pd', 'audio_afe_conn_pd', 'audio_a1sys_pd',
> 'audio_a2sys_pd'] is too short
>        2 (mediatek,mt6795-mmsys): 'assigned-clock-rates',
> 'assigned-clocks' do not match any of the regexes: '^pinctrl-[0-9]+$'
>        2 (mediatek,mt6795-mmc): Unevaluated properties are not allowed
> ('pinctrl-names' was unexpected)
>        2 (mediatek,mt6795-mmc): pinctrl-names: ['default'] is too short
>        2 (mediatek,mt6795-mmc): 'pinctrl-1' is a required property
>        2 (mediatek,mt6795-disp-ufoe): 'mediatek,gce-client-reg' does
> not match any of the regexes: '^pinctrl-[0-9]+$'
>        2 (mediatek,mt6795-disp-od): 'mediatek,gce-client-reg' does not
> match any of the regexes: '^pinctrl-[0-9]+$'
>        2 failed to match any schema with compatible:
> ['mediatek,mt8183_da7219_max98357']
>        2 failed to match any schema with compatible:
> ['mediatek,mt7622-scpsys', 'syscon']
>        2 failed to match any schema with compatible: ['mediatek,mt6797-scpsys']
>        2 failed to match any schema with compatible:
> ['mediatek,mt6380-regulator']
>        2 failed to match any schema with compatible: ['mediatek,mt2712-pcie']
>        1 (mediatek,mt8516-topckgen): compatible: 'oneOf' conditional
> failed, one must be fixed:
>        1 (mediatek,mt8516-apmixedsys): compatible: 'oneOf' conditional
> failed, one must be fixed:
>        1 (mediatek,mt8186-mt6366-rt5682s-max98360-sound): 'model' is a
> required property
>        1 (mediatek,mt8186-cci): 'proc-supply' is a required property
>        1 (mediatek,mt8183-pinctrl): 'i2c0', 'i2c1', 'i2c2', 'i2c3',
> 'i2c4', 'i2c5', 'mmc0', 'mmc0default', 'mmc1', 'mmc1default', 'pwm1',
> 'spi0', 'spi1', 'spi2', 'spi3', 'spi4', 'spi5' do not match any of the
> regexes: '-pins(-[a-z]+)?$', '^pinctrl-[0-9]+$'
>        1 (mediatek,mt8183-pinctrl): 'i2c0', 'i2c1', 'i2c2', 'i2c3',
> 'i2c4', 'i2c5', 'i2c6', 'keyboard' do not match any of the regexes:
> '-pins(-[a-z]+)?$', '^pinctrl-[0-9]+$'
>        1 (mediatek,mt8183-pinctrl): bt-pins-wakeup: 'piins-bt-wakeup'
> does not match any of the regexes: '^pinctrl-[0-9]+$', '^pins'
>        1 (mediatek,mt8173-pwrap): 'power-domains' does not match any of
> the regexes: '^pinctrl-[0-9]+$'
>        1 (mediatek,mt7622-pinctrl): 'asm-sel-hog' does not match any of
> the regexes: '-pins(-[a-z]+)?$', '^pinctrl-[0-9]+$'
>        1 (mediatek,mt7622-audsys): audio-controller:clocks: [[2, 2],
> [18, 80], [18, 81], [18, 107], [18, 108], [18, 89], [18, 90], [18,
> 91], [18, 92], [18, 95], [18, 96], [18, 97], [18, 98], [18, 103], [18,
> 104], [18, 105], [18, 106], [38, 8], [38, 9], [38, 10], [38, 11], [38,
> 4], [38, 5], [38, 6], [38, 7], [38, 14], [38, 15], [38, 39], [38, 40],
> [38, 0], [38, 46], [38, 17], [38, 18]] is too short
>        1 (mediatek,mt7622-audsys): audio-controller:clocks: [[2, 2],
> [18, 80], [18, 81], [18, 107], [18, 108], [18, 89], [18, 90], [18,
> 91], [18, 92], [18, 95], [18, 96], [18, 97], [18, 98], [18, 103], [18,
> 104], [18, 105], [18, 106], [37, 8], [37, 9], [37, 10], [37, 11], [37,
> 4], [37, 5], [37, 6], [37, 7], [37, 14], [37, 15], [37, 39], [37, 40],
> [37, 0], [37, 46], [37, 17], [37, 18]] is too short
>        1 (mediatek,mt7622-audio): clocks: [[2, 2], [18, 80], [18, 81],
> [18, 107], [18, 108], [18, 89], [18, 90], [18, 91], [18, 92], [18,
> 95], [18, 96], [18, 97], [18, 98], [18, 103], [18, 104], [18, 105],
> [18, 106], [38, 8], [38, 9], [38, 10], [38, 11], [38, 4], [38, 5],
> [38, 6], [38, 7], [38, 14], [38, 15], [38, 39], [38, 40], [38, 0],
> [38, 46], [38, 17], [38, 18]] is too short
>        1 (mediatek,mt7622-audio): clocks: [[2, 2], [18, 80], [18, 81],
> [18, 107], [18, 108], [18, 89], [18, 90], [18, 91], [18, 92], [18,
> 95], [18, 96], [18, 97], [18, 98], [18, 103], [18, 104], [18, 105],
> [18, 106], [37, 8], [37, 9], [37, 10], [37, 11], [37, 4], [37, 5],
> [37, 6], [37, 7], [37, 14], [37, 15], [37, 39], [37, 40], [37, 0],
> [37, 46], [37, 17], [37, 18]] is too short
>        1 (mediatek,mt7531): 'interrupts' is a dependency of
> 'interrupt-controller'
>        1 (mediatek,mt6357): 'adc' does not match any of the regexes:
> '^pinctrl-[0-9]+$'
>        1 (mediatek,mt6331): regulators:compatible: 'oneOf' conditional
> failed, one must be fixed:
>        1 (mediatek,mt6331-regulator): 'ldo-vio28' does not match any of
> the regexes: '^buck-v(core2|io18|dvfs11|dvfs12|dvfs13|dvfs14)$',
> '^ldo-(avdd32aud|vauxa32)$',
> '^ldo-v(dig18|emc33|ibr|mc|mch|mipi|rtc|sim1|sim2|sram|usb10)$',
> '^ldo-vcam(a|af|d|io)$', '^ldo-vgp[1234]$', '^ldo-vtcxo[12]$',
> '^pinctrl-[0-9]+$'
>        1 (mediatek,mt6331-regulator): ldo-vcamio:regulator-name:0:
> 'vcam_io' does not match '^vcam(a|_af|d|io)$'
>        1 (mediatek,mt6331): 'mt6332-led' does not match any of the
> regexes: '^pinctrl-[0-9]+$'
>        1 failed to match any schema with compatible:
> ['mediatek,mt6779-audio', 'syscon']
>        1 failed to match any schema with compatible:
> ['mediatek,mt2712-scpsys', 'syscon']

Powered by blists - more mailing lists