Re: [PATCH v4 7/9] serial: 8250: dw: Add support for DMA flow controlling devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

Thanks,
Miquèl




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux