[openib-general] (no subject)

[Roland Dreier]

 > mmm, I understand all the comments raised during the review were fixed
 > in the V3 post below, and now you say its both wrong and ugly... for
 > example what's wrong here?

I take back the wrong statement, I misread the patch just now.  But if
you don't think the patch is ugly then I don't think we're looking at
the same thing.

For example

 > +static int __devinit mthca_check_profile_value(int* pval, int pval_default){

and so on...