lkml.org 
[lkml]   [2020]   [Jul]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [net-next v4 2/6] net: marvell: prestera: Add PCI interface support
On Mon, Jul 27, 2020 at 5:11 PM Jiri Pirko <jiri@resnulli.us> wrote:
> Mon, Jul 27, 2020 at 03:29:17PM CEST, andy.shevchenko@gmail.com wrote:
> >On Mon, Jul 27, 2020 at 3:23 PM Vadym Kochan <vadym.kochan@plvision.eu> wrote:

...

> >> +err_prestera_dev_register:
> >> + free_irq(pci_irq_vector(pdev, 0), fw);
> >> +err_request_irq:
> >> + pci_free_irq_vectors(pdev);
> >> +err_irq_alloc:
> >> + destroy_workqueue(fw->wq);
> >> +err_wq_alloc:
> >> + prestera_fw_uninit(fw);
> >
> >> +err_prestera_fw_init:
> >> +err_pci_dev_alloc:
> >> +err_dma_mask:
> >
> >All three are useless.
>
> This is okay. It is symmetrical with init. err_what_you_init. It is all
> over the place.

We use multi-point return and these are inconsistent with the
approach. They simple LOCs without value.

--
With Best Regards,
Andy Shevchenko

\
 
 \ /
  Last update: 2020-07-27 17:35    [W:0.084 / U:1.172 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site