RE: [PATCH v6 0/8] Arasan SDHCI enhancements and ZynqMP Tap Delays Handling

From: Manish Narani <MNARANI@xilinx.com>
To: Ulf Hansson <ulf.hansson@linaro.org>, Rob Herring <robh+dt@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
	Adrian Hunter <adrian.hunter@intel.com>,
	Michal Simek <michals@xilinx.com>, Jolly Shah <JOLLYS@xilinx.com>,
	Rajan Vaja <RAJANV@xilinx.com>,
	Nava kishore Manne <navam@xilinx.com>,
	Moritz Fischer <mdf@kernel.org>,
	"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
	DTML <devicetree@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Linux ARM <linux-arm-kernel@lists.infradead.org>,
	git <git@xilinx.com>
Subject: RE: [PATCH v6 0/8] Arasan SDHCI enhancements and ZynqMP Tap Delays Handling
Date: Thu, 21 Nov 2019 07:08:56 +0000	[thread overview]
Message-ID: <MN2PR02MB6029E25E0DC26BD79DA58A4EC14E0@MN2PR02MB6029.namprd02.prod.outlook.com> (raw)
In-Reply-To: <CAPDyKFprcjgrcbT3jpT7pyM+FFWL8RAm5AtFAjVLcPMDiDzUvA@mail.gmail.com>

Hi Uffe, Rob and Adrian,

> -----Original Message-----
> From: Ulf Hansson <ulf.hansson@linaro.org>
> Sent: Wednesday, November 20, 2019 6:29 PM
> To: Manish Narani <MNARANI@xilinx.com>
> Cc: Rob Herring <robh+dt@kernel.org>; Mark Rutland
> <mark.rutland@arm.com>; Adrian Hunter <adrian.hunter@intel.com>;
> Michal Simek <michals@xilinx.com>; Jolly Shah <JOLLYS@xilinx.com>; Rajan
> Vaja <RAJANV@xilinx.com>; Nava kishore Manne <navam@xilinx.com>;
> Moritz Fischer <mdf@kernel.org>; linux-mmc@vger.kernel.org; DTML
> <devicetree@vger.kernel.org>; Linux Kernel Mailing List <linux-
> kernel@vger.kernel.org>; Linux ARM <linux-arm-
> kernel@lists.infradead.org>; git <git@xilinx.com>
> Subject: Re: [PATCH v6 0/8] Arasan SDHCI enhancements and ZynqMP Tap
> Delays Handling
> 
> On Wed, 20 Nov 2019 at 07:47, Manish Narani <manish.narani@xilinx.com>
> wrote:
> >
> > This patch series does the following:
> >  - Reorganize the Clock Handling in Arasan SD driver
> >  - Adds new sampling clock in Arasan SD driver
> >  - Adds support to set Clock Delays in SD Arasan Driver
> >  - Add SDIO Tap Delay handling in ZynqMP firmware driver
> >  - Add support for ZynqMP Tap Delays setting in Arasan SD driver
> >
> > Changes in v2:
> >         - Replaced the deprecated calls to clock framework APIs
> >         - Added support for dev_clk_get() call to work for SD card clock
> >         - Separated the clock data struct
> >         - Fragmented the patch series in smaller patches to make it more
> >           readable
> >
> > Changes in v3:
> >         - Reverted "Replaced the deprecated calls to clock framework APIs"
> >         - Removed devm_clk_get() call which was added in v2
> >
> > Changes in v4:
> >         - Made the Phase Delay properties Arasan specific
> >
> > Changes in v5:
> >         - Made Clock Phase Delay properties common
> >         - Moved documentation of them to the common mmc documentation.
> >
> > Changes in v6:
> >         - Clubbed all Clk Phase Delay properties' into a pattern
> >           Property
> >
> > Manish Narani (8):
> >   mmc: sdhci-of-arasan: Separate out clk related data to another
> >     structure
> >   dt-bindings: mmc: arasan: Update Documentation for the input clock
> >   mmc: sdhci-of-arasan: Add sampling clock for a phy to use
> >   dt-bindings: mmc: Add optional generic properties for mmc
> >   mmc: sdhci-of-arasan: Add support to set clock phase delays for SD
> >   firmware: xilinx: Add SDIO Tap Delay nodes
> >   dt-bindings: mmc: arasan: Document 'xlnx,zynqmp-8.9a' controller
> >   mmc: sdhci-of-arasan: Add support for ZynqMP Platform Tap Delays Setup
> >
> >  .../devicetree/bindings/mmc/arasan,sdhci.txt  |  25 +-
> >  .../bindings/mmc/mmc-controller.yaml          |  13 +
> >  drivers/mmc/host/sdhci-of-arasan.c            | 478 +++++++++++++++++-
> >  include/linux/firmware/xlnx-zynqmp.h          |  13 +-
> >  4 files changed, 497 insertions(+), 32 deletions(-)
> >
> > --
> > 2.17.1
> >
> 
> Applied for next, assuming Rob is okay with patch4, otherwise you need
> to send a fix on top, thanks!
> 
> Kind regards
> Uffe

Thank you so much for your reviews and continuous support! :)
Looking forward to work more with you on this.

Regards,
Manish