Commit d3deabe4 authored by Rob Clark's avatar Rob Clark
Browse files

drm/msm: Fix inverted WARN_ON() logic 



We want to WARN_ON() if info is NULL.

Suggested-by: default avatarKonrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Fixes: 0838fc3e ("drm/msm/adreno: Check for recognized GPU before bind")
Tested-by: default avatarNeil Armstrong <neil.armstrong@linaro.org>
Signed-off-by: default avatarRob Clark <robin.clark@oss.qualcomm.com>
Reported-by: default avatarAlexey Klimov <alexey.klimov@linaro.org>
Reviewed-by: default avatarKonrad Dybcio <konrad.dybcio@oss.qualcomm.com>
Patchwork: https://patchwork.freedesktop.org/patch/658631/
parent 0838fc3e

drivers/gpu/drm/msm/adreno/adreno_device.c

+1 −1
Original line number Diff line number Diff line
@@ -221,7 +221,7 @@ static int adreno_bind(struct device *dev, struct device *master, void *data) 


	info = adreno_info(config.chip_id);

	/* We shouldn't have gotten this far if we don't recognize the GPU: */

	if (!WARN_ON(info))

	if (WARN_ON(!info))

		return -ENXIO;



	config.info = info;