These were leftover from: commit 469ffe083665 ("[media] tda18271c2dd: Remove the CHK_ERROR macro") and commit 58d5eaec9f87 ("[media] drxd: Don't use a macro for CHK_ERROR ...") that programmatically removed the CHK_ERROR macro, which left behind a few self assignments that Clang warns about. These instances aren't errors. Signed-off-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> --- drivers/media/dvb-frontends/drxd_hard.c | 3 --- drivers/media/dvb-frontends/tda18271c2dd.c | 1 - 2 files changed, 4 deletions(-) diff --git a/drivers/media/dvb-frontends/drxd_hard.c b/drivers/media/dvb-frontends/drxd_hard.c index 0696bc62dcc9..ff18a0f7dc41 100644 --- a/drivers/media/dvb-frontends/drxd_hard.c +++ b/drivers/media/dvb-frontends/drxd_hard.c @@ -2140,7 +2140,6 @@ static int DRX_Start(struct drxd_state *state, s32 off) } break; } - status = status; if (status < 0) break; @@ -2251,7 +2250,6 @@ static int DRX_Start(struct drxd_state *state, s32 off) break; } - status = status; if (status < 0) break; @@ -2318,7 +2316,6 @@ static int DRX_Start(struct drxd_state *state, s32 off) } break; } - status = status; if (status < 0) break; diff --git a/drivers/media/dvb-frontends/tda18271c2dd.c b/drivers/media/dvb-frontends/tda18271c2dd.c index 2d2778be2d2f..45cd5ba0cf8a 100644 --- a/drivers/media/dvb-frontends/tda18271c2dd.c +++ b/drivers/media/dvb-frontends/tda18271c2dd.c @@ -674,7 +674,6 @@ static int PowerScan(struct tda_state *state, Count = 200000; wait = true; } - status = status; if (status < 0) break; if (CID_Gain >= CID_Target) { -- 2.15.1.504.g5279b80103-goog