> I am used to sending out one patch per each fix like this for each function.
> If you don't like me doing this I don't when sending patches for your
> subsystem(s).
I think that would be preferable.
> In addition would something like this fix your complains about not fixing
> error
> handling in bnx2x_dcbnl_update_applist.
> int bnx2x_dcbnl_update_applist(struct bnx2x *bp, bool delall) {
> int i, err = 0;
>
> for (i = 0; i < DCBX_MAX_APP_PROTOCOL && err == 0; i++) {
...
> err = dcb_setapp(bp->dev, &app);
> + if (err)
> + break;
Well, either that or retain the fact there's an error and return it at the end.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
