From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: Date: Mon, 30 Apr 2018 11:30:29 +0100 From: Lee Jones To: matthias.bgg@kernel.org Cc: ulrich.hecht+renesas@gmail.com, laurent.pinchart@ideasonboard.com, ck.hu@mediatek.com, p.zabel@pengutronix.de, airlied@linux.ie, robh+dt@kernel.org, mark.rutland@arm.com, mturquette@baylibre.com, sboyd@codeaurora.org, davem@davemloft.net, gregkh@linuxfoundation.org, mchehab@kernel.org, rdunlap@infradead.org, sean.wang@mediatek.com, linux-clk@vger.kernel.org, linux@armlinux.org.uk, matthias.bgg@gmail.com, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, Matthias Brugger Subject: Re: [v3 01/10] dt-bindings: mediatek: mmsys: Add support for mfd Message-ID: <20180430103029.GD5147@dell> References: <64ca273429d92f581568a48f3efe6035af65f579.1524820923.git.mbrugger@suse.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 In-Reply-To: <64ca273429d92f581568a48f3efe6035af65f579.1524820923.git.mbrugger@suse.com> List-ID: On Fri, 27 Apr 2018, matthias.bgg@kernel.org wrote: > From: Matthias Brugger > > Add binding description for the mmsys mfd for some Mediatek > devices. mmsys has some registers to control clock gates (which is > used in the clk driver) and some registers to set the routing and enable > the differnet blocks of the display subsystem. > > Signed-off-by: Matthias Brugger > --- > .../bindings/arm/mediatek/mediatek,mmsys.txt | 2 -- > .../bindings/display/mediatek/mediatek,disp.txt | 2 +- > .../devicetree/bindings/mfd/mediatek,mmsys.txt | 27 ++++++++++++++++++++++ > 3 files changed, 28 insertions(+), 3 deletions(-) > create mode 100644 Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt > > diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > index 4eb8bbe15c01..4468345f8b1a 100644 > --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > @@ -6,10 +6,8 @@ The Mediatek mmsys controller provides various clocks to the system. > Required Properties: > > - compatible: Should be one of: > - - "mediatek,mt2701-mmsys", "syscon" > - "mediatek,mt2712-mmsys", "syscon" > - "mediatek,mt6797-mmsys", "syscon" > - - "mediatek,mt8173-mmsys", "syscon" > - #clock-cells: Must be 1 > > The mmsys controller uses the common clk binding from > diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > index 383183a89164..85a3b4ec06cd 100644 > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > @@ -9,7 +9,7 @@ function block. > > All DISP device tree nodes must be siblings to the central MMSYS_CONFIG node. > For a description of the MMSYS_CONFIG binding, see > -Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt. > +Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt > > DISP function blocks > ==================== > diff --git a/Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt b/Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt > new file mode 100644 > index 000000000000..2331ae16917e > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt > @@ -0,0 +1,27 @@ > +MediaTek MMSYS Multifunction Device Driver What is "MMSYS"? "Multi-Function Driver"s are specific to Linux. What actually is the device? > +MMSYS is a multifunction device with the following sub modules: > +- clocks for the multi-media subsystem > +- central node for the DRM subsystem. > + > +This document describes the binding for MFD device. The MFD takes care to initailize > +the clock driver and the DRM driver. More info see > +Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > + > +Required properties: > +- compatible: Should be one of: > + - "mediatek,mt2701-mmsys", "syscon" > + - "mediatek,mt8173-mmsys", "syscon" > +- #clock-cells: Must be 1 > + > +Optional properties: > +- power-domains: list of powerdomains needed for the subsystem to work > + > +Example: > + > +mmsys: clock-controller@14000000 { > + compatible = "mediatek,mt8173-mmsys", "syscon"; > + reg = <0 0x14000000 0 0x1000>; > + power-domains = <&scpsys MT8173_POWER_DOMAIN_MM>; > + #clock-cells = <1>; > +}; -- Lee Jones [李琼斯] Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [v3 01/10] dt-bindings: mediatek: mmsys: Add support for mfd Date: Mon, 30 Apr 2018 11:30:29 +0100 Message-ID: <20180430103029.GD5147@dell> References: <64ca273429d92f581568a48f3efe6035af65f579.1524820923.git.mbrugger@suse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <64ca273429d92f581568a48f3efe6035af65f579.1524820923.git.mbrugger@suse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: matthias.bgg@kernel.org Cc: mark.rutland@arm.com, airlied@linux.ie, mturquette@baylibre.com, dri-devel@lists.freedesktop.org, laurent.pinchart@ideasonboard.com, ulrich.hecht+renesas@gmail.com, linux-clk@vger.kernel.org, linux@armlinux.org.uk, sean.wang@mediatek.com, robh+dt@kernel.org, linux-mediatek@lists.infradead.org, matthias.bgg@gmail.com, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, Matthias Brugger , gregkh@linuxfoundation.org, rdunlap@infradead.org, sboyd@codeaurora.org, linux-kernel@vger.kernel.org, davem@davemloft.net List-Id: linux-mediatek@lists.infradead.org T24gRnJpLCAyNyBBcHIgMjAxOCwgbWF0dGhpYXMuYmdnQGtlcm5lbC5vcmcgd3JvdGU6Cgo+IEZy b206IE1hdHRoaWFzIEJydWdnZXIgPG1icnVnZ2VyQHN1c2UuY29tPgo+IAo+IEFkZCBiaW5kaW5n IGRlc2NyaXB0aW9uIGZvciB0aGUgbW1zeXMgbWZkIGZvciBzb21lIE1lZGlhdGVrCj4gZGV2aWNl cy4gbW1zeXMgaGFzIHNvbWUgcmVnaXN0ZXJzIHRvIGNvbnRyb2wgY2xvY2sgZ2F0ZXMgKHdoaWNo IGlzCj4gdXNlZCBpbiB0aGUgY2xrIGRyaXZlcikgYW5kIHNvbWUgcmVnaXN0ZXJzIHRvIHNldCB0 aGUgcm91dGluZyBhbmQgZW5hYmxlCj4gdGhlIGRpZmZlcm5ldCBibG9ja3Mgb2YgdGhlIGRpc3Bs YXkgc3Vic3lzdGVtLgo+IAo+IFNpZ25lZC1vZmYtYnk6IE1hdHRoaWFzIEJydWdnZXIgPG1icnVn Z2VyQHN1c2UuY29tPgo+IC0tLQo+ICAuLi4vYmluZGluZ3MvYXJtL21lZGlhdGVrL21lZGlhdGVr LG1tc3lzLnR4dCAgICAgICB8ICAyIC0tCj4gIC4uLi9iaW5kaW5ncy9kaXNwbGF5L21lZGlhdGVr L21lZGlhdGVrLGRpc3AudHh0ICAgIHwgIDIgKy0KPiAgLi4uL2RldmljZXRyZWUvYmluZGluZ3Mv bWZkL21lZGlhdGVrLG1tc3lzLnR4dCAgICAgfCAyNyArKysrKysrKysrKysrKysrKysrKysrCj4g IDMgZmlsZXMgY2hhbmdlZCwgMjggaW5zZXJ0aW9ucygrKSwgMyBkZWxldGlvbnMoLSkKPiAgY3Jl YXRlIG1vZGUgMTAwNjQ0IERvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tZmQvbWVk aWF0ZWssbW1zeXMudHh0Cj4gCj4gZGlmZiAtLWdpdCBhL0RvY3VtZW50YXRpb24vZGV2aWNldHJl ZS9iaW5kaW5ncy9hcm0vbWVkaWF0ZWsvbWVkaWF0ZWssbW1zeXMudHh0IGIvRG9jdW1lbnRhdGlv bi9kZXZpY2V0cmVlL2JpbmRpbmdzL2FybS9tZWRpYXRlay9tZWRpYXRlayxtbXN5cy50eHQKPiBp bmRleCA0ZWI4YmJlMTVjMDEuLjQ0NjgzNDVmOGIxYSAxMDA2NDQKPiAtLS0gYS9Eb2N1bWVudGF0 aW9uL2RldmljZXRyZWUvYmluZGluZ3MvYXJtL21lZGlhdGVrL21lZGlhdGVrLG1tc3lzLnR4dAo+ ICsrKyBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9hcm0vbWVkaWF0ZWsvbWVk aWF0ZWssbW1zeXMudHh0Cj4gQEAgLTYsMTAgKzYsOCBAQCBUaGUgTWVkaWF0ZWsgbW1zeXMgY29u dHJvbGxlciBwcm92aWRlcyB2YXJpb3VzIGNsb2NrcyB0byB0aGUgc3lzdGVtLgo+ICBSZXF1aXJl ZCBQcm9wZXJ0aWVzOgo+ICAKPiAgLSBjb21wYXRpYmxlOiBTaG91bGQgYmUgb25lIG9mOgo+IC0J LSAibWVkaWF0ZWssbXQyNzAxLW1tc3lzIiwgInN5c2NvbiIKPiAgCS0gIm1lZGlhdGVrLG10Mjcx Mi1tbXN5cyIsICJzeXNjb24iCj4gIAktICJtZWRpYXRlayxtdDY3OTctbW1zeXMiLCAic3lzY29u Igo+IC0JLSAibWVkaWF0ZWssbXQ4MTczLW1tc3lzIiwgInN5c2NvbiIKPiAgLSAjY2xvY2stY2Vs bHM6IE11c3QgYmUgMQo+ICAKPiAgVGhlIG1tc3lzIGNvbnRyb2xsZXIgdXNlcyB0aGUgY29tbW9u IGNsayBiaW5kaW5nIGZyb20KPiBkaWZmIC0tZ2l0IGEvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVl L2JpbmRpbmdzL2Rpc3BsYXkvbWVkaWF0ZWsvbWVkaWF0ZWssZGlzcC50eHQgYi9Eb2N1bWVudGF0 aW9uL2RldmljZXRyZWUvYmluZGluZ3MvZGlzcGxheS9tZWRpYXRlay9tZWRpYXRlayxkaXNwLnR4 dAo+IGluZGV4IDM4MzE4M2E4OTE2NC4uODVhM2I0ZWMwNmNkIDEwMDY0NAo+IC0tLSBhL0RvY3Vt ZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9kaXNwbGF5L21lZGlhdGVrL21lZGlhdGVrLGRp c3AudHh0Cj4gKysrIGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Rpc3BsYXkv bWVkaWF0ZWsvbWVkaWF0ZWssZGlzcC50eHQKPiBAQCAtOSw3ICs5LDcgQEAgZnVuY3Rpb24gYmxv Y2suCj4gIAo+ICBBbGwgRElTUCBkZXZpY2UgdHJlZSBub2RlcyBtdXN0IGJlIHNpYmxpbmdzIHRv IHRoZSBjZW50cmFsIE1NU1lTX0NPTkZJRyBub2RlLgo+ICBGb3IgYSBkZXNjcmlwdGlvbiBvZiB0 aGUgTU1TWVNfQ09ORklHIGJpbmRpbmcsIHNlZQo+IC1Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUv YmluZGluZ3MvYXJtL21lZGlhdGVrL21lZGlhdGVrLG1tc3lzLnR4dC4KPiArRG9jdW1lbnRhdGlv bi9kZXZpY2V0cmVlL2JpbmRpbmdzL21mZC9tZWRpYXRlayxtbXN5cy50eHQKPiAgCj4gIERJU1Ag ZnVuY3Rpb24gYmxvY2tzCj4gID09PT09PT09PT09PT09PT09PT09Cj4gZGlmZiAtLWdpdCBhL0Rv Y3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5ncy9tZmQvbWVkaWF0ZWssbW1zeXMudHh0IGIv RG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL21mZC9tZWRpYXRlayxtbXN5cy50eHQK PiBuZXcgZmlsZSBtb2RlIDEwMDY0NAo+IGluZGV4IDAwMDAwMDAwMDAwMC4uMjMzMWFlMTY5MTdl Cj4gLS0tIC9kZXYvbnVsbAo+ICsrKyBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9iaW5kaW5n cy9tZmQvbWVkaWF0ZWssbW1zeXMudHh0Cj4gQEAgLTAsMCArMSwyNyBAQAo+ICtNZWRpYVRlayBN TVNZUyBNdWx0aWZ1bmN0aW9uIERldmljZSBEcml2ZXIKCldoYXQgaXMgIk1NU1lTIj8KCiJNdWx0 aS1GdW5jdGlvbiBEcml2ZXIicyBhcmUgc3BlY2lmaWMgdG8gTGludXguCgpXaGF0IGFjdHVhbGx5 IGlzIHRoZSBkZXZpY2U/Cgo+ICtNTVNZUyBpcyBhIG11bHRpZnVuY3Rpb24gZGV2aWNlIHdpdGgg dGhlIGZvbGxvd2luZyBzdWIgbW9kdWxlczoKPiArLSBjbG9ja3MgZm9yIHRoZSBtdWx0aS1tZWRp YSBzdWJzeXN0ZW0KPiArLSBjZW50cmFsIG5vZGUgZm9yIHRoZSBEUk0gc3Vic3lzdGVtLgo+ICsK PiArVGhpcyBkb2N1bWVudCBkZXNjcmliZXMgdGhlIGJpbmRpbmcgZm9yIE1GRCBkZXZpY2UuIFRo ZSBNRkQgdGFrZXMgY2FyZSB0byBpbml0YWlsaXplCj4gK3RoZSBjbG9jayBkcml2ZXIgYW5kIHRo ZSBEUk0gZHJpdmVyLiBNb3JlIGluZm8gc2VlCj4gK0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9i aW5kaW5ncy9kaXNwbGF5L21lZGlhdGVrL21lZGlhdGVrLGRpc3AudHh0Cj4gKwo+ICtSZXF1aXJl ZCBwcm9wZXJ0aWVzOgo+ICstIGNvbXBhdGlibGU6IFNob3VsZCBiZSBvbmUgb2Y6Cj4gKwktICJt ZWRpYXRlayxtdDI3MDEtbW1zeXMiLCAic3lzY29uIgo+ICsJLSAibWVkaWF0ZWssbXQ4MTczLW1t c3lzIiwgInN5c2NvbiIKPiArLSAjY2xvY2stY2VsbHM6IE11c3QgYmUgMQo+ICsKPiArT3B0aW9u YWwgcHJvcGVydGllczoKPiArLSBwb3dlci1kb21haW5zOiBsaXN0IG9mIHBvd2VyZG9tYWlucyBu ZWVkZWQgZm9yIHRoZSBzdWJzeXN0ZW0gdG8gd29yawo+ICsKPiArRXhhbXBsZToKPiArCj4gK21t c3lzOiBjbG9jay1jb250cm9sbGVyQDE0MDAwMDAwIHsKPiArCWNvbXBhdGlibGUgPSAibWVkaWF0 ZWssbXQ4MTczLW1tc3lzIiwgInN5c2NvbiI7Cj4gKwlyZWcgPSA8MCAweDE0MDAwMDAwIDAgMHgx MDAwPjsKPiArCXBvd2VyLWRvbWFpbnMgPSA8JnNjcHN5cyBNVDgxNzNfUE9XRVJfRE9NQUlOX01N PjsKPiArCSNjbG9jay1jZWxscyA9IDwxPjsKPiArfTsKCi0tIApMZWUgSm9uZXMgW+adjueQvOaW r10KTGluYXJvIFNlcnZpY2VzIFRlY2huaWNhbCBMZWFkCkxpbmFyby5vcmcg4pSCIE9wZW4gc291 cmNlIHNvZnR3YXJlIGZvciBBUk0gU29DcwpGb2xsb3cgTGluYXJvOiBGYWNlYm9vayB8IFR3aXR0 ZXIgfCBCbG9nCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f CmRyaS1kZXZlbCBtYWlsaW5nIGxpc3QKZHJpLWRldmVsQGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpo dHRwczovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2RyaS1kZXZlbAo= From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Mon, 30 Apr 2018 11:30:29 +0100 Subject: [v3 01/10] dt-bindings: mediatek: mmsys: Add support for mfd In-Reply-To: <64ca273429d92f581568a48f3efe6035af65f579.1524820923.git.mbrugger@suse.com> References: <64ca273429d92f581568a48f3efe6035af65f579.1524820923.git.mbrugger@suse.com> Message-ID: <20180430103029.GD5147@dell> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, 27 Apr 2018, matthias.bgg at kernel.org wrote: > From: Matthias Brugger > > Add binding description for the mmsys mfd for some Mediatek > devices. mmsys has some registers to control clock gates (which is > used in the clk driver) and some registers to set the routing and enable > the differnet blocks of the display subsystem. > > Signed-off-by: Matthias Brugger > --- > .../bindings/arm/mediatek/mediatek,mmsys.txt | 2 -- > .../bindings/display/mediatek/mediatek,disp.txt | 2 +- > .../devicetree/bindings/mfd/mediatek,mmsys.txt | 27 ++++++++++++++++++++++ > 3 files changed, 28 insertions(+), 3 deletions(-) > create mode 100644 Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt > > diff --git a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > index 4eb8bbe15c01..4468345f8b1a 100644 > --- a/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > +++ b/Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt > @@ -6,10 +6,8 @@ The Mediatek mmsys controller provides various clocks to the system. > Required Properties: > > - compatible: Should be one of: > - - "mediatek,mt2701-mmsys", "syscon" > - "mediatek,mt2712-mmsys", "syscon" > - "mediatek,mt6797-mmsys", "syscon" > - - "mediatek,mt8173-mmsys", "syscon" > - #clock-cells: Must be 1 > > The mmsys controller uses the common clk binding from > diff --git a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > index 383183a89164..85a3b4ec06cd 100644 > --- a/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > +++ b/Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > @@ -9,7 +9,7 @@ function block. > > All DISP device tree nodes must be siblings to the central MMSYS_CONFIG node. > For a description of the MMSYS_CONFIG binding, see > -Documentation/devicetree/bindings/arm/mediatek/mediatek,mmsys.txt. > +Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt > > DISP function blocks > ==================== > diff --git a/Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt b/Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt > new file mode 100644 > index 000000000000..2331ae16917e > --- /dev/null > +++ b/Documentation/devicetree/bindings/mfd/mediatek,mmsys.txt > @@ -0,0 +1,27 @@ > +MediaTek MMSYS Multifunction Device Driver What is "MMSYS"? "Multi-Function Driver"s are specific to Linux. What actually is the device? > +MMSYS is a multifunction device with the following sub modules: > +- clocks for the multi-media subsystem > +- central node for the DRM subsystem. > + > +This document describes the binding for MFD device. The MFD takes care to initailize > +the clock driver and the DRM driver. More info see > +Documentation/devicetree/bindings/display/mediatek/mediatek,disp.txt > + > +Required properties: > +- compatible: Should be one of: > + - "mediatek,mt2701-mmsys", "syscon" > + - "mediatek,mt8173-mmsys", "syscon" > +- #clock-cells: Must be 1 > + > +Optional properties: > +- power-domains: list of powerdomains needed for the subsystem to work > + > +Example: > + > +mmsys: clock-controller at 14000000 { > + compatible = "mediatek,mt8173-mmsys", "syscon"; > + reg = <0 0x14000000 0 0x1000>; > + power-domains = <&scpsys MT8173_POWER_DOMAIN_MM>; > + #clock-cells = <1>; > +}; -- Lee Jones [???] Linaro Services Technical Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog