Commit 66088749 authored Jun 30, 2017 by Dan Carpenter Committed by Laurent Pinchart Aug 03, 2017

drm: rcar-du: Remove an unneeded NULL check



"params" can't be NULL here.  The next lines assume that we either
hit the break statement of "params->mpixelclock == ~0UL".  The
inconsistent NULL checking makes static checkers complain.  I've just
removed the test.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>

parent f4112469

drivers/gpu/drm/rcar-du/rcar_dw_hdmi.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -45,7 +45,7 @@ static int rcar_hdmi_phy_configure(struct dw_hdmi *hdmi,
		{
		const struct rcar_hdmi_phy_params *params = rcar_hdmi_phy_params;

		for (; params && params->mpixelclock != ~0UL; ++params) {
		for (; params->mpixelclock != ~0UL; ++params) {
		if (mpixelclock <= params->mpixelclock)
		break;
		}

Admin message