[PATCH 2/3] dmaengine: dw_dmac: Enhance device tree support
Viresh Kumar
viresh.kumar at linaro.org
Fri Oct 12 10:47:49 EDT 2012
On 12 October 2012 20:11, Shevchenko, Andriy
<andriy.shevchenko at intel.com> wrote:
> On Fri, 2012-10-12 at 19:36 +0530, Viresh Kumar wrote:
> I meant is to use that constant instead of hard coding 4 everywhere.
> It's a maximum value, not the SoC specific.
Can be done.
>> + u32 val, arr[4];
> what about to use tmp name instead of val? It's really minor, but I
> think val name is little bit unrelated to a loop counter.
Its not minor at all, its major. Even when i was coding it, i thought
about renaming
it several times during this last patch. But couldn't find a better name.
I don't like tmp. Give me something better.
viresh
More information about the linux-arm-kernel
mailing list