+               if (0x44570140 != comp_type) {
+                       dev_err(&pdev->dev, "Unknown Synopsys component type: "
+                                       "0x%08x\n",   comp_type);
+                       r = -ENODEV;
+                       goto err_iounmap;
+               }
+       }
+       {

The general convention in linux is to do (if x = constant).
Old coding habits are difficult to get rid of ;-). I'll change it in the updated patchset, once all comments have been collected.

thanks,
jh
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to