> 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.