Re: [PATCH 1/1] PCI: Fix bug resulting in double hpmemsize being assigned to MMIO window

From: mika.westerberg@xxxxxxxxxxxxxxx
Date: Tue Nov 12 2019 - 09:24:09 EST


On Tue, Nov 12, 2019 at 02:17:35PM +0000, Nicholas Johnson wrote:
> > > if (!b_res)
> > > return;
> >
> > I think it may be good to comment here that skip the resources that are
> > assigned (->parent != NULL).
>
> Something like
>
> /* If resource is already assigned, nothing more to do. */

This one looks good to me.