Re: [PATCH v5 1/2] drm/bridge: Add sii902x driver

Am Freitag, den 03.06.2016, 10:38 +0100 schrieb Emil Velikov:

Hi Boris.

[...]

quoted

+
+       if (client->irq > 0) {

I was always confused which is the correct way to check this >= 0 vs >
0. DRM has both :-\
Do you have any suggestions, should be 'mass convert' DRM to use only
one of the two ?

IRQ 0 is "no assigned irq", so invalid. Checking for > 0 is the right
thing to do.

Regards,
LUcas

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html