On Sun, 2008-02-10 at 21:47 -0800, Joe Perches wrote: > - if (request_region(*base, 256, "aic7xxx") == 0) > + if (!request_region(*base, 256, "aic7xxx")) This patch is completely pointless. if (x == 0) and if (!x) mean identical things and there's no style standard preferring one form over another. There's a marginal preference for if (!x) over if (x == NULL) for pointers, but it's still up to a driver writer if they wish to use the marginally unpreferred form. James - To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
