linux-kernel - RE: [PATCH v6 1/2] dmaengine: Add Xilinx Central DMA DT Binding Documentation

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <C246CAC1457055469EF09E3A7AC4E11A4A4E7DD4@XAP-PVEXMBX01.xlnx.xilinx.com>
Date:	Thu, 24 Sep 2015 12:19:33 +0000
From:	Appana Durga Kedareswara Rao <appana.durga.rao@...inx.com>
To:	Arnd Bergmann <arnd@...db.de>
CC:	"robh+dt@...nel.org" <robh+dt@...nel.org>,
	"vinod.koul@...el.com" <vinod.koul@...el.com>,
	Anirudha Sarangi <anirudh@...inx.com>,
	"pawel.moll@....com" <pawel.moll@....com>,
	"mark.rutland@....com" <mark.rutland@....com>,
	"ijc+devicetree@...lion.org.uk" <ijc+devicetree@...lion.org.uk>,
	"galak@...eaurora.org" <galak@...eaurora.org>,
	Michal Simek <michals@...inx.com>,
	Soren Brinkmann <sorenb@...inx.com>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v6 1/2] dmaengine: Add Xilinx Central DMA DT Binding
 Documentation

Hi Arnd,
	
> -----Original Message-----
> From: Arnd Bergmann [mailto:arnd@...db.de]
> Sent: Monday, September 07, 2015 6:26 PM
> To: Appana Durga Kedareswara Rao
> Cc: robh+dt@...nel.org; vinod.koul@...el.com; Anirudha Sarangi;
> pawel.moll@....com; mark.rutland@....com; ijc+devicetree@...lion.org.uk;
> galak@...eaurora.org; Michal Simek; Soren Brinkmann; Appana Durga
> Kedareswara Rao; devicetree@...r.kernel.org; linux-arm-
> kernel@...ts.infradead.org; linux-kernel@...r.kernel.org
> Subject: Re: [PATCH v6 1/2] dmaengine: Add Xilinx Central DMA DT Binding
> Documentation
> 
> On Monday 07 September 2015 18:03:08 Kedareswara rao Appana wrote:
> > +- dma-channel child node: Should have only one channel
> >
> 
> What is the point of the child node if there is only one instance?

Sorry for the delay in reply I was on leave for the last couple weeks.

There are few reasons for these:
---> We (Xilinx) have 3 different DMA's and in order to make consistency b/w the three dma's used the child node concepts so that users can easily compare the drivers
---> It is a soft IP so features like (multiple channels, cyclic dma mode) may added in nearby future so for future driver changes it will be straight forward. 

Please let me know if it is wrong will fix it and will send the next version of the path.

Regards,
Kedar.

> 
> Could you just fold this into the main node instead?
> 
> (Sorry jumping in here without looking at the previous versions, I just returned to
> work from my parental leave).
> 
> 	Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/