Tian Tao
2020-Sep-23 00:55 UTC
[Nouveau] [PATCH v2] drm/nouveau/kms: Remove set but not used 'ret'
This addresses the following gcc warning with "make W=1": drivers/gpu/drm/nouveau/dispnv50/disp.c: In function ?nv50_mstm_prepare?: drivers/gpu/drm/nouveau/dispnv50/disp.c:1378:6: warning: variable ?ret? set but not used [-Wunused-but-set-variable] Signed-off-by: Tian Tao <tiantao6 at hisilicon.com> Reviewed-by: Lyude Paul <lyude at redhat.com> --- v2:Remove the braces around ret --- drivers/gpu/drm/nouveau/dispnv50/disp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/nouveau/dispnv50/disp.c b/drivers/gpu/drm/nouveau/dispnv50/disp.c index b111fe2..abc4f46 100644 --- a/drivers/gpu/drm/nouveau/dispnv50/disp.c +++ b/drivers/gpu/drm/nouveau/dispnv50/disp.c @@ -1379,6 +1379,8 @@ nv50_mstm_prepare(struct nv50_mstm *mstm) NV_ATOMIC(drm, "%s: mstm prepare\n", mstm->outp->base.base.name); ret = drm_dp_update_payload_part1(&mstm->mgr); + if (ret) + NV_ATOMIC(drm, "failed to update payload %d\n", ret); drm_for_each_encoder(encoder, mstm->outp->base.base.dev) { if (encoder->encoder_type == DRM_MODE_ENCODER_DPMST) { -- 2.7.4
Reasonably Related Threads
- [PATCH] drm/nouveau/kms: Remove set but not used 'ret'
- [PATCH] drm/nouveau/kms: Remove set but not used 'ret'
- [PATCH -next] gpu: nouveau: Remove set but not used variable
- [PATCH 2/6] drm/nouveau: Use drm_dp_get_payload_info() for getting payload/vcpi
- [WIP PATCH 10/15] drm/nouveau: Stop unsetting mstc->port, use malloc refs