This else statement wasn't indented so it was confusing. Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> diff --git a/drivers/gpu/drm/nouveau/nouveau_usif.c b/drivers/gpu/drm/nouveau/nouveau_usif.c index cb1182d..41c9b404 100644 --- a/drivers/gpu/drm/nouveau/nouveau_usif.c +++ b/drivers/gpu/drm/nouveau/nouveau_usif.c @@ -83,9 +83,10 @@ usif_notify(const void *header, u32 length, const void *data, u32 size) if (WARN_ON(!(ntfy = (void *)(unsigned long)rep->v0.token))) return NVIF_NOTIFY_DROP; BUG_ON(rep->v0.route != NVDRM_NOTIFY_USIF); - } else - if (WARN_ON(1)) - return NVIF_NOTIFY_DROP; + } else { + if (WARN_ON(1)) + return NVIF_NOTIFY_DROP; + } if (WARN_ON(!ntfy->p || ntfy->reply != (length + size))) return NVIF_NOTIFY_DROP; -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html