Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1519113

Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware properties

Show all headers | View raw

On Tue, 2016-11-08 at 15:36 +0200, Andy Shevchenko wrote:
> On Tue, 2016-11-08 at 12:22 +0000, Eugeniy Paltsev wrote:
> > 
> > On Mon, 2016-11-07 at 15:55 +0200, Andy Shevchenko wrote:
>  
> > 
> > > 
> > > > 
> > > > + * @only_quirks_used: Only read quirks (like "is_private" or
> > > > "is_memcpy") from
> > > > + *	platform data structure. Read other parameters from
> > > > device
> > > > tree
> > > > + *	node (if exists) or from hardware autoconfig
> > > > registers.
> > > Can you somehow be more clear that all listed quirks will be
> > > copied
> > > from
> > > platform data.
> > See comment below.
> > 
> > > 
> > >  
> > > > 
> > > >  
> > > >    * @is_nollp: The device channels does not support multi
> > > > block
> > > > transfers.
> > > >    * @chan_allocation_order: Allocate channels starting from 0
> > > > or
> > > > 7
> > > >    * @chan_priority: Set channel priority increasing from 0 to
> > > > 7
> > > > or
> > > > 7
> > > > to 0.
> > > > @@ -52,6 +55,7 @@ struct dw_dma_platform_data {
> > > >   	unsigned int	nr_channels;
> > > >   	bool		is_private;
> > > >   	bool		is_memcpy;
> > > >  
> > > > +	bool		only_quirks_used;
> > > Perhaps add if at the end of quirk list and name just 
> > >  
> > > > 
> > > >  
> > > >   	bool		is_nollp;
> > > ...here
> > >  
> > > bool use_quirks;
> What do think about shorten name?
> 
I don't know better short and understandable name for "use_quirks"
variable. You can suggest your ideas if you want.

> > 
> > I don't treat "is_nollp" as quirks like "is_private" or
> > "is_memcpy".
> > It is like general pdata field: we can easily read it from
> > autoconfig
> > registers (and we don't have any problem with that) in case of
> > pdata/device-tree absence (as opposed to quirks like "is_private"
> > or
> > "is_memcpy")
> > 
> > So, in PATCH v3 series "is_nollp" used as regular pdata field.
> I still would consider is_nollp as a quirk since nothing prevents to
> override the hardware value (see Intel Quark case).
> 
Do you mean this issue:
http://www.spinics.net/lists/linux-serial/msg22948.html
?

As I remember, we had problems with next code:
---------------------------->8--------------------------
channel_writel(dwc, LLP, DWC_LLP_LOC(0xffffffff));
dwc->nollp = DWC_LLP_LOC(channel_readl(dwc, LLP)) == 0;
channel_writel(dwc, LLP, 0);
---------------------------->8--------------------------
which was executed if we didn't use autoconfig registers.
This code doesn't used anymore.

And we don't have any problems with autoconfig registers!

So in case of Intel Quark we will read "nollp" parameter from pdata or
from autoconfig registers (in case of pdata absence). It should work
fine in both cases.
Please correct me if I'm wrong.

So, in my opinion, "is_nollp" should be used as regular pdata field.
-- 
 Paltsev Eugeniy

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware properties Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> - 2016-10-28 18:10 +0200
  [PATCH v3 2/3] dmaengine: DW DMAC: convert to unified device property API Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> - 2016-10-28 18:10 +0200
  [PATCH v3 1/3] dmaengine: DW DMAC: split pdata to hardware properties and platform quirks Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> - 2016-10-28 18:10 +0200
  [PATCH v3 3/3] dmaengine: DW DMAC: move "nollp" to "dwc->flags" Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> - 2016-10-28 18:10 +0200
  Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware  properties Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2016-10-28 18:20 +0200
  Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware  properties Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> - 2016-11-02 13:00 +0100
    Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware  properties Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2016-11-07 15:00 +0100
      Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware  properties Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> - 2016-11-08 13:30 +0100
        Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware  properties Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2016-11-08 14:40 +0100
          Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware  properties Eugeniy Paltsev <Eugeniy.Paltsev@synopsys.com> - 2016-11-10 17:30 +0100
            Re: [PATCH v3 0/3] dmaengine: DW DMAC: split pdata to hardware properties Andy Shevchenko <andy.shevchenko@gmail.com> - 2016-11-11 12:10 +0100

csiph-web