On Mon, 4 Apr 2022, Miquel Raynal wrote: > Hi Andy, Ilpo, > > andriy.shevchenko@xxxxxxxxxxxxxxx wrote on Fri, 1 Apr 2022 20:38:24 > +0300: > > > On Fri, Apr 01, 2022 at 01:56:49PM +0300, Ilpo Järvinen wrote: > > > On Wed, 30 Mar 2022, Miquel Raynal wrote: > > > > ... > > > > > > + writel(0, p->membase + RZN1_UART_RDMACR); > > > > + writel(1, p->membase + DW_UART_DMASA); > > > > > > Currently there is serial_out(), dw8250_writel_ext(), and a few writel()s > > > too for writing to registers. It would be nice to move towards more > > > homogeneous approach rather than adding more writel()s. > > > > Actually a good catch! > > > > > I suggest dw8250_writel_ext() is moved to dwlib.h. Then it could be used > > > here (and dw8250_readl_ext() too should be moved but IIRC there wasn't > > > any reads added by this series). > > > > I agree that for the sake of symmetry it's good to move both. > > I moved them both to dwlib.h as suggested. > > I had a look at the current uses of readb/l and writeb/l in dw.c but > converting all these function calls is not as straightforward as I > initially thought so I decided to limit myself to moving these helpers > and using them in my additions, I hope this is fine. Yes, I think it's enough for this series. -- i.
