Re: [PATCH v6 00/14] mmc: Add support to Marvell Xenon SD Host Controller
Hi Ulf, On lun., févr. 20 2017, Ulf Hansson wrote: > On 20 February 2017 at 17:59, Gregory CLEMENT > wrote: >> Hi Ulf, >> >> On mar., févr. 14 2017, Gregory CLEMENT >> wrote: >> >>> Hello, >>> >>> This the sixth version of the series adding support for the SDHCI >>> Xenon controller. It can be currently found on the Armada 37xx and the >>> Armada 7K/8K but will be also used in more Marvell SoC (and not only >>> the mvebu ones actually). >>> >>> Most of the following changes had been discussed on the mailing list, >>> except the new compatible string. >>> >> >> I think that with this version we addressed all the concerns expressed >> about the previous version. Do you think that you will be able to apply >> this series for 4.11 or do you still see any issues? >> >> Thanks, >> >> Gregory > > We are in the merge window for v4.11, so new material is out of the > question. I am preparing the PR to Linus as this very moment. v4.11-rc1 was released last week, do you want that I rebase the series ontop of it ? On our side we made several tests of this version of the driver on various hardware and everything works as expected. We only notice some configuration to update for the dts on some board but it didn't impact the driver itself. Gregory > > Anyway, I will look into the series as soon as I can. > > Kind regards > Uffe > > ___ > linux-arm-kernel mailing list > linux-arm-ker...@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com
Re: [PATCH v6 00/14] mmc: Add support to Marvell Xenon SD Host Controller
Hi Ulf, On 2017/2/21 3:32, Ulf Hansson wrote: > On 20 February 2017 at 17:59, Gregory CLEMENT > wrote: >> Hi Ulf, >> >> On mar., févr. 14 2017, Gregory CLEMENT >> wrote: >> >>> Hello, >>> >>> This the sixth version of the series adding support for the SDHCI >>> Xenon controller. It can be currently found on the Armada 37xx and the >>> Armada 7K/8K but will be also used in more Marvell SoC (and not only >>> the mvebu ones actually). >>> >>> Most of the following changes had been discussed on the mailing list, >>> except the new compatible string. >>> >> >> I think that with this version we addressed all the concerns expressed >> about the previous version. Do you think that you will be able to apply >> this series for 4.11 or do you still see any issues? >> >> Thanks, >> >> Gregory > > We are in the merge window for v4.11, so new material is out of the > question. I am preparing the PR to Linus as this very moment. > > Anyway, I will look into the series as soon as I can. Could you please take a look at our patch set? Very sorry for pushing you, but your comment is very important to us. Thank you. Best regards, Hu Ziji > > Kind regards > Uffe >
Re: [PATCH v6 00/14] mmc: Add support to Marvell Xenon SD Host Controller
On 20 February 2017 at 17:59, Gregory CLEMENT wrote: > Hi Ulf, > > On mar., févr. 14 2017, Gregory CLEMENT > wrote: > >> Hello, >> >> This the sixth version of the series adding support for the SDHCI >> Xenon controller. It can be currently found on the Armada 37xx and the >> Armada 7K/8K but will be also used in more Marvell SoC (and not only >> the mvebu ones actually). >> >> Most of the following changes had been discussed on the mailing list, >> except the new compatible string. >> > > I think that with this version we addressed all the concerns expressed > about the previous version. Do you think that you will be able to apply > this series for 4.11 or do you still see any issues? > > Thanks, > > Gregory We are in the merge window for v4.11, so new material is out of the question. I am preparing the PR to Linus as this very moment. Anyway, I will look into the series as soon as I can. Kind regards Uffe
Re: [PATCH v6 00/14] mmc: Add support to Marvell Xenon SD Host Controller
On Tue, Feb 14, 2017 at 06:01:40PM +0100, Gregory CLEMENT wrote: > Hello, > > This the sixth version of the series adding support for the SDHCI > Xenon controller. It can be currently found on the Armada 37xx and the > Armada 7K/8K but will be also used in more Marvell SoC (and not only > the mvebu ones actually). > > Most of the following changes had been discussed on the mailing list, > except the new compatible string. > > It occurred that the xenon controller was integrated in a different > way in the AP806 and the CP110 which both are part of the Armada 7K/8K > SoCs. For example the Xenon controller in CP110 doesn't not support > HS200. Seems to continnue to work on the Macchiatobin board. Tested-by: Russell King Thanks. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.
Re: [PATCH v6 00/14] mmc: Add support to Marvell Xenon SD Host Controller
Hi Ulf, On mar., févr. 14 2017, Gregory CLEMENT wrote: > Hello, > > This the sixth version of the series adding support for the SDHCI > Xenon controller. It can be currently found on the Armada 37xx and the > Armada 7K/8K but will be also used in more Marvell SoC (and not only > the mvebu ones actually). > > Most of the following changes had been discussed on the mailing list, > except the new compatible string. > I think that with this version we addressed all the concerns expressed about the previous version. Do you think that you will be able to apply this series for 4.11 or do you still see any issues? Thanks, Gregory > It occurred that the xenon controller was integrated in a different > way in the AP806 and the CP110 which both are part of the Armada 7K/8K > SoCs. For example the Xenon controller in CP110 doesn't not support > HS200. > > v5->v6: > > - Add a generic "mmc-card" parse in core layer. > > - Fix the spelling issues in Xenon dt binding doc and drivers. > > - Remove descriptions to common mmc properties from Xenon dt binding > doc. > > - Split compatible string "marvell,armada-8k-sdhci" into > "marvell,armada-ap806-sdhci" and "marvell,armada-cp110-sdhci". > > - Also updates the example in Xenon dt binding doc. > > - Remove unnecessary dependency on MMC_SDHCI from Xenon entry in > Kconfig. > > - Move Xenon specific dt parse into a separate function. > > - Adjust warnings and condition check in Xenon PHY setting, to remove > fragile hs200->hs400/hs400->hs200 sequence check function. > > - Enable PHY Slow Mode in MMC_TIMING_LEGACY timing if PHY Slow Mode is > required in dts. > > - Add a patch allowing dts backwards compatible for the clock > > v4->v5: > > - Remove the patch to export sdhci_execute_tuning(). It is already > exported in v4.10. > > - Introduce a patch adding a missing clock for the sdhci controller > present on the CP master for A7K/A8K. There is no build dependency > but obviously this patch is need to use the sdhci controller present > on the CP part. > > - Adjust Xenon return setup, to avoid being overwritten by >sdhci_add_host(). > > - Change Xenon register definition prefix to "XENON_". > > - Fix typos in Xenon driver and dt-binding docs. > > - Change compatible string "marvell,armada-7000-sdhci" to > "marvell,armada-8k-sdhci". Actually the Armada 7K SoCs are a subset > of the Armada 8K SoCs. Moreover, the use of the '000' is not > consistent with all the other compatible string already used for the > Armada 7K/8K family. > > - Added the Tested-by from Russell King on an Armada 8K based board. > > v3 -> v4: > For this version a few change have been done: > - fixes 2 bug reported by kbuild-bot > - remove extra of_node_put() > - convert 0 in false for function returning boolean > > - add a device tree node for the sdhci controller present on the CP > master for A7K/A8K. It also led to rename the sdhci0 node on AP to > ap_sdhci0 to make a distinction with the one present on CP master. > > v2 -> v3 > I think that now most (if not all) the remarks had been taking into > account since the second version. According to Ziji Hu, here are the > following changes: > " Changes in V3: > Adjust and improve Xenon DT bindings. Move some caps setting from driver > into > DT. Use mmc-card sub-node to represent eMMC type. > Remove PHY Sampling Fixed Delay Line scan in lower speed mode. > Improve Xenon probe and ->init_card() functions. > Export sdhci_enable_sdio_irq() and implement own SDIO IRQ control. > Split PHY patch into two smaller patches. > Temporarily remove AXI clock before its implementation is improved." > > Besides this changes I also > - Removed the sdhci-xenon-phy.h and moved its content in the > shc-xenon-phy.c file. > - Fixed the tuning-count usage > - Managed the error case for clk_prepare_enable > > For the record the change from v1 was: > " Changes in V2: > rebase on v4.9-rc2. > Re-write Xenon bindings. Ajust Xenon DT property naming. > Add a new DT property to indicate eMMC card type, instead of using > variable card_candidate. > Clear quirks SDHCI_QUIRK_MULTIBLOCK_READ_ACMD12 in Xenon platform data > Add support to HS400 retuning." > > Thanks, > > Gregory > > Gregory CLEMENT (4): > clk: apn806: Turn the eMMC clock as optional for dts backwards compatible > arm64: dts: marvell: add eMMC support for Armada 37xx > arm64: dts: marvell: add sdhci support for Armada 7K/8K > arm64: configs: enable SDHCI driver for Xenon > > Hu Ziji (9): > mmc: core: Add mmc-card dt sub-node parse in core layer > mmc: sdhci: Export sdhci_set_ios() from sdhci.c > mmc: sdhci: Export sdhci_start_signal_voltage_switch() in sdhci.c > mmc: sdhci: Export sdhci_enable_sdio_irq() from sdhci.c > dt: bindings: Add bindings for Marvell Xenon SD Host Controller > mmc: sdhci-xenon: Add Marvell Xenon SDHC core functionality > mmc: sdhci-xenon: Add support to PHYs of Marvell Xenon SDHC > mmc: sdhci-xenon: Add SoC P
