On Sun, Mar 24, 2019 at 2:05 PM Andy Shevchenko <andy.shevche...@gmail.com> wrote: > On Sun, Mar 24, 2019 at 12:55 AM Aditya Pakki <pakki...@umn.edu> wrote: > > > > pci_ioremap_bar could fail. The fix captures the failure and > > pass an error code upstream. This can avoid potential NULL > > pointer dereferences in the future. > > > > NAK. > This will break non-DMA case.
> > @@ -172,12 +172,14 @@ static void qrk_serial_setup_dma(struct lpss8250 > > *lpss, struct uart_port *port) Just to clarify, what you need is simple void __iomem *regs; chip->pdata = ... //move it here for better looking code chip->irq = ... chip->regs = pci_ioremap_bar(...); if (!chip->regs) return; and thank you for pointing to this. -- With Best Regards, Andy Shevchenko
