Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753961AbdGUO6T (ORCPT ); Fri, 21 Jul 2017 10:58:19 -0400 Received: from mail-pf0-f194.google.com ([209.85.192.194]:38838 "EHLO mail-pf0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751504AbdGUO4v (ORCPT ); Fri, 21 Jul 2017 10:56:51 -0400 Date: Fri, 21 Jul 2017 09:56:48 -0500 From: Rob Herring To: Ivan Mikhaylov Cc: Alistair Popple , Matt Porter , Adrian Hunter , Patrice Chotard , Ulf Hansson , linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org, Benjamin Herrenschmidt , Joel Stanley , Paul Mackerras , Michael Ellerman , linux-mmc@vger.kernel.org, kernel@stlinux.com Subject: Re: [PATCH v1 2/2] 44x/fsp2: enable eMMC arasan for fsp2 platform Message-ID: <20170721145648.eb35x26aq6nuqkpk@rob-hp-laptop> References: <20170630115331.73869-3-ivan@de.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170630115331.73869-3-ivan@de.ibm.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1465 Lines: 52 On Fri, Jun 30, 2017 at 02:53:31PM +0300, Ivan Mikhaylov wrote: > Add mmc0 changes for enabling arasan emmc and change > defconfig appropriately. > > Signed-off-by: Ivan Mikhaylov > --- > arch/powerpc/boot/dts/fsp2.dts | 33 +++++++++++++++++------------- > arch/powerpc/configs/44x/fsp2_defconfig | 2 + > 2 files changed, 21 insertions(+), 14 deletions(-) > > diff --git a/arch/powerpc/boot/dts/fsp2.dts b/arch/powerpc/boot/dts/fsp2.dts > index 475953a..6a63026 100644 > --- a/arch/powerpc/boot/dts/fsp2.dts > +++ b/arch/powerpc/boot/dts/fsp2.dts > @@ -52,6 +52,7 @@ > clocks { > mmc_clk: mmc_clk { > compatible = "fixed-clock"; > + #clock-cells = <0>; > clock-frequency = <50000000>; > clock-output-names = "mmc_clk"; > }; > @@ -359,20 +360,6 @@ > interrupts = <31 0x4 15 0x84>; > }; > > - mmc0: sdhci@020c0000 { > - compatible = "st,sdhci-stih407", "st,sdhci"; > - status = "disabled"; > - reg = <0x020c0000 0x20000>; > - reg-names = "mmc"; > - interrupt-parent = <&UIC1_3>; > - interrupts = <21 0x4 22 0x4>; > - interrupt-names = "mmcirq"; > - pinctrl-names = "default"; > - pinctrl-0 = <>; > - clock-names = "mmc"; > - clocks = <&mmc_clk>; > - }; > - > plb6 { > compatible = "ibm,plb6"; > #address-cells = <2>; > @@ -501,6 +488,24 @@ > /*RXDE*/ 4 &UIC1_2 13 0x4>; > }; > > + mmc0: sdhci@020c0000 { Use generic node names and drop leading 0: mmc@20c0000 Rob