Hello,

On Fri, 20 Jul 2018 10:15:53 +0100, Lorenzo Pieralisi wrote:

> >             switch (resource_type(res)) {
> >             case IORESOURCE_IO:
> > -                   advk_pcie_set_ob_win(pcie, 1,
> > -                                        upper_32_bits(res->start),
> > -                                        lower_32_bits(res->start),
> > -                                        0, 0xF8000000, 0,
> > -                                        lower_32_bits(res->start),
> > -                                        OB_PCIE_IO);
> >  -                  err = pci_remap_iospace(res, iobase);
> >  +                  err = devm_pci_remap_iospace(dev, res, iobase);
> >                     if (err) {
> >                             dev_warn(dev, "error %d: failed to map resource 
> > %pR\n",
> >                                      err, res);  
> 
> Hi Stephen,
> 
> thanks for fixing this up, the fix looks fine to me (Thomas please
> confirm), if it is I would ask Bjorn please to send it upstream when the
> time for the PCI pull request comes.

I confirm that the fix looks good to me. Thanks Stephen for fixing it
up.

Best regards,

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

Reply via email to