From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [2/2] dma: xilinx: dpdma: Add the Xilinx DisplayPort DMA engine driver From: Vinod Koul Message-Id: <20180129041927.GJ18649@localhost> Date: Mon, 29 Jan 2018 09:49:35 +0530 To: Hyun Kwon Cc: "dmaengine@vger.kernel.org" , "devicetree@vger.kernel.org" , Michal Simek , Tejas Upadhyay List-ID: T24gV2VkLCBKYW4gMjQsIDIwMTggYXQgMDU6NDU6MDBQTSArMDAwMCwgSHl1biBLd29uIHdyb3Rl OgoKPiA+ID4gPiA+ICtzdGF0aWMgaW50IHhpbGlueF9kcGRtYV9jb25maWcoc3RydWN0IGRtYV9j aGFuICpkY2hhbiwKPiA+ID4gPiA+ICsJCQkgICAgICAgc3RydWN0IGRtYV9zbGF2ZV9jb25maWcg KmNvbmZpZykKPiA+ID4gPiA+ICt7Cj4gPiA+ID4gPiArCWlmIChjb25maWctPmRpcmVjdGlvbiAh PSBETUFfTUVNX1RPX0RFVikKPiA+ID4gPiA+ICsJCXJldHVybiAtRUlOVkFMOwo+ID4gPiA+Cj4g PiA+ID4gPz8gV2h5IGFyZSB0aGUgY29uZmlnIHZhbHVlcyBub3QgdXNlZCwgaG93IGVsc2UgZG8g eW91IGNvbmZpZ3VyZSB0aGUKPiA+ID4gPiBjaGFubmVsPwo+ID4gPiA+Cj4gPiA+Cj4gPiA+IFRo ZXJlJ3Mgbm90IG11Y2ggY29uZmlndXJhdGlvbiB0byBiZSBkb25lLiBDaGFubmVscyBhcmUgcHJl dHR5IG11Y2gKPiA+IGlkZW50aWNhbAo+ID4gPiB3aXRoIGZpeGVkIGNvbmZpZ3VyYXRpb25zLgo+ ID4gCj4gPiBobW1tLCB5b3UgZG8gc3VwcG9ydCBETUFfU0xBVkUgcmlnaHQ/Cj4gPiAKPiAKPiBZ ZXMsIGFuZCBpdCdzIHNpbmdsZSBkaXJlY3Rpb24sIERNQV9NRU1fVE9fREVWLgoKT2theSBob3cg ZG8geW91IGZpbmQgdGhlIERNQSBkYXRhIHdpZHRoIGFuZCB0aGUgYnVyc3QgdG8gYXBwbHkgd2hl biB5b3UgRE1BCnRvIGRldmljZT8gQXJlIHRoZXNlIG5vdCBwcm9ncmFtbWFibGU/CgpJIHRoaW5r IHlvdSBuZWVkIHRvIGdvIGJhY2sgYW5kIGNoZWNrIHRoYXQgcGFydC4gTW9yZW92ZXIgYSBjbGll bnQgbWF5IHNlbmQKeW91IHBhcmFtZXRlcnMgYW5kIHByb2dyYW0gdGhlIERldmljZSBGSUZPLCB5 b3Ugbm90IGhvbm9yaW5nIHRob3NlIHdpbGwKcmVzdWx0IGluIGJhZCBkYXRhIHJlY2VpdmVkIG9u IERldmljZSwgYW0gc3VyZSB5b3UgZG9uJ3Qgd2FudCB0aGF0Cg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [PATCH 2/2] dma: xilinx: dpdma: Add the Xilinx DisplayPort DMA engine driver Date: Mon, 29 Jan 2018 09:49:35 +0530 Message-ID: <20180129041927.GJ18649@localhost> References: <1515204848-3493-1-git-send-email-hyun.kwon@xilinx.com> <1515204848-3493-2-git-send-email-hyun.kwon@xilinx.com> <20180112141355.GO18649@localhost> <20180124144059.GI18649@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Hyun Kwon Cc: "dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , Michal Simek , Tejas Upadhyay List-Id: devicetree@vger.kernel.org On Wed, Jan 24, 2018 at 05:45:00PM +0000, Hyun Kwon wrote: > > > > > +static int xilinx_dpdma_config(struct dma_chan *dchan, > > > > > + struct dma_slave_config *config) > > > > > +{ > > > > > + if (config->direction != DMA_MEM_TO_DEV) > > > > > + return -EINVAL; > > > > > > > > ?? Why are the config values not used, how else do you configure the > > > > channel? > > > > > > > > > > There's not much configuration to be done. Channels are pretty much > > identical > > > with fixed configurations. > > > > hmmm, you do support DMA_SLAVE right? > > > > Yes, and it's single direction, DMA_MEM_TO_DEV. Okay how do you find the DMA data width and the burst to apply when you DMA to device? Are these not programmable? I think you need to go back and check that part. Moreover a client may send you parameters and program the Device FIFO, you not honoring those will result in bad data received on Device, am sure you don't want that -- ~Vinod -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html