"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>
 {
        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;
        }