Re: [PATCH v3 3/7] drm/tinydrm: Add MIPI DBI support

From: Jani Nikula
Date: Mon Feb 06 2017 - 06:25:38 EST


On Tue, 31 Jan 2017, Noralf TrÃnnes <noralf@xxxxxxxxxxx> wrote:
> +static bool mipi_dbi_command_is_read(struct mipi_dbi *mipi, u8 cmd)

Okay this is one giant bikeshedding nitpick, but here goes anyway. Why
do you call MIPI DBI just "mipi", here and all around? Why not
"mipi_dbi" or "dbi"? We never talk about "vesa" displays or use that as
a shorthand when we refer to Display Port either.

BR,
Jani.


--
Jani Nikula, Intel Open Source Technology Center