On Fri, Oct 03, 2014 at 03:46:29PM -0700, David Miller wrote: > From: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > Date: Thu, 2 Oct 2014 14:22:19 +0300 > > > The tx_desc struct hold 8 __be64 values. The original code took a > > tx_desc pointer then casted it to an int pointer and then casted it to a > > u64 pointer. It was confusing and triggered some static checker > > warnings. > > > > I have changed the cxgb_pio_copy() to only take tx_desc pointers. This > > isn't really a loss of flexibility because anything else was buggy to > > begin with. > > > > Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> > > Please address the feedback you've received, resubmit this series, and actually > number this second change "2/2" instead of "1/2" :-) > Yes. Sorry for the delay. I'll send that this afternoon. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
