Hi Simon-san, 2018-05-02 0:30 GMT+09:00 Simon Horman <horms@xxxxxxxxxxxx>: > On Mon, Apr 30, 2018 at 04:48:16AM +0900, Yoshihiro Kaneko wrote: >> From: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> >> >> Add SDHI nodes to the DT of the r8a77965 SoC. >> >> Based on several similar patches of the R8A7796 device tree >> by Simon Horman <horms+renesas@xxxxxxxxxxxx>. >> >> Signed-off-by: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> >> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@xxxxxxxxx> > > This patch needs to be rebased on top of the devel branch of the renesas > tree. Otherwise it looks good to me. Thanks for your review. I will rebase this patch. > >> --- >> arch/arm64/boot/dts/renesas/r8a77965.dtsi | 68 ++++++++++++++++++++++--------- >> 1 file changed, 48 insertions(+), 20 deletions(-) >> >> diff --git a/arch/arm64/boot/dts/renesas/r8a77965.dtsi b/arch/arm64/boot/dts/renesas/r8a77965.dtsi >> index f0871fc..6860704 100644 >> --- a/arch/arm64/boot/dts/renesas/r8a77965.dtsi >> +++ b/arch/arm64/boot/dts/renesas/r8a77965.dtsi >> @@ -834,26 +834,6 @@ >> }; >> }; >> >> - sdhi0: sd@ee100000 { >> - reg = <0 0xee100000 0 0x2000>; >> - /* placeholder */ >> - }; >> - >> - sdhi1: sd@ee120000 { >> - reg = <0 0xee120000 0 0x2000>; >> - /* placeholder */ >> - }; >> - >> - sdhi2: sd@ee140000 { >> - reg = <0 0xee140000 0 0x2000>; >> - /* placeholder */ >> - }; >> - >> - sdhi3: sd@ee160000 { >> - reg = <0 0xee160000 0 0x2000>; >> - /* placeholder */ >> - }; >> - >> usb3_phy0: usb-phy@e65ee000 { >> reg = <0 0xe65ee000 0 0x90>; >> #phy-cells = <0>; >> @@ -874,5 +854,53 @@ >> reg = <0 0xe6020000 0 0x0c>; >> /* placeholder */ >> }; >> + >> + sdhi0: sd@ee100000 { >> + compatible = "renesas,sdhi-r8a77965", >> + "renesas,rcar-gen3-sdhi"; >> + reg = <0 0xee100000 0 0x2000>; >> + interrupts = <GIC_SPI 165 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&cpg CPG_MOD 314>; >> + max-frequency = <200000000>; >> + power-domains = <&sysc 32>; >> + resets = <&cpg 314>; >> + status = "disabled"; >> + }; >> + >> + sdhi1: sd@ee120000 { >> + compatible = "renesas,sdhi-r8a77965", >> + "renesas,rcar-gen3-sdhi"; >> + reg = <0 0xee120000 0 0x2000>; >> + interrupts = <GIC_SPI 166 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&cpg CPG_MOD 313>; >> + max-frequency = <200000000>; >> + power-domains = <&sysc 32>; >> + resets = <&cpg 313>; >> + status = "disabled"; >> + }; >> + >> + sdhi2: sd@ee140000 { >> + compatible = "renesas,sdhi-r8a77965", >> + "renesas,rcar-gen3-sdhi"; >> + reg = <0 0xee140000 0 0x2000>; >> + interrupts = <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&cpg CPG_MOD 312>; >> + max-frequency = <200000000>; >> + power-domains = <&sysc 32>; >> + resets = <&cpg 312>; >> + status = "disabled"; >> + }; >> + >> + sdhi3: sd@ee160000 { >> + compatible = "renesas,sdhi-r8a77965", >> + "renesas,rcar-gen3-sdhi"; >> + reg = <0 0xee160000 0 0x2000>; >> + interrupts = <GIC_SPI 168 IRQ_TYPE_LEVEL_HIGH>; >> + clocks = <&cpg CPG_MOD 311>; >> + max-frequency = <200000000>; >> + power-domains = <&sysc 32>; >> + resets = <&cpg 311>; >> + status = "disabled"; >> + }; >> }; >> }; >> -- >> 1.9.1 >> Best regards, Kaneko