Re: [PATCH] PCI: brcmstb: Add missing if statement

2020-09-21 Thread Jim Quinlan
On Mon, Sep 21, 2020 at 4:45 PM Alex Dewar wrote: > > brcm_pcie_resume() contains a return statement that was presumably > intended to have an "if (ret)" in front of it, otherwise the function > returns prematurely. Fix this. > > I don't know if this code was tested or not, but I assume that this

[PATCH] PCI: brcmstb: Add missing if statement

2020-09-21 Thread Alex Dewar
brcm_pcie_resume() contains a return statement that was presumably intended to have an "if (ret)" in front of it, otherwise the function returns prematurely. Fix this. I don't know if this code was tested or not, but I assume that this bug means that this driver will not resume properly. Fixes: a