* [PATCH v4 0/4] mediatek: watchdog: Fix compatible fallbacks
@ 2022-07-21 1:48 Allen-KH Cheng
2022-07-21 1:48 ` [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example Allen-KH Cheng
` (3 more replies)
0 siblings, 4 replies; 15+ messages in thread
From: Allen-KH Cheng @ 2022-07-21 1:48 UTC (permalink / raw)
To: Wim Van Sebroeck, Guenter Roeck, Krzysztof Kozlowski,
Matthias Brugger, Rob Herring, nfraprado
Cc: angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group,
devicetree, linux-arm-kernel, linux-kernel, linux-mediatek,
Allen-KH Cheng
Based on tag: next-20220720, linux-next/master
Beacase there are some PATCHs of mediatek wdt waiting to merge,
I drop "Convert binding to YAML" PATCH to not cause dtbs errors in
series v2.
We will send "watchdog: Convert binding to YAML" PATCH in the future.
changes since v3:
- fix commit title (from dt-binding to dt-bindings)
- Add RoB tags
changes since v2:
- remove "Convert binding to YAML" PATCH from series
- fix example
- change cover letter title
changes since v1:
- fix dtbs_check warnings in dts
- use enum instead of multiple const values
- remove timeout-sec
- rewrite examples
- rename schema to mediatek,wdt.yaml
Allen-KH Cheng (4):
dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example
arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt7986
arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt8195
arm64: dts: mediatek: Fix the watchdog node name
Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 9 ++++-----
arch/arm64/boot/dts/mediatek/mt7986a.dtsi | 3 +--
arch/arm64/boot/dts/mediatek/mt8195.dtsi | 3 +--
arch/arm64/boot/dts/mediatek/mt8516.dtsi | 2 +-
4 files changed, 7 insertions(+), 10 deletions(-)
--
2.18.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 15+ messages in thread* [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-07-21 1:48 [PATCH v4 0/4] mediatek: watchdog: Fix compatible fallbacks Allen-KH Cheng @ 2022-07-21 1:48 ` Allen-KH Cheng 2022-07-25 22:33 ` Rob Herring ` (2 more replies) 2022-07-21 1:48 ` [PATCH v4 2/4] arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt7986 Allen-KH Cheng ` (2 subsequent siblings) 3 siblings, 3 replies; 15+ messages in thread From: Allen-KH Cheng @ 2022-07-21 1:48 UTC (permalink / raw) To: Wim Van Sebroeck, Guenter Roeck, Krzysztof Kozlowski, Matthias Brugger, Rob Herring, nfraprado Cc: angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek, Allen-KH Cheng The watchdog timer of mt8186. mt8195 and mt7986 have their DT data. We should not use 'mediatek,mt6589-wdt' as fallback. For mediatek,wdt example of mt8183, We remove mediatek,mt6589-wdt fallback. Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for MediaTek MT8186") Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for Mediatek MT8195") Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for Mediatek MT7986") Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add #reset-cells") Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> --- Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt index 762c62e428ef..67ef991ec4cf 100644 --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt @@ -14,12 +14,12 @@ Required properties: "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for MT7622 "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for MT7623 "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629 - "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986 + "mediatek,mt7986-wdt": for MT7986 "mediatek,mt8183-wdt": for MT8183 - "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186 + "mediatek,mt8186-wdt": for MT8186 "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516 "mediatek,mt8192-wdt": for MT8192 - "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195 + "mediatek,mt8195-wdt": for MT8195 - reg : Specifies base physical address and size of the registers. @@ -32,8 +32,7 @@ Optional properties: Example: watchdog: watchdog@10007000 { - compatible = "mediatek,mt8183-wdt", - "mediatek,mt6589-wdt"; + compatible = "mediatek,mt8183-wdt"; mediatek,disable-extrst; reg = <0 0x10007000 0 0x100>; interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; -- 2.18.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-07-21 1:48 ` [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example Allen-KH Cheng @ 2022-07-25 22:33 ` Rob Herring 2022-08-01 9:29 ` Chen-Yu Tsai 2022-08-10 16:49 ` Krzysztof Kozlowski 2 siblings, 0 replies; 15+ messages in thread From: Rob Herring @ 2022-07-25 22:33 UTC (permalink / raw) To: Allen-KH Cheng Cc: Project_Global_Chrome_Upstream_Group, Wim Van Sebroeck, Guenter Roeck, Krzysztof Kozlowski, nfraprado, angelogioacchino.delregno, linux-arm-kernel, devicetree, linux-mediatek, Rob Herring, Matthias Brugger, linux-kernel On Thu, 21 Jul 2022 09:48:42 +0800, Allen-KH Cheng wrote: > The watchdog timer of mt8186. mt8195 and mt7986 have their DT data. > We should not use 'mediatek,mt6589-wdt' as fallback. > > For mediatek,wdt example of mt8183, We remove mediatek,mt6589-wdt fallback. > > Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for MediaTek MT8186") > Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for Mediatek MT8195") > Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for Mediatek MT7986") > Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add #reset-cells") > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > --- > Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > Acked-by: Rob Herring <robh@kernel.org> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-07-21 1:48 ` [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example Allen-KH Cheng 2022-07-25 22:33 ` Rob Herring @ 2022-08-01 9:29 ` Chen-Yu Tsai 2022-08-02 11:04 ` Matthias Brugger 2022-08-10 16:49 ` Krzysztof Kozlowski 2 siblings, 1 reply; 15+ messages in thread From: Chen-Yu Tsai @ 2022-08-01 9:29 UTC (permalink / raw) To: Allen-KH Cheng, Wim Van Sebroeck, Guenter Roeck, Rob Herring Cc: Krzysztof Kozlowski, Matthias Brugger, nfraprado, angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek On Thu, Jul 21, 2022 at 10:50 AM Allen-KH Cheng <allen-kh.cheng@mediatek.com> wrote: > > The watchdog timer of mt8186. mt8195 and mt7986 have their DT data. > We should not use 'mediatek,mt6589-wdt' as fallback. > > For mediatek,wdt example of mt8183, We remove mediatek,mt6589-wdt fallback. I think this needs some more information. Right now on the kernel side, mt6589-wdt provides just watchdog support. The SoC-specific compatibles that are touched by this patch provide reset controls in addition to the standard watchdog, which remains the same. If that is the case, then the fallback compatibles are correct. A fallback says that the new hardware is compatible with some older hardware, and can be run with the driver supporting that older hardware, likely with reduced functionality. As an example, if mt8195-wdt is backward compatible with mt6589-wdt, then it should run as mt6589-wdt, and would just be missing new functionality, in this case the reset controls. So either mt6589-wdt also contains a reset control that is not the same as the other newer chips, or has some other functionality that the other chips contain, and justifies the removal of the fallback, or this patch is incorrect. Note that mt2701-wdt and mt762*-wdt are still listed as compatible with mt6589-wdt. So I think a better explanation is required. Regards ChenYu > Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for MediaTek MT8186") > Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for Mediatek MT8195") > Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for Mediatek MT7986") > Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add #reset-cells") > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > --- > Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > index 762c62e428ef..67ef991ec4cf 100644 > --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > @@ -14,12 +14,12 @@ Required properties: > "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for MT7622 > "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for MT7623 > "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629 > - "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986 > + "mediatek,mt7986-wdt": for MT7986 > "mediatek,mt8183-wdt": for MT8183 > - "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186 > + "mediatek,mt8186-wdt": for MT8186 > "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516 > "mediatek,mt8192-wdt": for MT8192 > - "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195 > + "mediatek,mt8195-wdt": for MT8195 > > - reg : Specifies base physical address and size of the registers. > > @@ -32,8 +32,7 @@ Optional properties: > Example: > > watchdog: watchdog@10007000 { > - compatible = "mediatek,mt8183-wdt", > - "mediatek,mt6589-wdt"; > + compatible = "mediatek,mt8183-wdt"; > mediatek,disable-extrst; > reg = <0 0x10007000 0 0x100>; > interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; > -- > 2.18.0 > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-08-01 9:29 ` Chen-Yu Tsai @ 2022-08-02 11:04 ` Matthias Brugger 2022-08-03 9:23 ` Allen-KH Cheng 0 siblings, 1 reply; 15+ messages in thread From: Matthias Brugger @ 2022-08-02 11:04 UTC (permalink / raw) To: Chen-Yu Tsai, Allen-KH Cheng, Wim Van Sebroeck, Guenter Roeck, Rob Herring Cc: Krzysztof Kozlowski, nfraprado, angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek On 01/08/2022 11:29, Chen-Yu Tsai wrote: > On Thu, Jul 21, 2022 at 10:50 AM Allen-KH Cheng > <allen-kh.cheng@mediatek.com> wrote: >> >> The watchdog timer of mt8186. mt8195 and mt7986 have their DT data. >> We should not use 'mediatek,mt6589-wdt' as fallback. >> >> For mediatek,wdt example of mt8183, We remove mediatek,mt6589-wdt fallback. > > I think this needs some more information. > > Right now on the kernel side, mt6589-wdt provides just watchdog support. > The SoC-specific compatibles that are touched by this patch provide reset > controls in addition to the standard watchdog, which remains the same. > > If that is the case, then the fallback compatibles are correct. A fallback > says that the new hardware is compatible with some older hardware, and > can be run with the driver supporting that older hardware, likely with > reduced functionality. > My understanding is, that we add a fallback because although at the time we entered the compatible, the functionality of the device is the same as the fallback. Nonetheless we add a compatible specific for the device in case in the future we realize that the device has some functionality that is not and can not be covered by the fallback. This is the case here. Actually adding the fallback in the first place was wrong, because the driver since ever supports the extra function for the device, the reset. So this is a mere cleanup of the binding to reflect what was always present in the driver. Regards, Matthias > As an example, if mt8195-wdt is backward compatible with mt6589-wdt, > then it should run as mt6589-wdt, and would just be missing new > functionality, in this case the reset controls. > > So either mt6589-wdt also contains a reset control that is not the same > as the other newer chips, or has some other functionality that the other > chips contain, and justifies the removal of the fallback, or this patch > is incorrect. Note that mt2701-wdt and mt762*-wdt are still listed as > compatible with mt6589-wdt. So I think a better explanation is required. > > > Regards > ChenYu > > >> Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for MediaTek MT8186") >> Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for Mediatek MT8195") >> Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for Mediatek MT7986") >> Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add #reset-cells") >> Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> >> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> >> --- >> Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 9 ++++----- >> 1 file changed, 4 insertions(+), 5 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt >> index 762c62e428ef..67ef991ec4cf 100644 >> --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt >> +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt >> @@ -14,12 +14,12 @@ Required properties: >> "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for MT7622 >> "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for MT7623 >> "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629 >> - "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986 >> + "mediatek,mt7986-wdt": for MT7986 >> "mediatek,mt8183-wdt": for MT8183 >> - "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186 >> + "mediatek,mt8186-wdt": for MT8186 >> "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516 >> "mediatek,mt8192-wdt": for MT8192 >> - "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195 >> + "mediatek,mt8195-wdt": for MT8195 >> >> - reg : Specifies base physical address and size of the registers. >> >> @@ -32,8 +32,7 @@ Optional properties: >> Example: >> >> watchdog: watchdog@10007000 { >> - compatible = "mediatek,mt8183-wdt", >> - "mediatek,mt6589-wdt"; >> + compatible = "mediatek,mt8183-wdt"; >> mediatek,disable-extrst; >> reg = <0 0x10007000 0 0x100>; >> interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; >> -- >> 2.18.0 >> >> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-08-02 11:04 ` Matthias Brugger @ 2022-08-03 9:23 ` Allen-KH Cheng 2022-08-08 17:03 ` Nícolas F. R. A. Prado 0 siblings, 1 reply; 15+ messages in thread From: Allen-KH Cheng @ 2022-08-03 9:23 UTC (permalink / raw) To: Matthias Brugger, Chen-Yu Tsai, Wim Van Sebroeck, Guenter Roeck, Rob Herring Cc: Krzysztof Kozlowski, nfraprado, angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek Hi Chen-Yu and Mathias, Sincere apologies for the delayed response. On Tue, 2022-08-02 at 13:04 +0200, Matthias Brugger wrote: > > On 01/08/2022 11:29, Chen-Yu Tsai wrote: > > On Thu, Jul 21, 2022 at 10:50 AM Allen-KH Cheng > > <allen-kh.cheng@mediatek.com> wrote: > > > > > > The watchdog timer of mt8186. mt8195 and mt7986 have their DT > > > data. > > > We should not use 'mediatek,mt6589-wdt' as fallback. > > > > > > For mediatek,wdt example of mt8183, We remove mediatek,mt6589-wdt > > > fallback. > > > > I think this needs some more information. > > > > Right now on the kernel side, mt6589-wdt provides just watchdog > > support. > > The SoC-specific compatibles that are touched by this patch provide > > reset > > controls in addition to the standard watchdog, which remains the > > same. > > > > If that is the case, then the fallback compatibles are correct. A > > fallback > > says that the new hardware is compatible with some older hardware, > > and > > can be run with the driver supporting that older hardware, likely > > with > > reduced functionality. > > > > My understanding is, that we add a fallback because although at the > time we > entered the compatible, the functionality of the device is the same > as the > fallback. Nonetheless we add a compatible specific for the device in > case in the > future we realize that the device has some functionality that is not > and can not > be covered by the fallback. > > This is the case here. Actually adding the fallback in the first > place was > wrong, because the driver since ever supports the extra function for > the device, > the reset. > > So this is a mere cleanup of the binding to reflect what was always > present in > the driver. > > Regards, > Matthias > mt6589-wdt dosen't contains a reset control for other modules, like chen-yu mention "mt6589-wdt provides just watchdog support." For instance, there is a reset control in mt8195-wdt and we have a DT data to define its reset number of TOPRGU. I thought it's better not use mt6589-wdt as fallback. Please let me know if this works and if you have any suggestions or comments. Thanks, Allen > > As an example, if mt8195-wdt is backward compatible with mt6589- > > wdt, > > then it should run as mt6589-wdt, and would just be missing new > > functionality, in this case the reset controls. > > > > So either mt6589-wdt also contains a reset control that is not the > > same > > as the other newer chips, or has some other functionality that the > > other > > chips contain, and justifies the removal of the fallback, or this > > patch > > is incorrect. Note that mt2701-wdt and mt762*-wdt are still listed > > as > > compatible with mt6589-wdt. So I think a better explanation is > > required. > > > > > > Regards > > ChenYu > > > > > > > Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for > > > MediaTek MT8186") > > > Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for > > > Mediatek MT8195") > > > Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for > > > Mediatek MT7986") > > > Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add #reset- > > > cells") > > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > > Reviewed-by: AngeloGioacchino Del Regno < > > > angelogioacchino.delregno@collabora.com> > > > --- > > > Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 9 > > > ++++----- > > > 1 file changed, 4 insertions(+), 5 deletions(-) > > > > > > diff --git a/Documentation/devicetree/bindings/watchdog/mtk- > > > wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > > index 762c62e428ef..67ef991ec4cf 100644 > > > --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > > +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > > @@ -14,12 +14,12 @@ Required properties: > > > "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for MT7622 > > > "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for MT7623 > > > "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629 > > > - "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986 > > > + "mediatek,mt7986-wdt": for MT7986 > > > "mediatek,mt8183-wdt": for MT8183 > > > - "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186 > > > + "mediatek,mt8186-wdt": for MT8186 > > > "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516 > > > "mediatek,mt8192-wdt": for MT8192 > > > - "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195 > > > + "mediatek,mt8195-wdt": for MT8195 > > > > > > - reg : Specifies base physical address and size of the > > > registers. > > > > > > @@ -32,8 +32,7 @@ Optional properties: > > > Example: > > > > > > watchdog: watchdog@10007000 { > > > - compatible = "mediatek,mt8183-wdt", > > > - "mediatek,mt6589-wdt"; > > > + compatible = "mediatek,mt8183-wdt"; > > > mediatek,disable-extrst; > > > reg = <0 0x10007000 0 0x100>; > > > interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; > > > -- > > > 2.18.0 > > > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-08-03 9:23 ` Allen-KH Cheng @ 2022-08-08 17:03 ` Nícolas F. R. A. Prado 2022-08-10 12:58 ` Allen-KH Cheng 0 siblings, 1 reply; 15+ messages in thread From: Nícolas F. R. A. Prado @ 2022-08-08 17:03 UTC (permalink / raw) To: Allen-KH Cheng Cc: Matthias Brugger, Chen-Yu Tsai, Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski, angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek Hi, On Wed, Aug 03, 2022 at 05:23:57PM +0800, Allen-KH Cheng wrote: > Hi Chen-Yu and Mathias, > > Sincere apologies for the delayed response. > > On Tue, 2022-08-02 at 13:04 +0200, Matthias Brugger wrote: > > > > On 01/08/2022 11:29, Chen-Yu Tsai wrote: > > > On Thu, Jul 21, 2022 at 10:50 AM Allen-KH Cheng > > > <allen-kh.cheng@mediatek.com> wrote: > > > > > > > > The watchdog timer of mt8186. mt8195 and mt7986 have their DT > > > > data. > > > > We should not use 'mediatek,mt6589-wdt' as fallback. > > > > > > > > For mediatek,wdt example of mt8183, We remove mediatek,mt6589-wdt > > > > fallback. > > > > > > I think this needs some more information. > > > > > > Right now on the kernel side, mt6589-wdt provides just watchdog > > > support. > > > The SoC-specific compatibles that are touched by this patch provide > > > reset > > > controls in addition to the standard watchdog, which remains the > > > same. > > > > > > If that is the case, then the fallback compatibles are correct. A > > > fallback > > > says that the new hardware is compatible with some older hardware, > > > and > > > can be run with the driver supporting that older hardware, likely > > > with > > > reduced functionality. > > > > > > > My understanding is, that we add a fallback because although at the > > time we > > entered the compatible, the functionality of the device is the same > > as the > > fallback. Nonetheless we add a compatible specific for the device in > > case in the > > future we realize that the device has some functionality that is not > > and can not > > be covered by the fallback. > > > > This is the case here. Actually adding the fallback in the first > > place was > > wrong, because the driver since ever supports the extra function for > > the device, > > the reset. > > > > So this is a mere cleanup of the binding to reflect what was always > > present in > > the driver. > > > > Regards, > > Matthias > > > > mt6589-wdt dosen't contains a reset control for other modules, like > chen-yu mention "mt6589-wdt provides just watchdog support." > > For instance, there is a reset control in mt8195-wdt and we have a DT > data to define its reset number of TOPRGU. I thought it's better not > use mt6589-wdt as fallback. > > Please let me know if this works and if you have any suggestions or > comments. The only practical usecase that I can think of that relies on keeping the fallback compatibles is using the current DT with an older kernel that didn't yet support the specific watchdog compatibles. In this case, dropping the fallback compatibles would make the watchdog not work at all in such a kernel. I'm not sure how relevant/common of a usecase that would be, but maybe it's worth considering given that the advantage of removing the fallback compatible is purely aesthetic? Thanks, Nícolas > > Thanks, > Allen > > > > As an example, if mt8195-wdt is backward compatible with mt6589- > > > wdt, > > > then it should run as mt6589-wdt, and would just be missing new > > > functionality, in this case the reset controls. > > > > > > So either mt6589-wdt also contains a reset control that is not the > > > same > > > as the other newer chips, or has some other functionality that the > > > other > > > chips contain, and justifies the removal of the fallback, or this > > > patch > > > is incorrect. Note that mt2701-wdt and mt762*-wdt are still listed > > > as > > > compatible with mt6589-wdt. So I think a better explanation is > > > required. > > > > > > > > > Regards > > > ChenYu > > > > > > > > > > Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for > > > > MediaTek MT8186") > > > > Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for > > > > Mediatek MT8195") > > > > Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for > > > > Mediatek MT7986") > > > > Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add #reset- > > > > cells") > > > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > > > Reviewed-by: AngeloGioacchino Del Regno < > > > > angelogioacchino.delregno@collabora.com> > > > > --- > > > > Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 9 > > > > ++++----- > > > > 1 file changed, 4 insertions(+), 5 deletions(-) > > > > > > > > diff --git a/Documentation/devicetree/bindings/watchdog/mtk- > > > > wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > > > index 762c62e428ef..67ef991ec4cf 100644 > > > > --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > > > +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > > > @@ -14,12 +14,12 @@ Required properties: > > > > "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for MT7622 > > > > "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for MT7623 > > > > "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629 > > > > - "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for MT7986 > > > > + "mediatek,mt7986-wdt": for MT7986 > > > > "mediatek,mt8183-wdt": for MT8183 > > > > - "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for MT8186 > > > > + "mediatek,mt8186-wdt": for MT8186 > > > > "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516 > > > > "mediatek,mt8192-wdt": for MT8192 > > > > - "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for MT8195 > > > > + "mediatek,mt8195-wdt": for MT8195 > > > > > > > > - reg : Specifies base physical address and size of the > > > > registers. > > > > > > > > @@ -32,8 +32,7 @@ Optional properties: > > > > Example: > > > > > > > > watchdog: watchdog@10007000 { > > > > - compatible = "mediatek,mt8183-wdt", > > > > - "mediatek,mt6589-wdt"; > > > > + compatible = "mediatek,mt8183-wdt"; > > > > mediatek,disable-extrst; > > > > reg = <0 0x10007000 0 0x100>; > > > > interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; > > > > -- > > > > 2.18.0 > > > > > > > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-08-08 17:03 ` Nícolas F. R. A. Prado @ 2022-08-10 12:58 ` Allen-KH Cheng 2022-08-10 16:50 ` Krzysztof Kozlowski 0 siblings, 1 reply; 15+ messages in thread From: Allen-KH Cheng @ 2022-08-10 12:58 UTC (permalink / raw) To: Nícolas F. R. A. Prado Cc: Matthias Brugger, Chen-Yu Tsai, Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski, angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek Hi Nícolas, On Mon, 2022-08-08 at 13:03 -0400, Nícolas F. R. A. Prado wrote: > Hi, > > On Wed, Aug 03, 2022 at 05:23:57PM +0800, Allen-KH Cheng wrote: > > Hi Chen-Yu and Mathias, > > > > Sincere apologies for the delayed response. > > > > On Tue, 2022-08-02 at 13:04 +0200, Matthias Brugger wrote: > > > > > > On 01/08/2022 11:29, Chen-Yu Tsai wrote: > > > > On Thu, Jul 21, 2022 at 10:50 AM Allen-KH Cheng > > > > <allen-kh.cheng@mediatek.com> wrote: > > > > > > > > > > The watchdog timer of mt8186. mt8195 and mt7986 have their DT > > > > > data. > > > > > We should not use 'mediatek,mt6589-wdt' as fallback. > > > > > > > > > > For mediatek,wdt example of mt8183, We remove > > > > > mediatek,mt6589-wdt > > > > > fallback. > > > > > > > > I think this needs some more information. > > > > > > > > Right now on the kernel side, mt6589-wdt provides just watchdog > > > > support. > > > > The SoC-specific compatibles that are touched by this patch > > > > provide > > > > reset > > > > controls in addition to the standard watchdog, which remains > > > > the > > > > same. > > > > > > > > If that is the case, then the fallback compatibles are correct. > > > > A > > > > fallback > > > > says that the new hardware is compatible with some older > > > > hardware, > > > > and > > > > can be run with the driver supporting that older hardware, > > > > likely > > > > with > > > > reduced functionality. > > > > > > > > > > My understanding is, that we add a fallback because although at > > > the > > > time we > > > entered the compatible, the functionality of the device is the > > > same > > > as the > > > fallback. Nonetheless we add a compatible specific for the device > > > in > > > case in the > > > future we realize that the device has some functionality that is > > > not > > > and can not > > > be covered by the fallback. > > > > > > This is the case here. Actually adding the fallback in the first > > > place was > > > wrong, because the driver since ever supports the extra function > > > for > > > the device, > > > the reset. > > > > > > So this is a mere cleanup of the binding to reflect what was > > > always > > > present in > > > the driver. > > > > > > Regards, > > > Matthias > > > > > > > mt6589-wdt dosen't contains a reset control for other modules, like > > chen-yu mention "mt6589-wdt provides just watchdog support." > > > > For instance, there is a reset control in mt8195-wdt and we have a > > DT > > data to define its reset number of TOPRGU. I thought it's better > > not > > use mt6589-wdt as fallback. > > > > Please let me know if this works and if you have any suggestions or > > comments. > > The only practical usecase that I can think of that relies on keeping > the > fallback compatibles is using the current DT with an older kernel > that didn't > yet support the specific watchdog compatibles. In this case, dropping > the > fallback compatibles would make the watchdog not work at all in such > a kernel. > > I'm not sure how relevant/common of a usecase that would be, but > maybe it's > worth considering given that the advantage of removing the fallback > compatible > is purely aesthetic? > > Thanks, > Nícolas > Thanks for your comments. I agree the advantage of patch is aesthetic. Since I also want to send another "watchdog: Convert binding to YAML" PATCH, it's better let all wdt compatibles in the binding match the contents of mtk_wdt_dt_ids in drivers/watchdog/mtk_wdt.c static const struct of_device_id mtk_wdt_dt_ids[] = { { .compatible = "mediatek,mt2712-wdt", .data = &mt2712_data }, { .compatible = "mediatek,mt6589-wdt" }, { .compatible = "mediatek,mt7986-wdt", .data = &mt7986_data }, { .compatible = "mediatek,mt8183-wdt", .data = &mt8183_data }, { .compatible = "mediatek,mt8186-wdt", .data = &mt8186_data }, { .compatible = "mediatek,mt8192-wdt", .data = &mt8192_data }, { .compatible = "mediatek,mt8195-wdt", .data = &mt8195_data }, { /* sentinel */ } }; We have "mediatek,mt8186-wdt" "mediatek,mt8195-wdt" and "mediatek,mt7986-wdt" now and they have their DT data for the reset control. It's weird and unuseful to add "mediatek,mt6589-wdt" as fallback. Please kindly let me know if I missed anything BRs, Allen > > > > Thanks, > > Allen > > > > > > As an example, if mt8195-wdt is backward compatible with > > > > mt6589- > > > > wdt, > > > > then it should run as mt6589-wdt, and would just be missing new > > > > functionality, in this case the reset controls. > > > > > > > > So either mt6589-wdt also contains a reset control that is not > > > > the > > > > same > > > > as the other newer chips, or has some other functionality that > > > > the > > > > other > > > > chips contain, and justifies the removal of the fallback, or > > > > this > > > > patch > > > > is incorrect. Note that mt2701-wdt and mt762*-wdt are still > > > > listed > > > > as > > > > compatible with mt6589-wdt. So I think a better explanation is > > > > required. > > > > > > > > > > > > Regards > > > > ChenYu > > > > > > > > > > > > > Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for > > > > > MediaTek MT8186") > > > > > Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for > > > > > Mediatek MT8195") > > > > > Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for > > > > > Mediatek MT7986") > > > > > Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add > > > > > #reset- > > > > > cells") > > > > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> > > > > > Reviewed-by: AngeloGioacchino Del Regno < > > > > > angelogioacchino.delregno@collabora.com> > > > > > --- > > > > > Documentation/devicetree/bindings/watchdog/mtk-wdt.txt | 9 > > > > > ++++----- > > > > > 1 file changed, 4 insertions(+), 5 deletions(-) > > > > > > > > > > diff --git a/Documentation/devicetree/bindings/watchdog/mtk- > > > > > wdt.txt b/Documentation/devicetree/bindings/watchdog/mtk- > > > > > wdt.txt > > > > > index 762c62e428ef..67ef991ec4cf 100644 > > > > > --- a/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > > > > +++ b/Documentation/devicetree/bindings/watchdog/mtk-wdt.txt > > > > > @@ -14,12 +14,12 @@ Required properties: > > > > > "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for > > > > > MT7622 > > > > > "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for > > > > > MT7623 > > > > > "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for > > > > > MT7629 > > > > > - "mediatek,mt7986-wdt", "mediatek,mt6589-wdt": for > > > > > MT7986 > > > > > + "mediatek,mt7986-wdt": for MT7986 > > > > > "mediatek,mt8183-wdt": for MT8183 > > > > > - "mediatek,mt8186-wdt", "mediatek,mt6589-wdt": for > > > > > MT8186 > > > > > + "mediatek,mt8186-wdt": for MT8186 > > > > > "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for > > > > > MT8516 > > > > > "mediatek,mt8192-wdt": for MT8192 > > > > > - "mediatek,mt8195-wdt", "mediatek,mt6589-wdt": for > > > > > MT8195 > > > > > + "mediatek,mt8195-wdt": for MT8195 > > > > > > > > > > - reg : Specifies base physical address and size of the > > > > > registers. > > > > > > > > > > @@ -32,8 +32,7 @@ Optional properties: > > > > > Example: > > > > > > > > > > watchdog: watchdog@10007000 { > > > > > - compatible = "mediatek,mt8183-wdt", > > > > > - "mediatek,mt6589-wdt"; > > > > > + compatible = "mediatek,mt8183-wdt"; > > > > > mediatek,disable-extrst; > > > > > reg = <0 0x10007000 0 0x100>; > > > > > interrupts = <GIC_SPI 139 IRQ_TYPE_NONE>; > > > > > -- > > > > > 2.18.0 > > > > > > > > > > > > > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-08-10 12:58 ` Allen-KH Cheng @ 2022-08-10 16:50 ` Krzysztof Kozlowski 2022-08-11 9:56 ` Allen-KH Cheng 0 siblings, 1 reply; 15+ messages in thread From: Krzysztof Kozlowski @ 2022-08-10 16:50 UTC (permalink / raw) To: Allen-KH Cheng, Nícolas F. R. A. Prado Cc: Matthias Brugger, Chen-Yu Tsai, Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski, angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek On 10/08/2022 15:58, Allen-KH Cheng wrote: > I agree the advantage of patch is aesthetic. Since I also want to send > another "watchdog: Convert binding to YAML" PATCH, it's better let all > wdt compatibles in the binding match the contents of mtk_wdt_dt_ids in > drivers/watchdog/mtk_wdt.c > > static const struct of_device_id mtk_wdt_dt_ids[] = { > { .compatible = "mediatek,mt2712-wdt", .data = &mt2712_data }, > { .compatible = "mediatek,mt6589-wdt" }, > { .compatible = "mediatek,mt7986-wdt", .data = &mt7986_data }, > { .compatible = "mediatek,mt8183-wdt", .data = &mt8183_data }, > { .compatible = "mediatek,mt8186-wdt", .data = &mt8186_data }, > { .compatible = "mediatek,mt8192-wdt", .data = &mt8192_data }, > { .compatible = "mediatek,mt8195-wdt", .data = &mt8195_data }, > { /* sentinel */ } > }; > > We have "mediatek,mt8186-wdt" "mediatek,mt8195-wdt" and > "mediatek,mt7986-wdt" now and they have their DT data for the reset > control. > > It's weird and unuseful to add "mediatek,mt6589-wdt" as fallback. > > > Please kindly let me know if I missed anything How the driver arranges it should not be a reason to use or not to use specific fallback. Although Rob acked it, but I still think you did not provide valid reason for the change. Valid reason is usually the actual hardware (so they are actually not compatible with mt6589), not exactly how once someone did it in the driver. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-08-10 16:50 ` Krzysztof Kozlowski @ 2022-08-11 9:56 ` Allen-KH Cheng 0 siblings, 0 replies; 15+ messages in thread From: Allen-KH Cheng @ 2022-08-11 9:56 UTC (permalink / raw) To: Krzysztof Kozlowski, Nícolas F. R. A. Prado Cc: Matthias Brugger, Chen-Yu Tsai, Wim Van Sebroeck, Guenter Roeck, Rob Herring, Krzysztof Kozlowski, angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek he SoC-specific compatibles Hi Krzysztof, On Wed, 2022-08-10 at 19:50 +0300, Krzysztof Kozlowski wrote: > On 10/08/2022 15:58, Allen-KH Cheng wrote: > > I agree the advantage of patch is aesthetic. Since I also want to > > send > > another "watchdog: Convert binding to YAML" PATCH, it's better let > > all > > wdt compatibles in the binding match the contents of mtk_wdt_dt_ids > > in > > drivers/watchdog/mtk_wdt.c > > > > static const struct of_device_id mtk_wdt_dt_ids[] = { > > { .compatible = "mediatek,mt2712-wdt", .data = &mt2712_data }, > > { .compatible = "mediatek,mt6589-wdt" }, > > { .compatible = "mediatek,mt7986-wdt", .data = &mt7986_data }, > > { .compatible = "mediatek,mt8183-wdt", .data = &mt8183_data }, > > { .compatible = "mediatek,mt8186-wdt", .data = &mt8186_data }, > > { .compatible = "mediatek,mt8192-wdt", .data = &mt8192_data }, > > { .compatible = "mediatek,mt8195-wdt", .data = &mt8195_data }, > > { /* sentinel */ } > > }; > > > > We have "mediatek,mt8186-wdt" "mediatek,mt8195-wdt" and > > "mediatek,mt7986-wdt" now and they have their DT data for the reset > > control. > > > > It's weird and unuseful to add "mediatek,mt6589-wdt" as fallback. > > > > > > Please kindly let me know if I missed anything > > How the driver arranges it should not be a reason to use or not to > use > specific fallback. Although Rob acked it, but I still think you did > not > provide valid reason for the change. > > Valid reason is usually the actual hardware (so they are actually not > compatible with mt6589), not exactly how once someone did it in the > driver. > > Best regards, > Krzysztof Thank you for your detailed and clear explanation. "mediatek,mt6589-wdt" provides the mtk watchdog support and the SoC- specific compatibles is for reset controls to standard wdt. "mediatek,mt6589-wdt" is compatible with mt8186, mt8195 and mt7986 and just not support the reset controls. Based on the discussion of email thread, please drop my series. Thanks to everyone for your comments. Best regards, Allen _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-07-21 1:48 ` [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example Allen-KH Cheng 2022-07-25 22:33 ` Rob Herring 2022-08-01 9:29 ` Chen-Yu Tsai @ 2022-08-10 16:49 ` Krzysztof Kozlowski 2022-08-11 8:47 ` Allen-KH Cheng 2 siblings, 1 reply; 15+ messages in thread From: Krzysztof Kozlowski @ 2022-08-10 16:49 UTC (permalink / raw) To: Allen-KH Cheng, Wim Van Sebroeck, Guenter Roeck, Krzysztof Kozlowski, Matthias Brugger, Rob Herring, nfraprado Cc: angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek On 21/07/2022 04:48, Allen-KH Cheng wrote: > The watchdog timer of mt8186. mt8195 and mt7986 have their DT data. > We should not use 'mediatek,mt6589-wdt' as fallback. > > For mediatek,wdt example of mt8183, We remove mediatek,mt6589-wdt fallback. > > Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for MediaTek MT8186") > Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for Mediatek MT8195") > Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for Mediatek MT7986") > Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add #reset-cells") Missing spaces around SHA. Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example 2022-08-10 16:49 ` Krzysztof Kozlowski @ 2022-08-11 8:47 ` Allen-KH Cheng 0 siblings, 0 replies; 15+ messages in thread From: Allen-KH Cheng @ 2022-08-11 8:47 UTC (permalink / raw) To: Krzysztof Kozlowski, Wim Van Sebroeck, Guenter Roeck, Krzysztof Kozlowski, Matthias Brugger, Rob Herring, nfraprado Cc: angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek Hi Krzysztof, On Wed, 2022-08-10 at 19:49 +0300, Krzysztof Kozlowski wrote: > On 21/07/2022 04:48, Allen-KH Cheng wrote: > > The watchdog timer of mt8186. mt8195 and mt7986 have their DT data. > > We should not use 'mediatek,mt6589-wdt' as fallback. > > > > For mediatek,wdt example of mt8183, We remove mediatek,mt6589-wdt > > fallback. > > > > Fixes:a45b408a020b("dt-bindings: watchdog: Add compatible for > > MediaTek MT8186") > > Fixes:b326f2c85f3d("dt-bindings: watchdog: Add compatible for > > Mediatek MT8195") > > Fixes:41e73feb1024("dt-bindings: watchdog: Add compatible for > > Mediatek MT7986") > > Fixes:f43f97a0fc0e("dt-bindings: mediatek: mt8183: Add #reset- > > cells") > > Missing spaces around SHA. > > Best regards, > Krzysztof Thanks for your friendly reminder. I'll pay more attention in the next time. Thanks, Allen _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v4 2/4] arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt7986 2022-07-21 1:48 [PATCH v4 0/4] mediatek: watchdog: Fix compatible fallbacks Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example Allen-KH Cheng @ 2022-07-21 1:48 ` Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 3/4] arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt8195 Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 4/4] arm64: dts: mediatek: Fix the watchdog node name Allen-KH Cheng 3 siblings, 0 replies; 15+ messages in thread From: Allen-KH Cheng @ 2022-07-21 1:48 UTC (permalink / raw) To: Wim Van Sebroeck, Guenter Roeck, Krzysztof Kozlowski, Matthias Brugger, Rob Herring, nfraprado Cc: angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek, Allen-KH Cheng The watchdog driver for mt7986 relies on DT data. The fallback compatible "mediatek,mt6589-wdt" won't work. Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> --- arch/arm64/boot/dts/mediatek/mt7986a.dtsi | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/mediatek/mt7986a.dtsi b/arch/arm64/boot/dts/mediatek/mt7986a.dtsi index e3a407d03551..6ab8329c635f 100644 --- a/arch/arm64/boot/dts/mediatek/mt7986a.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt7986a.dtsi @@ -113,8 +113,7 @@ }; watchdog: watchdog@1001c000 { - compatible = "mediatek,mt7986-wdt", - "mediatek,mt6589-wdt"; + compatible = "mediatek,mt7986-wdt"; reg = <0 0x1001c000 0 0x1000>; interrupts = <GIC_SPI 110 IRQ_TYPE_LEVEL_HIGH>; #reset-cells = <1>; -- 2.18.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 3/4] arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt8195 2022-07-21 1:48 [PATCH v4 0/4] mediatek: watchdog: Fix compatible fallbacks Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 2/4] arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt7986 Allen-KH Cheng @ 2022-07-21 1:48 ` Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 4/4] arm64: dts: mediatek: Fix the watchdog node name Allen-KH Cheng 3 siblings, 0 replies; 15+ messages in thread From: Allen-KH Cheng @ 2022-07-21 1:48 UTC (permalink / raw) To: Wim Van Sebroeck, Guenter Roeck, Krzysztof Kozlowski, Matthias Brugger, Rob Herring, nfraprado Cc: angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek, Allen-KH Cheng The watchdog driver for mt8195 relies on DT data. The fallback compatible "mediatek,mt6589-wdt" won't work. Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> --- arch/arm64/boot/dts/mediatek/mt8195.dtsi | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/arm64/boot/dts/mediatek/mt8195.dtsi b/arch/arm64/boot/dts/mediatek/mt8195.dtsi index 066c14989708..31bf4bb97824 100644 --- a/arch/arm64/boot/dts/mediatek/mt8195.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8195.dtsi @@ -325,8 +325,7 @@ }; watchdog: watchdog@10007000 { - compatible = "mediatek,mt8195-wdt", - "mediatek,mt6589-wdt"; + compatible = "mediatek,mt8195-wdt"; reg = <0 0x10007000 0 0x100>; }; -- 2.18.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v4 4/4] arm64: dts: mediatek: Fix the watchdog node name 2022-07-21 1:48 [PATCH v4 0/4] mediatek: watchdog: Fix compatible fallbacks Allen-KH Cheng ` (2 preceding siblings ...) 2022-07-21 1:48 ` [PATCH v4 3/4] arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt8195 Allen-KH Cheng @ 2022-07-21 1:48 ` Allen-KH Cheng 3 siblings, 0 replies; 15+ messages in thread From: Allen-KH Cheng @ 2022-07-21 1:48 UTC (permalink / raw) To: Wim Van Sebroeck, Guenter Roeck, Krzysztof Kozlowski, Matthias Brugger, Rob Herring, nfraprado Cc: angelogioacchino.delregno, Project_Global_Chrome_Upstream_Group, devicetree, linux-arm-kernel, linux-kernel, linux-mediatek, Allen-KH Cheng The proper name is 'watchdog', not 'toprgu'. Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> --- arch/arm64/boot/dts/mediatek/mt8516.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/mediatek/mt8516.dtsi b/arch/arm64/boot/dts/mediatek/mt8516.dtsi index d1b67c82d761..fff59dc1b4d7 100644 --- a/arch/arm64/boot/dts/mediatek/mt8516.dtsi +++ b/arch/arm64/boot/dts/mediatek/mt8516.dtsi @@ -202,7 +202,7 @@ #clock-cells = <1>; }; - toprgu: toprgu@10007000 { + toprgu: watchdog@10007000 { compatible = "mediatek,mt8516-wdt", "mediatek,mt6589-wdt"; reg = <0 0x10007000 0 0x1000>; -- 2.18.0 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel ^ permalink raw reply related [flat|nested] 15+ messages in thread
end of thread, other threads:[~2022-08-11 10:19 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-07-21 1:48 [PATCH v4 0/4] mediatek: watchdog: Fix compatible fallbacks Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 1/4] dt-bindings: mediatek: watchdog: Fix compatible fallbacks and example Allen-KH Cheng 2022-07-25 22:33 ` Rob Herring 2022-08-01 9:29 ` Chen-Yu Tsai 2022-08-02 11:04 ` Matthias Brugger 2022-08-03 9:23 ` Allen-KH Cheng 2022-08-08 17:03 ` Nícolas F. R. A. Prado 2022-08-10 12:58 ` Allen-KH Cheng 2022-08-10 16:50 ` Krzysztof Kozlowski 2022-08-11 9:56 ` Allen-KH Cheng 2022-08-10 16:49 ` Krzysztof Kozlowski 2022-08-11 8:47 ` Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 2/4] arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt7986 Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 3/4] arm64: dts: mediatek: Remove mt6589 wdt fallback string from mt8195 Allen-KH Cheng 2022-07-21 1:48 ` [PATCH v4 4/4] arm64: dts: mediatek: Fix the watchdog node name Allen-KH Cheng
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).