On 10/06/2022 12:43, AngeloGioacchino Del Regno wrote: > Il 07/06/22 16:05, Johnson Wang ha scritto: >> Add devicetree binding of MediaTek CCI on MT8183 and MT8186. >> >> Signed-off-by: Jia-Wei Chang <jia-wei.chang@xxxxxxxxxxxx> >> Signed-off-by: Johnson Wang <johnson.wang@xxxxxxxxxxxx> >> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> >> Acked-by: Chanwoo Choi <cw00.choi@xxxxxxxxxxx> >> Reviewed-by: Chen-Yu Tsai <wenst@xxxxxxxxxxxx> >> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx> >> Acked-by: Georgi Djakov <djakov@xxxxxxxxxx> >> --- >> .../bindings/interconnect/mediatek,cci.yaml | 141 ++++++++++++++++++ > > Uhm, I've just noticed that this is being put in the bindings/interconnect folder. > The mediatek,cci is *not* an interconnect driver, but a devfreq one and is not > using any ICC APIs. > > Shouldn't this go to bindings/devfreq/ instead?! No, devfreq is not a hardware subsystem. Best regards, Krzysztof