Messages in this thread | | | Date | Thu, 2 Feb 2023 12:52:25 +0100 | Subject | Re: [PATCH] arm64: dts: mediatek: mt8195: Fix vdosys* compatible strings | From | Matthias Brugger <> |
| |
On 02/02/2023 12:42, AngeloGioacchino Del Regno wrote: > Il 02/02/23 11:40, Chen-Yu Tsai ha scritto: >> When vdosys1 was initially added, it was incorrectly assumed to be >> compatible with vdosys0, and thus both had the same mt8195-mmsys >> compatible attached. >> >> This has since been corrected in commit b237efd47df7 ("dt-bindings: >> arm: mediatek: mmsys: change compatible for MT8195") and commit >> 82219cfbef18 ("dt-bindings: arm: mediatek: mmsys: add vdosys1 compatible >> for MT8195"). The device tree needs to be fixed as well, otherwise >> the vdosys1 block fails to work, and causes its dependent power domain >> controller to not work either. >> >> Change the compatible string of vdosys1 to "mediatek,mt8195-vdosys1". >> While at it, also add the new "mediatek,mt8195-vdosys0" compatible to >> vdosys0. >> >> Fixes: 6aa5b46d1755 ("arm64: dts: mt8195: Add vdosys and vppsys clock nodes") >> Signed-off-by: Chen-Yu Tsai <wenst@chromium.org> > > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > Tested-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> >
Acked-by: Matthias Brugger <matthias.bgg@gmail.com>
Arnd, Olof, will you take it directly or shall I provide a pull request once -rc1 is out?
Regards, Matthias
>> --- >> >> Since we are at -rc6 and Matthias already sent out pull requests, I've >> CC-ed soc@ so that this may be picked up directly on top of them. This >> should be merged for -next. >> >> Thanks >> >
| |