On Thu, May 4, 2017 at 1:15 PM, Andy Shevchenko
<andy.shevchenko@xxxxxxxxx> wrote:
> On Thu, May 4, 2017 at 12:23 PM, Christian König
> <deathsimple@xxxxxxxxxxx> wrote:
>> Am 26.04.2017 um 19:00 schrieb Andy Shevchenko:
> static int ...xxx...(...)
> {
> unsigned int i;
> int ret;
>
> if (res->parent)
> release_resource(res);
>
dev_info() should be here. I commented on this earlier, just forgot.
> res->start = 0;
> res->end = 0;
> dev_info(&bridge->dev, "BAR %d: released %pR\n", i, res);
> return i;
> }
>
>
> struct pci_dev *next = bridge;
> ...
> do {
> bridge = next;
>
> ret = ...xxx...(...);
> if (ret)
Here, of course,
if (ret < 0)
> goto cleanup;
>
> next = bridge->bus ? bridge->bus->self : NULL;
> } while (next);
--
With Best Regards,
Andy Shevchenko
