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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html