On Mon, Nov 9, 2015 at 5:07 AM, Sinan Kaya <okaya@xxxxxxxxxxxxxx> wrote:
On 11/8/2015 3:09 PM, Andy Shevchenko wrote:
On Sun, Nov 8, 2015 at 6:52 AM, Sinan Kaya <okaya@xxxxxxxxxxxxxx> wrote:
This patch adds supporting utility functions
for selftest. The intention is to share the self
test code between different drivers.
Supported test cases include:
2. streaming DMA
3. coherent DMA
4. scatter-gather DMA
+ u32 i, j = 0;
Is i or j is going to be used for HW communication? No? What about
assignment to a values of type u32? No? Plain counters? Use plain
It's actually comment about your all patches I saw last week.
+ int err = 0;
+ int ret;
Any reason to have two instead of one of similar meaning?
Don't forget to check if it's redundant assignment (check in all your
patches as well).