Re: [PATCH v2 2/2] mmc: renesas_sdhi: Fix typo's

From: Geert Uytterhoeven
Date: Wed Jun 22 2022 - 14:11:57 EST


Hi Prabhakar,

On Wed, Jun 22, 2022 at 7:36 PM Lad Prabhakar
<prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote:
> Fix typo's,
> * difference -> different
> * alignment -> aligned
>
> While at it replaced 128-bytes -> 128 byte.
>
> Reported-by: Pavel Machek <pavel@xxxxxxx>
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>

Thanks for your patch!

Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>

> --- a/drivers/mmc/host/renesas_sdhi_internal_dmac.c
> +++ b/drivers/mmc/host/renesas_sdhi_internal_dmac.c
> @@ -355,7 +355,7 @@ renesas_sdhi_internal_dmac_map(struct tmio_mmc_host *host,
>
> data->host_cookie = cookie;
>
> - /* This DMAC cannot handle if buffer is not 128-bytes alignment */
> + /* This DMAC cannot handle if buffer is not 128 byte aligned */

128-byte? ;-)

> if (!IS_ALIGNED(sg_dma_address(data->sg), 128)) {
> renesas_sdhi_internal_dmac_unmap(host, data, cookie);
> return false;

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds