diff mbox

mmc: renesas_sdhi_internal_dmac: add R8A77980 to whitelist

Message ID 66c0b103-ed6e-36e0-74a6-9809ba55939c@cogentembedded.com (mailing list archive)
State Accepted
Delegated to: Geert Uytterhoeven
Headers show

Commit Message

Sergei Shtylyov April 19, 2018, 8:07 p.m. UTC
I've successfully tested eMMC on R8A77980/Condor. R8A77980 has a single
SDHI core anyway, so can't be a subject of the known RX DMA errata...

Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

---
This patch is against the 'next' branch of Ulf Hansson's 'mmc.git' repo.

 drivers/mmc/host/renesas_sdhi_internal_dmac.c |    1 +
 1 file changed, 1 insertion(+)

Comments

Wolfram Sang April 19, 2018, 8:09 p.m. UTC | #1
On Thu, Apr 19, 2018 at 11:07:44PM +0300, Sergei Shtylyov wrote:
> I've successfully tested eMMC on R8A77980/Condor. R8A77980 has a single
> SDHI core anyway, so can't be a subject of the known RX DMA errata...

Lucky you ;)

> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Ulf Hansson April 20, 2018, 7:08 a.m. UTC | #2
On 19 April 2018 at 22:09, Wolfram Sang <wsa@the-dreams.de> wrote:
> On Thu, Apr 19, 2018 at 11:07:44PM +0300, Sergei Shtylyov wrote:
>> I've successfully tested eMMC on R8A77980/Condor. R8A77980 has a single
>> SDHI core anyway, so can't be a subject of the known RX DMA errata...
>
> Lucky you ;)
>
>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>
> Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
>

Seems like this one also should go for fixes/stable then?

Kind regards
Uffe
Sergei Shtylyov April 20, 2018, 8:36 a.m. UTC | #3
Hello!

On 4/20/2018 10:08 AM, Ulf Hansson wrote:

>>> I've successfully tested eMMC on R8A77980/Condor. R8A77980 has a single
>>> SDHI core anyway, so can't be a subject of the known RX DMA errata...

    Should have been s/of/to/, I think.

>> Lucky you ;)
>>
>>> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>
>>
>> Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
>>
> Seems like this one also should go for fixes/stable then?

    In its current form it's only good for the next branch. Recasting/merging 
it for the fixes branch will result in the rejects in the next branch where 
you have already merged Wolfram's patches. The decision is up to you, of course...

> Kind regards
> Uffe

MBR, Sergei
Ulf Hansson April 23, 2018, 9:18 a.m. UTC | #4
On 19 April 2018 at 22:07, Sergei Shtylyov
<sergei.shtylyov@cogentembedded.com> wrote:
> I've successfully tested eMMC on R8A77980/Condor. R8A77980 has a single
> SDHI core anyway, so can't be a subject of the known RX DMA errata...
>
> Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>

Thanks, applied for next!

Kind regards
Uffe

>
> ---
> This patch is against the 'next' branch of Ulf Hansson's 'mmc.git' repo.
>
>  drivers/mmc/host/renesas_sdhi_internal_dmac.c |    1 +
>  1 file changed, 1 insertion(+)
>
> Index: mmc/drivers/mmc/host/renesas_sdhi_internal_dmac.c
> ===================================================================
> --- mmc.orig/drivers/mmc/host/renesas_sdhi_internal_dmac.c
> +++ mmc/drivers/mmc/host/renesas_sdhi_internal_dmac.c
> @@ -276,6 +276,7 @@ static const struct soc_device_attribute
>         /* generic ones */
>         { .soc_id = "r8a7795" },
>         { .soc_id = "r8a7796" },
> +       { .soc_id = "r8a77980" },
>         { .soc_id = "r8a77995" },
>         { /* sentinel */ }
>  };
diff mbox

Patch

Index: mmc/drivers/mmc/host/renesas_sdhi_internal_dmac.c
===================================================================
--- mmc.orig/drivers/mmc/host/renesas_sdhi_internal_dmac.c
+++ mmc/drivers/mmc/host/renesas_sdhi_internal_dmac.c
@@ -276,6 +276,7 @@  static const struct soc_device_attribute
 	/* generic ones */
 	{ .soc_id = "r8a7795" },
 	{ .soc_id = "r8a7796" },
+	{ .soc_id = "r8a77980" },
 	{ .soc_id = "r8a77995" },
 	{ /* sentinel */ }
 };