> Thanks, applied.  Overall this is very clean (one of the best
> formatted patches I've gotten in a while ;), but scripts/checkpatch.pl
> does complain:
> 
>     WARNING: __func__ should be used instead of gcc specific 
> __FUNCTION__
>     #83: FILE: drivers/infiniband/hw/nes/nes_nic.c:1504:
>     +       nes_debug(NES_DBG_NETDEV, "%s: %s\n", 
> __FUNCTION__, netdev->name);
> 
> I edited this by hand when I applied, but to partially answer your
> question from your other email, one thing to do when submitting
> patches is to run them through checkpatch.pl and consider the output.
> You don't have to fix everything it flags, but at least think about
> whether there's a good reason not to fix it (and there often is a good
> reason to ignore the false positives it flags).
> 
>  - R.
> 

Will do.  Thanks for the feedback.

Glenn
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to