[PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
I've successfully tested eMMC on the V3H Starter Kit board. R8A77970 SoC has a single SDHI core anyway, so can't be a subject to the known RX DMA errata... Signed-off-by: Sergei Shtylyov --- 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 @@ -293,6 +293,7 @@ static const struct soc_device_attribute { .soc_id = "r8a7795" }, { .soc_id = "r8a7796" }, { .soc_id = "r8a77965" }, + { .soc_id = "r8a77970" }, { .soc_id = "r8a77980" }, { .soc_id = "r8a77995" }, { /* sentinel */ }
Re: [PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
On Sat, Aug 18, 2018 at 09:08:26PM +0300, Sergei Shtylyov wrote: > I've successfully tested eMMC on the V3H Starter Kit board. R8A77970 SoC > has a single SDHI core anyway, so can't be a subject to the known RX DMA > errata... > > Signed-off-by: Sergei Shtylyov Nit: some strange double spaces and '...' in the commit message: Reviewed-by: Wolfram Sang And out of interest: was this an HS400 eMMC memory? Thanks! > --- > 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 > @@ -293,6 +293,7 @@ static const struct soc_device_attribute > { .soc_id = "r8a7795" }, > { .soc_id = "r8a7796" }, > { .soc_id = "r8a77965" }, > + { .soc_id = "r8a77970" }, > { .soc_id = "r8a77980" }, > { .soc_id = "r8a77995" }, > { /* sentinel */ } signature.asc Description: PGP signature
Re: [PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
On 08/20/2018 07:42 PM, Wolfram Sang wrote: >> I've successfully tested eMMC on the V3H Starter Kit board. R8A77970 SoC >> has a single SDHI core anyway, so can't be a subject to the known RX DMA >> errata... >> >> Signed-off-by: Sergei Shtylyov > > Nit: some strange double spaces and '...' in the commit message: Why are you so hostile to ...? :-) > Reviewed-by: Wolfram Sang > > And out of interest: was this an HS400 eMMC memory? I ndon';t have the datasheet handy but I don't think it matters -- the board only supplies 3.3V on both chip's voltage inputs... > Thanks! MBR, Sergei
Re: [PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
> > Nit: some strange double spaces and '...' in the commit message: > >Why are you so hostile to ...? :-) To me, it has a meaning: either something is to be continued or the sentence is intentionaly left open end, so the reader needs to complete it. I see neither fit here and I don't see any extra value over '.' Congrats, you got me bike-shedding. >I ndon';t have the datasheet handy but I don't think it matters -- the > board > only supplies 3.3V on both chip's voltage inputs... Okay. Pity, would have liked another test report for HS400. signature.asc Description: PGP signature
Re: [PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
On 08/20/2018 07:42 PM, Wolfram Sang wrote: >> I've successfully tested eMMC on the V3H Starter Kit board. R8A77970 SoC >> has a single SDHI core anyway, so can't be a subject to the known RX DMA >> errata... >> >> Signed-off-by: Sergei Shtylyov > > Nit: some strange double spaces and '...' in the commit message: > > Reviewed-by: Wolfram Sang Should I repost? Or you'd squash the fouble spaces and ellipsis while applying? BTW, will this patch go into the next or fixes branch? (I remember Ulf wanted to stick the analogous R8A77980 patch into fixes.) MBR, Sergei
Re: [PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
On Mon, Aug 20, 2018 at 10:44:18PM +0300, Sergei Shtylyov wrote: > On 08/20/2018 07:42 PM, Wolfram Sang wrote: > > >> I've successfully tested eMMC on the V3H Starter Kit board. R8A77970 SoC > >> has a single SDHI core anyway, so can't be a subject to the known RX DMA > >> errata... > >> > >> Signed-off-by: Sergei Shtylyov > > > > Nit: some strange double spaces and '...' in the commit message: > > > > Reviewed-by: Wolfram Sang > >Should I repost? Or you'd squash the fouble spaces and ellipsis while > applying? > >BTW, will this patch go into the next or fixes branch? (I remember Ulf > wanted to > stick the analogous R8A77980 patch into fixes.) I'll leave all these questions to Ulf. signature.asc Description: PGP signature
Re: [PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
On 08/20/2018 11:07 PM, Wolfram Sang wrote: I've successfully tested eMMC on the V3H Starter Kit board. R8A77970 SoC has a single SDHI core anyway, so can't be a subject to the known RX DMA errata... Signed-off-by: Sergei Shtylyov >>> >>> Nit: some strange double spaces and '...' in the commit message: >>> >>> Reviewed-by: Wolfram Sang >> >>Should I repost? Or you'd squash the fouble spaces and ellipsis while >> applying? >> >>BTW, will this patch go into the next or fixes branch? (I remember Ulf >> wanted to >> stick the analogous R8A77980 patch into fixes.) > > I'll leave all these questions to Ulf. Sorry, I forgot you don't have a separate repo. MBR, Sergei
Re: [PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
On 20 August 2018 at 21:44, Sergei Shtylyov wrote: > On 08/20/2018 07:42 PM, Wolfram Sang wrote: > >>> I've successfully tested eMMC on the V3H Starter Kit board. R8A77970 SoC >>> has a single SDHI core anyway, so can't be a subject to the known RX DMA >>> errata... >>> >>> Signed-off-by: Sergei Shtylyov >> >> Nit: some strange double spaces and '...' in the commit message: >> >> Reviewed-by: Wolfram Sang > >Should I repost? Or you'd squash the fouble spaces and ellipsis while > applying? No need to repost, I have amended the change. > >BTW, will this patch go into the next or fixes branch? (I remember Ulf > wanted to > stick the analogous R8A77980 patch into fixes.) Don't recall, but decided to queue this for v4.20. Please tell if you want me to do otherwise. Kind regards Uffe
Re: [PATCH] mmc: renesas_sdhi_internal_dmac: add R8A77970 to whitelist
On Sat, Aug 18, 2018 at 09:08:26PM +0300, Sergei Shtylyov wrote: > I've successfully tested eMMC on the V3H Starter Kit board. R8A77970 SoC > has a single SDHI core anyway, so can't be a subject to the known RX DMA > errata... > > Signed-off-by: Sergei Shtylyov Reviewed-by: Simon Horman