From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> To: Chen-Yu Tsai <wenst@chromium.org>, Matthias Brugger <matthias.bgg@gmail.com> Cc: "Rob Herring" <robh+dt@kernel.org>, "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, "Nícolas F . R . A . Prado" <nfraprado@collabora.com>, soc@kernel.org Subject: Re: [PATCH] arm64: dts: mediatek: mt8195: Fix vdosys* compatible strings Date: Thu, 2 Feb 2023 12:42:36 +0100 [thread overview] Message-ID: <4b5bb022-eb5d-2ed1-2123-eb9e502299dc@collabora.com> (raw) In-Reply-To: <20230202104014.2931517-1-wenst@chromium.org> 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> > --- > > 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 >
WARNING: multiple messages have this Message-ID (diff)
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> To: Chen-Yu Tsai <wenst@chromium.org>, Matthias Brugger <matthias.bgg@gmail.com> Cc: "Rob Herring" <robh+dt@kernel.org>, "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org, "Nícolas F . R . A . Prado" <nfraprado@collabora.com>, soc@kernel.org Subject: Re: [PATCH] arm64: dts: mediatek: mt8195: Fix vdosys* compatible strings Date: Thu, 2 Feb 2023 12:42:36 +0100 [thread overview] Message-ID: <4b5bb022-eb5d-2ed1-2123-eb9e502299dc@collabora.com> (raw) In-Reply-To: <20230202104014.2931517-1-wenst@chromium.org> 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> > --- > > 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 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-02-02 11:42 UTC|newest] Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-02-02 10:40 [PATCH] arm64: dts: mediatek: mt8195: Fix vdosys* compatible strings Chen-Yu Tsai 2023-02-02 10:40 ` Chen-Yu Tsai 2023-02-02 11:42 ` AngeloGioacchino Del Regno [this message] 2023-02-02 11:42 ` AngeloGioacchino Del Regno 2023-02-02 11:52 ` Matthias Brugger 2023-02-02 11:52 ` Matthias Brugger 2023-02-03 14:20 ` patchwork-bot+linux-soc
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=4b5bb022-eb5d-2ed1-2123-eb9e502299dc@collabora.com \ --to=angelogioacchino.delregno@collabora.com \ --cc=devicetree@vger.kernel.org \ --cc=krzysztof.kozlowski+dt@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mediatek@lists.infradead.org \ --cc=matthias.bgg@gmail.com \ --cc=nfraprado@collabora.com \ --cc=robh+dt@kernel.org \ --cc=soc@kernel.org \ --cc=wenst@chromium.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.