| Message ID | 1467882386-40544-2-git-send-email-dawei.chien@mediatek.com (mailing list archive) |
|---|---|
| State | Changes Requested |
| Delegated to: | Zhang Rui |
| Headers | show |
On Thursday 07 July 2016 02:36 PM, Dawei Chien wrote: > This adds the device tree binding documentation for the mediatek thermal > controller found on Mediatek MT2701. > > Signed-off-by: Dawei Chien <dawei.chien@mediatek.com> > --- > .../bindings/thermal/mediatek-thermal.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > index 81f9a51..bb55e79 100644 > --- a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > +++ b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > @@ -8,7 +8,7 @@ apmixedsys register space via AHB bus accesses, so a phandle to the APMIXEDSYS > is also needed. > > Required properties: > -- compatible: "mediatek,mt8173-thermal" > +- compatible: "mediatek,mt8173-thermal" or "mediatek,mt2701-thermal" - compatible : - "mediatek,mt8173-thermal" : For MT8173 family of SoCs - "mediatek,mt2701-thermal" : For MT2701 family of SoCs > - reg: Address range of the thermal controller > - interrupts: IRQ for the thermal controller > - clocks, clock-names: Clocks needed for the thermal controller. required > -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Dear Keerthy, On Thu, 2016-07-07 at 16:39 +0530, Keerthy wrote: > > On Thursday 07 July 2016 02:36 PM, Dawei Chien wrote: > > This adds the device tree binding documentation for the mediatek thermal > > controller found on Mediatek MT2701. > > > > Signed-off-by: Dawei Chien <dawei.chien@mediatek.com> > > --- > > .../bindings/thermal/mediatek-thermal.txt | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > > index 81f9a51..bb55e79 100644 > > --- a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > > +++ b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > > @@ -8,7 +8,7 @@ apmixedsys register space via AHB bus accesses, so a phandle to the APMIXEDSYS > > is also needed. > > > > Required properties: > > -- compatible: "mediatek,mt8173-thermal" > > +- compatible: "mediatek,mt8173-thermal" or "mediatek,mt2701-thermal" > > - compatible : > - "mediatek,mt8173-thermal" : For MT8173 family of SoCs > - "mediatek,mt2701-thermal" : For MT2701 family of SoCs Thank you, I will update it on next version. > > > - reg: Address range of the thermal controller > > - interrupts: IRQ for the thermal controller > > - clocks, clock-names: Clocks needed for the thermal controller. required > > -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 11/07/16 10:52, dawei chien wrote: > Dear Keerthy, > > On Thu, 2016-07-07 at 16:39 +0530, Keerthy wrote: >> >> On Thursday 07 July 2016 02:36 PM, Dawei Chien wrote: >>> This adds the device tree binding documentation for the mediatek thermal >>> controller found on Mediatek MT2701. >>> >>> Signed-off-by: Dawei Chien <dawei.chien@mediatek.com> >>> --- >>> .../bindings/thermal/mediatek-thermal.txt | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt >>> index 81f9a51..bb55e79 100644 >>> --- a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt >>> +++ b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt >>> @@ -8,7 +8,7 @@ apmixedsys register space via AHB bus accesses, so a phandle to the APMIXEDSYS >>> is also needed. >>> >>> Required properties: >>> -- compatible: "mediatek,mt8173-thermal" >>> +- compatible: "mediatek,mt8173-thermal" or "mediatek,mt2701-thermal" >> >> - compatible : >> - "mediatek,mt8173-thermal" : For MT8173 family of SoCs >> - "mediatek,mt2701-thermal" : For MT2701 family of SoCs > > Thank you, I will update it on next version. > Do you know about the compability to older SoCs (e.g. mt6589)? It might make sense to add mediatek,mtk-thermal if they are compatible or nearly compatible. >> >>> - reg: Address range of the thermal controller >>> - interrupts: IRQ for the thermal controller >>> - clocks, clock-names: Clocks needed for the thermal controller. required >>> > > -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Matthias, On Thu, 2016-08-11 at 17:48 +0200, Matthias Brugger wrote: > > On 11/07/16 10:52, dawei chien wrote: > > Dear Keerthy, > > > > On Thu, 2016-07-07 at 16:39 +0530, Keerthy wrote: > >> > >> On Thursday 07 July 2016 02:36 PM, Dawei Chien wrote: > >>> This adds the device tree binding documentation for the mediatek thermal > >>> controller found on Mediatek MT2701. > >>> > >>> Signed-off-by: Dawei Chien <dawei.chien@mediatek.com> > >>> --- > >>> .../bindings/thermal/mediatek-thermal.txt | 2 +- > >>> 1 file changed, 1 insertion(+), 1 deletion(-) > >>> > >>> diff --git a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > >>> index 81f9a51..bb55e79 100644 > >>> --- a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > >>> +++ b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt > >>> @@ -8,7 +8,7 @@ apmixedsys register space via AHB bus accesses, so a phandle to the APMIXEDSYS > >>> is also needed. > >>> > >>> Required properties: > >>> -- compatible: "mediatek,mt8173-thermal" > >>> +- compatible: "mediatek,mt8173-thermal" or "mediatek,mt2701-thermal" > >> > >> - compatible : > >> - "mediatek,mt8173-thermal" : For MT8173 family of SoCs > >> - "mediatek,mt2701-thermal" : For MT2701 family of SoCs > > > > Thank you, I will update it on next version. > > > > Do you know about the compability to older SoCs (e.g. mt6589)? > It might make sense to add mediatek,mtk-thermal if they are compatible > or nearly compatible. I agree with you that we should add mediatek,mtk-thermal for compatible or nearly compatible SoC. However, there is no more compatible SoC so far. Once we have such new nearly compatible SoC, we would update it on this binding document soon, thank you. BR, Dawei > >> > >>> - reg: Address range of the thermal controller > >>> - interrupts: IRQ for the thermal controller > >>> - clocks, clock-names: Clocks needed for the thermal controller. required > >>> > > > > -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt index 81f9a51..bb55e79 100644 --- a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt +++ b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt @@ -8,7 +8,7 @@ apmixedsys register space via AHB bus accesses, so a phandle to the APMIXEDSYS is also needed. Required properties: -- compatible: "mediatek,mt8173-thermal" +- compatible: "mediatek,mt8173-thermal" or "mediatek,mt2701-thermal" - reg: Address range of the thermal controller - interrupts: IRQ for the thermal controller - clocks, clock-names: Clocks needed for the thermal controller. required
This adds the device tree binding documentation for the mediatek thermal controller found on Mediatek MT2701. Signed-off-by: Dawei Chien <dawei.chien@mediatek.com> --- .../bindings/thermal/mediatek-thermal.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)