Re: [PATCHv2 linux dev-4.17 0/7] mmc/host: Add Aspeed SDIO driver

From: Alexander Amelkin <a.amelkin@yadro.com>
To: Ryan Chen <ryan_chen@aspeedtech.com>,
	"openbmc@lists.ozlabs.org" <openbmc@lists.ozlabs.org>
Subject: Re: [PATCHv2 linux dev-4.17 0/7] mmc/host: Add Aspeed SDIO driver
Date: Tue, 12 Mar 2019 15:16:41 +0300	[thread overview]
Message-ID: <a534e022-46c2-646c-9ea6-a1a2fd6ea7dd@yadro.com> (raw)
In-Reply-To: <00a601d4c5bf$c01a9520$404fbf60$@aspeedtech.com>

[-- Attachment #1.1: Type: text/plain, Size: 3310 bytes --]

Hi, Ryan!

How is it going? We've tried the original version of your patch and it didn't apply for us due to a typo. We've fixed the typo, but it still doesn't work. We're not sure yet why. Have you verified that old patch on any real hardware? When do you think you'll be able to come up with the new version of the patch?

Thank you.

Alexander.

16.02.2019 9:20, Ryan Chen wrote:
> Hello,
> 	I got some feedback from maintainer. I will modify for new architecture, hopeful, I will apply new patch at March.  
>
> Rayn
>
> -----Original Message-----
> From: openbmc [mailto:openbmc-bounces+ryan_chen=aspeedtech.com@lists.ozlabs.org] On Behalf Of Alexander Amelkin
> Sent: Friday, February 15, 2019 6:36 PM
> To: openbmc@lists.ozlabs.org
> Subject: Re: [PATCHv2 linux dev-4.17 0/7] mmc/host: Add Aspeed SDIO driver
>
> Guys, what's the fate of this patch set?
> Is going to be merged into openbmc linux?
> We have an SD card slot on our BMC card and so would like to know whether or not to await arrival of SDIO support from upstream or import this patch set locally.
>
> Thank you.
>
> 17.07.2018 10:26, Ryan Chen wrote:
>> This series implements Aspeed SDIO driver. In Aspeed SDIO controller 
>> with two slots and have a global register for interrupt status and 
>> other general information.
>>
>> v2
>> Change irqchip driver(irq-aspeed-sdhci-ic.c) to aspeed sdhci core 
>> driver aspeed-sdhci-core.c
>>
>> v0
>> For implements this, it will have have irq-aspeed-sdhci-ic.c for sdhci 
>> each slot irq dispatch, and then go to sdhci driver handle (sdhci.c).
>>
>> Another is sdhci-of-aspeed.c for specific function call.
>>
>>
>> Ryan Chen
>>
>>
>> Ryan Chen (7):
>>   clk: Aspeed: Modify clk-aspeed.c driver probe sequence
>>   clk: Aspeed: Add sdhci reset and clock
>>   mmc: Aspeed: Add Aspeed sdhci core driver
>>   mmc: Aspeed: Add driver for Aspeed sdhci
>>   dts: Aspeed: Add devicetree for Aspeed sdhci
>>   Documentation: mmc: add aspeed sdhci binding document
>>   configs: Aspeed: enable mmc host in defconfig
>>
>>  .../bindings/mmc/aspeed,aspeed-sdhci.txt           |  54 +++++++
>>  arch/arm/boot/dts/aspeed-ast2500-evb.dts           |  14 ++
>>  arch/arm/boot/dts/aspeed-g4.dtsi                   |  40 +++++
>>  arch/arm/boot/dts/aspeed-g5.dtsi                   |  40 +++++
>>  arch/arm/configs/aspeed_g5_defconfig               |   6 +
>>  drivers/clk/clk-aspeed.c                           |  51 +++---
>>  drivers/irqchip/Makefile                           |   2 +-
>>  drivers/mmc/host/Kconfig                           |  12 ++
>>  drivers/mmc/host/Makefile                          |   1 +
>>  drivers/mmc/host/aspeed-sdhci-core.c               | 147 +++++++++++++++++
>>  drivers/mmc/host/sdhci-of-aspeed.c                 | 178 +++++++++++++++++++++
>>  include/dt-bindings/clock/aspeed-clock.h           |   2 +-
>>  include/linux/mmc/sdhci-aspeed-data.h              |  28 ++++
>>  13 files changed, 553 insertions(+), 22 deletions(-)  create mode 
>> 100644 Documentation/devicetree/bindings/mmc/aspeed,aspeed-sdhci.txt
>>  create mode 100644 drivers/mmc/host/aspeed-sdhci-core.c
>>  create mode 100644 drivers/mmc/host/sdhci-of-aspeed.c
>>  create mode 100644 include/linux/mmc/sdhci-aspeed-data.h
>>

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]