Hello!
On 04/13/2018 05:10 PM, Wolfram Sang wrote:
> Early revisions of certain SoCs cannot do multiple DMA RX streams in
> parallel. To avoid data corruption, only allow one DMA RX channel and
> fall back to PIO, if needed.
>
> Signed-off-by: Wolfram Sang <wsa+rene...@sang-engineering.com>
> Reviewed-by: Yoshihiro Shimoda <yoshihiro.shimoda...@renesas.com>
> Tested-by: Nguyen Viet Dung <dung.nguyen...@renesas.com>
> ---
> drivers/mmc/host/renesas_sdhi_internal_dmac.c | 34
> ++++++++++++++++++++++++---
> 1 file changed, 31 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mmc/host/renesas_sdhi_internal_dmac.c
> b/drivers/mmc/host/renesas_sdhi_internal_dmac.c
> index 8e0acd197c43..c8d479375fb6 100644
> --- a/drivers/mmc/host/renesas_sdhi_internal_dmac.c
> +++ b/drivers/mmc/host/renesas_sdhi_internal_dmac.c
[...]
> @@ -251,18 +273,24 @@ static const struct tmio_mmc_dma_ops
> renesas_sdhi_internal_dmac_dma_ops = {
> * implementation as others may use a different implementation.
> */
> static const struct soc_device_attribute gen3_soc_whitelist[] = {
> - { .soc_id = "r8a7795", .revision = "ES1.*" },
> + { .soc_id = "r8a7795", .revision = "ES1.*",
> + .data = (void *)BIT(SDHI_INTERNAL_DMAC_ONE_RX_ONLY) },
> { .soc_id = "r8a7795", .revision = "ES2.0" },
> - { .soc_id = "r8a7796", .revision = "ES1.0" },
> + { .soc_id = "r8a7796", .revision = "ES1.0",
> + .data = (void *)BIT(SDHI_INTERNAL_DMAC_ONE_RX_ONLY) },
Again, could you fix the indentation while updating these 2 initializers?
checkpatch.pl complains about the spaces...
> { .soc_id = "r8a77995", .revision = "ES1.0" },
> { /* sentinel */ }
> };
MBR, Sergei
