From: "Andreas Färber" <afaerber@suse.de> To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>, ulf.hansson@linaro.org, robh+dt@kernel.org Cc: sboyd@kernel.org, linux-arm-kernel@lists.infradead.org, linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, thomas.liau@actions-semi.com, linux-actions@lists.infradead.org, linus.walleij@linaro.org, linux-clk@vger.kernel.org Subject: Re: [PATCH 2/7] dt-bindings: mmc: Add Actions Semi SD/MMC/SDIO controller binding Date: Mon, 10 Jun 2019 15:45:37 +0200 [thread overview] Message-ID: <5d164528-c797-5f94-f905-719d4f69542c@suse.de> (raw) In-Reply-To: <20190608195317.6336-3-manivannan.sadhasivam@linaro.org> Am 08.06.19 um 21:53 schrieb Manivannan Sadhasivam: > Add devicetree binding for Actions Semi Owl SoC's SD/MMC/SDIO controller. > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > --- > .../devicetree/bindings/mmc/owl-mmc.txt | 37 +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mmc/owl-mmc.txt Rob, should this be YAML now? > > diff --git a/Documentation/devicetree/bindings/mmc/owl-mmc.txt b/Documentation/devicetree/bindings/mmc/owl-mmc.txt > new file mode 100644 > index 000000000000..a702f8d66cec > --- /dev/null > +++ b/Documentation/devicetree/bindings/mmc/owl-mmc.txt > @@ -0,0 +1,37 @@ > +Actions Semi Owl SoCs SD/MMC/SDIO controller > + > +Required properties: > +- compatible: should be "actions,owl-mmc" > +- reg: offset and length of the register set for the device. > +- interrupts: single interrupt specifier. > +- clocks: single clock specifier of the controller clock. > +- resets: phandle to the reset line. > +- dma-names: should be "mmc". > +- dmas: single DMA channel specifier I recall the main blocker for MMC being regulators, i.e. the I²C attached multi-function PMIC. Yet I don't see any such required property here, nor any patch series implementing it. Seems like this relies on U-Boot having initialized SD/eMMC? Do you intend to make them optional or did you want to hold off merging this one until the rest is done? > + > +Optional properties: > +- pinctrl-names: pinctrl state names "default" must be defined. > +- pinctrl-0: phandle referencing pin configuration of the controller. > +- bus-width: see mmc.txt > +- cap-sd-highspeed: see mmc.txt > +- cap-mmc-highspeed: see mmc.txt > +- sd-uhs-sdr12: see mmc.txt > +- sd-uhs-sdr25: see mmc.txt > +- sd-uhs-sdr50: see mmc.txt > +- non-removable: see mmc.txt I'm not convinced duplicating common properties is a good idea here, in particular pinctrl. Regards, Andreas > + > +Example: > + > + mmc0: mmc@e0330000 { > + compatible = "actions,owl-mmc"; > + reg = <0x0 0xe0330000 0x0 0x4000>; > + interrupts = <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_SD0>; > + resets = <&cmu RESET_SD0>; > + dmas = <&dma 2>; > + dma-names = "mmc"; > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc0_default>; > + bus-width = <4>; > + cap-sd-highspeed; > + }; > -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Mary Higgins, Sri Rasiah HRB 21284 (AG Nürnberg)
WARNING: multiple messages have this Message-ID (diff)
From: "Andreas Färber" <afaerber@suse.de> To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>, ulf.hansson@linaro.org, robh+dt@kernel.org Cc: devicetree@vger.kernel.org, sboyd@kernel.org, linux-actions@lists.infradead.org, linus.walleij@linaro.org, linux-mmc@vger.kernel.org, linux-kernel@vger.kernel.org, thomas.liau@actions-semi.com, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 2/7] dt-bindings: mmc: Add Actions Semi SD/MMC/SDIO controller binding Date: Mon, 10 Jun 2019 15:45:37 +0200 [thread overview] Message-ID: <5d164528-c797-5f94-f905-719d4f69542c@suse.de> (raw) In-Reply-To: <20190608195317.6336-3-manivannan.sadhasivam@linaro.org> Am 08.06.19 um 21:53 schrieb Manivannan Sadhasivam: > Add devicetree binding for Actions Semi Owl SoC's SD/MMC/SDIO controller. > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > --- > .../devicetree/bindings/mmc/owl-mmc.txt | 37 +++++++++++++++++++ > 1 file changed, 37 insertions(+) > create mode 100644 Documentation/devicetree/bindings/mmc/owl-mmc.txt Rob, should this be YAML now? > > diff --git a/Documentation/devicetree/bindings/mmc/owl-mmc.txt b/Documentation/devicetree/bindings/mmc/owl-mmc.txt > new file mode 100644 > index 000000000000..a702f8d66cec > --- /dev/null > +++ b/Documentation/devicetree/bindings/mmc/owl-mmc.txt > @@ -0,0 +1,37 @@ > +Actions Semi Owl SoCs SD/MMC/SDIO controller > + > +Required properties: > +- compatible: should be "actions,owl-mmc" > +- reg: offset and length of the register set for the device. > +- interrupts: single interrupt specifier. > +- clocks: single clock specifier of the controller clock. > +- resets: phandle to the reset line. > +- dma-names: should be "mmc". > +- dmas: single DMA channel specifier I recall the main blocker for MMC being regulators, i.e. the I²C attached multi-function PMIC. Yet I don't see any such required property here, nor any patch series implementing it. Seems like this relies on U-Boot having initialized SD/eMMC? Do you intend to make them optional or did you want to hold off merging this one until the rest is done? > + > +Optional properties: > +- pinctrl-names: pinctrl state names "default" must be defined. > +- pinctrl-0: phandle referencing pin configuration of the controller. > +- bus-width: see mmc.txt > +- cap-sd-highspeed: see mmc.txt > +- cap-mmc-highspeed: see mmc.txt > +- sd-uhs-sdr12: see mmc.txt > +- sd-uhs-sdr25: see mmc.txt > +- sd-uhs-sdr50: see mmc.txt > +- non-removable: see mmc.txt I'm not convinced duplicating common properties is a good idea here, in particular pinctrl. Regards, Andreas > + > +Example: > + > + mmc0: mmc@e0330000 { > + compatible = "actions,owl-mmc"; > + reg = <0x0 0xe0330000 0x0 0x4000>; > + interrupts = <GIC_SPI 42 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&cmu CLK_SD0>; > + resets = <&cmu RESET_SD0>; > + dmas = <&dma 2>; > + dma-names = "mmc"; > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc0_default>; > + bus-width = <4>; > + cap-sd-highspeed; > + }; > -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Mary Higgins, Sri Rasiah HRB 21284 (AG Nürnberg) _______________________________________________ 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:[~2019-06-10 13:46 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-06-08 19:53 [PATCH 0/7] Add SD/MMC driver for Actions Semi S900 SoC Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam 2019-06-08 19:53 ` [PATCH 1/7] clk: actions: Fix factor clk struct member access Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam 2019-06-10 13:36 ` Andreas Färber 2019-06-10 13:36 ` Andreas Färber 2019-06-10 16:05 ` Manivannan Sadhasivam 2019-06-10 16:05 ` Manivannan Sadhasivam 2019-06-10 15:01 ` Stephen Boyd 2019-06-10 15:01 ` Stephen Boyd 2019-06-10 15:01 ` Stephen Boyd 2019-06-08 19:53 ` [PATCH 2/7] dt-bindings: mmc: Add Actions Semi SD/MMC/SDIO controller binding Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam 2019-06-10 13:45 ` Andreas Färber [this message] 2019-06-10 13:45 ` Andreas Färber 2019-06-10 16:04 ` Manivannan Sadhasivam 2019-06-10 16:04 ` Manivannan Sadhasivam 2019-07-09 2:16 ` Rob Herring 2019-07-09 2:16 ` Rob Herring 2019-06-08 19:53 ` [PATCH 3/7] arm64: dts: actions: Add MMC controller support for S900 Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam 2019-06-08 19:53 ` [PATCH 4/7] arm64: dts: actions: Add uSD and eMMC support for Bubblegum96 Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam 2019-06-10 14:08 ` Andreas Färber 2019-06-10 14:08 ` Andreas Färber 2019-06-10 16:11 ` Manivannan Sadhasivam 2019-06-10 16:11 ` Manivannan Sadhasivam 2019-06-08 19:53 ` [PATCH 5/7] mmc: Add Actions Semi Owl SoCs SD/MMC driver Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam 2019-07-22 13:41 ` Ulf Hansson 2019-07-22 13:41 ` Ulf Hansson 2019-08-21 2:26 ` Manivannan Sadhasivam 2019-08-21 2:26 ` Manivannan Sadhasivam 2019-06-08 19:53 ` [PATCH 6/7] MAINTAINERS: Add entry for Actions Semi SD/MMC driver and binding Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam 2019-06-08 19:53 ` [PATCH 7/7] arm64: configs: Enable Actions Semi platform in defconfig Manivannan Sadhasivam 2019-06-08 19:53 ` Manivannan Sadhasivam
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=5d164528-c797-5f94-f905-719d4f69542c@suse.de \ --to=afaerber@suse.de \ --cc=devicetree@vger.kernel.org \ --cc=linus.walleij@linaro.org \ --cc=linux-actions@lists.infradead.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mmc@vger.kernel.org \ --cc=manivannan.sadhasivam@linaro.org \ --cc=robh+dt@kernel.org \ --cc=sboyd@kernel.org \ --cc=thomas.liau@actions-semi.com \ --cc=ulf.hansson@linaro.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.