Thread (24 messages) 24 messages, 9 authors, 2010-08-10

[RFC] SDMA support for i.MX

flat view

From: Magnus Lilja <hidden>
Date: 2010-08-09 18:38:14

Hi Sascha,

On 2010-08-09 11:05, Sascha Hauer wrote:
Hi all,

Ever wondered what's hidden behind the tons of SDMA/iapi stuff in the
Freescale kernel? It's a nice and convenient to use DMA engine. The
following patches add support for it.

Nice!
git://git.pengutronix.de/git/imx/sdma-firmware.git

As the layout of the firmware is not given by fsl but by me it should
be part of the review.
I don't have any comments on the layout but some general comments on files in sdma-firmware.git:

Some files are missing copyright and license information.

Shouldn't the gen_one() function do a free(name) also when everything is successfull? I.e. before return 0.
Also, in case of errors after line 67, fclose(f) should be called.


Regards, Magnus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help