From: Andy Shevchenko Date: Thu, 17 Oct 2019 11:49:12 +0000 (+0300) Subject: drm/mipi_dbi: Use simple right shift instead of double negation X-Git-Tag: v5.5-rc1~128^2~24^2~21 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=dc6015cb1c834ce39c0d3ff09b3f1818b70ba9ad;p=thirdparty%2Flinux.git drm/mipi_dbi: Use simple right shift instead of double negation GCC complains about dubious bitwise OR operand: drivers/gpu/drm/drm_mipi_dbi.c:1024:49: warning: dubious: x | !y CC [M] drivers/gpu/drm/drm_mipi_dbi.o As long as buffer is consist of byte (u8) values, we may use simple right shift and satisfy compiler. It also reduces amount of operations needed. Signed-off-by: Andy Shevchenko Reviewed-by: Noralf Trønnes Tested-by: Noralf Trønnes Signed-off-by: Noralf Trønnes Link: https://patchwork.freedesktop.org/patch/msgid/20191017114912.61522-1-andriy.shevchenko@linux.intel.com --- diff --git a/drivers/gpu/drm/drm_mipi_dbi.c b/drivers/gpu/drm/drm_mipi_dbi.c index ccfb5b33c5e3f..e34058c721bec 100644 --- a/drivers/gpu/drm/drm_mipi_dbi.c +++ b/drivers/gpu/drm/drm_mipi_dbi.c @@ -1021,7 +1021,7 @@ static int mipi_dbi_typec3_command_read(struct mipi_dbi *dbi, u8 *cmd, unsigned int i; for (i = 0; i < len; i++) - data[i] = (buf[i] << 1) | !!(buf[i + 1] & BIT(7)); + data[i] = (buf[i] << 1) | (buf[i + 1] >> 7); } MIPI_DBI_DEBUG_COMMAND(*cmd, data, len);