| Message ID | 1526019083-20237-2-git-send-email-erin.lo@mediatek.com |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | Add basic support for Mediatek MT8183 SoC | expand |
On 05/11/2018 08:11 AM, Erin Lo wrote: > This adds dt-binding documentation for Mediatek MT8183. > Only include very basic items: cpu, gic and uart. > > Signed-off-by: Erin Lo <erin.lo@mediatek.com> > --- > Documentation/devicetree/bindings/arm/mediatek.txt | 4 ++++ > .../devicetree/bindings/interrupt-controller/mediatek,sysirq.txt | 1 + > Documentation/devicetree/bindings/serial/mtk-uart.txt | 1 + > 3 files changed, 6 insertions(+) > This touches different maintainers and the patches should go through their trees, so that no merge conflicts occur. I'm fine with take it through my tree but it's more work as we might need a stable branch that the other maintainers can merge. Apart I'd need the Acked-by from the maintainers. I'm fine with either way. > diff --git a/Documentation/devicetree/bindings/arm/mediatek.txt b/Documentation/devicetree/bindings/arm/mediatek.txt > index 7d21ab3..2754535 100644 > --- a/Documentation/devicetree/bindings/arm/mediatek.txt > +++ b/Documentation/devicetree/bindings/arm/mediatek.txt > @@ -19,6 +19,7 @@ compatible: Must contain one of > "mediatek,mt8127" > "mediatek,mt8135" > "mediatek,mt8173" > + "mediatek,mt8183" > > > Supported boards: > @@ -73,3 +74,6 @@ Supported boards: > - MTK mt8173 tablet EVB: > Required root node properties: > - compatible = "mediatek,mt8173-evb", "mediatek,mt8173"; > +- Evaluation board for MT8183: > + Required root node properties: > + - compatible = "mediatek,mt8183-evb", "mediatek,mt8183"; > diff --git a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > index 07bf0b9..5ff48a8 100644 > --- a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > +++ b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > @@ -5,6 +5,7 @@ interrupt. > > Required properties: > - compatible: should be > + "mediatek,mt8183-sysirq", "mediatek,mt6577-sysirq": for MT8183 > "mediatek,mt8173-sysirq", "mediatek,mt6577-sysirq": for MT8173 > "mediatek,mt8135-sysirq", "mediatek,mt6577-sysirq": for MT8135 > "mediatek,mt8127-sysirq", "mediatek,mt6577-sysirq": for MT8127 > diff --git a/Documentation/devicetree/bindings/serial/mtk-uart.txt b/Documentation/devicetree/bindings/serial/mtk-uart.txt > index f73abff..6e6d168 100644 > --- a/Documentation/devicetree/bindings/serial/mtk-uart.txt > +++ b/Documentation/devicetree/bindings/serial/mtk-uart.txt > @@ -15,6 +15,7 @@ Required properties: > * "mediatek,mt8127-uart" for MT8127 compatible UARTS > * "mediatek,mt8135-uart" for MT8135 compatible UARTS > * "mediatek,mt8173-uart" for MT8173 compatible UARTS > + * "mediatek,mt8183-uart" for MT8183 compatible UARTS If you add this compatible without a fallback that you will need an update in the driver. Is there really anything new in the serial console with regard to older devices? Regards, Matthias > * "mediatek,mt6577-uart" for MT6577 and all of the above > > - reg: The base address of the UART register bank. > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, 2018-05-11 at 10:33 +0200, Matthias Brugger wrote: > > On 05/11/2018 08:11 AM, Erin Lo wrote: > > This adds dt-binding documentation for Mediatek MT8183. > > Only include very basic items: cpu, gic and uart. > > > > Signed-off-by: Erin Lo <erin.lo@mediatek.com> > > --- > > Documentation/devicetree/bindings/arm/mediatek.txt | 4 ++++ > > .../devicetree/bindings/interrupt-controller/mediatek,sysirq.txt | 1 + > > Documentation/devicetree/bindings/serial/mtk-uart.txt | 1 + > > 3 files changed, 6 insertions(+) > > > > This touches different maintainers and the patches should go through their > trees, so that no merge conflicts occur. > I'm fine with take it through my tree but it's more work as we might need a > stable branch that the other maintainers can merge. Apart I'd need the Acked-by > from the maintainers. > > I'm fine with either way. > For prevent merge conflict and review convenient, I will split it into different patches in next round. > > diff --git a/Documentation/devicetree/bindings/arm/mediatek.txt b/Documentation/devicetree/bindings/arm/mediatek.txt > > index 7d21ab3..2754535 100644 > > --- a/Documentation/devicetree/bindings/arm/mediatek.txt > > +++ b/Documentation/devicetree/bindings/arm/mediatek.txt > > @@ -19,6 +19,7 @@ compatible: Must contain one of > > "mediatek,mt8127" > > "mediatek,mt8135" > > "mediatek,mt8173" > > + "mediatek,mt8183" > > > > > > Supported boards: > > @@ -73,3 +74,6 @@ Supported boards: > > - MTK mt8173 tablet EVB: > > Required root node properties: > > - compatible = "mediatek,mt8173-evb", "mediatek,mt8173"; > > +- Evaluation board for MT8183: > > + Required root node properties: > > + - compatible = "mediatek,mt8183-evb", "mediatek,mt8183"; > > diff --git a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > > index 07bf0b9..5ff48a8 100644 > > --- a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > > +++ b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt > > @@ -5,6 +5,7 @@ interrupt. > > > > Required properties: > > - compatible: should be > > + "mediatek,mt8183-sysirq", "mediatek,mt6577-sysirq": for MT8183 > > "mediatek,mt8173-sysirq", "mediatek,mt6577-sysirq": for MT8173 > > "mediatek,mt8135-sysirq", "mediatek,mt6577-sysirq": for MT8135 > > "mediatek,mt8127-sysirq", "mediatek,mt6577-sysirq": for MT8127 > > diff --git a/Documentation/devicetree/bindings/serial/mtk-uart.txt b/Documentation/devicetree/bindings/serial/mtk-uart.txt > > index f73abff..6e6d168 100644 > > --- a/Documentation/devicetree/bindings/serial/mtk-uart.txt > > +++ b/Documentation/devicetree/bindings/serial/mtk-uart.txt > > @@ -15,6 +15,7 @@ Required properties: > > * "mediatek,mt8127-uart" for MT8127 compatible UARTS > > * "mediatek,mt8135-uart" for MT8135 compatible UARTS > > * "mediatek,mt8173-uart" for MT8173 compatible UARTS > > + * "mediatek,mt8183-uart" for MT8183 compatible UARTS > > If you add this compatible without a fallback that you will need an update in > the driver. Is there really anything new in the serial console with regard to > older devices? > > Regards, > Matthias > No new function in MT8183..now How about this... "mediatek,mt8183-uart", "mediatek,mt6577-uart" for MT8183 compatible UARTS Regards, Erin > > * "mediatek,mt6577-uart" for MT6577 and all of the above > > > > - reg: The base address of the UART register bank. > > -- To unsubscribe from this list: send the line "unsubscribe devicetree" 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/arm/mediatek.txt b/Documentation/devicetree/bindings/arm/mediatek.txt index 7d21ab3..2754535 100644 --- a/Documentation/devicetree/bindings/arm/mediatek.txt +++ b/Documentation/devicetree/bindings/arm/mediatek.txt @@ -19,6 +19,7 @@ compatible: Must contain one of "mediatek,mt8127" "mediatek,mt8135" "mediatek,mt8173" + "mediatek,mt8183" Supported boards: @@ -73,3 +74,6 @@ Supported boards: - MTK mt8173 tablet EVB: Required root node properties: - compatible = "mediatek,mt8173-evb", "mediatek,mt8173"; +- Evaluation board for MT8183: + Required root node properties: + - compatible = "mediatek,mt8183-evb", "mediatek,mt8183"; diff --git a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt index 07bf0b9..5ff48a8 100644 --- a/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt +++ b/Documentation/devicetree/bindings/interrupt-controller/mediatek,sysirq.txt @@ -5,6 +5,7 @@ interrupt. Required properties: - compatible: should be + "mediatek,mt8183-sysirq", "mediatek,mt6577-sysirq": for MT8183 "mediatek,mt8173-sysirq", "mediatek,mt6577-sysirq": for MT8173 "mediatek,mt8135-sysirq", "mediatek,mt6577-sysirq": for MT8135 "mediatek,mt8127-sysirq", "mediatek,mt6577-sysirq": for MT8127 diff --git a/Documentation/devicetree/bindings/serial/mtk-uart.txt b/Documentation/devicetree/bindings/serial/mtk-uart.txt index f73abff..6e6d168 100644 --- a/Documentation/devicetree/bindings/serial/mtk-uart.txt +++ b/Documentation/devicetree/bindings/serial/mtk-uart.txt @@ -15,6 +15,7 @@ Required properties: * "mediatek,mt8127-uart" for MT8127 compatible UARTS * "mediatek,mt8135-uart" for MT8135 compatible UARTS * "mediatek,mt8173-uart" for MT8173 compatible UARTS + * "mediatek,mt8183-uart" for MT8183 compatible UARTS * "mediatek,mt6577-uart" for MT6577 and all of the above - reg: The base address of the UART register bank.
This adds dt-binding documentation for Mediatek MT8183. Only include very basic items: cpu, gic and uart. Signed-off-by: Erin Lo <erin.lo@mediatek.com> --- Documentation/devicetree/bindings/arm/mediatek.txt | 4 ++++ .../devicetree/bindings/interrupt-controller/mediatek,sysirq.txt | 1 + Documentation/devicetree/bindings/serial/mtk-uart.txt | 1 + 3 files changed, 6 insertions(+)