From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Subject: Re: [PATCH v6 0/3] mmc: sh_mobile_sdhi: fix missing r7s72100 clocks Date: Thu, 26 Jan 2017 11:30:13 +0100 Message-ID: <20170126103013.GB27721@verge.net.au> References: <20170125202810.16876-1-chris.brandt@renesas.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-renesas-soc-owner@vger.kernel.org To: Ulf Hansson Cc: Chris Brandt , Rob Herring , Mark Rutland , Wolfram Sang , Geert Uytterhoeven , "devicetree@vger.kernel.org" , "linux-mmc@vger.kernel.org" , Linux-Renesas List-Id: devicetree@vger.kernel.org On Thu, Jan 26, 2017 at 09:07:45AM +0100, Ulf Hansson wrote: > On 25 January 2017 at 21:28, Chris Brandt wrote: > > At first this started out as a simple typo fix, until I realized > > that the SDHI in the RZ/A1 has 2 clocks per channel and both need > > to be turned on/off. > > > > This patch series adds the ability to specify 2 clocks instead of > > just 1, and does so for the RZ/A1 r7s72100. > > > > This patch has been tested on an RZ/A1 RSK board. Card detect works > > fine as well as bind/unbind. > > > > v6: > > * more detailed descritpion in tmio_mmc.txt > > > > v5: > > * Take implementation details out of DT documentation and put into > > the driver code. > > > > v4: > > * No code changes, only text udpates to explain why there are 2 clocks > > and why we need to treat them as 1 clock. > > > > Chris Brandt (3): > > mmc: sh_mobile_sdhi: add support for 2 clocks > > mmc: sh_mobile_sdhi: explain clock bindings > > ARM: dts: r7s72100: update sdhi clock bindings > > > > Documentation/devicetree/bindings/mmc/tmio_mmc.txt | 13 ++++++++++++ > > arch/arm/boot/dts/r7s72100.dtsi | 17 +++++++++++----- > > drivers/mmc/host/sh_mobile_sdhi.c | 23 ++++++++++++++++++++++ > > include/dt-bindings/clock/r7s72100-clock.h | 6 ++++-- > > 4 files changed, 52 insertions(+), 7 deletions(-) > > > > -- > > 2.10.1 > > > > > > Thanks, applied patch1 and patch2 for next. Patch3 is for Simon, I > guess!? (And I can add Rob's ack afterwards). Thanks, I will take care of the 3rd patch. Chris, is it safe to apply the 3rd patch without the first 2 patches present?